search

Cary-Xx / pe

0 stars 0 forks source link

Every invalid command in grade tracker will result in exception of the app #5

Open Cary-Xx opened 5 years ago

Cary-Xx commented 5 years ago

Every invalid command in grade tracker will result in an exception, may consider improve it.

Screenshot 2019-11-15 at 13.00.34.png

nus-pe-bot commented 4 years ago

Team's Response

Screenshot 2019-11-17 at 12.41.50 AM.png

Invalid commands should be well-behaved as shown in the picture above. All invalid commands that do not exist are catched. It's unclear how the tester reached that state and stating EVERY invalid command causes uncatched exception is false.

Items for the Tester to Verify

:question: Issue response

Team chose [response.IssueUnclear]

Reason for disagreement: Sorry for the "every invalid command" i stated, the issue appears in the grade tracker section. For example, for add command, if any of the field in not valid, exception will be thrown and it can be seen on the terminal. Not sure if other commands and other sections have same problem. Team may consider catch the exception.