Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1747)

Unified Diff: scripts/slave/unittests/expect_tests/pipeline.py

Issue 220353003: Replace recipes_test.py with a new parallel expectation-based test runner. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: fix coverage race Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « scripts/slave/unittests/expect_tests/main.py ('k') | scripts/slave/unittests/expect_tests/serialize.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/unittests/expect_tests/pipeline.py
diff --git a/scripts/slave/unittests/expect_tests/pipeline.py b/scripts/slave/unittests/expect_tests/pipeline.py
new file mode 100644
index 0000000000000000000000000000000000000000..575ee063de11a2f2ca670c168225c21b64362ad7
--- /dev/null
+++ b/scripts/slave/unittests/expect_tests/pipeline.py
@@ -0,0 +1,175 @@
+# Copyright 2014 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+import Queue
+import glob
+import multiprocessing
+import re
+import signal
+
+from .type_definitions import (
+ Test, UnknownError, TestError, Result, ResultStageAbort)
+
+
+def gen_loop_process(gen, test_queue, result_queue, opts, kill_switch,
+ cover_ctx):
+ """Generate `Test`'s from |gen|, and feed them into |test_queue|.
+
+ Non-Test instances will be translated into `UnknownError` objects.
+
+ On completion, feed |opts.jobs| None objects into |test_queue|.
+
+ @param gen: generator yielding Test() instances.
+ @type test_queue: multiprocessing.Queue()
+ @type result_queue: multiprocessing.Queue()
+ @type opts: argparse.Namespace
+ @type kill_switch: multiprocessing.Event()
+ @type cover_ctx: cover.CoverageContext().create_subprocess_context()
+ """
+ matcher = re.compile(
+ '^%s$' % '|'.join('(?:%s)' % glob.fnmatch.translate(g)
+ for g in opts.test_glob if g[0] != '-'))
+ if matcher.pattern == '^$':
+ matcher = re.compile('^.*$')
+
+ neg_matcher = re.compile(
+ '^%s$' % '|'.join('(?:%s)' % glob.fnmatch.translate(g[1:])
+ for g in opts.test_glob if g[0] == '-'))
+
+ def generate_tests():
+ try:
+ for test in gen():
+ if kill_switch.is_set():
+ break
+
+ if not isinstance(test, Test):
+ result_queue.put_nowait(
+ UnknownError(
+ 'Got non-Test isinstance from generator: %r' % test))
+ continue
+
+ if not neg_matcher.match(test.name) and matcher.match(test.name):
+ yield test
+ except KeyboardInterrupt:
+ pass
+ finally:
+ for _ in xrange(opts.jobs):
+ test_queue.put_nowait(None)
+
+
+ next_stage = (result_queue if opts.handler.SKIP_RUNLOOP else test_queue)
+ with cover_ctx:
+ opts.handler.gen_stage_loop(opts, generate_tests(), next_stage.put_nowait,
+ result_queue.put_nowait)
+
+
+def run_loop_process(test_queue, result_queue, opts, kill_switch, cover_ctx):
+ """Consume `Test` instances from |test_queue|, run them, and yield the results
+ into opts.run_stage_loop().
+
+ Generates coverage data as a side-effect.
+ @type test_queue: multiprocessing.Queue()
+ @type result_queue: multiprocessing.Queue()
+ @type opts: argparse.Namespace
+ @type kill_switch: multiprocessing.Event()
+ @type cover_ctx: cover.CoverageContext().create_subprocess_context()
+ """
+ def generate_tests_results():
+ try:
+ while not kill_switch.is_set():
+ try:
+ test = test_queue.get(timeout=0.1)
+ if test is None:
+ break
+ except Queue.Empty:
+ continue
+
+ try:
+ result = test.run()
+ if not isinstance(result, Result):
+ result_queue.put_nowait(
+ TestError(test, 'Got non-Result instance from test: %r'
+ % result))
+ continue
+
+ yield test, result
+ except Exception as e:
+ # TODO(iannucci): include stacktrace
+ result_queue.put_nowait(TestError(test, str(e)))
+ except KeyboardInterrupt:
+ pass
+
+ with cover_ctx:
+ opts.handler.run_stage_loop(opts, generate_tests_results(),
+ result_queue.put_nowait)
+
+
+def result_loop(test_gen, cover_ctx, opts):
+ kill_switch = multiprocessing.Event()
+ def handle_killswitch(*_):
+ kill_switch.set()
+ # Reset the signal to DFL so that double ctrl-C kills us for sure.
+ signal.signal(signal.SIGINT, signal.SIG_DFL)
+ signal.signal(signal.SIGTERM, signal.SIG_DFL)
+ signal.signal(signal.SIGINT, handle_killswitch)
+ signal.signal(signal.SIGTERM, handle_killswitch)
+
+ test_queue = multiprocessing.Queue()
+ result_queue = multiprocessing.Queue()
+
+ test_gen_args = (
+ test_gen, test_queue, result_queue, opts, kill_switch, cover_ctx)
+
+ procs = []
+ if opts.handler.SKIP_RUNLOOP:
+ gen_loop_process(*test_gen_args)
+ else:
+ procs = [multiprocessing.Process(
+ target=gen_loop_process, args=test_gen_args)]
+
+ procs += [
+ multiprocessing.Process(
+ target=run_loop_process, args=(
+ test_queue, result_queue, opts, kill_switch, cover_ctx))
+ for _ in xrange(opts.jobs)
+ ]
+
+ for p in procs:
+ p.daemon = True
+ p.start()
+
+ error = False
+ try:
+ def generate_objects():
+ while not kill_switch.is_set():
+ while not kill_switch.is_set():
+ try:
+ yield result_queue.get(timeout=0.1)
+ except Queue.Empty:
+ break
+
+ if not any(p.is_alive() for p in procs):
+ break
+
+ # Get everything still in the queue. Still need timeout, but since nothing
+ # is going to be adding stuff to the queue, use a very short timeout.
+ while not kill_switch.is_set():
+ try:
+ yield result_queue.get(timeout=0.00001)
+ except Queue.Empty:
+ break
+
+ if kill_switch.is_set():
+ raise ResultStageAbort()
+ error = opts.handler.result_stage_loop(opts, generate_objects())
+ except ResultStageAbort:
+ pass
+
+ for p in procs:
+ p.join()
+
+ if not kill_switch.is_set() and not result_queue.empty():
+ error = True
+
+ return error, kill_switch.is_set()
« no previous file with comments | « scripts/slave/unittests/expect_tests/main.py ('k') | scripts/slave/unittests/expect_tests/serialize.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698