search

G13nd0n / pe

0 stars 0 forks source link

Command adding issue #3

Open G13nd0n opened 2 weeks ago

G13nd0n commented 2 weeks ago

image.png

image.png

There is issue with add course mapping. even though the course is present in the partner uni, seems like the software does not recognise its presence

Steps to replicate:

add cs2113 /pu uwa /coursepu cits3002

nus-se-bot commented 2 weeks ago

Team's Response

The program returns the expected and correct behaviour with the command input in the Steps to replicate add cs2113 /pu uwa /coursepu cits3002, as the course CITS3002, offered by UWA (University of Western Australia) is to be mapped to CS2105, instead of CS2113, which was an invalid mapping provided by the tester.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: image.png

It is stated that the user is able to add course mapping, however, it is not specified that the course mapping is required to be from the listing given by the developer, it is only mentioned that it is subjected to validation, which is also unclear regarding the form of validation.

Furthermore, students can always request for new module mapping from the university or be taking other modules for their minor such as Physics module. Hence, disallowing user to add in other map listing causes a feature flaw.