Make LightDispatchChain inherit from Chain

Author: cburgdorf

trinity/chains/light.py

@@ -1,5 +1,4 @@
 import asyncio
-import inspect
 from typing import (  # noqa: F401
     Any,
     Optional,
@@ -22,12 +21,7 @@
 )
 
 from evm.chains.base import (
-    AccountState,
-    BaseChain,
-)
-from evm.db.backends.base import BaseDB
-from evm.db.chain import (
-    BaseChainDB,
+    Chain,
 )
 from evm.db.header import (
     BaseHeaderDB,
@@ -37,17 +31,6 @@
 )
 from evm.rlp.headers import (
     BlockHeader,
-    HeaderParams,
-)
-from evm.rlp.receipts import (
-    Receipt
-)
-from evm.rlp.transactions import (
-    BaseTransaction,
-    BaseUnsignedTransaction,
-)
-from evm.vm.computation import (
-    BaseComputation
 )
 
 from p2p.lightchain import (
@@ -58,7 +41,7 @@
     from evm.vm.base import BaseVM  # noqa: F401
 
 
-class LightDispatchChain(BaseChain):
+class LightDispatchChain(Chain):
     """
     Provide the :class:`BaseChain` API, even though only a
     :class:`LightPeerChain` is syncing. Store results locally so that not
@@ -73,46 +56,6 @@ def __init__(self, headerdb: BaseHeaderDB, peer_chain: LightPeerChain) -> None:
         self._peer_chain = peer_chain
         self._peer_chain_loop = asyncio.get_event_loop()
 
-    #
-    # Helpers
-    #
-    @classmethod
-    def get_chaindb_class(cls) -> Type[BaseChainDB]:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    #
-    # Chain API
-    #
-    @classmethod
-    def from_genesis(cls,
-                     base_db: BaseDB,
-                     genesis_params: Dict[str, HeaderParams],
-                     genesis_state: AccountState=None) -> 'BaseChain':
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    @classmethod
-    def from_genesis_header(cls,
-                            base_db: BaseDB,
-                            genesis_header: BlockHeader) -> 'BaseChain':
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    def get_chain_at_block_parent(self, block: BaseBlock) -> 'BaseChain':
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    #
-    # VM API
-    #
-    def get_vm(self, header: BlockHeader=None) -> 'BaseVM':
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    #
-    # Header API
-    #
-    def create_header_from_parent(self,
-                                  parent_header: BlockHeader,
-                                  **header_params: HeaderParams) -> BlockHeader:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
     def get_block_header_by_hash(self, block_hash: Hash32) -> BlockHeader:
         return self._headerdb.get_block_header_by_hash(block_hash)
 
@@ -122,15 +65,6 @@ def get_canonical_head(self) -> BlockHeader:
     def get_score(self, block_hash: Hash32) -> int:
         return self._headerdb.get_score(block_hash)
 
-    #
-    # Block API
-    #
-    def get_ancestors(self, limit: int, header: BlockHeader=None) -> Iterator[BaseBlock]:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    def get_block(self) -> BaseBlock:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
     def get_block_by_hash(self, block_hash: Hash32) -> BaseBlock:
         header = self._headerdb.get_block_header_by_hash(block_hash)
         return self.get_block_by_header(header)
@@ -163,59 +97,6 @@ def get_canonical_block_by_number(self, block_number: BlockNumber) -> BaseBlock:
     def get_canonical_block_hash(self, block_number: int) -> Hash32:
         return self._headerdb.get_canonical_block_hash(block_number)
 
-    def build_block_with_transactions(
-            self, transactions: Tuple[BaseTransaction, ...], parent_header: BlockHeader) -> None:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    #
-    # Transaction API
-    #
-    def create_transaction(self, *args: Any, **kwargs: Any) -> BaseTransaction:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    def create_unsigned_transaction(self,
-                                    *args: Any,
-                                    **kwargs: Any) -> BaseUnsignedTransaction:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    def get_canonical_transaction(self, transaction_hash: Hash32) -> BaseTransaction:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    #
-    # Execution API
-    #
-    def apply_transaction(
-            self,
-            transaction: BaseTransaction) -> Tuple[BaseBlock, Receipt, BaseComputation]:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    def estimate_gas(self, transaction: BaseTransaction, at_header: BlockHeader=None) -> int:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    def import_block(self, block: BaseBlock, perform_validation: bool=True) -> BaseBlock:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    def mine_block(self, *args: Any, **kwargs: Any) -> BaseBlock:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    #
-    # Validation API
-    #
-    def validate_block(self, block: BaseBlock) -> None:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    def validate_gaslimit(self, header: BlockHeader) -> None:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    def validate_seal(self, header: BlockHeader) -> None:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    def validate_uncles(self, block: BaseBlock) -> None:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
-    def validate_chain(self, chain: Tuple[BlockHeader, ...]) -> None:
-        raise NotImplementedError("Chain classes must implement " + inspect.stack()[0][3])
-
     #
     # Async utils
     #