Skip to content

Commit ec2dcb3

Browse files
danceratopzclaude
andcommitted
docs: fix grammar in tests/shanghai (#46)
Fix incomplete sentence in TestMultipleWithdrawalsSameAddress docstring. The sentence "Test that multiple withdrawals can be sent to the same address in." was missing proper structure. Rewrote following the pattern used by TestUseValueInTx with self-contained summary and description. Co-authored-by: Claude <noreply@anthropic.com>
1 parent 4f24599 commit ec2dcb3

2 files changed

Lines changed: 6 additions & 4 deletions

File tree

repo_subpaths_for_grammar_check.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ Status legend: `[ ]` pending, `[~]` in progress, `[x]` complete
1414
- [x] `tests/osaka/**/*.py` (~43 files)
1515
- [x] `tests/benchmark/**/*.py` (~36 files)
1616
- [x] `tests/json_infra/**/*.py` (~22 files)
17-
- [ ] `tests/shanghai/**/*.py` (~19 files)
17+
- [x] `tests/shanghai/**/*.py` (~19 files)
1818
- [ ] `tests/byzantium/**/*.py` (~11 files)
1919
- [ ] `tests/istanbul/**/*.py`, `berlin/**/*.py` (~18 files)
2020
- [ ] `tests/constantinople/**/*.py`, `amsterdam/**/*.py` (~16 files)

tests/shanghai/eip4895_withdrawals/test_withdrawals.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -300,11 +300,13 @@ def test_withdrawals_root(
300300
@pytest.mark.parametrize("test_case", ["single_block", "multiple_blocks"])
301301
class TestMultipleWithdrawalsSameAddress:
302302
"""
303-
Test that multiple withdrawals can be sent to the same address in.
303+
Test that multiple withdrawals can be sent to the same address.
304304
305-
1. A single block.
305+
1. `single_block`: Test multiple withdrawals to the same address in a
306+
single block.
306307
307-
2. Multiple blocks.
308+
2. `multiple_blocks`: Test multiple withdrawals to the same address across
309+
multiple blocks.
308310
"""
309311

310312
@pytest.fixture

0 commit comments

Comments
 (0)