search

bluesky / suitcase-tiff

http://nsls-ii.github.io/suitcase
Other
3 stars 6 forks source link

FIX: in testing dont rely on the loaded file to decide how many images it should contain #10

Closed awalter-bnl closed 5 years ago

awalter-bnl commented 5 years ago

we found an fault in this that was not caught by the tests. Essentially the tests did not check how many images where saved into each file, this update fixes this and checks the input collector file for how many images should be contained in each file and compares this to the loaded files.

codecov-io commented 5 years ago

Codecov Report

Merging #10 into master will increase coverage by 1.37%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
+ Coverage   95.29%   96.66%   +1.37%     
==========================================
  Files           3        3              
  Lines          85      120      +35     
  Branches       14       26      +12     
==========================================
+ Hits           81      116      +35     
  Misses          2        2              
  Partials        2        2
Impacted Files Coverage Δ
suitcase/tiff/tests.py 100% <100%> (ø) :arrow_up:

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 7a5f84c...b99fb18. Read the comment docs.