search

T-Wan-Lin / pe

0 stars 0 forks source link

Extra parameters before `add` command not handled #4

Open T-Wan-Lin opened 1 year ago

T-Wan-Lin commented 1 year ago

Screenshot 2023-04-14 at 4.37.13 PM.png Extra parameters in invalid add command not handled, the wrong exception thrown.

soc-pe-bot commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Incorrect flag command error not handled

Screenshot 2023-04-14 at 4.48.11 PM.png

User may type in /n instead of n/ as per the UG, but this is not caught as an invalid format.

[original: nus-cs2113-AY2223S2/pe-interim#2061] [original labels: type.FunctionalityBug severity.Low]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

No details provided by team.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: These 2 bugs are 2 distinct cases of invalid user input. The first is adding extra parameters outside of the core command, the other is the wrong formatting of the flags. These 2 errors should have distinct exception cases and can be fixed independently.