DesmondTo / pe

0 stars 0 forks source link

Unmark feature behavior does not match that of user guide #9

Open DesmondTo opened 3 years ago

DesmondTo commented 3 years ago

Step to reproduce:

  1. Enter command unmark -n Candice da/

Expected:

image.png

image.png

When 0 date input specified, removed the absence period for the next seven days, as stated in the user guide Removed

Actual: Error message displayed does not match.

image.png

nus-pe-bot commented 3 years ago

Team's Response

It is pretty clear that no date input means no flag.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Mark feature behavior does not match the user guide

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


[original: nus-cs2103-AY2122S1/pe-interim#4432] [original labels: type.FunctionalityBug severity.Medium]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

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 duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]