-
Notifications
You must be signed in to change notification settings - Fork 5
test(mbt): Add Model Based Test for Emerald #201
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
base: erick/mbt-reset-timeouts
Are you sure you want to change the base?
Conversation
fa5b761 to
029e72f
Compare
This patch connects the newly added Quint specs to Emerald's code via the Quint Connect library.
029e72f to
d09bd51
Compare
| #[quint_run( | ||
| spec = "../../specs/emerald_mbt.qnt", | ||
| step = "step_no_failures", | ||
| max_samples = 32, | ||
| max_steps = 128 | ||
| )] | ||
| fn simulation_no_failures() -> impl Driver { | ||
| EmeraldDriver::default() | ||
| } | ||
|
|
||
| #[quint_run( | ||
| spec = "../../specs/emerald_mbt.qnt", | ||
| step = "step_with_failures", | ||
| max_samples = 32, | ||
| max_steps = 128 | ||
| )] | ||
| fn simulation_with_failures() -> impl Driver { | ||
| EmeraldDriver::default() | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
First uncached run took about 30m in the CI. Let me know if you want to generate fewer traces or reduce the number of steps to minimize test time.
bugarela
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good from my side!
This patch connects the newly added Quint specs to Emerald's code via the Quint Connect library.
PS: the best files to start reviewing are probable
tests.rsanddriver.rs.