singpath / classmentors

Class Mentors UI
http://www.classmentors.com
MIT License
0 stars 8 forks source link

Cohorts #45

Closed shiquan-lim closed 8 years ago

shiquan-lim commented 8 years ago

Implemented cohort functionality.

Use cases as follows:

Misc enhancements:

Re-sending pull due to change in name from cohort to cohorts which was causing failure in CI testing

edit Still failing. Not sure why

shiquan-lim commented 8 years ago

Not sure why checks are still failing.

There is some issue in this function

it('should update navbar with title', function() { expect(navBarService.update).to.have.been.calledOnce; expect(navBarService.update).to.have.been.calledWith('Cohorts'); });

in cohorts.specs.js

or in the initialising of the controller at

ctrl = new cohort.component.controller(navBarService);