Add tests for chainspec change

chainlib/cli/arg.py

@@ -57,7 +57,7 @@ class ArgumentParser(argparse.ArgumentParser):
         self.pos_args = []
 
 
-    def add_positional(self, name, type=str, help=None, append=False, required=True):
+    def add_positional(self, name, type=str, help=None, required=True):
         """Add a positional argument.
 
         Stdin piping will only be possible in the event a single positional argument is defined.
@@ -73,7 +73,7 @@ class ArgumentParser(argparse.ArgumentParser):
         :param required: If true, argument will be set to required
         :type required: bool
         """
-        self.pos_args.append((name, type, help, required, append,))
+        self.pos_args.append((name, type, help, required,))
 
 
     def parse_args(self, argv=sys.argv[1:]):
@@ -88,23 +88,13 @@ class ArgumentParser(argparse.ArgumentParser):
         """
         if len(self.pos_args) == 1:
             arg = self.pos_args[0]
-            if arg[4]:
-                self.add_argument(arg[0], nargs='*', type=arg[1], default=stdin_arg(), help=arg[2])
-            else:
-                self.add_argument(arg[0], nargs='?', type=arg[1], default=stdin_arg(), help=arg[2])
+            self.add_argument(arg[0], nargs='?', type=arg[1], default=stdin_arg(), help=arg[2])
         else:
             for arg in self.pos_args:
                 if arg[3]:
-                    if arg[4]:
-                        logg.debug('argumen')
-                        self.add_argument(arg[0], nargs='+', type=arg[1], help=arg[2])
-                    else:
-                        self.add_argument(arg[0], type=arg[1], help=arg[2])
+                    self.add_argument(arg[0], type=arg[1], help=arg[2])
                 else:
-                    if arg[4]:
-                        self.add_argument(arg[0], nargs='*', type=arg[1], help=arg[2])
-                    else:
-                        self.add_argument(arg[0], type=arg[1], help=arg[2])
+                    self.add_argument(arg[0], nargs='?', type=arg[1], help=arg[2])
         args = super(ArgumentParser, self).parse_args(args=argv)
 
         if args.dumpconfig:

chainlib/cli/wallet.py

@@ -1,6 +1,9 @@
 # standard imports
 import logging
 
+# external imports
+from crypto_dev_signer.keystore.dict import DictKeystore
+
 logg = logging.getLogger(__name__)
 
 
@@ -16,7 +19,7 @@ class Wallet:
     :todo: sign_transaction_to_rlp from chainlib-eth must be renamed to sign_transaction_to_wire, and included as part of signer interface
     """
     
-    def __init__(self, signer_cls, keystore=None, checksummer=None):
+    def __init__(self, signer_cls, keystore=DictKeystore(), checksummer=None):
         self.signer_constructor = signer_cls
         self.keystore = keystore
         self.signer = None

chainlib/connection.py

@@ -23,10 +23,7 @@ from .jsonrpc import (
         ErrorParser,
         )
 from .http import PreemptiveBasicAuthHandler
-from .error import (
-        JSONRPCException,
-        RPCException,
-        )
+from .error import JSONRPCException
 from .auth import Auth
 
 logg = logging.getLogger(__name__)
@@ -311,11 +308,8 @@ class JSONRPCHTTPConnection(HTTPConnection):
                     )
             ho = build_opener(handler)
             install_opener(ho)
-
-        try: 
-            r = urlopen(req, data=data.encode('utf-8'))
-        except URLError as e:
-            raise RPCException(e)
+       
+        r = urlopen(req, data=data.encode('utf-8'))
 
         result = json.load(r)
         logg.debug('(HTTP) recv {}'.format(result))

requirements.txt

@@ -1,3 +1,3 @@
-funga>=0.5.1a1,<0.6.0
+crypto-dev-signer>=0.4.15rc2,<=0.4.15
 pysha3==1.0.2
 hexathon~=0.0.1a8

setup.cfg

@@ -1,6 +1,6 @@
 [metadata]
 name = chainlib
-version = 0.0.10a6
+version = 0.0.10a1
 description = Generic blockchain access library and tooling
 author = Louis Holbrook
 author_email = dev@holbrook.no