yeohhq / pe

0 stars 0 forks source link

Command 'delstudent' name matches existing student but with whitespaces in input #3

Open yeohhq opened 3 years ago

yeohhq commented 3 years ago

This issue is a feature flaw about the 'delstudent' command.

Input: delstudent grp/g04 name/david chong matric/A0123456D (there are multiple whitespaces between 'david' and 'chong' above, but CATcher seems to not be able to display it)

Output: Screenshot 2020-11-13 at 4.41.57 PM.png

Rationale: Perhaps parser should be able to parse individual strings in the name input to match despite there being whitespaces in between as your other commands seem to be able to handle inputs with unnecessary whitespaces.

In this case, the input should match student at index 4.

nus-se-bot commented 3 years ago

Team's Response

Definition:

severity.High : A flaw that affects most users and causes major problems for users. i.e., makes the product almost unusable for most users.

Product can still be used despite this exception. If you try to enter another valid command after the exception occured, it will execute normally. The only inconvenience is that user needs to re-enter the command with the full name of the student.

severity.Low : A flaw that is unlikely to affect normal operations of the product. Appears only in very rare situations and causes a minor inconvenience only.

Dropping this to severity.Low because of the following reasons:

  1. delstudent INDEX is much more inutitive and much more likely to be used by our target users, hence this would appear in very rare situations when the tutor decides to use a less intuitive command.
  2. The only inconvenience that occurs as a result of this bug is that the user has to re-enter the command with the correct parameters

    Items for the Tester to Verify

    :question: Issue type

Team chose [type.FunctionalityBug] Originally [type.FeatureFlaw]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]