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

Unified Diff: scripts/slave/README.recipes.md

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: comments and fix 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 | « no previous file | scripts/slave/unittests/expect_tests/__init__.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/README.recipes.md
diff --git a/scripts/slave/README.recipes.md b/scripts/slave/README.recipes.md
index d191c78ec75814bbcb5e9eb181f812bd9336a89c..e5129fdbf6ad26e7392ce4ebc2b594b1349c5e46 100644
--- a/scripts/slave/README.recipes.md
+++ b/scripts/slave/README.recipes.md
@@ -342,7 +342,7 @@ Test data can contain any of the following keys:
assumed that the step succeeded with a retcode of 0.
The `api` passed to GenTests is confusingly **NOT** the same as the recipe api.
-It's actually an instance of `unittests/recipes_test.py:TestApi()`. This is
+It's actually an instance of `recipe_test_api.py:RecipeTestApi()`. This is
adimittedly pretty weak, and it would be great to have the test api
automatically created via modules. On the flip side, the test api is much less
necessary than the recipe api, so this transformation has not been designed yet.
@@ -613,10 +613,8 @@ Use `tools/show_me_the_modules.py`. It's super effective!
How do I run those tests you were talking about?
------------------------------------------------
To test all the recipes/apis, use `slave/unittests/recipes_test.py`. To set new
Vadim Sh. 2014/04/03 02:42:09 'use `slave/unittests/recipes_test.py`' Huh? Is it
iannucci 2014/04/03 04:31:40 oops :D
-expectations `slave/unittests/recipes_test.py --train`.
+expectations `slave/unittests/recipe_simulation_test.py train`.
-To test all the configs, use `slave/unittests/recipe_configs_test.py`. To set
-new expectations `slave/unittests/recipe_configs_test.py --train`.
Where's the docs on `*.py`?
« no previous file with comments | « no previous file | scripts/slave/unittests/expect_tests/__init__.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698