Skip to content

Add integration tests - #205

Merged
pablin-10 merged 7 commits into
mainfrom
pablo/integration_tests
Sep 7, 2026
Merged

Add integration tests#205
pablin-10 merged 7 commits into
mainfrom
pablo/integration_tests

Conversation

@pablin-10

Copy link
Copy Markdown
Contributor

No description provided.

@pablin-10
pablin-10 requested review from a team as code owners September 4, 2026 05:11

@ameba23 ameba23 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⭐ Great to have this as i have been checking this manually

Maybe put a note somewhere explaining that this test can be run manually from https://github.com/flashbots/devops/blob/main/.github/workflows/flashbox_integration_tests.yaml as i would not have guessed that if i didn't know.

Comment thread tests/integration/conftest.py Outdated
@pablin-10
pablin-10 force-pushed the pablo/integration_tests branch from 3836768 to 2c4d0c8 Compare September 4, 2026 18:27
@alexhulbert

alexhulbert commented Sep 4, 2026

Copy link
Copy Markdown
Member

lgtm! maybe these should be in the devops repo colocated with the yaml file that runs them? i have a half finished separate repo that does a whole bunch more integration testing for other repos like attest. eventually, i think all this stuff will go in there. but for now we can just keep it with devops and migrate to a separate organization-wide integration testing suite repo later

@pablin-10
pablin-10 merged commit 655d7cf into main Sep 7, 2026
@pablin-10
pablin-10 deleted the pablo/integration_tests branch September 7, 2026 13:17
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.

4 participants