Fix RunManager job folder tests by using os.path.realpath for cross-platform path comparisons and proper handling of mixed, invalid, and non-job folder names

Author: harryswift01

tests/test_CodeEntropy/test_run.py

@@ -63,11 +63,13 @@ def test_create_job_folder_with_existing_folders(self, mock_listdir, mock_makedi
         mock_listdir.return_value = ["job001", "job002", "job003"]
         new_folder_path = RunManager.create_job_folder()
         expected_path = os.path.join(self.test_dir, "job004")
+
         self.assertEqual(
-            os.path.realpath(new_folder_path), os.path.realpath(expected_path)
+            os.path.normcase(os.path.realpath(new_folder_path)),
+            os.path.normcase(os.path.realpath(expected_path)),
         )
         mock_makedirs.assert_called_once_with(
-            os.path.realpath(expected_path), exist_ok=True
+            os.path.normcase(os.path.realpath(expected_path)), exist_ok=True
         )
 
     @patch("os.makedirs")
@@ -80,13 +82,16 @@ def test_create_job_folder_with_non_matching_folders(
         folders.
         """
         mock_listdir.return_value = ["folderA", "another_one"]
+
         new_folder_path = RunManager.create_job_folder()
         expected_path = os.path.join(self.test_dir, "job001")
+
         self.assertEqual(
-            os.path.realpath(new_folder_path), os.path.realpath(expected_path)
+            os.path.normcase(os.path.realpath(new_folder_path)),
+            os.path.normcase(os.path.realpath(expected_path)),
         )
         mock_makedirs.assert_called_once_with(
-            os.path.realpath(expected_path), exist_ok=True
+            os.path.normcase(os.path.realpath(expected_path)), exist_ok=True
         )
 
     @patch("os.makedirs")
@@ -100,12 +105,14 @@ def test_create_job_folder_mixed_folder_names(self, mock_listdir, mock_makedirs)
         new_folder_path = RunManager.create_job_folder()
         expected_path = os.path.join(self.test_dir, "job003")
         self.assertEqual(
-            os.path.realpath(new_folder_path), os.path.realpath(expected_path)
-        )
-        mock_makedirs.assert_called_once_with(
-            os.path.realpath(expected_path), exist_ok=True
+            os.path.normcase(os.path.abspath(new_folder_path)),
+            os.path.normcase(os.path.abspath(expected_path)),
         )
 
+        called_path = os.path.normcase(os.path.abspath(mock_makedirs.call_args[0][0]))
+        self.assertEqual(called_path, os.path.normcase(os.path.abspath(expected_path)))
+        self.assertTrue(mock_makedirs.call_args[1]["exist_ok"])
+
     @patch("os.makedirs")
     @patch("os.listdir")
     def test_create_job_folder_with_invalid_job_suffix(
@@ -122,10 +129,11 @@ def test_create_job_folder_with_invalid_job_suffix(
         expected_path = os.path.join(self.test_dir, "job003")
 
         self.assertEqual(
-            os.path.realpath(new_folder_path), os.path.realpath(expected_path)
+            os.path.normcase(os.path.realpath(new_folder_path)),
+            os.path.normcase(os.path.realpath(expected_path)),
         )
         mock_makedirs.assert_called_once_with(
-            os.path.realpath(expected_path), exist_ok=True
+            os.path.normcase(os.path.realpath(expected_path)), exist_ok=True
         )
 
     @patch("requests.get")