-
Notifications
You must be signed in to change notification settings - Fork 5
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 noorul json patient edit #84
Add noorul json patient edit #84
Conversation
* 'master' of https://github.com/AY2021S1-CS2103T-W12-1/tp: Add ListRoomCommand and UI for both ListRoom and FindRoomCommand Add ListRoomCommand and UI for both ListRoom and FindRoomCommand Add ListRoomCommand and UI for both ListRoom and FindRoomCommand
… recognised previously but now has to.
Codecov Report
@@ Coverage Diff @@
## master #84 +/- ##
============================================
+ Coverage 67.07% 67.90% +0.82%
- Complexity 581 636 +55
============================================
Files 102 110 +8
Lines 2017 2203 +186
Branches 236 274 +38
============================================
+ Hits 1353 1496 +143
- Misses 588 617 +29
- Partials 76 90 +14
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
…ed not be recognised previously but now has
…ed not be recognised previously but now has to be Make changes to fix bugs. Pass checkstyle as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice Job! LGTM!
return roomNumber == room.roomNumber | ||
&& isOccupied == room.isOccupied | ||
&& taskList.equals(room.getTaskList()); | ||
if (taskList == null && patient == null) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice job to separate to more cases!
@@ -25,7 +25,9 @@ public void addTask_nullTask_throwsNullPointerException() { | |||
public void addTask_validTask_success() { | |||
Room room = new RoomBuilder(ROOM_PATIENT_ALICE_NO_TASK).build(); | |||
room.addTask(REMIND_PATIENT); | |||
|
|||
System.out.println(room.getTaskList().getInternalList().size()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remember to remove the system.out.print after doing the "actual" testing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, thanks for noticing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some minor changes to be made. good job and effort!
Please review my code for the part to includes reading and saving the Patient details in a class.