The tests for the plotting functions are still only on SNP frequencies. Is that good enough?

@alimanfoo, I might have found a solution that works for what I wanted to do (namely, import shared testing functions for plotting frequencies into the various test_*_frq.pys) by telling pytest to ignore them when it reaches their file (it means that the tests now return the number of "passed", "skipped", "failed" tests (as well as warnings). I checked that when the tests are actually run when called in the relevant file, they would show a failure if they failed (i.e., they are not skipped when explicitly called).

@alimanfoo, I might have found a solution that works for what I wanted to do (namely, import shared testing functions for plotting frequencies into the various test_*_frq.pys) by telling pytest to ignore them when it reaches their file (it means that the tests now return the number of "passed", "skipped", "failed" tests (as well as warnings). I checked that when the tests are actually run when called in the relevant file, they would show a failure if they failed (i.e., they are not skipped when explicitly called).

Thanks @jonbrenas. As suggested in the code review above, it might be even better to name these utility functions like "check_..." instead of "test_..." then no need for skipping.

If the importing of functions between test modules seems to work then I guess we're OK! I still don't understand the mechanics of how pytest imports test modules. But if you've confirmed it works then no objection.

Thanks @alimanfoo. This all makes a lot of sense.

Codecov Report

Attention: Patch coverage is 97.00000% with 6 lines in your changes missing coverage. Please review.

Project coverage is 94.96%. Comparing base (cc1ce67) to head (8c0a432).
Report is 123 commits behind head on master.

@@            Coverage Diff             @@
##           master     #690      +/-   ##
==========================================
+ Coverage   94.57%   94.96%   +0.39%     
==========================================
  Files          40       45       +5     
  Lines        4187     4612     +425     
==========================================
+ Hits         3960     4380     +420     
- Misses        227      232       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.