Actually added pretest support

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 contextlib
 import imp
 import inspect
 import logging
 import multiprocessing
 import os
@@ skipping 381 matching lines @@
   This function is meant to be run as a dedicated process. Calling it twice
   in the same process is not supported.
 
   Args:
     cover_ctx:
     kill_switch (multiprocessing.Event):
     result_queue (multiprocessing.Queue):
     opts: output of argparse.ArgumentParser.parse_args (see main.py)
     processing_context (ProcessingContext): the task to perform.
     """
+  pretest_filename = os.path.join(processing_context.cwd, '.expect_tests.py')

[agable, 2015/02/03 04:23:48]

In the documentation you committed previously, you named this file
".expect_tests_pretest.py".

[agable, 2015/02/03 04:27:59]

Also, this implementation requires the same .expect_tests.py file to be in every
subdirectory of appengine/. This makes me sad, but I understand why it is that
way.

+  if os.path.isfile(pretest_filename):
+    execfile(pretest_filename)
+
   sys.path.insert(0, processing_context.cwd)
 
   # This flag is set when test generation has finished.
   test_gen_finished = multiprocessing.Event()
   test_queue = multiprocessing.Queue()
 
   with TempDir() as temp_dir:
     test_gen_args = (
         processing_context.testing_contexts, test_queue, result_queue, opts,
         kill_switch, cover_ctx, temp_dir
@@ skipping 110 matching lines @@
 
     if procs:
       error = opts.handler.result_stage_loop(opts, generate_objects(procs))
   except ResultStageAbort:
     pass
 
   if not kill_switch.is_set() and not result_queue.empty():
     error = True
 
   return error, kill_switch.is_set()