nus-cs2103-AY2021S1 / pe-dev-response

0 stars 0 forks source link

Wrong Error message output for addAttendance #4968

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

It shows me details of adding participation when attempting to add attendance with invalid attendance week.

Steps to replicate: 1) Add a module 2) Add a TG 3) Add a Student 4) Attempt to add attendance for a student with an invalid week number, e.g. addAttendance 1 week/2 week/20 5) The following error message is shown:

image.png


[original: nus-cs2103-AY2021S1/pe-interim#4835]

euzintan commented 3 years ago

Team's Response

Thanks for your feedback. This error message is from our attendance class which covers both attendance and participation scores.

The error message does serve its purpose in pointing out that "Attendance covers week 1 to 13" which is what is wrong with the tester's input.

Duplicate status (if any):

--