STY: Apply ruff/pycodestyle rule PLW1508

PLW1508 Invalid type for environment variable default; expected `str` or `None`

Author: DimitriPapadopoulos

niworkflows/interfaces/tests/test_plotting.py

@@ -34,7 +34,7 @@
 
 def test_cifti_carpetplot():
     """Exercise extraction of timeseries from CIFTI2."""
-    save_artifacts = os.getenv('SAVE_CIRCLE_ARTIFACTS', False)
+    save_artifacts = os.getenv('SAVE_CIRCLE_ARTIFACTS')
 
     cifti_file = load_test_data(
         'sub-01_task-mixedgamblestask_run-02_space-fsLR_den-91k_bold.dtseries.nii'
@@ -54,7 +54,7 @@ def test_cifti_carpetplot():
 
 def test_nifti_carpetplot():
     """Exercise extraction of timeseries from CIFTI2."""
-    save_artifacts = os.getenv('SAVE_CIRCLE_ARTIFACTS', False)
+    save_artifacts = os.getenv('SAVE_CIRCLE_ARTIFACTS')
 
     nifti_file = load_test_data('sub-ds205s03_task-functionallocalizer_run-01_bold_volreg.nii.gz')
     seg_file = load_test_data('sub-ds205s03_task-functionallocalizer_run-01_bold_parc.nii.gz')

niworkflows/tests/test_confounds.py

@@ -37,7 +37,7 @@
 def _smoke_test_report(report_interface, artifact_name):
     out_report = report_interface.run().outputs.out_file
 
-    save_artifacts = os.getenv('SAVE_CIRCLE_ARTIFACTS', False)
+    save_artifacts = os.getenv('SAVE_CIRCLE_ARTIFACTS')
     if save_artifacts:
         copy(out_report, os.path.join(save_artifacts, artifact_name))
     assert os.path.isfile(out_report), f'Report "{out_report}" does not exist'

niworkflows/tests/test_registration.py

@@ -47,7 +47,7 @@ def _smoke_test_report(report_interface, artifact_name):
         res = pe.Node(report_interface, name='smoke_test', base_dir=tmpdir).run()
         out_report = res.outputs.out_report
 
-        save_artifacts = os.getenv('SAVE_CIRCLE_ARTIFACTS', False)
+        save_artifacts = os.getenv('SAVE_CIRCLE_ARTIFACTS')
         if save_artifacts:
             copy(out_report, os.path.join(save_artifacts, artifact_name))
         assert os.path.isfile(out_report), 'Report does not exist'

niworkflows/tests/test_segmentation.py

@@ -46,7 +46,7 @@ def _smoke_test_report(report_interface, artifact_name):
         res = pe.Node(report_interface, name='smoke_test', base_dir=tmpdir).run()
         out_report = res.outputs.out_report
 
-        save_artifacts = os.getenv('SAVE_CIRCLE_ARTIFACTS', False)
+        save_artifacts = os.getenv('SAVE_CIRCLE_ARTIFACTS')
         if save_artifacts:
             copy(out_report, os.path.join(save_artifacts, artifact_name))
         assert os.path.isfile(out_report), f'Report "{out_report}" does not exist'

niworkflows/tests/test_viz.py

@@ -43,7 +43,7 @@
 @pytest.mark.parametrize('sorting', [None, 'ward', 'linkage'])
 def test_carpetplot(tr, sorting):
     """Write a carpetplot"""
-    save_artifacts = os.getenv('SAVE_CIRCLE_ARTIFACTS', False)
+    save_artifacts = os.getenv('SAVE_CIRCLE_ARTIFACTS')
 
     rng = np.random.default_rng(2010)
 
@@ -137,7 +137,7 @@ def test_carpetplot(tr, sorting):
 )
 def test_fmriplot(input_files):
     """Exercise the fMRIPlot class."""
-    save_artifacts = os.getenv('SAVE_CIRCLE_ARTIFACTS', False)
+    save_artifacts = os.getenv('SAVE_CIRCLE_ARTIFACTS')
     rng = np.random.default_rng(2010)
 
     in_file = os.path.join(datadir, input_files[0])