zengzihui / pe

0 stars 0 forks source link

Inconsistent error message for attd command when involving multiple indexes (same issue as the att command) #5

Open zengzihui opened 4 months ago

zengzihui commented 4 months ago

In the attd command which is used to unmark the attendance, it has the same issue as the issue I raised for the att command. There are inconsistent error message each time.

image.png

image.png

nus-pe-script commented 4 months ago

Team's Response

Bug deemed as NotInScope as it is less important than the work that has been done already (data validation).

image.png

Data validation is indeed in place for this command, which causes the error messages to be thrown. While it is indeed inconsistent, it is not invalid (The attendance is indeed not marked for Alex Yeoh, and the attendance is indeed not marked for Bernice Yu). The error messages are factual.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]