diff options
| author | stephana <stephana@google.com> | 2017-01-13 13:16:40 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2017-01-13 13:16:40 -0800 |
| commit | 38c2705c697cd9a67b02ead1d601610d7af96524 (patch) | |
| tree | 434dc7672e680de8119ba24c44545be8dadd2d47 | |
| parent | 98d00b230aa14924dfde2eba3aaf69f86d18a45b (diff) | |
| download | pdfium-38c2705c697cd9a67b02ead1d601610d7af96524.tar.xz | |
Ensures that the gold output directory existschromium/2983chromium/2982chromium/2981
This will land after this CL
https://chromium-review.googlesource.com/c/424575/
as a way of making sure it works as intended.
TBR=borenet
BUG=skia:5973
Review-Url: https://codereview.chromium.org/2629943002
| -rw-r--r-- | testing/tools/gold.py | 4 | ||||
| -rw-r--r-- | testing/tools/test_runner.py | 9 |
2 files changed, 9 insertions, 4 deletions
diff --git a/testing/tools/gold.py b/testing/tools/gold.py index fda63b6deb..7598caffdc 100644 --- a/testing/tools/gold.py +++ b/testing/tools/gold.py @@ -67,6 +67,10 @@ class GoldResults(object): self._results = [] self._outputDir = outputDir + # make sure the output directory exists. + if not os.path.exists(outputDir): + os.makedirs(outputDir) + def AddTestResult(self, testName, md5Hash, outputImagePath): # Copy the image to <output_dir>/<md5Hash>.<image_extension> imgExt = os.path.splitext(outputImagePath)[1].lstrip(".") diff --git a/testing/tools/test_runner.py b/testing/tools/test_runner.py index fad7a9c66a..92db9118dd 100644 --- a/testing/tools/test_runner.py +++ b/testing/tools/test_runner.py @@ -127,10 +127,11 @@ class TestRunner: def HandleResult(self, input_filename, input_path, result): if self.gold_results: success, image_paths = result - for img_path, md5_hash in image_paths: - # the output filename (without extension becomes the test name) - test_name = os.path.splitext(os.path.split(img_path)[1])[0] - self.gold_results.AddTestResult(test_name, md5_hash, img_path) + if image_paths: + for img_path, md5_hash in image_paths: + # the output filename (without extension becomes the test name) + test_name = os.path.splitext(os.path.split(img_path)[1])[0] + self.gold_results.AddTestResult(test_name, md5_hash, img_path) if self.test_suppressor.IsResultSuppressed(input_filename): if result: |