AY2425S1-CS2103T-F12-3 / tp

MIT License
3 stars 5 forks source link

Update and re-enabled some test-cases that were previously disabled #77

Closed yong-zhuo closed 1 day ago

yong-zhuo commented 1 day ago

Updated and fix the following test cases due to the new parameters for Person:

Also updated some classes to include logging and fix code quality issues

codecov[bot] commented 1 day ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Files with missing lines Coverage Δ Complexity Δ
...main/java/seedu/address/commons/util/DateUtil.java 90.00% <100.00%> (+1.11%) 4.00 <2.00> (+1.00)
.../java/seedu/address/logic/commands/AddCommand.java 100.00% <100.00%> (ø) 8.00 <0.00> (ø)
...a/seedu/address/logic/parser/AddCommandParser.java 100.00% <100.00%> (+5.55%) 6.00 <1.00> (+2.00)
...n/java/seedu/address/model/person/DateOfBirth.java 93.33% <ø> (ø) 7.00 <0.00> (ø)
src/main/java/seedu/address/model/person/Nric.java 92.85% <100.00%> (-0.48%) 6.00 <0.00> (ø)

... and 7 files with indirect coverage changes