search

IshaaanVyas / pe

0 stars 0 forks source link

Missing methods #18

Open IshaaanVyas opened 2 years ago

IshaaanVyas commented 2 years ago

image.png

what methods are being called here?

nus-pe-script commented 2 years ago

Team's Response

The function called was left out for simplicity but can be included to be clearer.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.High]

Reason for disagreement: If the method names are removed for simplicity, might as well just not show that in the sequence diagram at all. Not having the method names can confuse new developers a lot and makes this sequence diagram not very useful as it would be better to read through the code itself. I consider this to be at least of Medium severity.