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] Add more test coverage for listing files with PROPFIND #38981

Closed
dpakach opened this issue Jul 13, 2021 · 1 comment · Fixed by #38995
Closed

[Tests-only] Add more test coverage for listing files with PROPFIND #38981

dpakach opened this issue Jul 13, 2021 · 1 comment · Fixed by #38995
Assignees
Labels

Comments

@dpakach
Copy link
Contributor

dpakach commented Jul 13, 2021

We mostly have tests that test the existence of each files which sends request to the dav endpoint of each file/folder. But we dont have tests that try to get the entire file list with webdav.

We need to add tests to get the list of files from webdav

  • Test getting files list on user files, public link, trashbin
  • Use Depth Header to get the nested files/folders
@dpakach dpakach self-assigned this Jul 13, 2021
@phil-davis
Copy link
Contributor

Note: the lack of API test coverage of PROPFIND was noticed in owncloud/web tests, where we happened to use this kind of PROPFIND API in webdavHelper.js owncloud/web#5511

I would have been nicer if the regression in reva-OCIS was noticed by core API tests. So adding coverage here is good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants