nighoggDatatype / pe

0 stars 0 forks source link

Borrow command does not throw error upon being given multiple books: #8

Open nighoggDatatype opened 3 years ago

nighoggDatatype commented 3 years ago

To recreate:

Start a fresh copy of the software with default data
Run borrow r/Alex Yu bc/1999999999 bc/1999999998

Expected Results: Rejected for inputting too many barcodes

Actual Results: Accepts, and silently only process the last barcode. This behaviour can lead to much confusion and mistracking.

This scenario in particular is a reasonable expectation of the borrow command, but can be worked around if caught on time, resulting in the medium rating.

nus-pe-bot commented 3 years ago

Team's Response

According to our UG, if there are multiple occurrences of the same command parameter, our app will only keep track of the last one.

And btw it is not relevant to the documentation quality.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

[ ] I disagree

Reason for disagreement: [replace this with your reason]