search

AY2324S1-CS2103T-T08-1 / tp

MIT License
0 stars 5 forks source link

Fix Add Doctor Bugs #102

Closed kimshitong closed 11 months ago

kimshitong commented 11 months ago

Modified Delimiter

Overridden Hashcode

codecov[bot] commented 11 months ago

Codecov Report

Merging #102 (8a2ca12) into master (39f6eb5) will increase coverage by 1.40%. Report is 103 commits behind head on master. The diff coverage is 100.00%.

:exclamation: Current head 8a2ca12 differs from pull request most recent head bb461d0. Consider uploading reports for the commit bb461d0 to get more accurate results

@@             Coverage Diff              @@
##             master     #102      +/-   ##
============================================
+ Coverage     78.37%   79.78%   +1.40%     
- Complexity      584      707     +123     
============================================
  Files            86      104      +18     
  Lines          1716     2093     +377     
  Branches        170      214      +44     
============================================
+ Hits           1345     1670     +325     
- Misses          331      384      +53     
+ Partials         40       39       -1
Files Coverage Δ
...seedu/address/logic/commands/AddDoctorCommand.java 100.00% <100.00%> (ø)
...ain/java/seedu/address/logic/parser/CliSyntax.java 90.90% <100.00%> (+5.19%) :arrow_up:
...el/person/NameContainsKeywordsDoctorPredicate.java 100.00% <100.00%> (ø)

... and 35 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more