nus-cs2113-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Clashes not shown when allocating classes #890

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

There are clashes in the courses but the allocate function treat the allocation as successful, without any clashes.

Steps to reproduce

  1. Start and input 2
  2. Input add then input cs2113
  3. Input add then input gec1007
  4. Input allocate
  5. Input print

Expected

A message showing me that there are clashed modules and cannot be allocate all modules successfully, as shown in the print command's timetable.

Actual

image.png


[original: nus-cs2113-AY2223S1/pe-interim#858] [original labels: severity.Medium type.FunctionalityBug]

cheehengk commented 1 year ago

Team's Response

The two modules entered by the tester originally clashed due to fixed lessons of the respective modules. As per the UG (see screenshot below), the allocate feature does not resolve existing clashes. It simply allocates undetermined lesson slots.

image_2022-11-14_00-29-41.png

Duplicate status (if any):

--