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

Error message doesnt correspond to error in command #9

Open
tangzhenen opened this issue Nov 17, 2023 · 1 comment
Open

Error message doesnt correspond to error in command #9

tangzhenen opened this issue Nov 17, 2023 · 1 comment

Comments

@tangzhenen
Copy link
Owner

tangzhenen commented Nov 17, 2023

6.PNG
i missed out a dash in my command but the error message talked about the date instead

@nus-pe-script
Copy link

nus-pe-script commented Nov 20, 2023

Team's Response

Intended behaviour, there is another command find DATE, where DATE has to be a Localdate in format of yyyy-MM-dd.

The parser interpreted the DATE as String "sleep" and therefore threw the correct error of incorrect date provided, as such it is intended behavior.

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: i do not see how this intended behaviour helps the user with giving a correct command as there was something wrong with the command itself instead of the date


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