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

Side by Side Diff: recipes/top.py

Issue 1421843006: Add simple depends_on API. (Closed) Base URL: https://chromium.googlesource.com/external/github.com/luci/recipes-py@master
Patch Set: Created 5 years, 1 month 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 unified diff | Download patch
« no previous file with comments | « recipes/next_prime.py ('k') | recipes/top.expected/basic.json » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 from recipe_engine.recipe_api import Property
2 from recipe_engine import config
3
4 DEPS = [
5 'recipe_engine/properties',
6 ]
7
8 PROPERTIES = {
9 'to_pass': Property(),
10 }
11
12 RETURN_SCHEMA = config.ReturnSchema(
13 result=config.Single(int),
14 )
15
16 def RunSteps(api, to_pass):
17 res = api.depend_on('bottom', {'number': to_pass})
estaab 2015/11/10 02:55:12 Since to_pass is just a plain Property() does that
martiniss 2015/11/13 00:17:12 Technically yes, and naive simulation tests won't
18 return RETURN_SCHEMA(**res)
estaab 2015/11/10 02:55:12 Would "return {'result': res['result']}" also be v
martiniss 2015/11/13 00:17:12 Yes.
19
20 def GenTests(api):
21 yield (
22 api.test('basic') +
23 api.properties(to_pass=3)
24 )
OLDNEW
« no previous file with comments | « recipes/next_prime.py ('k') | recipes/top.expected/basic.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698