mozillascience / PaperBadger

Issuing badges to credit authors for their work on academic papers
https://badges.mozillascience.org/
Mozilla Public License 2.0
95 stars 45 forks source link

separates unit and integration tests; npm test still runs them all #175

Closed josmas closed 8 years ago

josmas commented 8 years ago

Because the integration tests run against a real server on the Internet, it would be great to be able to run unit tests (that talk to nothing; no network, no db (or mocked db)) on their own, because they are super fast and they can be run very often (they could be hooked into the commit flow at some point).

npm test still runs all tests. Instead of calling on the other two tasks, I prefer to run all of them in the same process, so that the runner reports on all tests (instead of having two report outputs, and having to scroll up on terminal).

abbycabs commented 8 years ago

Oh sweet! This is a bit of a bigger pr -- I'll take a closer look tomorrow afternoon :)

josmas commented 8 years ago

@acabunoc no rush! thanks for the other merges!

abbycabs commented 8 years ago

A+ work, thank you! R+