docs: fix grammar in tests/frontier/**/*.py

repo_subpaths_for_grammar_check.md

@@ -9,7 +9,7 @@ Status legend: `[ ]` pending, `[~]` in progress, `[x]` complete
 - [x] `tests/static/**/*.py` (~75 files)
 - [x] `tests/unscheduled/**/*.py` (~73 files)
 - [x] `tests/prague/**/*.py` (~69 files)
-- [ ] `tests/frontier/**/*.py` (~50 files)
+- [x] `tests/frontier/**/*.py` (~50 files)
 - [ ] `tests/cancun/**/*.py` (~44 files)
 - [ ] `tests/osaka/**/*.py` (~43 files)
 - [ ] `tests/benchmark/**/*.py` (~36 files)

tests/frontier/create/test_create_deposit_oog.py

@@ -1,5 +1,5 @@
 """
-The test CREATE's behavior when running out of gas for code deposit.
+Test CREATE's behavior when running out of gas for code deposit.
 """
 
 import pytest

tests/frontier/create/test_create_one_byte.py

@@ -1,6 +1,6 @@
 """
-The test calls CREATE in a loop deploying 1-byte contracts with all possible
-byte values, records in storage the values that failed to deploy.
+Test calling CREATE in a loop to deploy 1-byte contracts with all possible
+byte values, recording in storage the values that failed to deploy.
 """
 
 import pytest

tests/frontier/create/test_create_suicide_store.py

@@ -1,6 +1,6 @@
 """
-Test dynamically created address is still callable and perform storage
-operations after being called for self destruct in a call.
+Test that a dynamically created address is still callable and can perform
+storage operations after being called for selfdestruct.
 """
 
 from enum import IntEnum
@@ -49,8 +49,8 @@ def test_create_suicide_store(
     create_opcode: Op,
 ) -> None:
     """
-    Create dynamic contract that suicides, then called to push some storage
-    and then called to return that storage value.
+    Create a dynamic contract that self-destructs, then call it to store some
+    data and then call it again to return that storage value.
     """
     tload_support = fork.valid_opcodes().count(Op.TLOAD)
     subcall_storage = 0x12

tests/frontier/scenarios/common.py

@@ -37,9 +37,10 @@ class ScenarioExpectOpcode(Enum):
 @dataclass
 class ScenarioEnvironment:
     """
-    Scenario evm environment Each scenario must define an environment on which
-    program is executed This is so post state verification could check results
-    of evm opcodes.
+    Scenario EVM environment.
+
+    Each scenario must define an environment on which the program is executed.
+    This is so post state verification can check the results of EVM opcodes.
     """
 
     code_address: Address  # Op.ADDRESS, address scope for program
@@ -247,8 +248,9 @@ def make_gas_hash_contract(pre: Alloc) -> Address:
 
 def make_invalid_opcode_contract(pre: Alloc, fork: Fork) -> Address:
     """
-    Deploy a contract that will execute any asked byte as an opcode from
-    calldataload Deploy 20 empty stack elements. Jump to opcode instruction. if
+    Deploy a contract that will execute any asked byte as an opcode.
+
+    Deploy 20 empty stack elements. Jump to opcode instruction. If it
     worked, return 0.
     """
     invalid_opcode_caller = pre.deploy_contract(

tests/frontier/validation/test_header.py

@@ -28,7 +28,7 @@ def test_gas_limit_below_minimum(
     env: Environment,
 ) -> None:
     """
-    Tests that a block with a gas limit below limit throws an error.
+    Tests that a block with a gas limit below the minimum throws an error.
     """
     modified_fields = {"gas_limit": gas_limit}
     env.gas_limit = ZeroPaddedHexNumber(5000)

tests/frontier/validation/test_transaction.py

@@ -21,7 +21,7 @@ def test_tx_gas_limit(
     env: Environment,
 ) -> None:
     """
-    Tests that if a tx gas limit is higher than the block with a gas limit,
+    Tests that if a tx gas limit is higher than the block gas limit,
     an exception is raised.
     """
     sender = pre.fund_eoa()