search

AY2324S1-CS2103T-T08-1 / tp

MIT License
0 stars 5 forks source link

Refactor data classes #127

Closed derekjxtan closed 11 months ago

derekjxtan commented 11 months ago

Closes #126

  1. Added a Person class to tie functionality of Patient and Doctor together
  2. Added a Data class which has Remarks and Tags
codecov[bot] commented 11 months ago

Codecov Report

Merging #127 (afcfbf9) into master (32f0476) will increase coverage by 0.24%. Report is 12 commits behind head on master. The diff coverage is 92.85%.

@@             Coverage Diff              @@
##             master     #127      +/-   ##
============================================
+ Coverage     79.18%   79.42%   +0.24%     
- Complexity      649      668      +19     
============================================
  Files            98      103       +5     
  Lines          1931     1983      +52     
  Branches        197      199       +2     
============================================
+ Hits           1529     1575      +46     
- Misses          370      374       +4     
- Partials         32       34       +2
Files Coverage Δ
src/main/java/seedu/address/model/Data.java 100.00% <100.00%> (ø)
src/main/java/seedu/address/model/Database.java 98.48% <ø> (ø)
.../main/java/seedu/address/model/UniqueItemList.java 96.22% <ø> (ø)
...a/seedu/address/model/appointment/Appointment.java 97.67% <ø> (ø)
...c/main/java/seedu/address/model/person/Person.java 100.00% <100.00%> (ø)
...java/seedu/address/model/person/doctor/Doctor.java 88.88% <100.00%> (-11.12%) :arrow_down:
...va/seedu/address/model/person/patient/Patient.java 100.00% <100.00%> (ø)
...c/main/java/seedu/address/model/remark/Remark.java 71.42% <71.42%> (ø)

... and 3 files with indirect coverage changes

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