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

Introduce end-to-end testing crate #1557

Merged
merged 14 commits into from
Feb 18, 2025
Merged

Introduce end-to-end testing crate #1557

merged 14 commits into from
Feb 18, 2025

Conversation

agryaznov
Copy link
Contributor

This adds crate to be used as our integration testing framework.

Currently we have a smoke test which basically does the same as tc-cli smoke-test 2 3 but in the form of a normal integration test. Just run:

cargo test --tests smoke

and it will

  1. Spawn the environment, starting all the needed containers.
  2. Run the test.
  3. Tear down the environment, stopping all the containers.

Next thing to be done is to add GMP assets bridge e2e test.

@agryaznov agryaznov requested a review from dvc94ch February 17, 2025 20:16
@agryaznov
Copy link
Contributor Author

@penumbra23 can we add Foundry to the GHA runner? We need forge for the integration test.

@@ -7,6 +7,7 @@ concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: true
jobs:
if: false
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@penumbra23 I disable this one because this PR introduces integration test which does the same

@agryaznov
Copy link
Contributor Author

Last left failing CI job has nothing to do with the changes introduced in this pr and has been around for a while. It's (hopefully) going to be fixed with #1538

@agryaznov agryaznov merged commit d10c12b into development Feb 18, 2025
12 of 13 checks passed
@agryaznov agryaznov deleted the ag/e2e-tests branch February 18, 2025 22:16
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 this pull request may close these issues.

2 participants