owncloud / core

:cloud: ownCloud web server core (Files, DAV, etc.)
https://owncloud.com
GNU Affero General Public License v3.0
8.37k stars 2.06k forks source link

Acceptance tests for sharing when the receiver has a matching file/folder #35414

Open phil-davis opened 5 years ago

phil-davis commented 5 years ago

Background

Currently the sharing acceptance tests mostly use files and folders that are in the "standard" acceptance test webUI or API skeleton. e.g. user1 and user2 both have simple-folder. When user1 shares simple-folder with user2 then user2 receives a folder called simple-folder (2). So we have "accidentally" created lots of test scenarios for the case where the share receiver already has a matching file or folder name.

As part of speeding up CI, we are reducing the use of the "standard" skeleton (it takes significant time to create for every user of every scenario). Scenarios will (soon) usually result in the share receiver not already having a matching file or folder. So the (2) naming thing will not get tested much.

To Do

Create explicit test scenarios for sharing where the share receiver already has a matching file or folder name.

sakshamgurung commented 3 years ago

@phil-davis I have come with following scenario lists that can be added. Please see if these are relevant:

phil-davis commented 3 years ago

@sakshamgurung That seems good.

The scenarios could mix up file and folder - one share is for a folder "x.y" and the other share is for a file "x.y". Brian should be able to see both received shares somehow.

Also try with different case - "x.y" and "X.Y" to make sure that there is no problem. I think that 2 shares like that should be nicely received separately.