Add friendly error message when GenTests is missing or misspelled

recipe_engine/test.py

Index: recipe_engine/test.py
diff --git a/recipe_engine/test.py b/recipe_engine/test.py
index 2924b206e684ff3b3738f9885ab45175148460fa..95d688b108c36ce398e8a3a9d49a2f53e76864a6 100644
--- a/recipe_engine/test.py
+++ b/recipe_engine/test.py
@@ -397,6 +397,9 @@ def get_tests(test_filter=None):
         # TODO(phajdan.jr): move expectation tree outside of the recipe tree.
         expect_dir = os.path.join(root, '%s.expected' % name)
 
+        if not recipe.gen_tests:
+          raise Exception('Missing or misspelled GenTests function.')

[iannucci, 2017/04/27 19:28:33]

Can you do the same for RunSteps too?

[Paweł Hajdan Jr., 2017/05/08 18:03:51]

Done.

Turns out it already works, just added a test for it.

+
         # Immediately convert to list to force running the generator under
         # coverage context. Otherwise coverage would only report executing
         # the function definition, not GenTests body.