Open Punpun1643 opened 2 years ago
No details provided by team.
[The team marked this bug as a duplicate of the following bug]
Delete command in DG inconsistent with UG
In DG instructions for manual testing, under person delete, the sample commands provided were
delete
instead ofpersondelete
. Can be very confusing for testers and future developers, as we don't see the expected behaviour
[original: nus-cs2103-AY2122S2/pe-interim#2986] [original labels: severity.Medium type.DocumentationBug]
[This is the team's response to the above 'original' bug]
The severity of this issue is Low because it is a flaw that is unlikely to affect normal operations of the product. Appears only in very rare situations and causes minor inconvenience to the tester and developer, not the user.
Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)
Reason for disagreement: [replace this with your explanation]
Team chose [severity.Low
]
Originally [severity.Medium
]
Reason for disagreement: This case is very similar to one of the questions in the PE Readiness quiz that we have attempted. Please refer to this question:
Firstly, I think the team's response is invalid. Just because the DG is meant for the tester does not mean that they could lower the severity to Low
. Testers are users of the product too so long as they are using the app. Hence, I think their argument is not justified.
Secondly, based on the question in the quiz, I think the reason is the same that in the worst case, the tester would never be able to delete a person because there are no other instructions on delete
command (e.g. delete jobs). The testers may never find out how to delete a person. It is not the tester's job to find out why the command is wrong or what should be the correct command.
Moreover, if the team were to argue that the DG is for the testers only, then is the UG meant for the users only? Then how should the testers find out what the correct command is if the testers should use the DG only (because it is meant for them as what the team has claimed)? If that is the case, the more reason I would think this should be a Medium
severity bug as there is no other way for the testers to find out how they should delete a person. That is why I think that the argument that "DG is for testers and not users" is really not justified.
Explanation: Step 1
list
Step 2
delete 1
Output: nothing happens
Severity: Medium
Justification: User can't delete an employee from the app, this means that the app is not a full CRUD app as it is unable to support delete command. Is this command meant to delete jobs instead? then how do i delete employee.
It is stated in the DG that this is how we delete employee.
Edit: I later found out that the DG for manual testing is written wrongly. It should have been deleting job not employee. Took me around 5 mins to figure this out. This is an inconvenince for someone who has never used this app before, and likely to happen to everyone who has never used the app hence Medium severity.