search

owncloud / ocis

:atom_symbol: ownCloud Infinite Scale Stack
https://doc.owncloud.com/ocis/next/
Apache License 2.0
1.4k stars 183 forks source link

[QA] API test for `REPORT` request to webdav endpoint #9991

Closed PrajwolAmatya closed 1 month ago

PrajwolAmatya commented 2 months ago

Description

It seems there might not be full test coverage for REPORT request for endpoint /remote.php/dav/spaces/ with and without spaceUUID,

Both seems to behave the same way.

TODO:

Related bug reports:

prashant-gurung899 commented 2 months ago

@saw-jan @PrajwolAmatya

With remote.php

As per my research in OCIS-tests, i have found that tests cases for endpoints with remote.php are covered for all dav-paths in tests/acceptance/features/apiSearch1/search.feature:

I haven't found test coverage for endpoint without {space-id} i.e https://localhost:9200/remote.php/dav/spaces

Without remote.php

For the following endpoints without remote.php :

I think it should be discussed on how to optimally implement these cases

saw-jan commented 1 month ago
  • [ ] Check for the related test coverages
  • [ ] Add test coverages for these endpoint

Tests now use these paths for REPORT request: (#10044) https://localhost:9200/remote.php/webdav https://localhost:9200/remote.php/dav/files https://localhost:9200/remote.php/dav/spaces

And as per https://github.com/owncloud/ocis/issues/7104#issuecomment-1711310103, we really don't need to test with space-id in the request path.

  • [ ] Check with and without remote.php in dav-path

This will be done in another PR with different implementation.

And I want to move bug reports coverage in ticket https://github.com/owncloud/ocis/issues/9948

So maybe we can close this one? CC @prashant-gurung899 @amrita-shrestha @Salipa-Gurung @PrajwolAmatya

saw-jan commented 1 month ago

I am closing this here.