[Tests-Only] Make API tests pass on EOS

[haribhandari07]

Description

Right now ocis behaves differently (partly) with EOS storage. This PR makes files permissions tests to pass on EOS

Related Issue

• Part of https://github.com/owncloud/ocis/issues/253

Motivation and Context

How Has This Been Tested?

• CI
• [tests-only] try test changes from core/37575 ocis#362
• [tests-only] try test changes from core/37575 ocis-reva#343

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Database schema changes (next release will require increase of minor version instead of patch)
• Breaking change (fix or feature that would cause existing functionality to change)
• Technical debt
• Tests only (no source changes)

Checklist:

• Code changes
• Unit tests added
• Acceptance tests added
• Documentation ticket raised:
• Changelog item, see TEMPLATE

[codecov]

Codecov Report

Merging #37575 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #37575   +/-   ##
=========================================
  Coverage     64.70%   64.70%           
  Complexity    19350    19350           
=========================================
  Files          1281     1281           
  Lines         75607    75607           
  Branches       1333     1333           
=========================================
  Hits          48920    48920           
  Misses        26295    26295           
  Partials        392      392           

Flag	Coverage Δ	Complexity Δ
#javascript	54.03% <ø> (ø)	0.00 <ø> (ø)
#phpunit	65.88% <ø> (ø)	19350.00 <ø> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a13d708...e29cd0a. Read the comment docs.

[codecov]

Codecov Report

Merging #37575 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #37575   +/-   ##
=========================================
  Coverage     64.74%   64.74%           
  Complexity    19360    19360           
=========================================
  Files          1281     1281           
  Lines         75622    75622           
  Branches       1333     1333           
=========================================
  Hits          48959    48959           
  Misses        26271    26271           
  Partials        392      392           

Flag	Coverage Δ	Complexity Δ
#javascript	54.03% <ø> (ø)	0.00 <ø> (ø)
#phpunit	65.92% <ø> (ø)	19360.00 <ø> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2aacc87...3e0f7ed. Read the comment docs.

[dpakach]

LGTM 👍

[individual-it]

I'm taking over this PR

[individual-it]

rebased and fixed conflicts

I worked myself on this PR

[phil-davis]

There are also 2 occurrences of [a-f0-9:]{1,32} in apiWebdavOperations/search.feature - probably those should also be made more flexible.

Maybe we should have a substituteInLineCodes item for this string. We keep changing it a bit and have to search-replace throughout the feature files. Maybe %etag_regex% could expand to the regex?

[individual-it]

making it more flexible is currently an issue because when checking the substitution in the headers we also run preg_quote to be able to mix the substitution with regular expressions see f4beb74#diff-22732c7ce419762b047562109c5dff15R154
so if the substitution is a regular expression itself there are too many quotes

[individual-it]

some tests fail in the ocis repo because of new enabled tests in 21803fd