naijie2108 / pe

0 stars 0 forks source link

Add mapping extra arguments exception message #4

Open naijie2108 opened 2 years ago

naijie2108 commented 2 years ago

image.png

Error message says that index mapping does not exists even though the problem is with extra arguments to the command.

As shown in the picture, when the last argument is removed, the command goes through.

The same error can be seen across other commands such as find /code, but is not duplicated

nus-se-script commented 2 years ago

Team's Response

"1 32214" is read as 1 argument and there are no universities matching that input hence there is no universities. We accept the input as the correct format hence there is no parser format error.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: If "1 3214" was read as 1 argument, shouldn't the error message have been "invalid index" or something similar? since the command only expects integers as its arguments.

Furthermore, from a user's standpoint, the error here still lies in the fact that there was an extra argument since there is no mention that indexes can contain spaces(which doesn't make sense either since university and module index are integers), so "3214wer" should have been treated as an extra argument