Remove default value for defaults

This is a dangerous practice becuse the default values could be mutated. And it should always be passed in.
Supercedes #286

src/rocker/core.py

@@ -117,7 +117,7 @@ def check_args_for_activation(cls, cli_args):
         return True if cli_args.get(cls.get_name()) else False
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         raise NotImplementedError
 
 

src/rocker/extensions.py

@@ -54,7 +54,7 @@ def get_docker_args(self, cliargs):
         return args
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument('--devices',
             default=defaults.get('devices', None),
             nargs='*',
@@ -84,7 +84,7 @@ def get_snippet(self, cliargs):
         return empy_expand(snippet, self.get_environment_subs())
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument(name_to_argument(DevHelpers.get_name()),
             action='store_true',
             default=defaults.get('dev_helpers', None),
@@ -110,7 +110,7 @@ def get_docker_args(self, cliargs):
         return args
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument('--hostname', default=defaults.get('hostname', ''),
                             help='Hostname of the container.')
 
@@ -133,7 +133,7 @@ def get_docker_args(self, cliargs):
         return args
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument('--name', default=defaults.get('name', ''),
                             help='Name of the container.')
 
@@ -156,7 +156,7 @@ def get_docker_args(self, cliargs):
         return args
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         client = get_docker_client()
         parser.add_argument('--network', choices=[n['Name'] for n in client.networks()],
             default=defaults.get('network', None),
@@ -182,7 +182,7 @@ def get_docker_args(self, cliargs):
         return ' '.join(args)
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument('--expose',
             default=defaults.get('expose', None),
             action='append',
@@ -208,7 +208,7 @@ def get_docker_args(self, cliargs):
         return ' '.join(args)
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument('--port',
             default=defaults.get('port', None),
             action='append',
@@ -246,7 +246,7 @@ def get_docker_args(self, cliargs):
         return args % self.get_environment_subs()
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument(name_to_argument(PulseAudio.get_name()),
             action='store_true',
             default=defaults.get(PulseAudio.get_name(), None),
@@ -265,7 +265,7 @@ def get_docker_args(self, cliargs):
         return ' -v %s:%s ' % (Path.home(), Path.home())
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument(name_to_argument(HomeDir.get_name()),
             action='store_true',
             default=defaults.get(HomeDir.get_name(), None),
@@ -321,7 +321,7 @@ def get_snippet(self, cliargs):
         return empy_expand(snippet, substitutions)
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument(name_to_argument(User.get_name()),
             action='store_true',
             default=defaults.get('user', None),
@@ -377,7 +377,7 @@ def get_docker_args(self, cli_args):
         return ' '.join(args)
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument('--env', '-e',
             metavar='NAME[=VALUE]',
             type=str,
@@ -413,7 +413,7 @@ def get_docker_args(self, cli_args):
         return ' --privileged'
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument(name_to_argument(Privileged.get_name()),
                             action='store_true',
                             default=defaults.get(Privileged.get_name(), None),
@@ -440,7 +440,7 @@ def get_docker_args(self, cliargs):
         return ' '.join(args)
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument(name_to_argument(GroupAdd.get_name()),
             default=defaults.get(GroupAdd.get_name(), None),
             action='append',

src/rocker/git_extension.py

@@ -46,7 +46,7 @@ def get_docker_args(self, cli_args):
         return args
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument('--git',
             action='store_true',
             default=defaults.get(Git.get_name(), None),

src/rocker/nvidia_extension.py

@@ -71,7 +71,7 @@ def precondition_environment(self, cliargs):
             raise ex
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument(name_to_argument(X11.get_name()),
             action='store_true',
             default=defaults.get(X11.get_name(), None),
@@ -134,7 +134,7 @@ def get_docker_args(self, cliargs):
         return "  --runtime=nvidia"
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument(name_to_argument(Nvidia.get_name()),
             choices=['auto', 'runtime', 'gpus'],
             nargs='?',
@@ -196,7 +196,7 @@ def get_docker_args(self, cliargs):
         # Runtime requires --nvidia option too
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument(name_to_argument(Cuda.get_name()),
             action='store_true',
             default=defaults.get('cuda', None),

src/rocker/rmw_extension.py

@@ -72,7 +72,7 @@ def get_snippet(self, cliargs):
         return empy_expand(snippet, data)
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument(name_to_argument(RMW.get_name()),
             default=defaults.get('rmw', None),
             nargs=1,

src/rocker/ssh_extension.py

@@ -46,7 +46,7 @@ def get_docker_args(self, cli_args):
         return args
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument('--ssh',
             action='store_true',
             default=defaults.get(Ssh.get_name(), None),

src/rocker/volume_extension.py

@@ -61,7 +61,7 @@ def get_docker_args(self, cli_args):
         return ' '.join(args)
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument(Volume.ARG_ROCKER_VOLUME,
             metavar='HOST-DIR[:CONTAINER-DIR[:OPTIONS]]',
             type=str,

test/test_file_writing.py

@@ -58,7 +58,7 @@ def get_files(self, cliargs):
 
 
     @staticmethod
-    def register_arguments(parser, defaults={}):
+    def register_arguments(parser, defaults):
         parser.add_argument('--test-file-injection',
             action='store_true',
             default=defaults.get('test_file_injection', False),