-
Notifications
You must be signed in to change notification settings - Fork 155
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] add test download previos version #6038
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
Results for oCISFiles1 https://drone.owncloud.com/owncloud/web/20384/56/1 |
Results for oC10Files1 https://drone.owncloud.com/owncloud/web/20386/17/1
|
Results for oCISSharingInternalUsers1 https://drone.owncloud.com/owncloud/web/20393/59/1
|
Results for oC10SharingAccept https://drone.owncloud.com/owncloud/web/20393/16/1
|
Results for oCISFiles1 https://drone.owncloud.com/owncloud/web/20393/56/1 |
Results for oC10Files1 https://drone.owncloud.com/owncloud/web/20421/17/1 |
Results for oCISTrashbinUploadMoveJourney https://drone.owncloud.com/owncloud/web/20426/67/1
|
Results for oCISFiles1 https://drone.owncloud.com/owncloud/web/20426/56/1 |
6b68047
to
a6976ef
Compare
Results for oC10Files1 https://drone.owncloud.com/owncloud/web/20427/17/1 |
@ScharfViktor could you investigate why the tests failed here? :) |
Strange things happen here) |
I think if you search issues in oCIS/web you should find mentions for this. @individual-it and @phil-davis also should be able to provide more insights on this topic since it for sure already popped up in CI |
versions tests have often issues because in oC10 a version is only saved once per second. If a file is updated multiple times within one second it will only generate one version. |
the solution will be improved soon to be even more stable, see #6081 and then it should be ported over to the middleware |
@individual-it @ScharfViktor what's the current state here? |
a6976ef
to
5edce9b
Compare
@ScharfViktor could you soft reset your changes |
Yeah sure, I'll do it. I just have to wait until the CI is green. |
Results for oCISFiles2 https://drone.owncloud.com/owncloud/web/20985/56/1 |
That doesn't make too much sense since it would need to run once again after you did it anyways, no? |
5edce9b
to
dd075a6
Compare
The test steps
|
Results for oC10SharingAccept https://drone.owncloud.com/owncloud/web/20990/15/1
|
Results for oC10Files2 https://drone.owncloud.com/owncloud/web/20990/17/1 |
Results for oC10Files2 https://drone.owncloud.com/owncloud/web/21013/17/1 |
Results for oCISFiles2 https://drone.owncloud.com/owncloud/web/21071/56/1 |
0f3bb33
to
65775df
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fine for me, not sure if we need to sort out expected failures after this has been merged
A test for existing error 2261 is written
And added to expected-failures