search

lightz96 / pe

0 stars 0 forks source link

Unable to view a valid module #9

Open lightz96 opened 4 years ago

lightz96 commented 4 years ago

UG does not mentioned that module is case sensitive. Therefore, the command showmod m/cs2103 should work. Furthermore, no error message is shown.

image.png

nus-pe-bot commented 4 years ago

Team's Response

Hi, we accept that the UG should have mentioned that v1.4 does not support case-insensitive entries for module codes.

We have lowered the severity to medium as it only causes a occasional inconvenience to users but the product is still able to operate normally.

However, we were unable to reproduce the situation where no error message is shown, as it does show an error message in our own tests:

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.CannotReproduce]

Reason for disagreement: The team accepted the fact that UG should have mentioned that showmod does not support case-insensitive entries for module codes.

They should have accepted this issue as a documentation bug.

I realized that the bug only occurs when you execute the same invalid command again (does not need to be directly after).

However, did the team try to execute the same commands multiple times? (As stated in the instruction: response.CannotReproduce: You are unable to reproduce the behavior reported in the bug after multiple tries). It should produce the same behavior as stated in the issue.

E.g.

showmod m/cs2103

showmod m/cs2103

It should result in NullPointerException as shown below: image.png

The expected behavior should show module CS2103 on the browser regarding of the number of times the same command was executed.

:question: Issue severity

Team chose [severity.Medium]. Originally [severity.High].

Reason for disagreement: