search

sagarsureka / pe

0 stars 0 forks source link

Unexpected behaviour of find command #7

Open sagarsureka opened 3 years ago

sagarsureka commented 3 years ago

The app currently contains these exercises: (notice 1 and 3) Screenshot 2020-11-13 at 2.25.13 PM.png

When executing the command find k/sit k/pull, the expected behaviour as per the ug is both exercises 1 and 3 should be listed but only exercise 3 of the original list is listed.

Screenshot 2020-11-13 at 2.25.21 PM.png

guide screenshot:

Screenshot 2020-11-13 at 2.29.02 PM.png

nus-pe-bot commented 3 years ago

Team's Response

If users enter multiple headers (k/ in this issue), only the last header will be accepted (k/pull in this issue). If users would like to enter multiple keywords, they should enter all keywords after ONE header and divide them using whitespace. (similar to the behavior of find command in AB3) I did fail to mention this in UG. Sorry about this : (

image.png

Items for the Tester to Verify

:question: Issue type

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

Reason for disagreement: A functionality bug is when a functionality does not work as expected. A documentation bug is one where there is some issue purely in the documentation. In this case, due to a missing constraint in the user guide the functionality is not working as expected. Hence, I feel this is a functionality bug.

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]