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

Unified Diff: scripts/slave/recipes/chromium.py

Issue 23889036: Refactor the way that TestApi works so that it is actually useful. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Yeeeeaaaahhhhhh! Created 7 years, 3 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
Index: scripts/slave/recipes/chromium.py
diff --git a/scripts/slave/recipes/chromium.py b/scripts/slave/recipes/chromium.py
index 4223f4967f81d0ffba396380d1035f724110596e..a1c02c71036f791912fba6a15c12e2f2e1f86895 100644
--- a/scripts/slave/recipes/chromium.py
+++ b/scripts/slave/recipes/chromium.py
@@ -5,6 +5,7 @@
DEPS = [
'chromium',
'gclient',
+ 'platform',
'properties',
]
@@ -28,7 +29,9 @@ def GenSteps(api):
def GenTests(api):
for plat in ('win', 'mac', 'linux'):
for bits in (32, 64):
- yield 'basic_%s_%s' % (plat, bits), {
- 'mock': {'platform': {'name': plat}},
- 'properties': {'TARGET_BITS': bits},
- }
+ yield (
+ api.Test('basic_%s_%s' % (plat, bits)) +
+ api.Properties(TARGET_BITS=bits) +
+ api.platform(plat, bits)
+ )
+

Powered by Google App Engine
This is Rietveld 408576698