get code coverage up to 100%

Level / memdown

In-memory abstract-leveldown store for Node.js and browsers.

MIT License

287 stars 37 forks source link

get code coverage up to 100% #53

Closed nolanlawson closed 8 years ago

nolanlawson commented 8 years ago

There were just a few changes necessary to get it to 100%. Also there was quite a bit of dead code.

coveralls commented 8 years ago

Coverage increased (+6.9%) to 100.0% when pulling 45407b43273af9db8bd1b563ad469dbee072fe4e on code-coverage into 8cf887082218e72b946ceb7194ef98b5e73b3658 on master.

juliangruber commented 8 years ago

thank you for your work on this!

nolanlawson commented 8 years ago

Added @juliangruber's suggestions and removed the setImmediate -> immediate change per discussion in #52

coveralls commented 8 years ago

Coverage increased (+6.9%) to 100.0% when pulling cc7829c1fc29d121ef58bfb1c6bf4c33deb47ea5 on code-coverage into 8cf887082218e72b946ceb7194ef98b5e73b3658 on master.

nolanlawson commented 8 years ago

localtunnel is not having a good day today, which is why the tests keep failing. will wait for a green tho

nolanlawson commented 8 years ago

I have restarted this like 10 times and localtunnel still keeps failing. :confused:

nolanlawson commented 8 years ago

wooooooot