search

V4Vern / pe

0 stars 0 forks source link

Lack of validation for quote command #4

Open V4Vern opened 4 weeks ago

V4Vern commented 4 weeks ago

Description:

Inproper validation for quote command (minor issue)

Steps to Reproduce:

quote //s.2 quote 24342 quote 2w1

Expected Result: It should print out an error messsage

Actual Result: It prints out the quote message

image.png

nus-pe-script commented 3 weeks ago

Team's Response

I think this shouldn't be an issue, we have specified that such parameters would be ignored in our user guide.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I feel that it should be not-in-scope because it's already specified in the UG. However, the error messages are not very consistent between different commands. For example, "suggest" does not take in parameters but it does print out error messages as show. However, "quote" command" does not appear to do so.