Add tests - Githubissues

GritLearning / grit

Grit Learning app

5 stars 1 forks source link

Add tests #48

Open eoinkelly opened 10 years ago

eoinkelly commented 10 years ago

We have ignored tests up to now but since MVP has stablilised, now is the time to add them. Currently I favour Mocha as test runner. This issue can be closed when we have at least one unit test and one E2E test that can be run.

eoinkelly commented 10 years ago

We now have tests setup

jasmine + karma for unit tests
jasmine + protractor for e2e tests

I decided to go with jasmine as it seems to be the most used in the angular community. The next step in resolving this bug will be to actually test the code we have.