search

xlzior / pe

0 stars 0 forks source link

Success message for deleting classes is wrong (does not list out deleted classes correctly) #3

Open xlzior opened 2 years ago

xlzior commented 2 years ago

Steps to reproduce

image.png

I started with these 4 classes: CS1101S S08K - S11K. They all have the same time (I believe this is necessary to reproduce the bug)

Then I ran the command rmc 1-4

Actual

image.png

Expected

It should list out the 4 distinct classes I deleted, but instead it copies S11K 4 times

nus-pe-script commented 2 years ago

Team's Response

The command still behaves as expected, it deletes the lessons from the index 1 to 4. It is only a cosmetic error in the return message, does not affect the operations or cause inconvenience since there is no subsequent actions that the user has to take. Also, it is unlikely to have multiple lessons on the same day and the same time. The inputs are not likely to occur for actual usage. This instance only occurs for this specific case which is not common.

Hence, bug is accepted but at the severity level of very low.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: [replace this with your explanation]