Weiennn / pe

0 stars 0 forks source link

Sequence diagram missing argument #11

Open Weiennn opened 1 year ago

Weiennn commented 1 year ago

The sequence diagram omits the argument for execute(), should it be execute(model) instead?

image.png

I found this in at least 5 sequence diagrams, which may qualify this for a higher severity.

nus-pe-script commented 1 year ago

Team's Response

We decided that omitting the parameter of the execute command does not lose any relevant information since the model already plays a role in the sequence diagram.

From the textbook:

lol.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]