search

pr4aveen / pe

0 stars 0 forks source link

Extended lifeline in DG #13

Open pr4aveen opened 4 years ago

pr4aveen commented 4 years ago

In the test mechanism sequence diagram, there is a stray lifeline present. Should this be there or removed? I feel that this information might be useful for future developers to know

Screenshot 2020-11-13 at 5.31.42 PM.png

nus-pe-bot commented 4 years ago

Team's Response

Hi, thank you for the question. The lifeline is supposed to be there as updatedFlashcard replaces the existing flashcard as shown in the sequence diagram, existing flashcard is subsequently destroyed.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I'm rejecting this as I'm not too sure whether the notation is correct here. If the lifeline should be there, then why is it only present in one of the alt blocks?

image.png

I'm not sure if this is a plantuml issue. If it is, please ignore my disagreement.