search

darrentde / pe

0 stars 0 forks source link

No error thrown when invalid parameters for predict command #3

Open darrentde opened 1 year ago

darrentde commented 1 year ago

image.png

image.png

image.png

image.png

No error thrown to notify users to find out whether their input is valid

Refer to page 24

nus-pe-bot commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

predict command does not return error message upon invalid parameter input

Replication: predict n/Bernice Yu s/MATH diff/3.5aasd
There is no error message indicating what I should do if my parameter in diff/ is incorrect; there is no response from the app for this, unlike if I got n/ or s/ parameters wrong.

image.png

[original: nus-cs2103-AY2223S1/pe-interim#2526] [original labels: type.FeatureFlaw severity.Low]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

We do not expect users to input non-numeric characters for difficulty. The UG clearly specifies that the value must be between 0 and 5.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Even if alphabets are not part of the parameters inputted, there should be a prompt by Watson to alert the users that the input should be in the range of 0 - 5.