update reva

[C0rby]

Updated reva to latest edge.
This includes the implementation for #2986

[ownclouders]

💥 Acceptance test localApiTests-apiSpaces-ocis failed. Further test are cancelled...

[rhafer]

boom Acceptance test Web-Tests-ocis-smoke-ocis-storage-1 failed. Further test are cancelled...

@C0rby I think this should be fixed if you rebase to latest master

[C0rby]

Thanks!

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[phil-davis]

The JSON response to a "list spaces" request no longer includes the structure "owner - user - id". A space has users who have the "manager" role. There is not a single "owner" of a space.

I adjusted the failing test so that it looks for the "manager" role permission.

Note: the requirement of these sort of tests (existing tests, and the test that I adjusted) are rather brittle, because they specify the index "0", "1" etc where the permission/role appears in the array of user permissions/roles. That breaks if the permissions array is returned in a different order, which is not a good thing. We are thinking about how best to refactor that so that tests are less brittle see issue #3179

[C0rby]

Thanks @phil-davis for taking care. I wanted to take a look today but since you've already dealt with it we can just continue with the oCIS release. 👍 💪