search

kangxinwang / pe

0 stars 0 forks source link

Does not account for error in duplicates of course type #1

Open kangxinwang opened 3 years ago

kangxinwang commented 3 years ago

There should also be an error that there is an overflow of parameters since the second course code is wrong. The group only takes into account of the first time -c is used and not the second time.

bug1.1.png

nus-pe-bot commented 3 years ago

Team's Response

We agree that the bug is a functionality bug but disagree with the severity medium.

This is because in the scenario that you put -c thesjef THEN -c cs2103, our program will note that thesjef is an invalid module code. In the event of accidental duplicate, the user will not be greatly affected. Furthermore, the flaw is also unlikely to affect normal operations of the product, since only the first value is considered.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your reason]