Skip to content

refactor(tests): Proposed patch for bloatnet SSTORE tests - #1

Merged
gballet merged 3 commits into
gballet:bloatnet-test-SSTOREfrom
fselmo:patch-bloatnet-sstore
Sep 2, 2025
Merged

refactor(tests): Proposed patch for bloatnet SSTORE tests#1
gballet merged 3 commits into
gballet:bloatnet-test-SSTOREfrom
fselmo:patch-bloatnet-sstore

Conversation

@fselmo

@fselmo fselmo commented Aug 29, 2025

Copy link
Copy Markdown

Proposed patch from review on ethereum#2040

@gballet

gballet commented Aug 29, 2025

Copy link
Copy Markdown
Owner

ah just saw that, it didn't appear on mobile :(

Comment thread tests/benchmark/test_bloatnet.py Outdated
Comment thread tests/benchmark/test_bloatnet.py Outdated
Comment thread tests/benchmark/test_bloatnet.py Outdated
Comment thread tests/benchmark/test_bloatnet.py
Comment thread tests/benchmark/test_bloatnet.py
Comment thread tests/benchmark/test_bloatnet.py
@fselmo
fselmo force-pushed the patch-bloatnet-sstore branch from 307ed10 to 5273df9 Compare August 29, 2025 15:57
Comment thread tests/benchmark/test_bloatnet.py Outdated
Comment thread tests/benchmark/test_bloatnet.py Outdated
fselmo added a commit to fselmo/execution-spec-tests that referenced this pull request Sep 1, 2025
fselmo added a commit to fselmo/execution-spec-tests that referenced this pull request Sep 1, 2025
PR: gballet#1
Signed-off-by: fselmo <fselmo2@gmail.com>

@LouisTsai-Csie LouisTsai-Csie left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread tests/benchmark/test_bloatnet.py
Comment thread tests/benchmark/test_bloatnet.py Outdated
Comment thread tests/benchmark/test_bloatnet.py Outdated
Comment thread tests/benchmark/test_bloatnet.py Outdated
Comment thread tests/benchmark/test_bloatnet.py Outdated
@gballet
gballet merged commit b3aa527 into gballet:bloatnet-test-SSTORE Sep 2, 2025
@fselmo
fselmo deleted the patch-bloatnet-sstore branch September 2, 2025 15:01
@fselmo

fselmo commented Sep 2, 2025

Copy link
Copy Markdown
Author

@gballet thanks for making the changes and cleaning it up 👌🏼

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants