nus-cs2113-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Not adding 's/' tag after 't/' tag will cause program to crash #316

Open nus-pe-bot opened 11 months ago

nus-pe-bot commented 11 months ago

When I tried to add the step buy ingredients in t/1, i did not put s/ before 'buy ingredients', and this caused a number format exception to be thrown.

tag bug.png


[original: nus-cs2113-AY2324S1/pe-interim#255] [original labels: type.FunctionalityBug severity.Medium]

kaijie0102 commented 11 months ago

Team's Response

We chose severity as Medium because it is a rare case which most users will not face.

1) User guide explicitly stated that steps have to have s/ 2) User guide explicitly stated that tags are to be either 1 2 3 or 4 (in this case tag="1 buy ingredients")

But we did not think of handling this error and our app crashed, hence we are accepting the bug as medium severity.

Duplicate status (if any):

--