[JOSS review] Test coverage

delve-team / delve

PyTorch model training and layer saturation monitor

https://delve-docs.readthedocs.io

MIT License

79 stars 13 forks source link

[JOSS review] Test coverage #57

Closed dionhaefner closed 2 years ago

dionhaefner commented 2 years ago

I suggest adding a service like codecov to see how much is actually covered by tests, and adding a badge to the README. There's no shame in not reaching 100% coverage, but if you don't measure it you won't know whether your tests work as intended.

(This is a part of the ongoing review at openjournals/joss-reviews#3992)

MLRichter commented 2 years ago

Currently, this is implemented in this PR alongside some additional tests to raise the overall coverage: https://github.com/delve-team/delve/pull/63