-
Notifications
You must be signed in to change notification settings - Fork 186
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
enable sharing tests #518
enable sharing tests #518
Comments
Also see owncloud/ocis-reva#415 - which is a more general issue to enable as many API test scenarios as "reasonable" so that they are run in CI, listed in expected-failures and "ready to be made to pass" |
Note: PR owncloud/core#37774 did enable a small number tests that happened to pass. |
core PR owncloud/core#37881 has the way forward for this. It creates new sharing scenarios that have This issue can now be assigned to someone to do similar with the remaining sharing API tests. |
I moved this to ToDo in Sprint 20-17 because there seems be no other sprint to choose. |
@haribhandari07 please also assign anyone else who is working on this. |
Questions to discuss:
Given parameter "shareapi_auto_accept_share" of app "core" has been set to "yes"
set some flag, so that the tests know to accept shares without it being mentioned in the GherkinShares
folder.%SharesFolder%
in front of every item that is expected to be a Share, in ocis-tests substitute that toShares
. This would not work for tests that expect the receiving item to ass(2)
to the name if an item with the same name existsCC @phil-davis @dpakach @PVince81 @kulmann
The text was updated successfully, but these errors were encountered: