0liverkong / pe

0 stars 0 forks source link

No error message shown for invalid input for create-group #3

Open 0liverkong opened 2 months ago

0liverkong commented 2 months ago

To replicate:

  1. Run "create-group CS2103 -cm #1 -cm Alex" (invalid as #1 and Alex both refer to the same person)

Expected: either an error is thrown informing the user of the duplicate reference, or the duplicate reference is resolved and Alex is added normally.

Actual: No error message, hitting enter has no response

nus-se-script commented 2 months ago

Team's Response

Thank you for reporting the issue.

This detection was never planned / implemented, and it was never mentioned in the documentation that such a message exists.

Moreover, implementing this detection feature requires way higher effort. Considering the user could detect the issue easily by observing the group list and checking the command history, this feature has a lower priority than what we have already implemented. Thus we will be marking this issue as "Not in Scope".

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: > This detection was never planned / implemented, and it was never mentioned in the documentation that such a message exists.

This is not true - in particular, functionality for detecting duplicates was implemented. The log book shows a "DuplicateCourseMateException" thrown. It has been detected by your code but went unhandled. This is entirely within the scope of your project, as 1. the group has created a custom error for it and 2. has used it elsewhere in your code, as seen from your AddMemberCommand. Screenshot 2024-04-25 at 1.45.55 AM.png