search

maxxyx96 / pe

0 stars 0 forks source link

Command /delete with whitespaces not handled properly. #2

Open maxxyx96 opened 5 years ago

maxxyx96 commented 5 years ago

Command /delete done [whitespaces] leads to a long bug report

image.png

nus-pe-bot commented 4 years ago

Team's Response

The use of whitespaces results in an exception being raised. But the use of trailing or leading whitespaces seems rare in real world use although a valid flaw.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.Low]. Originally [severity.Medium].

Reason for disagreement: It should be medium as user wouldn't want to see all the message. Instead it should contain the exception like "Please do not use spacing, please follow the command exactly without spacing" etc. The exception message should not overwhelm the user.

[Update: This bug was potentially the reason why i had a lot of subsequent bugs reported with Hustler. I believe this is because the exception was not properly handled.]

:question: Issue type

Team chose [type.FunctionalityBug]. Originally [type.FeatureFlaw].

Reason for disagreement: [replace this with your reason]