CS2103-AY1819S2-T12-3 / main

MediTabs: Medicine Stock Taking Application
https://cs2103-ay1819s2-t12-3.github.io/main/
MIT License
2 stars 4 forks source link

'warning' command displays wrong error message #169

Closed nus-se-bot closed 5 years ago

nus-se-bot commented 5 years ago

Describe the bug Wrong error message displayed. Should state the range of integer values accepted (i.e. state max number should be ...), not tell me that the threshold should only contain numbers

To Reproduce Steps to reproduce the behavior:

  1. Type warning q/9999999999999999
  2. Get an error
  3. But its a wrong error

Expected behavior Should display an error like "Maximum threshold should be ..." instead of "Threshold should only contain numbers"

Screenshots Untitled


Reported by: @reinsheer Severity: Medium

[original: nus-cs2103-AY1819S2/pe-dry-run#637]

flxffy commented 5 years ago

Fixed with pull request #179