frisciliasultan / pe

0 stars 0 forks source link

List should throw error if moduleBook is empty #5

Open frisciliasultan opened 3 years ago

frisciliasultan commented 3 years ago

Steps to Reproduce: clear then list

Expected: Error message saying book is empty

Actual: Success message

Screenshots:

Screenshot 2021-04-16 at 3.01.47 PM.png

nus-pe-bot commented 3 years ago

Team's Response

Thank you for pointing this out. We believe that no error message should be thrown as the message is trivially true as there are no tasks. Hence, saying "listed all tasks" is not wrong.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: There are some instances that users could forget that they have already cleared the modulebook and hence are still under the impression that their modulebook is still filled, if for example they have not used the modulebook for a long time. Therefore, user would expect to see some tasks when they type list, and the success message validates it, giving the impression that there are tasks that have been listed, but the UI does not show anything. Therefore, this may cause confusion to the user as they would think that the app does not function as expected. Therefore, it is important to indicate to the user that there is no task available for display. However, it is indeed a rare occasion and it does not affect usage very much as user can still use the commands, and therefore it should be VeryLow.