Various minor improvements to recipe_simulation_test.py.

scripts/slave/unittests/expect_tests/pipeline.py

 # 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
 import traceback
 
 from .type_definitions import (
-    Test, UnknownError, TestError, Result, ResultStageAbort)
+    Test, UnknownError, TestError, NoMatchingTestsError,
+    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()
   """
+  # Implicitly append '*'' to globs that don't specify it.
+  globs = ['%s%s' % (g, '*' if '*' not in g else '') for g in opts.test_glob]
+
   matcher = re.compile(
       '^%s$' % '|'.join('(?:%s)' % glob.fnmatch.translate(g)
-                        for g in opts.test_glob if g[0] != '-'))
+                        for g in globs 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] == '-'))
+                        for g in globs if g[0] == '-'))
 
   def generate_tests():
     paths_seen = set()
+    seen_tests = False
     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
 
         test_path = test.expect_path()
         if test_path in paths_seen:
           result_queue.put_nowait(
               TestError(test, 'Duplicate expectation path!'))
         else:
           paths_seen.add(test_path)
           if not neg_matcher.match(test.name) and matcher.match(test.name):
+            seen_tests = True
             yield test
+      if not seen_tests:
+        result_queue.put_nowait(NoMatchingTestsError())
     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,
@@ skipping 101 matching lines @@
   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()