Skip to content
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

Implement test model subcommand #66

Closed
andrey-kuprianov opened this issue Jul 28, 2022 · 0 comments · Fixed by #122
Closed

Implement test model subcommand #66

andrey-kuprianov opened this issue Jul 28, 2022 · 0 comments · Fixed by #122
Assignees

Comments

@andrey-kuprianov
Copy link
Contributor

This is the follow-up issue for #27 (and PR #61), as well as for #65, which implement generation, execution, and reporting for single ITF traces. This issue depends on #54 and #55, which implement the necessary programmatic interface from the Model module.

The tasks of this issue are:

This is the simplest and preferable route for Atomkrat prototype.

Alternative route for Atomkraft MVP

The alternative route is to generate another kind of Pytest using @mbt decorator, and refer from it directly to the model being used. But this route is:

  • more complicated, because it will require the modification of @mbt decorator in modelator, in order to adopt the decorator to the changed modelator API, with TOML configs.
  • will not lead to reproducible Python scripts, as the Pytest test will depend on the model, and not on already generated trace.

Implementation of the alternative route will require a more deep integration with Modelator, and implementation of caching there, in order to achieve the speed and reproducibility of the first route. When implemented though, this will be the main and preferable way of Atomkraft operation, because the users will not need to concern themselves with the intermediate phase (ITF traces), if they don't want to. It will be a direct route from a model and a test assertion, to the execution of multiple generated traces against the testnet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants