search

nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Addgroup feature as given in UG examples does not add student with NUSNET ID parameter #5815

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

Steps to reproduce:
addstudent -n Jonas Chow -i E0123456
addstudent -n Jonas Chow -i E0123457
addgroup -g F01A -n Jonas Chow -i E0123457

Expected result:
Expected to add Jonas Chow with id E0123457 to group F01A

Actual result
"The student "Jonas Chow" needs to be allocated manually using ID due to duplicate naming."

Screenshots

Screen Shot 2021-11-12 at 4.17.08 PM.png

[original: nus-cs2103-AY2122S1/pe-interim#5792] [original labels: severity.Medium type.DocumentationBug]

ganhongyao commented 2 years ago

Team's Response

It is likely that the tester here is the same as that of #5841.

  1. The command that the tester used is an invalid one in the first place. If the tester is trying to add Jonas Chow with ID E0123457 into the group, only the name OR the ID should be specified. This is specified in the User Guide (see below). In this case, the tester specified both. The result that the tester expected is therefore not the correct result to be expected.

image.png

  1. The actual result is correct, as the -n Jonas Chow argument matches multiple students. The student must then be specified by using the ID only.

Duplicate status (if any):

--