ZweZeya / pe

0 stars 0 forks source link

Closing help window on full screen crashes the app #2

Open ZweZeya opened 1 week ago

ZweZeya commented 1 week ago

Steps to replicate:

  1. Put the main window to full screen
  2. execute the help command
  3. Help window should pop up as full screen
  4. Close the help window
  5. App crashes

You can try again if this does not happen on the first try.

image.png

nus-se-script commented 1 week ago

Team's Response

https://drive.google.com/file/d/1gVYxsItnZt3SOREA19C-KSe3ND5K71H3/view?usp=sharing

The above is tested using a MAC OS, assuming the tester is using the same system, and as shown it does not crash. We have likewise tested on linux and windows and encountered no such issues.

It is likely an issue that sporadically happened on the user's machine. Likely the user has ran out of RAM, causing the application to close.

Either ways, we are unable to replicate the bug after extensive hours of testing with different computers.

Items for the Tester to Verify

:question: Issue response

Team chose [response.CannotReproduce]

Reason for disagreement: View the bug here

I could easily reproduce this bug. For reference, I am using a Macbook Pro with M1 Pro chip and hence, it is unlikely do to RAM issues since this is a simple Java application and I was not trying any stress testing.


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.High`] - [x] I disagree **Reason for disagreement:** Previously I have put this as `high` but after a thorough review, I have considered putting this to `medium`. Here are the reason(s): 1. An app crash is a fatal issue that will make it unusable for many users. Furthermore, using a full screen is common because the user may want to view more items on the list. Given that `help` is also a common command, an app crash may occur frequently, inconveniencing many users. 2. Provided that I am using MacOS, I cannot guarantee that this issue occurs in other OSs. Hence, this does not warrant a severity `high`. Nevertheless, this should also not be a `low` because, MacOS is a common OS and this issue could affect many users.