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

Side by Side Diff: frog/scripts/buildbot_annotated_steps.py

Issue 9418045: Support for native in leg, and start moving native tests to a specific test suite. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 8 years, 10 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
« no previous file with comments | « frog/presubmit.py ('k') | frog/tests/frog/src/NativeClassAvoidsHiddenNameFrogTest.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/python 1 #!/usr/bin/python
2 2
3 # Copyright (c) 2011 The Chromium Authors. All rights reserved. 3 # Copyright (c) 2011 The Chromium Authors. All rights reserved.
4 # Use of this source code is governed by a BSD-style license that can be 4 # Use of this source code is governed by a BSD-style license that can be
5 # found in the LICENSE file. 5 # found in the LICENSE file.
6 6
7 """Dart frog buildbot steps 7 """Dart frog buildbot steps
8 8
9 Runs tests for the frog compiler (running on the vm or the self-hosting version) 9 Runs tests for the frog compiler (running on the vm or the self-hosting version)
10 """ 10 """
(...skipping 129 matching lines...) Expand 10 before | Expand all | Expand 10 after
140 - browser: one of the browsers, see GetBuildInfo 140 - browser: one of the browsers, see GetBuildInfo
141 - flags: extra flags to pass to test.dart 141 - flags: extra flags to pass to test.dart
142 """ 142 """
143 143
144 # Make sure we are in the frog directory 144 # Make sure we are in the frog directory
145 os.chdir(FROG_PATH) 145 os.chdir(FROG_PATH)
146 146
147 if arch != 'frogium': # frog and frogsh 147 if arch != 'frogium': # frog and frogsh
148 TestStep("frog", mode, system, arch, [], flags) 148 TestStep("frog", mode, system, arch, [], flags)
149 TestStep("frog_extra", mode, system, 149 TestStep("frog_extra", mode, system,
150 arch, ['frog', 'peg', 'css'], flags) 150 arch, ['frog', 'frog_native', 'peg', 'css'], flags)
151 151
152 if arch == 'frogsh': 152 if arch == 'frogsh':
153 # There is no need to run these tests both for frog and frogsh. 153 # There is no need to run these tests both for frog and frogsh.
154 154
155 TestStep("leg", mode, system, 'leg', [], flags) 155 TestStep("leg", mode, system, 'leg', [], flags)
156 TestStep("leg_extra", mode, system, 'leg', ['leg_only'], flags) 156 TestStep("leg_extra", mode, system, 'leg', ['leg_only'], flags)
ahe 2012/02/22 22:08:47 You need to pass in 'frog_native' here. Otherwise
157 # Leg isn't self-hosted (yet) so we run the leg unit tests on the VM. 157 # Leg isn't self-hosted (yet) so we run the leg unit tests on the VM.
158 TestStep("leg_extra", mode, system, 'vm', ['leg'], flags) 158 TestStep("leg_extra", mode, system, 'vm', ['leg'], flags)
159 159
160 else: 160 else:
161 tests = ['client', 'language', 'corelib', 'isolate', 'frog', 'peg', 'css'] 161 tests = ['client', 'language', 'corelib', 'isolate', 'frog',
162 'frog_native', 'peg', 'css']
162 163
163 # TODO(jmesserly): make DumpRenderTree more like other browser tests, so 164 # TODO(jmesserly): make DumpRenderTree more like other browser tests, so
164 # we don't have this translation step. See dartbug.com/1158. 165 # we don't have this translation step. See dartbug.com/1158.
165 # Ideally we can run most Chrome tests in DumpRenderTree because it's more 166 # Ideally we can run most Chrome tests in DumpRenderTree because it's more
166 # debuggable, but still have some tests run the full browser. 167 # debuggable, but still have some tests run the full browser.
167 # Also: we don't have DumpRenderTree on Windows yet 168 # Also: we don't have DumpRenderTree on Windows yet
168 if browser == 'chrome' and not system.startswith('win'): 169 if browser == 'chrome' and not system.startswith('win'):
169 TestStep('browser', mode, system, 'frogium', tests, flags) 170 TestStep('browser', mode, system, 'frogium', tests, flags)
170 else: 171 else:
171 TestStep(browser, mode, system, 'webdriver', tests, 172 TestStep(browser, mode, system, 'webdriver', tests,
(...skipping 27 matching lines...) Expand all
199 200
200 status = TestFrog(arch, mode, system, browser, ['--checked']) 201 status = TestFrog(arch, mode, system, browser, ['--checked'])
201 if status != 0: 202 if status != 0:
202 print '@@@STEP_FAILURE@@@' 203 print '@@@STEP_FAILURE@@@'
203 204
204 return status 205 return status
205 206
206 207
207 if __name__ == '__main__': 208 if __name__ == '__main__':
208 sys.exit(main()) 209 sys.exit(main())
OLDNEW
« no previous file with comments | « frog/presubmit.py ('k') | frog/tests/frog/src/NativeClassAvoidsHiddenNameFrogTest.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698