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

[tests-only] test issue 35974 #38867

Closed
wants to merge 6 commits into from

Conversation

phil-davis
Copy link
Contributor

@phil-davis phil-davis commented Jun 19, 2021

Related issue: #35974

@phil-davis phil-davis self-assigned this Jun 19, 2021
@ownclouders
Copy link
Contributor

💥 Acceptance tests apiTrashbin-mariadb10.2-php7.4 failed. The build is cancelled...

@phil-davis
Copy link
Contributor Author

https://drone.owncloud.com/owncloud/core/30741/4/13
The test show 2 different behaviors:

  1. The file still has "PARENT file content" - the restore from trashbin did nothing, the original text is still in the file or
  2. The file has the expected restored content "file to delete" and the over-written file is in the trashbin.

They each happen about 50% of the time.

Behavior (2) seems the most reasonable one.

https://drone.owncloud.com/owncloud/core/30743/4/13
The scenario is now written to expect behavior (2). That is passing abut 50% of the time.
And 50% of the time the content of the file has not been restored - the original content is still there.

@ownclouders
Copy link
Contributor

💥 Acceptance tests apiTrashbin-mariadb10.2-php7.4 failed. The build is cancelled...

1 similar comment
@ownclouders
Copy link
Contributor

💥 Acceptance tests apiTrashbin-mariadb10.2-php7.4 failed. The build is cancelled...

@phil-davis
Copy link
Contributor Author

phil-davis commented Jun 19, 2021

https://drone.owncloud.com/owncloud/core/30744/4/13
That is another variation of behavior (2) - this time it expects both of:
a) the previous content is in the trashbin
b) the restored content is in the file

(a) is maybe not a requirement - we do not normally put an old "version" in the trashbin when it is overwritten by new content. We would usually save it in "versions"

@ownclouders
Copy link
Contributor

💥 Acceptance tests apiTrashbin-mariadb10.2-php7.4 failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests apiTrashbin-mariadb10.2-php7.4 failed. The build is cancelled...

@phil-davis
Copy link
Contributor Author

https://drone.owncloud.com/owncloud/core/30747/4/13
I put extra sleep(5) before the checks in Then steps and still the same result - about half the time the updated content gets restored, and half the time not. So it does not seem to be a test timing issue.

@phil-davis
Copy link
Contributor Author

This demonstrates that the issue is still happening.

@phil-davis phil-davis closed this Sep 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants