Arnaxx54 / pe

0 stars 0 forks source link

`List` command returns incorrect message #17

Open Arnaxx54 opened 4 days ago

Arnaxx54 commented 4 days ago

image.png

The list command returns a success message even when there are no persons to list.

Expected: Return an error message saying there are no persons to display.

nus-pe-script commented 21 hours ago

Team's Response

Thank you for the bug report. However, this is expected for a command to only deliver what you see is what you get. Empty list is a valid output. When we run SQL select statement on an empty table, does the DB server return an error?

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Thank you for your feedback. I would like to clarify that the issue lies within the user feedback message and not the output of an empty list. If a list of persons does not exist, the feedback message 'Listed all persons' is inaccurate and confusing. Additionally, the user guide does not acknowledge this situation.

To draw parallels with the find command, this error would be similar to returning 'customers listed!' when there are no potential matches.

The comparison with a DB server is invalid in this situation since they are completely different applications.