search

nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Instruction Testing not thorough enough #5960

Open nus-se-script opened 1 year ago

nus-se-script commented 1 year ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

Most of the commands were not covered and some key test cases such as add commands were not included. It's quite limiting as a means to test the application. For instance, I started with an empty list and had to navigate the UG for more test commands. The commands felt like they weren't specified to test edge cases.

[original: nus-cs2103-AY2324S1/pe-interim#5924] [original labels: type.DocumentationBug severity.Medium]

ylyma commented 1 year ago

Team's Response

image.png

As mentioned in the CS2103T website, the tests that we included in this section is to give some guidance to chart a path through the features. We did not cover the AB3 features, such as the Add command, because its function remains similar to the original. We also did not give a long list of test cases to show all possible variations, as it is not required. It should be the tester coming up with the variations. Furthermore, this bug does not affect the functionality of the application at all and should not warrant a medium.

Duplicate status (if any):

--