AY2122S1-CS2103T-T09-1 / tp

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

Improve test coverage #221

Closed qreoct closed 2 years ago

qreoct commented 2 years ago

Improved test coverage to 100% in:

Part of #86

codecov-commenter commented 2 years ago

Codecov Report

Merging #221 (0a41b0a) into master (2ec0385) will increase coverage by 1.67%. The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #221      +/-   ##
============================================
+ Coverage     62.30%   63.97%   +1.67%     
- Complexity      663      690      +27     
============================================
  Files            94       94              
  Lines          2573     2579       +6     
  Branches        335      337       +2     
============================================
+ Hits           1603     1650      +47     
+ Misses          870      835      -35     
+ Partials        100       94       -6     
Impacted Files Coverage Δ
...ava/seedu/address/logic/commands/AliasCommand.java 96.15% <100.00%> (+0.32%) :arrow_up:
src/main/java/seedu/address/model/tag/Tag.java 91.66% <0.00%> (-8.34%) :arrow_down:
...in/java/seedu/address/logic/parser/ParserUtil.java 78.12% <0.00%> (-2.73%) :arrow_down:
.../seedu/address/model/studyspot/OperatingHours.java 90.24% <0.00%> (+2.43%) :arrow_up:
...u/address/model/studyspot/UniqueStudySpotList.java 97.56% <0.00%> (+2.43%) :arrow_up:
...rc/main/java/seedu/address/model/StudyTracker.java 91.07% <0.00%> (+5.35%) :arrow_up:
.../java/seedu/address/model/studyspot/StudySpot.java 98.71% <0.00%> (+6.41%) :arrow_up:
.../main/java/seedu/address/model/studyspot/Name.java 92.30% <0.00%> (+7.69%) :arrow_up:
...ain/java/seedu/address/model/studyspot/Rating.java 90.00% <0.00%> (+10.00%) :arrow_up:
...in/java/seedu/address/model/studyspot/Address.java 90.00% <0.00%> (+10.00%) :arrow_up:
... and 5 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 2ec0385...0a41b0a. Read the comment docs.

qreoct commented 2 years ago

Thank you for the review! I've made the changes