search

kyhjonathan / pe

0 stars 0 forks source link

Program accepts reducing savings amount values that are past 2 decimal values. #5

Open kyhjonathan opened 7 months ago

kyhjonathan commented 7 months ago

When there is an existing savings amount, when user tries to reduce savings of an amount that has more than 2 decimal values, it accepts the input.

Steps to reproduce:

  1. Have existing savings in program (in my case category:gifts)
  2. type in "reduce savings c/Gifts a/0.000001"
  3. output: Savings reduced successfully for category: Gifts

This is inconsistent when add expenses, as it will throw an exception if more than 2 decimal values are given. However for reducing savings, no exception is thrown.

Suggestion: Allow for program to only take in 2 decimal values.

image.png

soc-se-bot commented 6 months ago

Team's Response

Thank you for pointing out this error, but it is quite unrealistic that this error would occur too much as no one is going to be spending $0.000001 realistically

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]