search

Garden-AI / garden

https://garden-ai.readthedocs.io
MIT License
21 stars 4 forks source link

Add small-scale tests of core modules #530

Closed hholb closed 3 months ago

hholb commented 3 months ago

Related: #507

Overview

This PR add small-scale test coverage of the the core modules that make garden work.

Discussion

This PR includes a sizeable refactor of the notebook_metadata.py module to make it easier to get better test coverage. I didn't change any of the underlying logic, mostly moved stuff around into helper functions.

The tests of the containers module are mostly from the old test suite. The containers module and notebook CLI commands are where the biggest gaps in coverage are after this PR. Integration tests that cover these modules will come in another PR.

Testing

Lots of new tests, plus some old ones!

📚 Documentation preview 📚: https://garden-ai--530.org.readthedocs.build/en/530/

codecov[bot] commented 3 months ago

Codecov Report

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

Project coverage is 83.70%. Comparing base (ff790ca) to head (b02b39d). Report is 16 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #530 +/- ## ========================================== + Coverage 75.46% 83.70% +8.23% ========================================== Files 48 53 +5 Lines 3298 3806 +508 ========================================== + Hits 2489 3186 +697 + Misses 809 620 -189 ```

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

🚨 Try these New Features: