search

LowTL / pe

0 stars 0 forks source link

[GPALIST] `clear` function unknown with empty list #9

Open LowTL opened 5 months ago

LowTL commented 5 months ago

Incorrect exception handling: The clear command is unrecognised after clear has already been used previously.

image.png

image.png

soc-se-bot commented 5 months ago

Team's Response

According to the UG, the 'clear' function is a sub function of del. The user should access del then can use clear

The 'Original' Bug

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

[GPALIST] Inconsistency with UG

In the UG, I was told to use del to delete the whole list, but in the program, the command given was clear. The clear command was correct.

image.png

image.png

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

Their Response to the 'Original' Bug

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

image.png As mentioned in the UG, del is just a function for u to choose either delete specific one or to clear the whole list depending on your input. Hence this is not the bug as the tester is not following the prompt.

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:** Neither the UG nor DG have made any mention of `clear` as a sub-command of `del`.
## :question: Issue type Team chose [`type.DocumentationBug`] Originally [`type.FeatureFlaw`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [x] I disagree **Reason for disagreement:** The lack of mention of a sub-function would lead to confusion and inconvenience of users.