search

kaitinghh / pe

0 stars 0 forks source link

Clear command prerequisite different from app behaviour #15

Open kaitinghh opened 7 months ago

kaitinghh commented 7 months ago

image.png

The pre-requisite is not reflective of the behaviour on the app. When I try to clear the bookkeeper when it is empty, the command still succeeds with output "BookKeeper has been cleared!"

nus-pe-script commented 7 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

Realistically, if the application was cleared without any entries in it, the Ui and functionalities would remain unchanged, which is confusing for the user. Hence, as a prerequisite, we wanted to specify that only if you have at least one client or order, then the clear function would work as expected.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your reason]