Closed ejtovar closed 4 years ago
Merging #1227 into master will increase coverage by
0.07%
. The diff coverage is60.00%
.
@@ Coverage Diff @@
## master #1227 +/- ##
==========================================
+ Coverage 52.22% 52.29% +0.07%
==========================================
Files 531 531
Lines 109522 109521 -1
==========================================
+ Hits 57196 57274 +78
+ Misses 52326 52247 -79
Impacted Files | Coverage Δ | |
---|---|---|
proteus/tests/SWFlow/santos_step.py | 92.77% <0.00%> (+92.77%) |
:arrow_up: |
proteus/tests/SWFlow/solitary_wave.py | 93.75% <ø> (-0.16%) |
:arrow_down: |
proteus/tests/SWFlow/test_SWFlows.py | 100.00% <100.00%> (ø) |
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 27634d3...3a57337. Read the comment docs.
Mandatory Checklist
Please ensure that the following criteria are met:
As a general rule of thumb, try to follow PEP8 guidelines.
Description
Updating the SWFlow test suite to cover the
santos_step.py
test (along with updating the description in file). Also removed unnecessary lines insolitary_wave.py
New file
proteus/tests/SWFlow/comparison_files/comparison_santos_step_h_t2.csv -- 4kBs