commit 96eff7c52d6767b440ce887583e27885663fa77c
author Lei Zhang <thestig@chromium.org> Thu Sep 19 21:08:58 2019 +0000
committer Chromium commit bot <commit-bot@chromium.org> Thu Sep 19 21:08:58 2019 +0000
tree a956c210842be6186c48648b6d2122ad8cc4836d
parent af79ecc8fd07ee7042931f7b3c6ffe6c47b4a3f1

Clear the working directory in the test runner.

Otherwise, if an existing working directory is cluttered with random
leftover files, that may affect the test runner's behavior.

Change-Id: I19d06c4b6bb917c88f50c4cf03b35faa35586b2e
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/60950
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>

testing/tools/test_runner.py

diff --git a/testing/tools/test_runner.py b/testing/tools/test_runner.py
index c8b217d..74a86b1 100644
--- a/testing/tools/test_runner.py
+++ b/testing/tools/test_runner.py
@@ -257,8 +257,8 @@
       return 1
 
     self.working_dir = finder.WorkingDir(os.path.join('testing', self.test_dir))
-    if not os.path.exists(self.working_dir):
-      os.makedirs(self.working_dir)
+    shutil.rmtree(self.working_dir, ignore_errors=True)
+    os.makedirs(self.working_dir)
 
     self.feature_string = subprocess.check_output([self.pdfium_test_path,
                                                    '--show-config'])