albertarielw / pe

0 stars 0 forks source link

Cannot find client (case sensitivty issue) #8

Open albertarielw opened 1 year ago

albertarielw commented 1 year ago

Steps to take: enter: edit -p 1 ic/bobby (assuming you have client named Bobby)

Expected: adds client named bobby or Bobby. String matching must be exact in terms of case sensitivity this means that user has to remember client named down to which characters are capital or lower case

Actual: cannot find bobby error message

Screenshot:

image.png

nus-pe-script commented 1 year ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

image.png

Stated in UG, interested clients argument is case-sensitive

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]