search

cheeseong2001 / pe

0 stars 0 forks source link

Empty input during test report causes next question to be explained instead of invalid command #5

Open cheeseong2001 opened 3 months ago

cheeseong2001 commented 3 months ago

image.png

As per picture shown, when inputting nothing and pressing only enter, the explanation goes into the next question. When inputting hello (which is an invalid command), it also goes into the explanation for the next question.

Perhaps there should be checks to check that the commands acceptable are only exp or explanation or exit only? If there are other acceptable inputs, please specify

soc-se-bot commented 3 months ago

Team's Response

yes , maybe a while loop check can make the control flow more strict

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: This bug is not cosmetic in nature. The bug comes from the fact that it accepts any command even though it explicitly mentioned in the instructions that explanation, exp and exit are acceptable.

What if the user accidentally presses enter without typing anything? It goes to the next question and there is no back command to return to the previous questions.

Now, I am thinking that the severity should be severity.Medium since it could hinder the users without a back command if they ever go to the next wrong question accidentally.