yucongkoo / pe

0 stars 0 forks source link

Bug in use case 7 - included internal behaviours #24

Open yucongkoo opened 11 months ago

yucongkoo commented 11 months ago

Notice the following sentence in use case 7:

InsuraHub deletes the person

Screenshot 2023-11-17 at 5.36.42 PM.png

This is an internal behaviour and should not be included in use cases.

nus-se-script commented 11 months ago

Team's Response

It is not an implementation detail. This is just stating what the application will do and not providing the exact details to how it is being deleted.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: > It is not an implementation detail.This is just stating what the application will do and not providing the exact details to how it is being deleted.

I do agree with you that this is not an implementation detail, however, I did not say that it is an implementation detail. Rather, I said that it is an internal detail.

More specifically justifying my statement, the system updating itself is an internal detail which is not externally visible by the user. This case is somewhat similar to the following counter-example provided in the textbook, where the system does some internal action(saving a file to cache). Screenshot 2023-11-21 at 1.29.44 PM.png


## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [x] I disagree **Reason for disagreement:** I do not think this is an cosmetic issue, rather, it is including internal details that should not be included in use cases(so technically this use case is not a correct use case)