evedaktyl / pe

0 stars 0 forks source link

Policy Command Shows Incorrect Premium Value in UI #8

Open evedaktyl opened 5 months ago

evedaktyl commented 5 months ago


video:https://raw.githubusercontent.com/evedaktyl/pe/main/files/7f8b3836-9113-4d0e-92ac-6075b564561c.mov

As seen in the screen recording above, the premium value stored does not match the input premium value.

nus-pe-bot commented 5 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Using large premium sizes cause the program to round it up

Screenshot 2024-04-19 at 4.52.54 PM.png

Screenshot 2024-04-19 at 4.53.09 PM.png

Unsure if this is intended, but I dont think an insurance program should ever change the premium sizes, even if its just 1 dollar, they could get sued for this


[original: nus-cs2103-AY2324S2/pe-interim#2635] [original labels: type.FunctionalityBug severity.Low]

Their Response to the 'Original' Bug

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

This is a case of extreme user behaviour. In real life, insurance premiums will not come close to 15 digits which is around a quadrillion dollars. The course website mentions to consider such cases as not bugs.

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`] - [x] I disagree **Reason for disagreement:** ![image.png](https://raw.githubusercontent.com/evedaktyl/pe/main/files/3f1b9686-29c2-413d-a631-e8533dfed61c.png) It may not be an extreme user behaviour if the user is using another currency. The UG also does not explicitly state that the premium value must be in SGD. Even if the application explicitly specifies that the premium value must be in SGD, it is still possible for a user to enter a 15-digit premium value by mistake (15 digits is not as extreme as 30 digits).
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.High`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]