findTeacher accepts invalid fields

[juliussneezer04]

Expected: findTeacher throws error for invalid field 'i/' OR show teachers with involvement 'Math' (See example)

Actual: findTeacher does not display any teachers and shows no errors

[nus-pe-bot]

Team's Response

This command is not supposed to show teachers with involvement 'Math' as findTeacher is only supposed to search by name.

This is the intended behaviour, this just means that there are no teachers with the name i/Math. It's not in scope for the findTeacher command to make sure the user's input makes sense since any search query is a valid search query.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

• [x] I disagree

Reason for disagreement: # Reason:

• The program does not need to make sense of every query, rather it should ensure queries are not invalid.
• The point of prefixes like "i/" or "n/" are to signify parameters, and when used in commands that don't expect them, they should raise an error instead of allowing the invalid query to pass.
• It is in the scope for any command to throw errors for invalid inputs

---