zxisatree / pe

0 stars 0 forks source link

Add command with valid address gives error message #8

Open zxisatree opened 1 year ago

zxisatree commented 1 year ago

Running the command add n/e2 p/91234567 d/Sales pr/4000 3 a/1/2 l/Downing Street displays an error, even though the address is valid by both the UG and the application's requirements. This is a concern as certain addresses contain slashes.

image.png

nus-se-script commented 1 year ago

Team's Response

Firstly, slashes are generally accepted in the input, so any reasonable input like "Block 123 #01/12" would work, as would "Block 123 m/test" or "Block 123l/test". Secondly, the tester's input is not something that any user would want to enter, so we are rejecting this bug report.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I'm not really sure what the team means by "reasonable input". How do you define a "reasonable input"? Why is the tester's input in particular "not something that any user would want to enter"? The response states that "Block 123l/test" is a "reasonable input". Is "reasonable input" defined to be "only the inputs that our application is able to accept at the point in testing"? I am not sure why a user would want to enter "Block 123l/test" but not "Block 123 l/test".


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]