Add extra reliability for multi-threaded test runs

Author: mdmintz

seleniumbase/core/download_helper.py

@@ -35,7 +35,10 @@ def reset_downloads_folder():
 
 def reset_downloads_folder_assistant(archived_downloads_folder):
     if not os.path.exists(archived_downloads_folder):
-        os.makedirs(archived_downloads_folder)
+        try:
+            os.makedirs(archived_downloads_folder)
+        except Exception:
+            pass  # Should only be reachable during multi-threaded test runs
     new_archived_downloads_sub_folder = "%s/downloads_%s" % (
         archived_downloads_folder, int(time.time()))
     if os.path.exists(downloads_path):

seleniumbase/core/report_helper.py

@@ -66,7 +66,10 @@ def clear_out_old_report_logs(archive_past_runs=True, get_log_folder=False):
     abs_path = os.path.abspath('.')
     file_path = abs_path + "/%s" % LATEST_REPORT_DIR
     if not os.path.exists(file_path):
-        os.makedirs(file_path)
+        try:
+            os.makedirs(file_path)
+        except Exception:
+            pass  # Should only be reachable during multi-threaded runs
 
     if archive_past_runs:
         archive_timestamp = int(time.time())

seleniumbase/fixtures/base_case.py

@@ -3025,12 +3025,19 @@ def tearDown(self):
                         self.save_screenshot_after_test):
                     test_logpath = self.log_path + "/" + test_id
                     if not os.path.exists(test_logpath):
-                        os.makedirs(test_logpath)
+                        try:
+                            os.makedirs(test_logpath)
+                        except Exception:
+                            pass  # Only reachable during multi-threaded runs
                     log_helper.log_screenshot(test_logpath, self.driver)
+                    self.__add_pytest_html_extra()
                 if self.with_testing_base and has_exception:
                     test_logpath = self.log_path + "/" + test_id
                     if not os.path.exists(test_logpath):
-                        os.makedirs(test_logpath)
+                        try:
+                            os.makedirs(test_logpath)
+                        except Exception:
+                            pass  # Only reachable during multi-threaded runs
                     if ((not self.with_screen_shots) and (
                             not self.with_basic_test_info) and (
                             not self.with_page_source)):
@@ -3096,7 +3103,10 @@ def tearDown(self):
                                         self._testMethodName)
                 test_logpath = "latest_logs/" + test_id
                 if not os.path.exists(test_logpath):
-                    os.makedirs(test_logpath)
+                    try:
+                        os.makedirs(test_logpath)
+                    except Exception:
+                        pass  # Only reachable during multi-threaded runs
                 log_helper.log_test_failure_data(
                     self, test_logpath, self.driver, self.browser)
                 if len(self._drivers_list) > 0:
@@ -3108,7 +3118,10 @@ def tearDown(self):
                                         self._testMethodName)
                 test_logpath = "latest_logs/" + test_id
                 if not os.path.exists(test_logpath):
-                    os.makedirs(test_logpath)
+                    try:
+                        os.makedirs(test_logpath)
+                    except Exception:
+                        pass  # Only reachable during multi-threaded runs
                 log_helper.log_screenshot(test_logpath, self.driver)
             # Finally close all open browser windows
             self.__quit_all_drivers()