commit 3da80f5ecd98c9b3f46c23b85854b8f96025e565
author mtklein <mtklein@chromium.org> Wed Jul 27 04:14:07 2016 -0700
committer Commit bot <commit-bot@chromium.org> Wed Jul 27 04:14:07 2016 -0700
tree cc46733e43f16882841d29a3fb60c4911396a79b
parent eb560284c5a44389cd3f3b304839ea34a3e73398

Only run recipe simulation tests if infra/ changes.

This should lessen the need for mortals to have coverage.py,
and saves 1.8s (of 3.6s) on my desktop for this non-infra change.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2187713002

Review-Url: https://codereview.chromium.org/2187713002

PRESUBMIT.py

diff --git a/PRESUBMIT.py b/PRESUBMIT.py
index a3af9e4..e0e01fb 100644
--- a/PRESUBMIT.py
+++ b/PRESUBMIT.py
@@ -172,6 +172,10 @@
 def _RecipeSimulationTest(input_api, output_api):
   """Run the recipe simulation test."""
   results = []
+  if not any(f.LocalPath().startswith('infra')
+             for f in input_api.AffectedFiles()):
+    return results
+
   recipes_py = os.path.join('infra', 'bots', 'recipes.py')
   cmd = ['python', recipes_py, 'simulation_test']
   try: