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

enable sharing tests #518

Closed
2 tasks
individual-it opened this issue Sep 3, 2020 · 8 comments · Fixed by owncloud/web#4103 or owncloud/core#38052
Closed
2 tasks

enable sharing tests #518

individual-it opened this issue Sep 3, 2020 · 8 comments · Fixed by owncloud/web#4103 or owncloud/core#38052
Assignees

Comments

@individual-it
Copy link
Member

individual-it commented Sep 3, 2020

  • core API tests
  • Phoenix UI tests

Questions to discuss:

  1. in ocis every share needs to be accepted first, how should we deal with that in the tests
    1. make the step 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 Gherkin
    2. duplicate tests
  2. in OCIS shares are accessible via the Shares folder.
    1. make the test be aware of that, so that it does not need to be mentioned in the Gherkin. But how to deal with when the tests check the existence of files/folders? The test code would have to know that this particular item is to be expected in the Shares folder or outside.
    2. add %SharesFolder% in front of every item that is expected to be a Share, in ocis-tests substitute that to Shares. This would not work for tests that expect the receiving item to ass (2) to the name if an item with the same name exists
    3. duplicate tests

CC @phil-davis @dpakach @PVince81 @kulmann

@phil-davis
Copy link
Contributor

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"

@phil-davis
Copy link
Contributor

Note: PR owncloud/core#37774 did enable a small number tests that happened to pass.

@phil-davis
Copy link
Contributor

core PR owncloud/core#37881 has the way forward for this. It creates new sharing scenarios that have share_folder set to Shares and auto-accept of shares off. That corresponds to the default OCIS settings. Those new scenarios run on both ownCloud10 and OCIS. See the PR for a detailed description of what is done with tags, scenario background, accepting received shares etc.

This issue can now be assigned to someone to do similar with the remaining sharing API tests.

@phil-davis
Copy link
Contributor

I moved this to ToDo in Sprint 20-17 because there seems be no other sprint to choose.

@phil-davis
Copy link
Contributor

@haribhandari07 please also assign anyone else who is working on this.

@dpakach
Copy link
Contributor

dpakach commented Sep 23, 2020

Phoenix WebUI Tests

@dpakach
Copy link
Contributor

dpakach commented Oct 16, 2020

Webui tests to enable in CI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants