search

Joellimjr / pe

0 stars 0 forks source link

switching timer and exiting causes crash #9

Open Joellimjr opened 3 months ago

Joellimjr commented 3 months ago

image.png

running the focus swtich command and immediately using exit causes the program to crash

nus-se-bot commented 3 months ago

Team's Response

It was a bug we failed to catch.

image.png

However, by using the command exit, it indicates that the user intends to exit the application, which ultimately terminates the app. Hence, following the description for the severity level, this bug would satisfy severity.Low. While it does cause some minor inconveniences like the failure to print goodbye message, it will not affect normal operations of the product.

The 'Original' Bug

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

Fatal error when using exit after focus switch

image.png

Entered focus switch followed by exit and the app crashed, showing me the above error. Bug occurs when I switch the timer to "Count down timer".

image.png

Using focus switch twice to switch back to the "Counter up timer" followed by exit does not crash the app.

In both cases, no other focus functionality was used i.e. had just started the app.

image.png

Using switch to get to "Count down timer" then start and stop, finally exit does not produce this error.

image.png

Following from the previous example, running start and stop once in "Count down timer" mode prevents this bug from happening ever again.

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

Their Response to the 'Original' Bug

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

It was a bug we failed to catch.

image.png

However, by using the command exit, it indicates that the user intends to exit the application, which ultimately terminates the app. Hence, following the description for the severity level, this bug would satisfy severity.Low. While it does cause some minor inconveniences like the failure to print goodbye message, it will not affect normal operations of the product.

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