groupfolders icon indicating copy to clipboard operation
groupfolders copied to clipboard

Add E2E tests for copying and moving shared groupfolders

Open danxuliu opened this issue 1 year ago โ€ข 1 comments

This pull request adds E2E tests for the issue fixed in nextcloud/server#47847 (some scenarios are fixed by that pull request, other scenarios work fine already but were added for completion).

~~[skip ci] was used for now until the fix is merged, as otherwise running one of the scenarios would recursively copy the data directory of the Nextcloud server and maybe cause issues in CI.~~

danxuliu avatar Sep 09 '24 13:09 danxuliu

They seem to be failing though

artonge avatar Dec 02 '24 15:12 artonge

@danxuliu can you rebase and check if everything passes now?

provokateurin avatar Jun 27 '25 07:06 provokateurin

In the rebase the helper functions were updated to the latest server version (so they now take into account the change in how internal shares are created since Nextcloud 31) and it was no longer needed to add one of them (createFolder) as it was added to the cypress code of groupfolders in the meantime. The tests themselves were not modified.

The tests were run locally several times and they always passed, let's see what CI says :crossed_fingers:

danxuliu avatar Jun 29 '25 22:06 danxuliu

The tests were run locally several times and they always passed, let's see what CI says ๐Ÿคž

Well, all tests failed :shrug: The problems seem to be that showing the list of possible shares takes more than 4 seconds and... that the session is lost while creating the share so a new one is started, but the following requests fail as being unauthenticated due to the new session not being logged in? I do not have any idea of what is going on :smiling_face_with_tear:

danxuliu avatar Jun 30 '25 02:06 danxuliu

I used saveState/restoreState to reduce the amount of duplicate operations during the setup phase. I also raised the timeout, it seems to work for now.

artonge avatar Sep 08 '25 10:09 artonge