Fix old usages of BlockHeader() and .from_parent()

Direct invocation of the header object is not allowed anymore, because
different VMs now have different header types (as of EIP-1559 in
London). So we must always invoke header creation through the VM.

Remove the BlockHeader() init in test_blockchain.

Removed HeaderChain, which seems to be completely unused in the code
base.

Ugly fix for header db, because it only tests chains that are
London-style header. Should test a variety of configs.

Author: carver

eth/constants.py

@@ -161,6 +161,9 @@
 GENESIS_NONCE = b'\x00\x00\x00\x00\x00\x00\x00B'  # 0x42 encoded as big-endian-integer
 GENESIS_MIX_HASH = ZERO_HASH32
 GENESIS_EXTRA_DATA = b''
+GENESIS_BLOOM = 0
+GENESIS_GAS_USED = 0
+
 #
 # Sha3 Keccak
 #

eth/tools/fixtures/__init__.py

@@ -8,6 +8,7 @@
 )
 from .helpers import (  # noqa: F401
     new_chain_from_fixture,
+    genesis_fields_from_fixture,
     genesis_params_from_fixture,
     apply_fixture_block_to_chain,
     setup_state,

eth/tools/fixtures/helpers.py

@@ -24,6 +24,7 @@
     StateAPI,
     VirtualMachineAPI,
 )
+from eth import constants
 from eth.db.atomic import AtomicDB
 from eth.chains.mainnet import (
     MainnetDAOValidatorVM,
@@ -165,7 +166,11 @@ def chain_vm_configuration(fixture: Dict[str, Any]) -> Iterable[Tuple[int, Type[
         raise ValueError(f"Network {network} does not match any known VM rules")
 
 
-def genesis_params_from_fixture(fixture: Dict[str, Any]) -> Dict[str, Any]:
+def genesis_fields_from_fixture(fixture: Dict[str, Any]) -> Dict[str, Any]:
+    """
+    Convert all genesis fields in a fixture to a dictionary of header fields and values.
+    """
+
     return {
         'parent_hash': fixture['genesisBlockHeader']['parentHash'],
         'uncles_hash': fixture['genesisBlockHeader']['uncleHash'],
@@ -185,6 +190,34 @@ def genesis_params_from_fixture(fixture: Dict[str, Any]) -> Dict[str, Any]:
     }
 
 
+def genesis_params_from_fixture(fixture: Dict[str, Any]) -> Dict[str, Any]:
+    """
+    Convert a genesis fixture into a dict of the configurable header fields and values.
+
+    Some fields cannot be explicitly set when creating a new header, like
+    parent_hash, which is automatically set to the empty hash.
+    """
+
+    params = genesis_fields_from_fixture(fixture)
+
+    # Confirm that (currently) non-configurable defaults are set correctly,
+    #   then remove them because they cannot be configured on the header.
+    defaults = (
+        ('parent_hash', constants.GENESIS_PARENT_HASH),
+        ('uncles_hash', constants.EMPTY_UNCLE_HASH),
+        ('bloom', constants.GENESIS_BLOOM),
+        ('block_number', constants.GENESIS_BLOCK_NUMBER),
+        ('gas_used', constants.GENESIS_GAS_USED),
+    )
+
+    for key, default_val in defaults:
+        supplied_val = params.pop(key)
+        if supplied_val != default_val:
+            raise ValueError(f"Unexpected genesis {key}: {supplied_val}, expected: {default_val}")
+
+    return params
+
+
 def new_chain_from_fixture(fixture: Dict[str, Any],
                            chain_cls: Type[ChainAPI] = MainnetChain) -> ChainAPI:
     base_db = AtomicDB()

eth/tools/rlp.py

@@ -8,11 +8,6 @@
 )
 
 
-assert_imported_genesis_header_unchanged = replace_exceptions({
-    ValidationError: AssertionError,
-})(validate_rlp_equal(obj_a_name='genesis header', obj_b_name='imported header'))
-
-
 assert_mined_block_unchanged = replace_exceptions({
     ValidationError: AssertionError,
 })(validate_rlp_equal(obj_a_name='provided block', obj_b_name='executed block'))