Revert: Make LightDispatchChain inherit from Chain

This reverts commit 7aaf4b6.

Author: carver

trinity/chains/light.py

@@ -1,4 +1,5 @@
 import asyncio
+import inspect
 from typing import (  # noqa: F401
     Any,
     Optional,
@@ -21,7 +22,12 @@
 )
 
 from evm.chains.base import (
-    Chain,
+    AccountState,
+    BaseChain,
+)
+from evm.db.backends.base import BaseDB
+from evm.db.chain import (
+    BaseChainDB,
 )
 from evm.db.header import (
     BaseHeaderDB,
@@ -31,6 +37,17 @@
 )
 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 (
@@ -41,7 +58,7 @@
     from evm.vm.base import BaseVM  # noqa: F401
 
 
-class LightDispatchChain(Chain):
+class LightDispatchChain(BaseChain):
     """
     Provide the :class:`BaseChain` API, even though only a
     :class:`LightPeerChain` is syncing. Store results locally so that not
@@ -56,6 +73,46 @@ 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)
 
@@ -65,6 +122,15 @@ 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)
@@ -97,6 +163,59 @@ 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
     #