search

kyhjonathan / pe

0 stars 0 forks source link

Shows invalid command when the number of people sharing is high #6

Open kyhjonathan opened 7 months ago

kyhjonathan commented 7 months ago

Steps to reproduce:

  1. type in "add shared bill a/123123 n/999999999999999999 d/testing shared bill23"
  2. output: "For input string: "999999999999999999" (Invalid command)"

image.png

nus-se-bot commented 6 months ago

Team's Response

Thank you for bringing to our attention to a potential bug. In the User Guide, it is stated that the command for adding a shared bill should be add shared bill a/AMOUNT n/NUMBER_OF_PEOPLE d/DESCRIPTION. However, the example shown in the screenshot omits the 'description' part of the command, which understandably leads to incorrect syntax and causing an 'invalid command' error.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Apologies on the incorrect input for the screenshot. During the testing phase I must have found out the bug and tried to replicate it for the screenshot but did not type in the description for the screenshot.

According to CS2113 website: If you cannot reproduce the bug based on the info given by the tester you are still expected to make a reasonable attempt to go beyond the information provided by the tester to reproduce the bug, if there is clear evidence of something wrong.

I replicated this test by typing in the description and the error is still present.

image.png

As seen above the command works for n =99, but when n=high number, it does not work.