owncloud / product

ownCloud Product Backlog
GNU Lesser General Public License v3.0
0 stars 1 forks source link

Review local API tests that were skipped on OCIS storage #225

Open phil-davis opened 4 years ago

phil-davis commented 4 years ago

In owncloud/ocis and owncloud/ocis-reva there are local API test scenarios. Those scenarios demonstrate existing bugs. When the bug is fixed, the bug-demo scenario should be deleted and there should be some scenario(s) in the core API tests that demonstrate the correct behavior and start passing.

In https://github.com/owncloud/ocis/pull/556/commits/b953b5e546a6ed0f4199e14457227dda41826ea0 various local API test scenarios were skipped on the new "ocis" storage. They were skipped because they "fail" - i.e. the behavior has changed. Either:

phil-davis commented 3 years ago

I removed my assignment because I have not got to doing this! Someone else should be able to look through the scenarios and sort out what to do.

ocis-reva is gone since I made this issue. So ignore that.

Look through the "local" API tests in https://github.com/owncloud/ocis/tree/master/tests/acceptance/features

If the scenario now demonstrates the correct behaviour, then:

If the scenario is still demonstrating "bad behaviour", then find and check the "good behaviour" scenario in owncloud/core. Also check the referenced issue. Update any references that are out-of-date.