Skip to content
This repository has been archived by the owner on Jun 18, 2021. It is now read-only.

export toStateMachine from Test.StateMachine.Lockstep.NAry #372

Merged

Conversation

duog
Copy link
Contributor

@duog duog commented May 11, 2021

The immediate motivation for this is to get access to forAllCommands,
to facilitate labeling of test cases.

@edsko
Copy link
Contributor

edsko commented May 11, 2021

LGTM. This provides users with an escape hatch for functionality that isn't (yet) implemented in Lockstep -- though adding support for labelling to Lockstep itself would certainly be a valuable addition.

@duog duog force-pushed the feature/lockstep-expose-toStateMachine branch 2 times, most recently from 8d44754 to af98898 Compare May 11, 2021 08:55
@duog
Copy link
Contributor Author

duog commented May 11, 2021

I'm still fiddling with this, the type of Test.StateMachineQuickCheck.Lockstep.Simple.toStateMachine is in terms of NAry.At, which is unsatisfactory.

@duog duog force-pushed the feature/lockstep-expose-toStateMachine branch 4 times, most recently from be699e4 to a0566a1 Compare May 11, 2021 09:17
The immediate motivation for this is to get access to `forAllCommands`,
to facilitate labeling of test cases.

Signed-off-by: Douglas Wilson <douglas.wilson@gmail.com>
@duog duog force-pushed the feature/lockstep-expose-toStateMachine branch from a0566a1 to 6b33a31 Compare May 11, 2021 09:17
@duog duog changed the title export toStateMachine from Test.StateMachine.Lockstep.{NAry,Simple} export toStateMachine from Test.StateMachine.Lockstep.NAry May 11, 2021
@duog
Copy link
Contributor Author

duog commented May 11, 2021

Indeed, let's just expose Nary.toStateMachine for now

@stevana stevana merged commit 644e3ac into advancedtelematic:master May 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants