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

Abstract Engine (MiniMina) for integration tests #14621

Open
wants to merge 103 commits into
base: develop
Choose a base branch
from

Conversation

joaosreis
Copy link
Member

This PR introduces the abstract engine for integration testing and general improvements.

The general idea is that the abstract engine should be agnostic to the network runner. It operates by using network runner-specific config files (e.g. the mock config located in ./integration_tests/) and running bash commands based on the config. The stdout of the network runner commands is captured by the abstract engine and parsed into the response types defined in config_util.ml, which are subsequently used to manage Lucy's internal network state. Network runner responses must conform to return_spec.json. Node interactions still proceed via GQL.

The other major departures from the GCP engine are the abstract engine's generation of a topology file to be consumed by the network runner and the usage of pre-generated network and libp2p keys to boost performance.

  • adds abstract engine module in src/lib/integration_test_abstract_engine
  • adds topology file generation
  • adds usage of pre-generated network and libp2p keys
  • adds config and return spec

@joaosreis
Copy link
Member Author

!ci-build-me

@joaosreis joaosreis marked this pull request as ready for review November 28, 2023 11:19
@joaosreis joaosreis requested a review from a team as a code owner November 28, 2023 11:19
@joaosreis joaosreis self-assigned this Nov 28, 2023
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.

4 participants