[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

pydra/engine/specs.py

@@ -162,11 +162,17 @@ def check_fields_input_spec(self):
             mdata = fld.metadata
             # checking if the mandatory field is provided
             if getattr(self, fld.name) is attr.NOTHING:
-                if mdata.get("mandatory"):    
+                if mdata.get("mandatory"):
                     # checking if the mandatory field is provided elsewhere in the xor list
                     in_exclusion_list = mdata.get("xor") is not None
-                    alreday_populated = in_exclusion_list and [getattr(self, el) for el in mdata["xor"] if (getattr(self, el) is not attr.NOTHING)]
-                    if alreday_populated: #another input satisfies mandatory attribute via xor condition
+                    alreday_populated = in_exclusion_list and [
+                        getattr(self, el)
+                        for el in mdata["xor"]
+                        if (getattr(self, el) is not attr.NOTHING)
+                    ]
+                    if (
+                        alreday_populated
+                    ):  # another input satisfies mandatory attribute via xor condition
                         continue
                     else:
                         raise AttributeError(

pydra/engine/tests/test_specs.py

@@ -16,14 +16,12 @@
     LazyField,
     ShellOutSpec,
 )
-from ..task import (
-    TaskBase,
-    ShellCommandTask
-)
+from ..task import TaskBase, ShellCommandTask
 from ..helpers import make_klass
 import pytest
 import attr
 
+
 def test_basespec():
     spec = BaseSpec()
     assert (
@@ -372,40 +370,43 @@ def test_input_file_hash_5(tmpdir):
     hash3 = inputs(in_file=[{"file": file_diffcontent, "int": 3}]).hash
     assert hash1 != hash3
 
+
 class SimpleTask(ShellCommandTask):
-    input_fields=[
-            (
-                "input_1",
-                str,
-                {
-                    "help_string": "help",
-                    "mandatory": True,
-                    "xor": ("input_1", "input_2", "input_3"),
-                }
-            ),
-            (
-                "input_2",
-                bool,
-                {
-                    "help_string": "help",
-                    "mandatory": True,
-                    "argstr": "--i2",
-                    "xor": ("input_1", "input_2", "input_3"),
-                }
-            ),
-            (
-                "input_3",
-                bool,
-                {
-                    "help_string": "help",
-                    "mandatory": True,
-                    "xor": ("input_1", "input_2", "input_3"),
-                }
-            )
+    input_fields = [
+        (
+            "input_1",
+            str,
+            {
+                "help_string": "help",
+                "mandatory": True,
+                "xor": ("input_1", "input_2", "input_3"),
+            },
+        ),
+        (
+            "input_2",
+            bool,
+            {
+                "help_string": "help",
+                "mandatory": True,
+                "argstr": "--i2",
+                "xor": ("input_1", "input_2", "input_3"),
+            },
+        ),
+        (
+            "input_3",
+            bool,
+            {
+                "help_string": "help",
+                "mandatory": True,
+                "xor": ("input_1", "input_2", "input_3"),
+            },
+        ),
     ]
     task_input_spec = SpecInfo(name="Input", fields=input_fields, bases=(ShellSpec,))
     task_output_fields = []
-    task_output_spec = SpecInfo(name="Output", fields=task_output_fields, bases=(ShellOutSpec,))
+    task_output_spec = SpecInfo(
+        name="Output", fields=task_output_fields, bases=(ShellOutSpec,)
+    )
 
     input_spec = task_input_spec
     output_spec = task_output_spec
@@ -415,7 +416,7 @@ class SimpleTask(ShellCommandTask):
 def test_task_inputs_mandatory_with_xOR_one_mandatory_is_OK():
     """input spec with mandatory inputs"""
     task = SimpleTask()
-    task.inputs.input_1 = 'Input1'
+    task.inputs.input_1 = "Input1"
     task.inputs.input_2 = attr.NOTHING
     task.inputs.check_fields_input_spec()
 
@@ -443,23 +444,27 @@ def test_task_inputs_mandatory_with_xOR_zero_mandatory_raises_error():
 def test_task_inputs_mandatory_with_xOR_two_mandatories_raises_error():
     """input spec with mandatory inputs"""
     task = SimpleTask()
-    task.inputs.input_1 = 'Input1'
+    task.inputs.input_1 = "Input1"
     task.inputs.input_2 = True
-    
+
     with pytest.raises(Exception) as excinfo:
         task.inputs.check_fields_input_spec()
-    assert "input_2 is mutually exclusive with ('input_1', 'input_2'" in str(excinfo.value)
+    assert "input_2 is mutually exclusive with ('input_1', 'input_2'" in str(
+        excinfo.value
+    )
     assert excinfo.type is AttributeError
 
 
 def test_task_inputs_mandatory_with_xOR_3_mandatories_raises_error():
     """input spec with mandatory inputs"""
     task = SimpleTask()
-    task.inputs.input_1 = 'Input1'
+    task.inputs.input_1 = "Input1"
     task.inputs.input_2 = True
     task.inputs.input_3 = False
-    
+
     with pytest.raises(Exception) as excinfo:
         task.inputs.check_fields_input_spec()
-    assert "input_2 is mutually exclusive with ('input_1', 'input_2', 'input_3'" in str(excinfo.value)
-    assert excinfo.type is AttributeError
+    assert "input_2 is mutually exclusive with ('input_1', 'input_2', 'input_3'" in str(
+        excinfo.value
+    )
+    assert excinfo.type is AttributeError