ama-chi / pe

0 stars 0 forks source link

Tag function allows duplicate tags on edited stores #5

Open ama-chi opened 1 year ago

ama-chi commented 1 year ago

To reproduce:

  1. edit a store

Screenshot 2022-11-11 at 5.02.25 PM.png

  1. Add a tag that already exists to that store

Expected: Tag should be rejected

Screenshot 2022-11-11 at 5.04.30 PM.png

Actual: Tag accepted

Screenshot 2022-11-11 at 5.03.19 PM.png

Now store contains 2 duplicate tags, with the visual bug mentioned in an earlier issue.

Screenshot 2022-11-11 at 5.04.03 PM.png

nus-se-script commented 1 year ago

Team's Response

The UG indicates that each tag should be preceded by a -t prefix as follows:

image.png

image.png

From the images, we believe the tester has incorrectly input multiple tags preceded by only one -t prefix, causing multiple words to be parsed as an individual tag, even though he intended for each word to be individual tags.

There are no duplicate tags, and we believe this is evident by how each tag is displayed in its own red box. Hence, the application is behaving as expected.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]