SimNee / pe

0 stars 0 forks source link

Help command allows arguments #6

Open SimNee opened 4 years ago

SimNee commented 4 years ago

Screenshot 2019-11-15 at 4.39.27 PM.png

An exception should be thrown if user tried, help aaa.

nus-pe-bot commented 4 years ago

Team's Response

We believe there is no compelling reason for the application to throw an exception in such a scenario.

In terms of the back end, the internal application state is unaffected and the help command behaviour remains identical regardless of what the user supplies as "arguments" (since those are simply ignored anyway as they do not affect how the help command executes).

In terms of the user experience, we also think the fact that help behaves exactly the same way regardless of the supplied arguments should be quite obvious to the average user after a few tries.

Thus we believe that throwing an exception serves no useful purpose here. In addition, this behaviour is inherited from AB3.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]