search

SharlynLui / pe

0 stars 0 forks source link

Mismatch in status of the topics #11

Open SharlynLui opened 5 months ago

SharlynLui commented 5 months ago

image.png

Under "list", topic 2 is unattempted. However, when viewing results, it shows that I have a result of 200% for topic 2.

nus-pe-bot commented 5 months ago

Team's Response

Thanks for pointing this out. Through many attempts to reproduce the issue, we found out that the Storage class incorrectly initiates all the classes associated to the user results, as well as the topic attempt status.

Duplicate as the root cause of the reported bug is the same.

The 'Original' Bug

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

Index out of bound at the end of results

image.png

Text at the end of results shows "index out of bounds". Can consider reviewing code to not print outside of bound.

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

Their Response to the 'Original' Bug

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

Thanks for pointing this out. Through many attempts to reproduce the issue, we found out that the Storage class incorrectly initiates all the classes associated to the user results.

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: This bug should not be a duplicate of the original bug because they are not of the same severity even though I agree that solving the bigger bug might also solve the original bug.

The original bug focused on the extra output from the app. This current bug focuses on the function of tracking the attempted topic and score promised by the app not delivered which might be due to the same reason why there was an extra output.

Given the explanation of why the bug is of medium severity below, it should not be considered a duplicate to avoid inheriting the same severity.

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** ![image.png](https://raw.githubusercontent.com/SharlynLui/pe/main/files/bbe8d98e-f141-4080-967c-44e1771cb718.png) **Why is it not low:** It is a flaw that affects the normal operation of the product as "exiting the game and resuming mid-way" is a valid operation of the product as seen from the UG. Thus, it is also not very rare as it is likely that users will use the app's stated features. It is a clear bug as I have attempted both **topics 1 and 2** as seen from the image attached in the bug above, but the status for topic 2 is shown as **not attempted**. It also shows that the score for topic 2 is **200%** which does not make sense. ![image.png](https://raw.githubusercontent.com/SharlynLui/pe/main/files/9cfa59ca-62c8-4d37-ae8b-43582a4964a8.png) **Why it is medium:** User will not have their attempt recorded and will not be able to have an accurate score to reflect their understanding of the topics attempted which is the purpose of the app. This causes inconvenience to users who try to exit the app midway and try to resume their progress after and affects the functionality of the app.