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 test scenarios for trying to reshare when user is in a group excluded from sharing #33824

Closed
5 tasks done
phil-davis opened this issue Dec 7, 2018 · 6 comments
Closed
5 tasks done

Comments

@phil-davis
Copy link
Contributor

phil-davis commented Dec 7, 2018

There are already some acceptance test scenarios that check that, for a user in a group that is excluded from sharing, that user cannot create a share of their own file or folder.
tests/acceptance/features/webUISharingInternalGroups/shareWithGroups.feature

  Scenario: user tries to share a file in a group which is blacklisted from sharing
  Scenario: user tries to share a folder in a group which is blacklisted from sharing

None of these should be possible.

Ref: https://github.com/owncloud/enterprise/issues/3037

@ownclouders
Copy link
Contributor

GitMate.io thinks possibly related issues are #33745 (Add tests for excluding group from receiving share), #10307 (Exclude Groups from Sharing), #17346 (Exclude group from sharing after a share was created), #31490 (Exclude users from sharing if they are a member of any excluded group), and #31696 (Acceptance tests for exclude users from sharing changes).

@paurakhsharma
Copy link
Member

@phil-davis Turns out owncloud/enterprise#3037 is only UI bug where sharing dialog box shows field to enter the name of sharee. PR #33865 doesn't use UI to verify that. It only checks if the user is not allowed to share. New PR needs to be made to verify owncloud/enterprise#3037 where we should use UI to verify this.

@phil-davis
Copy link
Contributor Author

Yes, the webUI is probably using its "private" API. The public sharing API is working, the new tests demonstrate that. I guess the private API used by the webUI will be not doing the needed checks.

@paurakhsharma
Copy link
Member

@phil-davis Do we close this now?

@phil-davis
Copy link
Contributor Author

Done - tests for attempting to reshare have been merged.

@phil-davis
Copy link
Contributor Author

I raised issue #33918 to deal with the webUI issue.

@lock lock bot locked as resolved and limited conversation to collaborators Dec 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants