DeepRegNet / DeepReg

Medical image registration using deep learning
Apache License 2.0
564 stars 76 forks source link

379 identify all code not covered by unit test #639

Closed mathpluscode closed 3 years ago

mathpluscode commented 3 years ago

Description

Cover or ignore some tests, so that test coverage is 100%.

Fixes #379

Type of change

What types of changes does your code introduce to DeepReg?

Please check the boxes that apply after submitting the pull request.

Checklist

Please check the boxes that apply after submitting the pull request.

If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

mathpluscode commented 3 years ago

Another quick MR to improve test coverage to 100% @YipengHu @kate-sann5100

codecov[bot] commented 3 years ago

Codecov Report

Merging #639 (20e7a26) into main (4bfd1d8) will increase coverage by 0.32%. The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##             main      #639      +/-   ##
===========================================
+ Coverage   99.67%   100.00%   +0.32%     
===========================================
  Files          33        33              
  Lines        2175      2173       -2     
===========================================
+ Hits         2168      2173       +5     
+ Misses          7         0       -7     
Impacted Files Coverage Δ
deepreg/dataset/loader/grouped_loader.py 100.00% <ø> (+0.87%) :arrow_up:
deepreg/dataset/loader/h5_loader.py 100.00% <ø> (+1.47%) :arrow_up:
deepreg/dataset/loader/nifti_loader.py 100.00% <ø> (+1.47%) :arrow_up:
deepreg/predict.py 100.00% <ø> (+1.06%) :arrow_up:
deepreg/warp.py 100.00% <100.00%> (+5.40%) :arrow_up:
deepreg/model/layer.py 100.00% <0.00%> (+0.41%) :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 4bfd1d8...20e7a26. Read the comment docs.

scrutinizer-notifier commented 3 years ago

The inspection completed: 13 new issues, 7 updated code elements