Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add test for hairdresser and print #278

Merged
merged 12 commits into from
Nov 7, 2020

Conversation

zyifanz
Copy link

@zyifanz zyifanz commented Nov 7, 2020

Close #273

@zyifanz zyifanz added this to the v1.4 milestone Nov 7, 2020
@zyifanz zyifanz self-assigned this Nov 7, 2020
@codecov-io
Copy link

Codecov Report

Merging #278 (22f78d2) into master (8d787c8) will increase coverage by 10.31%.
The diff coverage is 76.92%.

Impacted file tree graph

@@              Coverage Diff              @@
##             master     #278       +/-   ##
=============================================
+ Coverage     37.73%   48.05%   +10.31%     
- Complexity      493      606      +113     
=============================================
  Files           149      149               
  Lines          2862     2878       +16     
  Branches        323      326        +3     
=============================================
+ Hits           1080     1383      +303     
+ Misses         1715     1402      -313     
- Partials         67       93       +26     
Impacted Files Coverage Δ Complexity Δ
src/main/java/seedu/address/model/IdCounter.java 86.66% <55.55%> (-13.34%) 15.00 <3.00> (+3.00) ⬇️
src/main/java/seedu/address/model/UserPrefs.java 80.55% <83.33%> (+3.88%) 11.00 <3.00> (+2.00)
...ava/seedu/address/logic/commands/PrintCommand.java 83.17% <100.00%> (+83.17%) 21.00 <3.00> (+21.00)
...c/main/java/seedu/address/model/person/Person.java 70.27% <0.00%> (+2.70%) 12.00% <0.00%> (+1.00%)
...ain/java/seedu/address/model/UniqueEntityList.java 90.69% <0.00%> (+4.65%) 20.00% <0.00%> (+1.00%)
...u/address/model/specialisation/Specialisation.java 53.84% <0.00%> (+7.69%) 12.00% <0.00%> (+3.00%)
...c/main/java/seedu/address/model/person/Gender.java 72.72% <0.00%> (+9.09%) 9.00% <0.00%> (+2.00%)
...dress/model/appointment/UniqueAppointmentList.java 16.27% <0.00%> (+9.30%) 4.00% <0.00%> (+2.00%)
src/main/java/seedu/address/model/tag/Tag.java 80.00% <0.00%> (+10.00%) 4.00% <0.00%> (+1.00%)
...ava/seedu/address/model/person/client/Address.java 70.00% <0.00%> (+10.00%) 5.00% <0.00%> (+1.00%)
... and 21 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 8d787c8...22f78d2. Read the comment docs.

@iamjamestan iamjamestan merged commit 6ebf42a into AY2021S1-CS2103T-T15-1:master Nov 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add test for Hairdresser and Print
3 participants