change out_file behavior on Zeropad

Author: Zach Lindsey

nipype/interfaces/afni/tests/test_extra_Zeropad.py

@@ -0,0 +1,25 @@
+from pathlib import Path
+from nipype.interfaces.afni import Zeropad
+from nipype.testing.fixtures import create_files_in_directory
+
+
+def test_zeropad_handles_outfile_default(create_files_in_directory):
+    filelist, outdir = create_files_in_directory
+    zp = Zeropad(I=1)
+    zp.inputs.in_files = filelist[0]
+
+    result = zp.run()
+
+    assert (Path(outdir) / "zeropad+tlrc.BRIK").exists()
+    assert Path(result.outputs.out_file).name == "zeropad+tlrc.BRIK"
+
+
+def test_zeropad_handles_outfile_specified_nii_gz(create_files_in_directory):
+    filelist, outdir = create_files_in_directory
+    zp = Zeropad(I=1, out_file="padded.nii.gz")
+    zp.inputs.in_files = filelist[0]
+
+    result = zp.run()
+
+    assert (Path(outdir) / "padded.nii.gz").exists()
+    assert Path(result.outputs.out_file).name == "padded.nii.gz"

nipype/interfaces/afni/utils.py

@@ -3391,7 +3391,6 @@ class ZeropadInputSpec(AFNICommandInputSpec):
         copyfile=False,
     )
     out_file = File(
-        name_template="zeropad",
         desc="output dataset prefix name (default 'zeropad')",
         argstr="-prefix %s",
     )
@@ -3497,3 +3496,11 @@ class Zeropad(AFNICommand):
     _cmd = "3dZeropad"
     input_spec = ZeropadInputSpec
     output_spec = AFNICommandOutputSpec
+
+    def _list_outputs(self):
+        out_file = getattr(self.inputs, "out_file")
+
+        if not isdefined(out_file):
+            out_file = "zeropad+tlrc.BRIK"
+
+        return {"out_file": out_file}