-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Fixed flakes and added parallelization to ChainComponents tests #15537
Conversation
dff9e0e
to
0ba8baa
Compare
AER Report: CI Coreaer_workflow , commit , Detect Changes , Clean Go Tidy & Generate , Scheduled Run Frequency , Flakeguard Root Project / Get Tests To Run , GolangCI Lint (core/scripts) , Core Tests (go_core_tests) , GolangCI Lint (.) , test-scripts , Core Tests (go_core_tests_integration) , GolangCI Lint (integration-tests) , Core Tests (go_core_ccip_deployment_tests) , GolangCI Lint (deployment) , Core Tests (go_core_fuzz) , Core Tests (go_core_race_tests) , Flakeguard Deployment Project / Get Tests To Run , Flakeguard Root Project / Run Tests , Flakeguard Root Project / Report , Flakeguard Deployment Project / Run Tests , Flakeguard Deployment Project / Report , lint , SonarQube Scan , Flakey Test Detection 1. File not found error during SonarScanner execution:[SonarQube Scan]Source of Error:
Why: The error occurred because the SonarScanner attempted to access a file ( Suggested fix: Ensure that the AER Report: Operator UI CI ran successfully ✅ |
b4c5545
to
f12df7a
Compare
536d752
to
146dd9f
Compare
Flakeguard SummaryRan new or updated tests between View Flaky Detector Details | Compare Changes Found Flaky Tests ❌
ArtifactsFor detailed logs of the failed tests, please refer to the artifact failed-test-results-with-logs.json. |
96ccc8a
to
41b47e6
Compare
ba0f7a2
to
a1f48db
Compare
t.Run("RunChainComponentsEvmTests", func(t *testing.T) { | ||
t.Parallel() | ||
it := &EVMChainComponentsInterfaceTester[*testing.T]{Helper: helper, DeployLock: &deployLock} | ||
// These tests are broken in develop as well, so disable them for now |
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.
CC @poopoothegorilla Can you create an epic for general Chain Components work that is not urgent, but has to be done? Maybe just have 1 for now and then have them per chain
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.
Created this epic https://smartcontract-it.atlassian.net/browse/NONEVM-1084 used the word "Hardening" which some other projects use for this type of work
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.
Good job!
6450e32
to
ae33309
Compare
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.
great job with this! 💪
ae33309
to
30edf9d
Compare
Quality Gate passedIssues Measures |
30edf9d
to
7d11d5a
Compare
This PR changes the ChainComponents EVM testing interface to support parallel tests, enabling isolated test state and removing potential race conditions.
Local verification to confirm the tests are flake free
Total runs: 500
Failures: 0
Pass rate: 100.00%
Average test time: 3m11.98371491s
Total time: 26h39m51.85745504s
Requires
Supports