ernestlim8 / ped Goto Github PK

Message should reflect that a floating point value should not be used instead.

worker-add n/John hp/98765432 a/21 Lower Kent Ridge Rd, Singapore 119077 r/Cashier p/0 is valid where it should throw an error.

Small bug in error message, should be Adds a worker to McScheduler instead.
Similar errors are observed in other error messages.

2 unique workers should not have the same phone number

Command result message is difficult to read because of small size, requires user to scroll.

Perhaps if the time is not given, we can assume that the worker is not free for trhe whole day (set to full instead)

No formatting for pay, it is not rounded off to 2decimal places, and there is no dollar sign. It is unclear on what this value is referring to without reading the documentation.

Worker 10 is not aligned.

Wrong error message when assigning another role for the same slot. Should be john is on leave

Incorrect display of 2 of the same roles in the same shift. Instead, consider throwing an duplicate error message.

Janitor in the test cases are not predefined, consider renaming to Cleaner instead.

No details provided.

it is not stated in the UG what the number of hours for AM and PM are, making it difficult to test the values.

Error in UG, w/WORKER_INDEX should be compulsory and not be encased in square brackets

Command is long and difficult to execute. Similarly observed for other commands that take in too many inputs.

worker-delete has an incorrectly formatted error message.

There should be a limit on the hourly pay.