summaryrefslogtreecommitdiff
path: root/testing/tools/run_corpus_tests.py
diff options
context:
space:
mode:
authordsinclair <dsinclair@chromium.org>2016-04-28 06:17:40 -0700
committerCommit bot <commit-bot@chromium.org>2016-04-28 06:17:40 -0700
commit3b5cb78353f92ad7d23ace4e452ed26acf9aeaa7 (patch)
treedd0c2972880da75840a94c9dbe538607aaa10c05 /testing/tools/run_corpus_tests.py
parentbb0d446df18ee34504a165f3fc96fbb81b274f31 (diff)
downloadpdfium-3b5cb78353f92ad7d23ace4e452ed26acf9aeaa7.tar.xz
Remove output redirect from corpus test runner.
The corpus tests redirect output to a string but they just output it later anyway. This CL removes the output redirect and removes the redirect_output flag which will always be False now. Review-Url: https://codereview.chromium.org/1927633002
Diffstat (limited to 'testing/tools/run_corpus_tests.py')
-rwxr-xr-xtesting/tools/run_corpus_tests.py22
1 files changed, 6 insertions, 16 deletions
diff --git a/testing/tools/run_corpus_tests.py b/testing/tools/run_corpus_tests.py
index 82c3d2aefe..2c3c191c88 100755
--- a/testing/tools/run_corpus_tests.py
+++ b/testing/tools/run_corpus_tests.py
@@ -26,8 +26,7 @@ class KeyboardInterruptError(Exception): pass
# c_dir - "path/to/a/b/c"
def test_one_file(input_filename, source_dir, working_dir,
- pdfium_test_path, image_differ, drmem_wrapper,
- redirect_output=False):
+ pdfium_test_path, image_differ, drmem_wrapper):
input_path = os.path.join(source_dir, input_filename)
pdf_path = os.path.join(working_dir, input_filename)
# Remove any existing generated images from previous runs.
@@ -45,29 +44,21 @@ def test_one_file(input_filename, source_dir, working_dir,
os.path.splitext(input_filename)[0])
cmd_to_run.extend([pdfium_test_path, '--png', pdf_path])
# run test
- error = common.RunCommand(cmd_to_run, redirect_output)
+ error = common.RunCommand(cmd_to_run)
if error:
print "FAILURE: " + input_filename + "; " + str(error)
return False
- return not image_differ.HasDifferences(input_filename, source_dir,
- working_dir, redirect_output)
+ return not image_differ.HasDifferences(input_filename, source_dir, working_dir)
def test_one_file_parallel(working_dir, pdfium_test_path, image_differ,
test_case):
"""Wrapper function to call test_one_file() and redirect output to stdout."""
try:
- old_stdout = sys.stdout
- old_stderr = sys.stderr
- sys.stdout = cStringIO.StringIO()
- sys.stderr = sys.stdout
input_filename, source_dir = test_case
result = test_one_file(input_filename, source_dir, working_dir,
- pdfium_test_path, image_differ, "", True);
- output = sys.stdout
- sys.stdout = old_stdout
- sys.stderr = old_stderr
- return (result, output.getvalue(), input_filename, source_dir)
+ pdfium_test_path, image_differ, "");
+ return (result, input_filename, source_dir)
except KeyboardInterrupt:
raise KeyboardInterruptError()
@@ -138,9 +129,8 @@ def main():
pdfium_test_path, image_differ)
worker_results = pool.imap(worker_func, test_cases)
for worker_result in worker_results:
- result, output, input_filename, source_dir = worker_result
+ result, input_filename, source_dir = worker_result
input_path = os.path.join(source_dir, input_filename)
- sys.stdout.write(output)
handle_result(test_suppressor, input_filename, input_path, result,
surprises, failures)
pool.close()