ocaml-multicore / multicoretests

PBT testsuite and libraries for testing multicore OCaml
https://ocaml-multicore.github.io/multicoretests/
BSD 2-Clause "Simplified" License
37 stars 16 forks source link

Add Lin_domain.stress_test #443

Closed jmid closed 5 months ago

jmid commented 5 months ago

This PR adds Lin_domain.stress_{prop,test} These are useful to test that parallel misuse does not cause any crashes.

For the testsuite we add stress tests of modules containing only negative parallel tests, that tend to find a counterexample fast, and hence not perform much parallel stress testing.

While experimenting with the branch, this mode has already been useful to help create a consistent reproducer of the Dynlink issue #307.

I tried cleaning up the git-history this time, so it should read relatively cleanly @shym :wink: Can you give it a brief look, e.g., just the first 4 commits?

jmid commented 5 months ago

CI summary:

The two MinGW failures nicely makes the case for this PR :tada:

Out of 35 workflows 5 failed with 3 genuine issues and 2 CI issues

jmid commented 5 months ago

CI summary for merge to main:

Out of 36 workflows 6 failed with 4 genuine issues and 2 internal multicoretests-ci failures