search

steemit / libcrypto-js

Cryptographic utilities for Steem in Javascript. Zero dependencies.
5 stars 17 forks source link

Code coverage #4

Open jnordberg opened 6 years ago

jnordberg commented 6 years ago

Add some code coverage tool to make sure our tests cover all usecases

goldibex commented 6 years ago

I'm in for this. Do you have a preference on tooling? For choice it would be something lightweight that integrates with node-tape.

Also, I assume we're taking on code coverage for "our" code (i.e. any js file in this repo), not the vendored sjcl, right?

jnordberg commented 6 years ago

I like nyc/istanbul https://github.com/istanbuljs/nyc, works fine with tape as well.

Yeah, our code. Wouldn't hurt to see what's covered in sjcl though but I don't think we should enforce any coverage inside there