datalad / datalad-fuse

DataLad extension to provide FUSE file system access
Other
1 stars 4 forks source link

Set required git configuration for initiated test repository #113

Closed TheChymera closed 4 months ago

TheChymera commented 4 months ago

Preventing issues like → https://github.com/datalad/datalad-fuse/issues/112

I really think it would be best to set these configs when initializing the new repo, since this is required by git and tests may run as a sandbox user.

yarikoptic commented 4 months ago

Thanks!

We have tmp_home fixture https://github.com/datalad/datalad-fuse/blob/HEAD/datalad_fuse/tests/conftest.py#L65 so just try using it (add to test signature) and it should then work for you.

Filed

so we can ensure this at CI level, and might provide a fix there unless you beat me to it here

codecov[bot] commented 4 months ago

Codecov Report

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

Project coverage is 91.05%. Comparing base (02200a1) to head (808d184).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #113 +/- ## ========================================== + Coverage 91.02% 91.05% +0.02% ========================================== Files 10 10 Lines 624 626 +2 ========================================== + Hits 568 570 +2 Misses 56 56 ```

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