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

Side by Side Diff: scripts/slave/recipes/v8.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: License headers 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 DEPS = [ 5 DEPS = [
6 'v8', 6 'v8',
7 'platform',
8 'properties',
7 ] 9 ]
8 10
9 def GenSteps(api): 11 def GenSteps(api):
10 api.v8.set_config('v8', optional=True) 12 api.v8.set_config('v8', optional=True)
11 13
12 yield api.v8.checkout() 14 yield api.v8.checkout()
13 yield api.v8.runhooks() 15 yield api.v8.runhooks()
14 yield api.v8.compile() 16 yield api.v8.compile()
15 17
16 # Tests. 18 # Tests.
17 # TODO(machenbach): Implement the tests. 19 # TODO(machenbach): Implement the tests.
18 20
19 def GenTests(_api): 21 def GenTests(api):
20 for bits in [32, 64]: 22 for bits in [32, 64]:
21 for build_config in ['Release', 'Debug']: 23 for build_config in ['Release', 'Debug']:
22 yield '%s%s' % (build_config, bits), { 24 yield (
23 'properties': { 25 api.test('%s%s' % (build_config, bits)) +
24 'build_config': build_config, 26 api.properties(build_config=build_config, bits=bits)
25 'bits': bits, 27 )
26 },
27 }
28 28
29 for build_config in ['Release', 'Debug']: 29 for build_config in ['Release', 'Debug']:
30 yield 'arm_%s' % (build_config), { 30 yield (
31 'properties': { 31 api.test('arm_%s' % build_config) +
32 'build_config': build_config, 32 api.properties(build_config=build_config, target_arch='arm')
33 'target_arch': 'arm', 33 )
34 },
35 }
36 34
37 yield 'default_platform', { 35 yield (
38 'mock': { 36 api.test('mips_target') +
39 'platform': { 37 api.properties(build_config='Release', target_arch='mips')
40 'name': 'linux', 38 )
41 'bits': 64,
42 }
43 },
44 }
45 39
46 yield 'clobber', { 40 yield api.test('default_platform') + api.platform('linux', 64)
47 'properties': {
48 'clobber': '',
49 },
50 }
51 41
42 yield api.test('clobber') + api.properties(clobber='')
OLDNEW
« no previous file with comments | « scripts/slave/recipes/run_presubmit.py ('k') | scripts/slave/recipes/v8.expected/mips_target.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698