Add suite with alcotest

[gaborigloi]

And test Valid_ref_list using alcotest.

Signed-off-by: Gabor Igloi gabor.igloi@citrix.com

[gaborigloi]

This depends on xs-opam being released.

[edwintorok]

I think you can make both of these run as part of runtest if you put alcotest in another folder

[gaborigloi]

How can I do that?

[edwintorok]

See https://discuss.ocaml.org/t/jbuilder-support-for-multiple-test-executables/825 you can in fact have two runtest in same jbuild file.

[gaborigloi]

Thanks, that's great, I'll use two!

[lindig]

xs-opam has been released. Can we merge this?

[mseri]

Let's rerun travis first, once we have a build with the next xs-opam uploaded to the xs-buildenv

[gaborigloi]

Coveralls doesn't work yet

[mseri]

Same issue as message-switch... There is finally a plan upstream to fix the ppx_bisect/coverage situation. Until that is complete you only need to install by hand the test dependencies in the .coverage script. It's a matter of adding alcotest to the opam install -y bisect_ppx ... line:

xen-api/.travis-opam-coverage.sh

Line 36 in e50f875

opam install -y bisect_ppx ocveralls

We don't have the issue with oUnit because until we rationalise the build, it is considered a required dependency and not a test one.

[gaborigloi]

How about using opam install --build-test?

[mseri]

Let's try it. It did not work when I tried on message-switch, in that case I even tried setting the variables and installing bisect before running it trying to avoid running the tests twice (once by opam and the other for the coverage) but without much success

[gaborigloi]

Oh, that doesn't work because that tries to install the test deps of the dependencies transitively

[coveralls]

Coverage decreased (-0.004%) to 17.659% when pulling 0aa2e31 on gaborigloi:alcotest into 818b7e0 on xapi-project:master.