AY2122S1-CS2103T-T09-1 / tp

https://ay2122s1-cs2103t-t09-1.github.io/tp/
MIT License
4 stars 5 forks source link

Improve test coverage in Logic #230

Closed mslevis closed 2 years ago

codecov-commenter commented 2 years ago

Codecov Report

Merging #230 (1600a3c) into master (7a57a63) will increase coverage by 2.76%. The diff coverage is 25.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #230      +/-   ##
============================================
+ Coverage     64.98%   67.74%   +2.76%     
- Complexity      717      742      +25     
============================================
  Files            94       94              
  Lines          2590     2586       -4     
  Branches        348      348              
============================================
+ Hits           1683     1752      +69     
+ Misses          818      768      -50     
+ Partials         89       66      -23     
Impacted Files Coverage Δ
...in/java/seedu/address/logic/parser/ParserUtil.java 96.77% <0.00%> (+18.64%) :arrow_up:
...va/seedu/address/model/studyspot/StudiedHours.java 85.71% <0.00%> (-4.29%) :arrow_down:
.../seedu/address/logic/parser/ListCommandParser.java 95.83% <100.00%> (+30.44%) :arrow_up:
...main/java/seedu/address/model/amenity/Amenity.java 95.23% <0.00%> (+4.76%) :arrow_up:
src/main/java/seedu/address/model/tag/Tag.java 100.00% <0.00%> (+8.33%) :arrow_up:
.../address/storage/JsonSerializableStudyTracker.java 100.00% <0.00%> (+11.11%) :arrow_up:
...java/seedu/address/logic/commands/EditCommand.java 86.69% <0.00%> (+13.30%) :arrow_up:
...va/seedu/address/logic/commands/DeleteCommand.java 100.00% <0.00%> (+25.00%) :arrow_up:
... and 1 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 7a57a63...1600a3c. Read the comment docs.