Closed staheri14 closed 1 week ago
The CI failures are irrelevant to this PR and are due to #3592 which hopefully get fixed by #3594
The recent changes in benchmark.go
focus on resolving the issue of transaction clients failing to start in e2e benchmark tests. Specifically, the functionality that initiated transaction clients after the testnet setup has been removed. This change addresses the state management issue, ensuring that transaction clients are only handled in valid states.
File | Change Summary |
---|---|
test/e2e/benchmark/benchmark.go |
Removed code that started transaction clients post testnet setup to address state mismatch errors. This ensures transaction clients are not erroneously initiated. |
Objective (Issue #) | Addressed | Explanation |
---|---|---|
Fix txsim start failure in benchmark tests (#3589) | ✅ |
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Closes #3589 Remnant of #3505
With the changes in this PR, the benchamrk test runs successfully.