search

trash-bin99 / pe

0 stars 0 forks source link

Possible missing extension for UC03 #11

Open trash-bin99 opened 2 years ago

trash-bin99 commented 2 years ago

image.png

The use case does not address the scenario when a contact has already been added to the specified folder. If it is considered to be an invalid index, then it should have been more clearly specified in the extensions 3a, that an invalid index includes a contact that already exists inside the folder.

nus-pe-bot commented 2 years ago

Team's Response

According to the examples in the website, use cases do not need to contain nitty-gritty details.

image.png

We have also included information in the UG.

Minimize repeating information that are already mentioned in the UG.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I do agree that we should avoid repeating information, but it should be crucial for all possible extensions to be present in the use case, as missing out one might cause potential developers reading the DG to miss out on potential bugs or missing functionality. Additionally, the extension I mentioned is not a nitty-gritty detail, but a possible and likely scenario that will happen if the user forgets that he has previously added a contact to a folder. Therefore, it is a valid extension that should have been addressed in the use case.