Added more-legitimate tests + code coverage #1

Merged

brianreavis merged 3 commits from tests into master

2014-07-31 17:17:24 -07:00

brianreavis commented

2014-07-31 12:02:51 -07:00

(Migrated from github.com)

I added a better test suite that's more automated than having to read console.log's to see if something is or isn't working. The tests can be run with either of the following:

$ make test
$ npm test

I also added configuration for running them automatically on TravisCI (so that future PR's will report test results + coverage). For this, you'll need to sign up and enable tinycache on TravisCI and Coveralls.io (both are free). For a demo, checkout the naturalatlas fork of tinycache.

This PR also includes a bugfix for misses() – it wasn't reporting misses caused by non-existent keys.

I added a better test suite that's more automated than having to read console.log's to see if something is or isn't working. The tests can be run with either of the following: ``` sh $ make test $ npm test ``` I also added configuration for running them automatically on TravisCI (so that future PR's will report test results + coverage). For this, you'll need to sign up and enable tinycache on [TravisCI](https://travis-ci.org/) and [Coveralls.io](https://coveralls.io) (both are free). For a demo, checkout the [naturalatlas fork](https://github.com/naturalatlas/tinycache) of tinycache. This PR also includes a bugfix for `misses()` – it wasn't reporting misses caused by non-existent keys.

andyburke commented

2014-07-31 17:25:12 -07:00

(Migrated from github.com)

Thanks, great PR. I signed up for Travis and Coveralls.