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

Side by Side Diff: tools/bots/compiler.py

Issue 226243002: Run language tests on ie11, too to ensure it's not the test framework problem. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 8 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 | « no previous file | no next file » | 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) 2012, the Dart project authors. Please see the AUTHORS file 3 # Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
4 # for details. All rights reserved. Use of this source code is governed by a 4 # for details. All rights reserved. Use of this source code is governed by a
5 # BSD-style license that can be found in the LICENSE file. 5 # BSD-style license that can be found in the LICENSE file.
6 6
7 """ 7 """
8 Dart2js buildbot steps 8 Dart2js buildbot steps
9 9
10 Runs tests for the dart2js compiler. 10 Runs tests for the dart2js compiler.
(...skipping 118 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 system == 'linux' and platform.system() != 'Linux'): 129 system == 'linux' and platform.system() != 'Linux'):
130 print ('Error: You cannot emulate a buildbot with a platform different ' 130 print ('Error: You cannot emulate a buildbot with a platform different '
131 'from your own.') 131 'from your own.')
132 return None 132 return None
133 return bot.BuildInfo(compiler, runtime, mode, system, checked, host_checked, 133 return bot.BuildInfo(compiler, runtime, mode, system, checked, host_checked,
134 minified, shard_index, total_shards, is_buildbot, 134 minified, shard_index, total_shards, is_buildbot,
135 test_set, csp, arch, dart2js_full) 135 test_set, csp, arch, dart2js_full)
136 136
137 137
138 def NeedsXterm(compiler, runtime): 138 def NeedsXterm(compiler, runtime):
139 return runtime in ['ie9', 'ie10', 'ie11', 'chrome', 'safari', 'opera', 139 return runtime in ['ie9', 'ie10', 'ie11', 'chrome', 'safari', 'opera',
140 'ff', 'drt', 'dartium'] 140 'ff', 'drt', 'dartium']
141 141
142 142
143 def TestStepName(name, runtime, flags): 143 def TestStepName(name, runtime, flags):
144 # Filter out flags with '=' as this breaks the /stats feature of the 144 # Filter out flags with '=' as this breaks the /stats feature of the
145 # build bot. 145 # build bot.
146 flags = [x for x in flags if not '=' in x] 146 flags = [x for x in flags if not '=' in x]
147 step_name = '%s-%s tests %s' % (name, runtime, ' '.join(flags)) 147 step_name = '%s-%s tests %s' % (name, runtime, ' '.join(flags))
148 return step_name.strip() 148 return step_name.strip()
149 149
(...skipping 105 matching lines...) Expand 10 before | Expand all | Expand 10 after
255 if runtime == 'd8': 255 if runtime == 'd8':
256 # The dart2js compiler isn't self-hosted (yet) so we run its 256 # The dart2js compiler isn't self-hosted (yet) so we run its
257 # unit tests on the VM. We avoid doing this on the builders 257 # unit tests on the VM. We avoid doing this on the builders
258 # that run the browser tests to cut down on the cycle time. 258 # that run the browser tests to cut down on the cycle time.
259 unit_test_flags = [flag for flag in flags if flag.startswith('--shard')] 259 unit_test_flags = [flag for flag in flags if flag.startswith('--shard')]
260 # Run the unit tests in checked mode (the VM's checked mode). 260 # Run the unit tests in checked mode (the VM's checked mode).
261 unit_test_flags.append('--checked') 261 unit_test_flags.append('--checked')
262 TestStep("dart2js_unit", mode, system, 'none', 'vm', ['dart2js'], 262 TestStep("dart2js_unit", mode, system, 'none', 'vm', ['dart2js'],
263 unit_test_flags, arch) 263 unit_test_flags, arch)
264 264
265 if compiler == 'dart2js' and runtime in ['ie10', 'ie11']: 265 if compiler == 'dart2js' and runtime in ['ie10']:
266 TestStep(compiler, mode, system, compiler, runtime, 266 TestStep(compiler, mode, system, compiler, runtime,
267 ['html', 'pkg', 'samples'], flags, arch) 267 ['html', 'pkg', 'samples'], flags, arch)
268 elif compiler == 'dart2js' and runtime in ['ie11']:
269 #TODO(efortuna): Remove this branch.
270 TestStep(compiler, mode, system, compiler, runtime,
271 ['html', 'pkg', 'samples', 'language'], flags, arch)
268 else: 272 else:
269 # Run the default set of test suites. 273 # Run the default set of test suites.
270 TestStep(compiler, mode, system, compiler, 274 TestStep(compiler, mode, system, compiler,
271 runtime, [], flags, arch) 275 runtime, [], flags, arch)
272 276
273 if compiler == 'dart2js': 277 if compiler == 'dart2js':
274 # TODO(kasperl): Consider running peg and css tests too. 278 # TODO(kasperl): Consider running peg and css tests too.
275 extras = ['dart2js_extra', 'dart2js_native'] 279 extras = ['dart2js_extra', 'dart2js_native']
276 extras_flags = flags 280 extras_flags = flags
277 if (system == 'linux' 281 if (system == 'linux'
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
372 """ 376 """
373 with bot.BuildStep('Build SDK'): 377 with bot.BuildStep('Build SDK'):
374 args = [sys.executable, './tools/build.py', '--mode=' + build_info.mode, 378 args = [sys.executable, './tools/build.py', '--mode=' + build_info.mode,
375 '--arch=' + build_info.arch, 'dart2js_bot'] 379 '--arch=' + build_info.arch, 'dart2js_bot']
376 print 'Build SDK and d8: %s' % (' '.join(args)) 380 print 'Build SDK and d8: %s' % (' '.join(args))
377 bot.RunProcess(args) 381 bot.RunProcess(args)
378 382
379 383
380 if __name__ == '__main__': 384 if __name__ == '__main__':
381 bot.RunBot(GetBuildInfo, RunCompilerTests, build_step=BuildCompiler) 385 bot.RunBot(GetBuildInfo, RunCompilerTests, build_step=BuildCompiler)
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698