tezos-checker / checker

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

Include checkerMain and checkerEntrypoints in the coverage report #166

Closed gkaracha closed 3 years ago

gkaracha commented 3 years ago

Closes #163.

I got a little confused by this one, since checkerMain.ml and checkerEntrypoints.ml are both already listed in src/dune after all. Apparently bisect_ppx is a little lazy: having src/checkerEntrypoints.ml and src/checkerMain.ml around is not enough, but the testsuite must have some dependency on them for them to be included on the report. Simply opening them doesn't do it either. So, this PR:

gkaracha commented 3 years ago

Locally I witness a test coverage reduction from 89.33% 83.25%.

gkaracha commented 3 years ago

Thanks for the review! :pray: