Skip to content

[Dune] Add domain flag to ortac-dune qcheck-stm #329

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

n-osborne
Copy link
Collaborator

@n-osborne n-osborne commented Jun 18, 2025

Make Ortac/Dune support the new --domain flag for Ortac/QCheck-STM.

This PR is based on #316 and #328. Please consider only the five last commits.

@n-osborne n-osborne force-pushed the dune-rules-qcheck-stm-domain branch from a314d20 to bcc3008 Compare June 18, 2025 16:18
@n-osborne n-osborne force-pushed the dune-rules-qcheck-stm-domain branch from bcc3008 to cefbf18 Compare July 8, 2025 12:47
@n-osborne n-osborne force-pushed the dune-rules-qcheck-stm-domain branch from cefbf18 to de76e23 Compare July 16, 2025 12:00
In particular, this runtime ignores the report build by the generated
`ortac_postcond` function and uses the initial `cmd_show`.
@n-osborne n-osborne force-pushed the dune-rules-qcheck-stm-domain branch from de76e23 to 605e9b7 Compare July 18, 2025 15:41
Default is false and generates test in a sequential context.
This first version makes the following design choice: when testing in a
concurrent context, we don't store the new SUTs created during the
tests.

This weakens the tests of the sequential prefix compared to the
sequential tests.

The rational is about ease of implementation, allowing for tesing with
domains without having to differentiate between commands ran in the
sequential prefix and those ran in one of the spawned domain.
In order to have `build_if` available.
@n-osborne n-osborne force-pushed the dune-rules-qcheck-stm-domain branch from 605e9b7 to e3409e1 Compare August 21, 2025 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant