search

tezos-checker / checker

An in-development "robocoin" system for the Tezos blockchain
24 stars 16 forks source link

Start adding tests for checkerEntrypoints.ml #174

Closed gkaracha closed 3 years ago

gkaracha commented 3 years ago

This PR includes only tests for the failing cases (i.e., trying to operate on the checker state while it's still unsealed). I wanted to add tests for the succeeding cases too, but this one got already big enough without them, so I'll open a separate PR for those cases.

Test coverage also increases (from 84.32% to 85.59%).

purcell commented 3 years ago

Nice! I pushed a further commit to reduce the duplication, since it appeared that the tests so far are all the same shape.

purcell commented 3 years ago

Merging in the interest of generally landing stuff into master ASAP

gkaracha commented 3 years ago

Great, thanks!