Closed m-Peter closed 2 months ago
[!WARNING]
Rate limit exceeded
@sideninja has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 17 minutes and 50 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between 1ec615135195ac5d7f155861115ada4325888430 and 4d880805735e2b464542f7c1c7d3fe5cec38f5dc.
The recent changes implement a configurable minimum gas price across various test files, moving away from a hardcoded gas price of 0
. This update enhances the realism of end-to-end tests by simulating more accurate transaction costs, significantly impacting test outcomes. Additionally, improvements in code readability and maintainability were made by formatting numeric literals and ensuring consistency in parameter usage.
Files | Change Summary |
---|---|
tests/helpers.go, tests/web3js/config.js | Updated GasPrice from 0 to 150 ; added minGasPrice parameter to the configuration. |
tests/web3js/*.js (multiple files) | Replaced hardcoded gasPrice of 0 with conf.minGasPrice for enhanced configurability; improved numeric formatting. |
tests/web3js/eth_transfer_between_eoa_accounts_test.js | Modified balance assertions for clearer validation after transfers. |
tests/web3js/helpers.js | Changed gasPrice in deployContract to use conf.minGasPrice , enhancing deployment functionality. |
sequenceDiagram
participant Test as Test Script
participant Config as Config Module
participant Network as Ethereum Network
Test->>Config: Retrieve minGasPrice
Config-->>Test: Return minGasPrice
Test->>Network: Send transaction with gasPrice=minGasPrice
Network-->>Test: Return transaction result
Objective | Addressed | Explanation |
---|---|---|
Update E2E tests to run with a configured minimum gas price (#[393]) | ✅ |
🐰 In the meadow where the bunnies play,
A gas price rises like the sun's bright ray.
Configured anew, our tests will now gleam,
With realistic costs, they'll flow like a dream!
Hop along, dear friends, to a world more fair,
Where transactions dance without a single care! 🌼✨
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: https://github.com/onflow/flow-evm-gateway/issues/393
Description
Make sure that our test suite is as close as possible to real-world setups.
For contributor use:
master
branchFiles changed
in the Github PR explorerSummary by CodeRabbit
New Features
Bug Fixes
Style