Use os.path.join to create file names.

test/basic_test.py

@@ -140,12 +140,12 @@ def test_xml_configurations(tmp_path):
         mask.text = str(os.path.join(CURRENT_DIR, "data", "sts_images"))
 
     # Save as temporary xml file.
-    tree.write(tmp_path / "temp_test_config_data.xml")
+    tree.write(os.path.join(tmp_path, "temp_test_config_data.xml"))
 
     data = extract_features(
         write_features=False,
         export_features=True,
-        image=str(tmp_path / "temp_test_config_data.xml"),
+        image=os.path.join(tmp_path, "temp_test_config_data.xml"),
         settings=os.path.join(CURRENT_DIR, "data", "configuration_files", "test_config_settings.xml")
     )
 

test/configuration_test.py

@@ -23,8 +23,7 @@ def test_general_settings_configuration(tmp_path):
     from mirp.settings.import_config_parameters import create_settings_object
     from mirp.settings.generic import SettingsClass
 
-    temp_file = tmp_path / "settings.xml"
-
+    temp_file = os.path.join(tmp_path, "settings.xml")
     get_settings_xml(tmp_path)
 
     settings_definitions = get_general_settings()
@@ -85,8 +84,7 @@ def test_post_processing_settings_configuration(tmp_path):
     from mirp.settings.import_config_parameters import create_settings_object
     from mirp.settings.generic import SettingsClass
 
-    temp_file = tmp_path / "settings.xml"
-
+    temp_file = os.path.join(tmp_path, "settings.xml")
     get_settings_xml(tmp_path)
 
     settings_definitions = get_post_processing_settings()
@@ -156,8 +154,7 @@ def test_interpolation_settings_configuration(tmp_path):
     from mirp.settings.import_config_parameters import create_settings_object
     from mirp.settings.generic import SettingsClass
 
-    temp_file = tmp_path / "settings.xml"
-
+    temp_file = os.path.join(tmp_path, "settings.xml")
     get_settings_xml(tmp_path)
 
     # All default settings.
@@ -263,8 +260,7 @@ def test_perturbation_settings_configuration(tmp_path):
     from mirp.settings.import_config_parameters import create_settings_object
     from mirp.settings.generic import SettingsClass
 
-    temp_file = tmp_path / "settings.xml"
-
+    temp_file = os.path.join(tmp_path, "settings.xml")
     get_settings_xml(tmp_path)
 
     settings_definitions = get_perturbation_settings()
@@ -334,8 +330,7 @@ def test_mask_resegmentation_settings_configuration(tmp_path):
     from mirp.settings.import_config_parameters import create_settings_object
     from mirp.settings.generic import SettingsClass
 
-    temp_file = tmp_path / "settings.xml"
-
+    temp_file = os.path.join(tmp_path, "settings.xml")
     get_settings_xml(tmp_path)
 
     settings_definitions = get_mask_resegmentation_settings()
@@ -405,8 +400,7 @@ def test_feature_extraction_settings_configuration(tmp_path):
     from mirp.settings.import_config_parameters import create_settings_object
     from mirp.settings.generic import SettingsClass
 
-    temp_file = tmp_path / "settings.xml"
-
+    temp_file = os.path.join(tmp_path, "settings.xml")
     get_settings_xml(tmp_path)
 
     settings_definitions = get_feature_extraction_settings()
@@ -479,8 +473,7 @@ def test_image_transformation_settings_configuration(tmp_path):
     from mirp.settings.import_config_parameters import create_settings_object
     from mirp.settings.generic import SettingsClass
 
-    temp_file = tmp_path / "settings.xml"
-
+    temp_file = os.path.join(tmp_path, "settings.xml")
     get_settings_xml(tmp_path)
 
     settings_definitions = get_image_transformation_settings()

test/export_image_and_mask_test.py

@@ -15,7 +15,7 @@ def test_basic_ct_feature_extraction(tmp_path):
 
     extract_features_and_images(
         write_features=True,
-        export_features=True,
+        export_features=False,
         write_images=True,
         export_images=False,
         write_dir=tmp_path,
@@ -25,7 +25,7 @@ def test_basic_ct_feature_extraction(tmp_path):
     )
 
     # Check that files exist.
-    file_names = [file for file in os.listdir(tmp_path) if (tmp_path / file).is_file()]
+    file_names = [file for file in os.listdir(tmp_path) if os.path.isfile(os.path.join(tmp_path, file))]
 
     assert len(file_names) == 3
     assert len([file for file in file_names if file.endswith(".csv")]) == 1

test/import_image_and_mask_test.py

@@ -683,7 +683,7 @@ def test_failure_multiple_image_and_mask_import_data_xml(tmp_path):
     from xml.etree import ElementTree as ElemTree
     from mirp import get_data_xml
 
-    target_file = str(tmp_path / "data.xml")
+    target_file = os.path.join(tmp_path, "data.xml")
 
     # Start with a clean slate.
     if os.path.exists(target_file):

test/read_image_and_mask_test.py

@@ -394,10 +394,11 @@ def test_read_dicom_image_and_mask_data_xml(tmp_path):
         mask.text = str(os.path.join(CURRENT_DIR, "data", "sts_images"))
 
     # Save as temporary xml file.
-    tree.write(tmp_path / "temp_test_config_data.xml")
+    file = os.path.join(tmp_path, "temp_test_config_data.xml")
+    tree.write(file)
 
     image_list = import_image_and_mask(
-        image=str(tmp_path / "temp_test_config_data.xml")
+        image=file
     )
 
     image, roi_list = read_image_and_masks(image=image_list[0])

test/xml_file_copy_test.py

@@ -5,7 +5,7 @@
 
 
 def test_copy_settings_xml(tmp_path):
-    target_file = tmp_path / "settings.xml"
+    target_file = os.path.join(tmp_path, "settings.xml")
 
     # Start with a clean slate.
     if os.path.exists(target_file):
@@ -20,7 +20,7 @@ def test_copy_settings_xml(tmp_path):
 
 
 def test_copy_data_xml(tmp_path):
-    target_file = tmp_path / "data.xml"
+    target_file = os.path.join(tmp_path, "data.xml")
 
     # Start with a clean slate.
     if os.path.exists(target_file):