Closed Silvernitro closed 4 years ago
Merging #175 into master will increase coverage by
0.02%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #175 +/- ##
============================================
+ Coverage 75.40% 75.43% +0.02%
- Complexity 565 566 +1
============================================
Files 93 93
Lines 1720 1722 +2
Branches 175 176 +1
============================================
+ Hits 1297 1299 +2
Misses 341 341
Partials 82 82
Impacted Files | Coverage Δ | Complexity Δ | |
---|---|---|---|
...main/java/seedu/address/commons/core/Messages.java | 0.00% <ø> (ø) |
0.00 <0.00> (ø) |
|
...java/seedu/address/logic/commands/EditCommand.java | 95.38% <100.00%> (+0.14%) |
14.00 <0.00> (+1.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 af938e3...95b6e72. Read the comment docs.
Closes #152
Changes
EditCommand
, check if the edited module is the same as the original one. If it is, display an error message instead of continuing to "edit" itEditCommandTest
to retrieve modules from the test model, so I've abstracted it out to 2 fxns