AY2425S1-CS2103T-F14b-1 / tp

https://ay2425s1-cs2103t-f14b-1.github.io/tp/
MIT License
1 stars 5 forks source link

[PE-D][Tester B] Wrong error message for very large family size #207

Open nus-se-script opened 3 hours ago

nus-se-script commented 3 hours ago

Expected: The error message displayed should indicate the limit to which the app can take the family size input Actual: The error message is Family size should be a positive integer when the input family size is indeed a positive integer Steps to reproduce:

  1. Input command: add n/John Doe p/98765432 e/johnd@example.com a/311, Clementi Ave 2, #02-25 dob/1999-03-09 famsize/300000000000000000000000000000000000000000000000000000000 income/1000
  2. Error message displayed is Family size should be a positive integer

image.png


Labels: severity.Medium type.FeatureFlaw original: Virusrwj223/ped#5

LeongWZ commented 2 hours ago

Perhaps the message could be changed to Family size should be a positive integer not exceeding 2^31 - 1