Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add grpc integration tests for the ocis and the owncloud storage drivers #1514

Merged
merged 24 commits into from
Mar 16, 2021

Conversation

aduffeck
Copy link
Contributor

@aduffeck aduffeck commented Mar 4, 2021

This PR adds a test suite which tests the grpc interface to the storage provider with the ocis and the owncloud drivers.

It also fixes a few issues found while writing the tests:

  • 28063fd fixes searching users by opaque id in the demo user provider
  • 39e04dc makes the GetPath permission translate to a "r" ace permission
  • 0c96aae fixes the user and group ids in the grants created from an ace
  • defc5de makes the owncloud driver return CODE_NOT_FOUND when listing the non-existent shared folder

@aduffeck aduffeck force-pushed the grpc-tests branch 3 times, most recently from 9e5fda7 to 429af59 Compare March 11, 2021 14:09
@aduffeck aduffeck changed the title Add grpc integration tests for the ocis and the owncloud storage providers [tests-only] Add grpc integration tests for the ocis and the owncloud storage providers Mar 11, 2021
@aduffeck aduffeck changed the title [tests-only] Add grpc integration tests for the ocis and the owncloud storage providers [tests-only] Add grpc integration tests for the ocis and the owncloud storage drivers Mar 11, 2021
@aduffeck aduffeck marked this pull request as ready for review March 11, 2021 14:25
@aduffeck aduffeck requested a review from labkode as a code owner March 11, 2021 14:25
@aduffeck aduffeck changed the title [tests-only] Add grpc integration tests for the ocis and the owncloud storage drivers Add grpc integration tests for the ocis and the owncloud storage drivers Mar 11, 2021
butonic
butonic previously approved these changes Mar 11, 2021
@ishank011 ishank011 merged commit 881b15e into cs3org:master Mar 16, 2021
@ishank011
Copy link
Contributor

@aduffeck I merged the PR without rebasing, which breaks the expected tests sync, as well introduces some lint issues, sorry about that! Can you create another PR to fix these? Thanks!

ffurano pushed a commit to ffurano/reva that referenced this pull request Apr 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants