Fix a buggy samples_dir path construction when HOME contains a remote path

Author: Thierry RAMORASOAVINA

khiops/core/internals/runner.py

@@ -14,7 +14,6 @@
 
 import io
 import os
-import pathlib
 import platform
 import shlex
 import shutil
@@ -65,7 +64,7 @@ def get_default_samples_dir():
     elif platform.system() == "Windows" and "PUBLIC" in os.environ:
         samples_dir = os.path.join(os.environ["PUBLIC"], "khiops_data", "samples")
     else:
-        samples_dir = str(pathlib.Path.home() / "khiops_data" / "samples")
+        samples_dir = os.path.join(os.environ["HOME"], "khiops_data", "samples")
     return samples_dir
 
 

tests/test_remote_access.py

@@ -227,6 +227,27 @@ def test_train_predictor_fail_and_log_with_remote_access(self):
             self.assertTrue(fs.exists(log_file_path), f"Path: {log_file_path}")
             fs.remove(log_file_path)
 
+        def test_samples_dir_inferred_from_remote_home(self):
+            """Test either samples_dir is correctly inferred
+            using a remote path in HOME
+            """
+
+            # Save initial state
+            # This runner has remote paths (for root_temp_dir for example)
+            initial_runner = kh.get_runner()
+            initial_home = os.environ.get("HOME")
+
+            # Set a remote path to HOME
+            os.environ["HOME"] = initial_runner.root_temp_dir
+            test_runner = KhiopsLocalRunner()
+            kh.set_runner(test_runner)
+
+            print(f"Inferred sample_dir is {test_runner.samples_dir}")
+
+            # Restore initial state
+            os.environ["HOME"] = initial_home
+            kh.set_runner(initial_runner)
+
 
 class KhiopsS3RemoteFileTests(KhiopsRemoteAccessTestsContainer.KhiopsRemoteAccessTests):
     """Integration tests with Amazon S3 filesystems"""