[test] Only cleanup temp directory when running CI

This causes all kind of issues if you have more than one emscripten
checkout active at a given time (which I often do these days thanks the
git worktrees and AI agents).

The tests themselves should never leak temp files. We even have
`EMTEST_DETECT_TEMPFILE_LEAKS` (which we set during CI) that ensure this
is the case.

I think the parallel test runner can leak temp directories if its killed
at the wrong time, but #25615 should help with that.

See #25614 and #26110

Author: sbc100

test/runner.py

@@ -570,9 +570,9 @@ def configure():
   browser_common.configure_test_browser()
 
 
-def cleanup_emscripten_temp():
-  """Deletes all files and directories under Emscripten
-  that look like they might have been created by Emscripten."""
+def cleanup_temp_directory():
+  """Deletes all files and directories in TEMP_DIR that look like they
+  might have been created by Emscripten."""
   for entry in os.listdir(shared.TEMP_DIR):
     if entry.startswith(('emtest_', 'emscripten_')):
       entry = os.path.join(shared.TEMP_DIR, entry)
@@ -718,8 +718,10 @@ def set_env(name, option_value):
 
   # Remove any old test files before starting the run. Skip cleanup when we're running in debug mode
   # where we want to preserve any files created (e.g. emscripten.lock from shared.py).
-  if not (shared.DEBUG or common.EMTEST_SAVE_DIR):
-    cleanup_emscripten_temp()
+  # Note: We don't do this except in the CI environment, since it prevents multiple
+  # emscirpten checkout from running tests at the same time.
+  if os.getenv('CI') and not (shared.DEBUG or common.EMTEST_SAVE_DIR):
+    cleanup_temp_directory()
   utils.delete_file(common.flaky_tests_log_filename)
 
   browser_common.init(options.force_browser_process_termination)