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] Run litmus CI for both owncloud and ocis storage #1166

Merged
merged 1 commit into from
Sep 17, 2020
Merged

[Tests-Only] Run litmus CI for both owncloud and ocis storage #1166

merged 1 commit into from
Sep 17, 2020

Conversation

phil-davis
Copy link
Contributor

@phil-davis phil-davis commented Sep 14, 2020

Test both ocis and owncloud storage with litmus.

(after this is merged, I will then work on running the API tests with ocis storage as well as owncloud storage)

@phil-davis phil-davis self-assigned this Sep 14, 2020
@phil-davis
Copy link
Contributor Author

phil-davis commented Sep 14, 2020

PR #1155 introduced ocis storage.

@butonic this PR runs litmus tests on both owncloud and ocis storage. owncloud passes (like it already did). ocis storage fails:

old webdav:
https://cloud.drone.io/cs3org/reva/2525/3/5

...
10. move_coll............. 
10. move_coll............. FAIL (collection COPY `/remote.php/webdav/litmus/mvdest/' to `/remote.php/webdav/litmus/mvdest2/', depth infinity: 500 Internal Server Error)

11. move_cleanup.......... 
11. move_cleanup.......... pass

12. finish................ 
12. finish................ pass
<- summary for `copymove': of 13 tests run: 12 passed, 1 failed. 92.3%

new webdav:
https://cloud.drone.io/cs3org/reva/2525/5/5

...
10. move_coll............. 
10. move_coll............. FAIL (collection COPY `/remote.php/dav/files/4c510ada-c86b-4815-8820-42cdf82c3d51/litmus/mvdest/' to `/remote.php/dav/files/4c510ada-c86b-4815-8820-42cdf82c3d51/litmus/mvdest2/', depth infinity: 500 Internal Server Error)

11. move_cleanup.......... 
11. move_cleanup.......... pass

12. finish................ 
12. finish................ pass
<- summary for `copymove': of 13 tests run: 12 passed, 1 failed. 92.3%

Is there a known issue with copying/moving a collection/folder?

@phil-davis
Copy link
Contributor Author

I cherry-picked from PR #1177 into here - to demonstrate that litmus now passes for ocis storage.

@phil-davis
Copy link
Contributor Author

phil-davis commented Sep 17, 2020

Now litmus fails in a different way: https://cloud.drone.io/cs3org/reva/2566/3/5 (ocis old webdav)

13. propreplace........... 
13. propreplace........... FAIL (PROPPATCH on `/remote.php/webdav/litmus/prop2': 500 Internal Server Error)

14. propget............... 
14. propget............... FAIL (Property {http://example.com/neon/litmus/}prop9 had value value9, expected newvalue9)

15. propnullns............ 
15. propnullns............ pass

16. propget............... 
16. propget............... pass

17. propremoveset......... 
17. propremoveset......... FAIL (PROPPATCH remove then set)

18. propget............... 
18. propget............... FAIL (No value given for property {http://example.com/neon/litmus/}removeset)

And https://cloud.drone.io/cs3org/reva/2566/5/5 has the same fails for ocis new webdav.

This is after PR #1177 "fix ocis move" was merged.

I raised issue #1178

This was referenced Sep 17, 2020
@phil-davis
Copy link
Contributor Author

PR #1179 fixed the fails. I rebased, so this should pass now.

@phil-davis phil-davis marked this pull request as ready for review September 17, 2020 11:54
@phil-davis
Copy link
Contributor Author

@ishank011 CI should pass here now - please review.

@ishank011 ishank011 merged commit fd2b257 into cs3org:master Sep 17, 2020
@phil-davis phil-davis deleted the dual-storage-litmus-pipelines branch September 17, 2020 15:57
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