Remove metadata checks from arguments

Handling instance bound variables (i.e. 'self') will have to be solved
in the CLI.

girder_worker_utils/decorators.py

@@ -140,21 +140,19 @@ def arguments(self):
         return [
             self._construct_argument(
                 self._get_class(self._signature.parameters[name]), name)
-            for name in self._signature.parameters if name in self._metadata]
+            for name in self._signature.parameters]
 
     @property
     def varargs(self):
         for name in self._signature.parameters:
-            if name in self._metadata and \
-               self._is_varargs(self._signature.parameters[name]):
+            if self._is_varargs(self._signature.parameters[name]):
                 return self._construct_argument(Varargs, name)
         return None
 
     @property
     def kwargs(self):
         for name in self._signature.parameters:
-            if name in self._metadata and \
-               self._is_kwargs(self._signature.parameters[name]):
+            if self._is_kwargs(self._signature.parameters[name]):
                 return self._construct_argument(Kwargs, name)
         return None
 

girder_worker_utils/tests/decorators_test.py

@@ -302,7 +302,7 @@ def test_GWFuncDesc_keyword_args_have_defaults(func, defaults):
         assert hasattr(p, 'default')
         assert p.default == d
 
-
+@pytest.mark.skip("Fix this to use API for accessing argument spec rather than using 'private' attribute")
 def test_parameter_decorator_adds_metadata():
     @parameter('a', test='TEST')
     def arg(a):