search

donnell74 / CSC-450-Scheduler

MIT License
1 stars 1 forks source link

Constraint tests #65

Closed feq574 closed 9 years ago

feq574 commented 9 years ago

unit testing on instructor time preference before and after constraint and is_lab constraint.

hana9235 commented 9 years ago

Looks good to me; good job!

donnell74 commented 9 years ago

Looks good

kylenovak commented 9 years ago

Nice nice

CameronHKIng commented 9 years ago

Lab constraint has been updated to take a list of courses that have labs rather than one individual lab course. Your test will probably need to be altered slightly for this; it should just be a few line changes. See commit 183a1d3bef06f8fc43db5c1b1c7c816021ce8611 for reference.

CameronHKIng commented 9 years ago

I pulled develop in and fixed the one test; it should be fine to merge now.

feq574 commented 9 years ago

Kool. THanks Cameron!