DesmondTo / pe

0 stars 0 forks source link

Mark feature behavior does not match the user guide #8

Open DesmondTo opened 2 years ago

DesmondTo commented 2 years ago

Step to reproduce:

  1. Enter the command: mark -n Candice da/

Expected:

image.png

As shown in the screenshot, no date will mark the absence period to be the next 7 days

Actual:

Not marked. Error message displayed.

image.png

nus-pe-bot commented 2 years ago

Team's Response

No date input means no date prefix attached. In this case a date input of an incorrect format was provided. It is not considered no date, but a wrong date period entered.

image.png

As mentioned here, this is a flag input, and the date is expected, however the flag input provided did not have the appropriate date attached, it is not considered no date input.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: image.png

Then, it is still a valid bug, since you can just say No date input means no date prefix attached. Putting 0, 1, or 2 date inputs can confuse the reader as 1 or 2 date inputs requires da/ flag, it will be natural for the reader to think that 0 date input requires da/ flag as well. However, I agree that severity should be low as it actually works for no date.


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]