RM bad test case from writeReport, unset gas limit is allowed and def…#22343
RM bad test case from writeReport, unset gas limit is allowed and def…#22343
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
👋 ilija42, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
|
✅ No conflicts with other open PRs targeting |
There was a problem hiding this comment.
Pull request overview
Risk Rating: LOW
This PR updates the CRE EVM v2 regression system tests to remove a WriteReport negative test case that no longer matches the intended behavior (unset/zero gas limit being allowed and defaulting to a maximum).
Changes:
- Removed the unused
expectedWriteReportInvalidGasconstant. - Removed the
WriteReport - invalid gasnegative test case for"0"(zero gas), leaving only the clearly-invalid “too low” and “too high” cases.
|
|





…aults to max
Requires
Supports