Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Functionality bug: View does not reject incorrect dates #5

Open
Jiacheng-y opened this issue Nov 11, 2022 · 1 comment
Open

Functionality bug: View does not reject incorrect dates #5

Jiacheng-y opened this issue Nov 11, 2022 · 1 comment

Comments

@Jiacheng-y
Copy link
Owner

Jiacheng-y commented Nov 11, 2022

To reproduce the bug,

  1. input the following command into the command line:
    view t/i mo/0000-12
  2. hit enter

Expected: an error message to inform user that the year input is wrong and year 0000 does not exist.

However,

the following display is shown although year 0000 does not exist and is no way a meaningful input for a finance management application in the 21st century.

Screen Shot 2022-11-11 at 17.07.15 PM.png

@nus-se-script
Copy link

nus-se-script commented Nov 14, 2022

Team's Response

Intended, this is to avoid overzealous input validation and cater for strange ways that the users may use the product as advised by prof.
for example: users may want to use year 0000 to indicate a special expense or a special income. We do not think it is necessary to limit the year that they can input to improve flexibility of the application so the users can use it however they desire to
Screenshot 2022-11-12 at 9.59.53 PM.png

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: Thank you for the response.

However, there is no warning for the potentially invalid input of view t/i mo/0000-12 either. Perhaps type.FeatureFlaw would be a more appropriate tag for this bug.


❓ Issue severity

Team chose [severity.Low]
Originally [severity.Medium]

  • I disagree

Reason for disagreement: [replace this with your explanation]


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants