search

GanapathySanathBalaji / pe

0 stars 0 forks source link

Wrong error message shown when wrong format is used for deletep command #3

Open GanapathySanathBalaji opened 4 years ago

GanapathySanathBalaji commented 4 years ago

The error message shown isn't the correct one.

deletep \ index 2

image.png

nus-pe-bot commented 4 years ago

Team's Response

Rejected justification:

This is the correct behaviour. Please refer to the UG screenshot below. Section 4.1.3 explains the deletep functionality.

It is stated that the format should be \index , not \ index. In addition, this is compulsory as highlighted below.

Because the wrong format is provided \ index, HAMS inteprets this as a dummy field and ignores it, as noted in Section 3.

Because no date is received by HAMS, it prompts the user to input a valid integer that is within range of the list.

image.png

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]