search

tasoc / photometry

The TASOC Photometry Pipeline
https://tasoc.dk
GNU General Public License v3.0
18 stars 4 forks source link

Tests of command-line interfaces #23

Closed rhandberg closed 4 years ago

rhandberg commented 4 years ago

CLIs should be tested as well as the underlying functionality.

codecov[bot] commented 4 years ago

Codecov Report

Merging #23 into devel will increase coverage by 1.70%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##            devel      #23      +/-   ##
==========================================
+ Coverage   81.21%   82.91%   +1.70%     
==========================================
  Files          27       32       +5     
  Lines        3678     4085     +407     
==========================================
+ Hits         2987     3387     +400     
- Misses        691      698       +7
Impacted Files Coverage Δ
photometry/catalog.py 79.62% <ø> (ø)
run_tessphot_mpi.py 93.54% <ø> (ø)
make_todo.py 100.00% <0.00%> (ø)
run_prepare_photometry.py 90.32% <0.00%> (ø)
run_ffimovie.py 93.03% <0.00%> (ø)
run_tessphot.py 87.01% <0.00%> (ø)
photometry/BasePhotometry.py 94.48% <0.00%> (+0.12%) :arrow_up:
photometry/todolist.py 85.61% <0.00%> (+0.35%) :arrow_up:
photometry/image_motion.py 95.12% <0.00%> (+0.60%) :arrow_up:
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f43e9dd...f513ee8. Read the comment docs.

rhandberg commented 4 years ago

Now in a state where it can be merged. There are still some CLIs that are not being tested, but this is either because they can not be tested or not in a state where they should be tested yet.