search

AdChain / AdChainRegistry

adChain registry smart contracts
Apache License 2.0
30 stars 10 forks source link

Test coverage sucks #12

Closed skmgoldin closed 6 years ago

skmgoldin commented 7 years ago

We need coverage of every single function with positive cases, negative cases and stateful cases.

skmgoldin commented 7 years ago

The Function: exit block is a decent example, but even that needs stateful tests. I'll stub them in. https://github.com/AdChain/AdChainRegistry/blob/5ddd33b633acf6c49622d7d5c43938eb02e871b2/test/registry.js#L180-L271

skmgoldin commented 6 years ago

Test coverage no longer sucks. It is now merely incomplete.