search

tjiarong / pe

0 stars 0 forks source link

Add Dish - Improper input accepted #2

Open tjiarong opened 2 years ago

tjiarong commented 2 years ago

image.png

A fullstop behind a money value is still parsed and let through, even though it is not in proper monetary format or specified that a fullstop is alllowed in UG

nus-pe-bot commented 2 years ago

Team's Response

Thanks for reporting the issue.

Although 30. is not a "complete" double, however, it's reasonable to infer that the input value is 30.00 since the last character is . (similar to .30, .44).

Thus, we consider the program runs as expected in this scenario.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I disagree that this is merely an issue of inferred double, but rather incomplete parsing which could lead to further complication.

Also, nowhere in your UG was it stated that such "inferred" inputs are allowed. You are leaving it up to Java to handle the acceptance of such input value.

:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]