search

AY2122S1-CS2103T-T09-1 / tp

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

Improve coverage for ParserUtil #236

Closed limdanqi closed 2 years ago

limdanqi commented 2 years ago

Resolves #237

codecov-commenter commented 2 years ago

Codecov Report

Merging #236 (d91801a) into master (39ade2d) will increase coverage by 0.44%. The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #236      +/-   ##
============================================
+ Coverage     67.74%   68.19%   +0.44%     
- Complexity      742      745       +3     
============================================
  Files            94       94              
  Lines          2586     2581       -5     
  Branches        348      346       -2     
============================================
+ Hits           1752     1760       +8     
+ Misses          768      763       -5     
+ Partials         66       58       -8
Impacted Files Coverage Δ
...in/java/seedu/address/logic/parser/ParserUtil.java 96.77% <100.00%> (ø)
.../java/seedu/address/logic/commands/LogCommand.java 97.33% <0.00%> (+1.33%) :arrow_up:
...java/seedu/address/logic/commands/EditCommand.java 88.66% <0.00%> (+1.97%) :arrow_up:
...main/java/seedu/address/model/amenity/Amenity.java 100.00% <0.00%> (+4.76%) :arrow_up:
...seedu/address/logic/parser/StudyTrackerParser.java 100.00% <0.00%> (+5.71%) :arrow_up:
.../seedu/address/logic/parser/EditCommandParser.java 97.36% <0.00%> (+7.36%) :arrow_up:
...eedu/address/logic/parser/DeleteCommandParser.java 87.50% <0.00%> (+14.77%) :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 39ade2d...d91801a. Read the comment docs.