nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Different error messages for invalid integer index #1444

Open nus-se-script opened 11 months ago

nus-se-script commented 11 months ago

Steps:

  1. When typing the command select 9 for a list size of 8, the error message displayed is "the course index provided is invalid" as shown in the screenshot below.

image.png

  1. However, when typing the same command select 999999999999999, the error message displayed is "Invalid command format" but the expected output should still be "the course index provided is invalid".

image.png

Perhaps can handle the errors when there is an integer overflow?


[original: nus-cs2103-AY2324S1/pe-interim#2127] [original labels: severity.Low type.FeatureFlaw]

phiphi-tan commented 11 months ago

Team's Response

image.png

As seen in the screenshot, this is considered extreme user behaviour. It is unrealistic to assume a TA to have courses/students anywhere close to the tested value.

Duplicate status (if any):

--