search

audeering / audbackend

Manage file storage on different backends
https://audeering.github.io/audbackend/
Other
3 stars 0 forks source link

Fix audbackend.interface.Maven.ls() on sub-path #210

Closed hagenw closed 3 months ago

hagenw commented 3 months ago

Closes #207

This updates the tests of audbackend.interface.Maven and audbackend.interface.Versioned to include files in sub-paths that are named as the sub-path, namely /sub/sub.ext and /sub/sub/sub.ext. Those tests pass for audbackend.interface.Versioned, but failed for audbackend.interface.Maven due to #207.

audbackend.interface.Maven.ls() is then fixed, by removing results from the returned list of paths for a given sub-path, that contain files. This is neccessary compared to audbackend.interface.Versioned as audbackend.interface.Maven also stores files in sub-path, e.g. /file/1.0.0/file-1.0.0.ext.

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.0%. Comparing base (d317956) to head (4994857).

Additional details and impacted files | [Files](https://app.codecov.io/gh/audeering/audbackend/pull/210?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=audeering) | Coverage Δ | | |---|---|---| | [audbackend/core/interface/maven.py](https://app.codecov.io/gh/audeering/audbackend/pull/210?src=pr&el=tree&filepath=audbackend%2Fcore%2Finterface%2Fmaven.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=audeering#diff-YXVkYmFja2VuZC9jb3JlL2ludGVyZmFjZS9tYXZlbi5weQ==) | `100.0% <100.0%> (ø)` | |