fix warning and tests

Author: bpinsard

heudiconv/convert.py

@@ -61,9 +61,7 @@ class PopulateIntendedForOpts(TypedDict, total=False):
 
 
 LOCKFILE = "heudiconv.lock"
-DW_IMAGE_IN_FMAP_FOLDER_WARNING = (
-    "Diffusion-weighted image saved in non dwi folder ({folder})"
-)
+DW_IMAGE_WRONG_SUFFIX_WARNING = "Diffusion-weighted image saved as non dwi ({prefix})"
 lgr = logging.getLogger(__name__)
 
 
@@ -976,9 +974,7 @@ def save_converted_files(
                     os.remove(ftr)
                 lgr.debug("%s and %s were removed since not dwi", bvecs, bvals)
             else:
-                lgr.warning(
-                    DW_IMAGE_IN_FMAP_FOLDER_WARNING.format(folder=prefix_dirname)
-                )
+                lgr.warning(DW_IMAGE_WRONG_SUFFIX_WARNING.format(prefix=prefix))
                 lgr.warning(
                     ".bvec and .bval files will be generated. This is NOT BIDS compliant"
                 )

heudiconv/tests/test_convert.py

@@ -13,7 +13,7 @@
 from heudiconv.cli.run import main as runner
 import heudiconv.convert
 from heudiconv.convert import (
-    DW_IMAGE_IN_FMAP_FOLDER_WARNING,
+    DW_IMAGE_WRONG_SUFFIX_WARNING,
     bvals_are_zero,
     update_complex_name,
     update_multiecho_name,
@@ -170,8 +170,8 @@ def test_b0dwi_for_fmap(tmp_path: Path, caplog: pytest.LogCaptureFixture) -> Non
 
     # assert that it raised a warning that the fmap directory will contain
     # bvec and bval files.
-    expected_msg = DW_IMAGE_IN_FMAP_FOLDER_WARNING.format(
-        folder=op.join(tmp_path, f"sub-{subID}", "fmap")
+    expected_msg = DW_IMAGE_WRONG_SUFFIX_WARNING.format(
+        prefix=op.join(tmp_path, f"sub-{subID}", "fmap", f"sub-{subID}_acq-b0dwi_epi")
     )
     assert any(expected_msg in c.message for c in caplog.records)