search

meteor / todos

The example app "Todos", written following the Meteor Guide
Other
535 stars 367 forks source link

Fix tests #210

Closed GeoffreyBooth closed 7 years ago

GeoffreyBooth commented 7 years ago

So at some point, the collections were renamed from Lists and Todos to lists and todos, but no one updated the tests to reflect this. With this update the tests (as run via rm -Rf .meteor/local/plugin-cache; meteor npm test) pass.

About that command: I seem to be hit hard by https://github.com/meteor/todos/issues/198 / https://github.com/DispatchMe/meteor-mocha-phantomjs/issues/33, and the solution in this comment was the only way I could get the tests to run. This should really be fixed. The CI server isn’t even running the real tests, it’s running a single test that looks like a stub.

hwillson commented 7 years ago

Awesome, thanks @GeoffreyBooth! Yes, issue #198 is a real pain :-( ...