search

pra-navi / pe

0 stars 0 forks source link

No warning of using PlantUML #22

Open pra-navi opened 7 months ago

pra-navi commented 7 months ago

image.png

There is no warning that the image is inaccurate as a result of using PlantUML and that by right after the cross marker the lifeline should be cut off as well.

This is in every UML diagram and will heavily confuse readers on the implementation.

soc-se-bot commented 7 months ago

Team's Response

We did provide a warning for the first instance of a sequence diagram in the dg but we failed to provide the reminder again for the subsequent diagrams. This was because we did not want to repeat the same information over and over. The below screenshot shows the warning message.

Screenshot 2023-11-19 at 12.16.46 AM.png

This is certainly not a high severity issue. We believe that this is a cosmetic issue and hence veryLow as it is simply a good to have warning for each sequence diagram but we did not prioritise this in this version. Even if the developer reading the diagram did not know of the limitation of plantuml he/she would not change their implementation of the various methods because of this since Java does not require memory allocation and the dereferencing of objects after use

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.High`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]