search

Johnson-Yee / pe

0 stars 0 forks source link

Sequence diagram error #23

Open Johnson-Yee opened 3 years ago

Johnson-Yee commented 3 years ago

Arrows pointing to no where and returning from edge of the box

image.png

image.png

nus-pe-bot commented 3 years ago

Team's Response

Thanks for pointing this out. However, I am downgrading this to a low severity as this does not severely hinders the readability of the sequence diagram.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: I believe that this diagram is very ambiguous and that the moment the arrow points out of the diagram, it becomes very unclear. When ControlManager runs execute(), it should know what object the execute() method belongs to. This hinders readibility and clear understanding of the sequence diagram. Hence, I believe that the severity of this bug should be medium.