ethereum / ethereum/execution-specs

chore(tests): Remove hard-coded gas limits from system contract tests

Open
#3,071 3 comments 0 reactions 0 assignees View on GitHub
stale
Dominant language
Python
Stars
1.2k
Forks
505
Avg merge
2d 14h
Merged PRs (30d)
116

Description

Could we add an issue and try to remove the hardcode value in follow-up PRs?

_Originally posted by @LouisTsai-Csie in https://github.com/ethereum/execution-specs/pull/2989#discussion_r3457025425_

Also fix: https://github.com/ethereum/execution-specs/pull/3070#discussion_r3511427154

Contributor guide

Open the contributing guide

Research direction

Start by locating the system contract tests and their hard-coded gas limits, then review the discussions in PRs 2989 and 3070 for the affected cases. The work is complete when those tests no longer rely on hard-coded gas limits and the test suite passes.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
testing-qa
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
64/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.