nus-pe-bot
commented
3 years ago Team's Response
No response provided.
The 'Original' Bug
[The team marked this bug as a duplicate of the following bug]
crash
When a element is remove program should say "file is corrupted" but it just crashed.
[original: nus-cs2113-AY2021S2/pe-interim#116] [original labels: severity.High type.FunctionalityBug]
Their Response to the 'Original' Bug
[This is the team's response to the above 'original' bug]
The data file is supposed to maintained, only read and written to by the program itself. And users are not supposed to touch the data file. This error occurs because the tester deliberately change the file, which a normal user will never do so in normal use case.
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)
- [x] I disagree
Reason for disagreement: As you have 3 text files, each text file is a point of failure. This belongs to another text file.
:question: Issue severity
Team chose [severity.Low]
Originally [severity.High]
- [x] I disagree
Reason for disagreement: We cannot assume that the file will always be un-corrupted. In the scenario where the file is corrupted, the app will not be able to run at all because it has not capture this exception and the user will not know about it. System crashed, so it should be high severity.
File crash when one of the element is deleted in goals.txt Should output "File corrupted"