Add test coverage for commits to repo information tests

aai-institute / lakefs-spec

An fsspec implementation for the lakeFS project.

http://lakefs-spec.org/

Apache License 2.0

37 stars 4 forks source link

Add test coverage for commits to repo information tests #244

Closed nicholasjng closed 6 months ago

nicholasjng commented 6 months ago

Informative file system APIs like fs.info(), fs.ls() and fs.exists() should work on branches as well as commits, since they carry a meaning for commits as well, as opposed to state-changing APIs like fs.rm().

For this reason, check that these APIs perform the same as for a branch input by checking them against a branch and the branch@HEAD commit subsequently, comparing the results. Result equality is expected.

Closes #242.

codecov[bot] commented 6 months ago

Codecov Report

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

Comparison is base (03dffc7) 93.39% compared to head (814bcb1) 93.39%.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #244 +/- ## ======================================= Coverage 93.39% 93.39% ======================================= Files 5 5 Lines 409 409 Branches 70 70 ======================================= Hits 382 382 Misses 18 18 Partials 9 9 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.