Closed anhductn2001 closed 1 month ago
[!WARNING]
Rate limit exceeded
@anhductn2001 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 0 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 10a6bd7dfce2f527b721218c10f67fcc2995dbe9 and e12fad910c277761e5959bed865c40dff2a65d2e.
The recent changes primarily focus on updating version numbers in the README.md
file and modifying test assertions across multiple test files to include the subtraction of a bridgingFee
from balance calculations. This ensures more accurate and clear error messages. Additionally, new declarations for amt
and bridgingFee
are added in tests/setup.go
to standardize fee calculations across tests.
File(s) | Change Summary |
---|---|
README.md |
Updated version matrix for E2E Tests, Dymension, Rollapp-EVM, Rollapp-Wasm, and Relayer. |
tests/eibc_ack_error_test.go |
Updated assertion messages to include bridgingFee in balance calculations. |
tests/eibc_corrupted_memo_test.go |
Adjusted balance assertion to subtract bridgingFee from transferAmount . |
tests/eibc_fulfillment_test.go |
Updated error messages in assertions to include bridgingFee in expected values. |
tests/eibc_invariant_test.go |
Updated error messages in assertions to include bridgingFee in expected values. |
tests/eibc_nobalance_test.go |
Adjusted balance verification to subtract bridgingFee from the transferred amount. |
tests/eibc_not_fulfillment_test.go |
Updated error message formatting in assertions. |
tests/ibc_grace_period_test.go |
Updated balance assertion logic to subtract bridgingFee from transferAmount in various test cases. |
tests/ibc_pfm_with_grace_period_test.go |
Modified balance comparison to subtract bridgingFee from the transfer amount. |
tests/ibc_transfer_test.go |
Adjusted balance assertion after a timeout to subtract bridgingFee from the transferred amount. |
tests/rollapp_genesis_event_test.go |
Changed balance assertion logic to account for fees deducted from the transfer amount. |
tests/rollapp_upgrade_test.go |
Changed balance calculations in multiple test functions to include bridgingFee . |
tests/setup.go |
Added declarations for amt and bridgingFee to standardize fee calculations across tests. |
Amidst the code, where tests align,
A bridging fee now marks the sign.
Balances checked with careful eye,
Ensuring numbers never lie.
Versions updated, tests refined,
In the realm of code, progress shines.
🐰✨
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?
Summary by CodeRabbit
Documentation
Tests
bridgingFee
deductions.Chores
amt
andbridgingFee
in test setup.