tyuyang / pe

0 stars 0 forks source link

Bug: Error thrown is does not represent the actual error #3

Open tyuyang opened 1 year ago

tyuyang commented 1 year ago

To reproduce Type in : add ground chicken /qty 10000000000000000000000000000000000000 Expected Either this /qty is too large! or accept quantity with no issues. A huge number is still a number Actual Error thrown says argument entered is not a number. image.png

nus-se-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]

Large positive numbers in the number field are not allowed

image.png

Large positive numbers in the flags that require numbers are not detected as numbers. This occurs in most of the commands that require some form of numbers. e.g. add, remove, recipe need , make, recipe favourite, recipe unfavourite


[original: nus-cs2113-AY2223S2/pe-interim#819] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

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

Downgraded to very low because incorrect error message is a typo/cosmetic issue

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 severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]