chewterence / pe

0 stars 0 forks source link

DG Incomplete instructions for manual testing: "viewMembers" #14

Open chewterence opened 3 years ago

chewterence commented 3 years ago

Expected: Lists all members within HEY MATEz. Did not mention that the status message would change even though it changed.

image.png

nus-pe-bot commented 3 years ago

Team's Response

Expected outcome is listed under expected, which lists all members. Command message also specifies that it has listed members. Hence stating the command message is not significant and conveys the same expected result

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Thank you for the response.

While i agree that "Command message also specifies that it has listed members.", but the status message has changed.

Consistency wise, compared to the rest of the DG instructions for manual testing, status message being updated is reflected in the instructions. image.png

This could cause a slight confusion for the developer reading the instructions for manual testing, is the status message expected to change? or not? Perhaps the status message would only be updated for deleting a task, and maybe I am not expecting the status message to change when viewing members. This remains unclear in the instructions for manual testing.

Hence this should be accepted as a "very low" severity documentation bug.