search

jessen11 / pe

0 stars 0 forks source link

Application does not accept floats #3

Open jessen11 opened 3 years ago

jessen11 commented 3 years ago

add property n/Bishan t/Hdb a/Blk 150 Bishan Street 11 #02-101 p/570150 d/30-06-2021 ca/20,2

App gives an error message

Note the following conditions for specifying an asking price:

  1. The dollar sign is optional. (ok)
  2. There should not be any leading zeros in the number specified. (ok)
  3. The asking price should not be negative. (ok)
  4. Either do not use commas at all or be consistent in the usage of commas throughout, where each comma should separate every three digits from the back. (what if the price is not an integer) E.g. $1000000 or $1,000,000 are valid but $1000,000 or $100,00,00 are not valid.
nus-pe-bot commented 3 years ago

Team's Response

From the provided test input ca/20,2, this input is invalid because it was stated that each comma should separate every three digits from the back. Since the input 20,2 does not follow the required format, there is an error message produced.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I disagree, as I believe it is not the Function that are being questioned. Rather, it is the feature. Because of this, as a Singapore Citizen, it is possible for me to find a decimal number as a nominal (Example: $235,45), as Singaporeans have cents.

:question: Issue type

Team chose [type.FunctionalityBug] Originally [type.FeatureFlaw]

Reason for disagreement: This is a feature bug. I believe that functionally, this is correct.