Dniv-ra / pe

0 stars 0 forks source link

Adding Infinity as a expense breaks the code #5

Open Dniv-ra opened 2 years ago

Dniv-ra commented 2 years ago

Screenshot 2021-11-12 at 4.38.03 PM.png

While adding infinity with a small i the code correctly prompts to reenter the details but for Infinity the code breaks with an exception.

nus-se-script commented 2 years ago

Team's Response

Duplicate of #1882

Downgraded to Low - deliberate sabotage

The 'Original' Bug

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

Adding NaN as a expense amount breaks the code

Screenshot 2021-11-12 at 4.34.54 PM.png

The application adds NaN but when it has to save the data the code breaks throwing an exception The NaN here is case sensitive (only breaks for NaN not Nan or nan)


[original: nus-cs2113-AY2122S1/pe-interim#1939] [original labels: severity.Medium type.FunctionalityBug]

Their Response to the 'Original' Bug

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

NaN is an undefined number - not considered reasonable user input. deliberate sabotage is not a bug.

Downgraded to Low - deliberate sabotage

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 reason]


:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]


:question: Issue severity

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

Reason for disagreement: [replace this with your reason]