From 80302c77a854e3252bb4c1dc6f09ea3097536092 Mon Sep 17 00:00:00 2001 From: Ryan Harrison Date: Thu, 10 May 2018 18:27:25 +0000 Subject: Use test_dir instead of 'pdfium' for source type The source type value Gold results is just a logical grouping label. Having a different value for each type will allow the Gold web UI to have a set of labels at the top that allows you to easily switch between specific test tool results. Currently the test scripts are using the test directories 'corpus', 'javascript', and 'pixel', which seem like reasonable labels. BUG=pdfium:1073 Change-Id: I893a1afeb3b87ffb84948c18ae3052b2dd6216ea Reviewed-on: https://pdfium-review.googlesource.com/32350 Commit-Queue: Ryan Harrison Reviewed-by: Henrique Nakashima --- testing/tools/test_runner.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/testing/tools/test_runner.py b/testing/tools/test_runner.py index 5acda86f54..3fa1c79e04 100644 --- a/testing/tools/test_runner.py +++ b/testing/tools/test_runner.py @@ -37,7 +37,12 @@ def TestOneFileParallel(this, test_case): class TestRunner: def __init__(self, dirname): + # Currently the only used directories are corpus, javascript, and pixel, + # which all correspond directly to the type for the test being run. In the + # future if there are tests that don't have this clean correspondence, then + # an argument for the type will need to be added. self.test_dir = dirname + self.test_type = dirname self.enforce_expected_images = False self.oneshot_renderer = False @@ -272,7 +277,7 @@ class TestRunner: # Collect Gold results if an output directory was named. self.gold_results = None if self.options.gold_output_dir: - self.gold_results = gold.GoldResults('pdfium', + self.gold_results = gold.GoldResults(self.test_type, self.options.gold_output_dir, self.options.gold_properties, self.options.gold_key, -- cgit v1.2.3