vigonometry / pe

0 stars 0 forks source link

[Developer Guide] NFR pertaining to sorting maximum time is not scoped clearly #10

Open vigonometry opened 1 year ago

vigonometry commented 1 year ago

Details

Expected behavior

Screenshots / Screen recordings

Screenshot 2023-04-14 at 3.30.27 PM.png

System Details

nus-se-bot commented 1 year ago

Team's Response

It should be clear that we are talking about time complexity ...

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: Screenshot 2023-04-18 at 3.00.35 PM.png

Referring to the quality of requirements section on the CS2103/T module website a well-defined requirement should be testable. Simply stating "has a time of O(nlogn)" does not make this verifiable. It has to be associated with a quantifiable/testable metric that makes it easy to ascertain the verity of their assertions.

As such I will not be accepting this response as their NFR has been poorly scoped resulting in testers finding it tough to verify their claims. It should thus remain a DocumentationBug.