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

Side by Side Diff: tools/testing/test_configuration.py

Issue 8679025: Fix handling of VMOptions= in multi tests (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 9 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 | 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 # Copyright (c) 2011, the Dart project authors. Please see the AUTHORS file 1 # Copyright (c) 2011, the Dart project authors. Please see the AUTHORS file
2 # for details. All rights reserved. Use of this source code is governed by a 2 # for details. All rights reserved. Use of this source code is governed by a
3 # BSD-style license that can be found in the LICENSE file. 3 # BSD-style license that can be found in the LICENSE file.
4 4
5 """Common Testconfiguration subclasses used to define a class of tests.""" 5 """Common Testconfiguration subclasses used to define a class of tests."""
6 6
7 import atexit 7 import atexit
8 import fileinput 8 import fileinput
9 import os 9 import os
10 import re 10 import re
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
81 return [test_case.BrowserTestCase( 81 return [test_case.BrowserTestCase(
82 self.context, test_path, filename, False, mode, arch, component, 82 self.context, test_path, filename, False, mode, arch, component,
83 self.flags)] 83 self.flags)]
84 else: 84 else:
85 tests = [] 85 tests = []
86 if tags: 86 if tags:
87 for tag in sorted(tags): 87 for tag in sorted(tags):
88 kind, test_source = tags[tag] 88 kind, test_source = tags[tag]
89 if not self.Contains(path, test_path + [tag]): 89 if not self.Contains(path, test_path + [tag]):
90 continue 90 continue
91 tests.append(test_case.MultiTestCase(self.context, 91 if vm_options_list:
92 test_path + [tag], 92 for options in vm_options_list:
93 test_source, 93 tests.append(test_case.MultiTestCase(self.context,
94 kind, 94 test_path + [tag],
95 mode, arch, component, 95 test_source,
96 self.flags)) 96 kind,
97 mode, arch, component,
98 options + self.flags))
99 else:
100 tests.append(test_case.MultiTestCase(self.context,
101 test_path + [tag],
102 test_source,
103 kind,
104 mode, arch, component,
105 self.flags))
97 else: 106 else:
98 if vm_options_list: 107 if vm_options_list:
99 for options in vm_options_list: 108 for options in vm_options_list:
100 tests.append(test_case.StandardTestCase(self.context, 109 tests.append(test_case.StandardTestCase(self.context,
101 test_path, filename, mode, arch, component, 110 test_path, filename, mode, arch, component,
102 options + self.flags)) 111 options + self.flags))
103 else: 112 else:
104 tests.append(test_case.StandardTestCase(self.context, 113 tests.append(test_case.StandardTestCase(self.context,
105 test_path, filename, mode, arch, component, self.flags)) 114 test_path, filename, mode, arch, component, self.flags))
106 return tests 115 return tests
(...skipping 199 matching lines...) Expand 10 before | Expand all | Expand 10 after
306 315
307 def GetTestStatus(self, sections, defs): 316 def GetTestStatus(self, sections, defs):
308 status = os.path.join(self.root, 'dartc.status') 317 status = os.path.join(self.root, 'dartc.status')
309 if os.path.exists(status): 318 if os.path.exists(status):
310 test.ReadConfigurationInto(status, sections, defs) 319 test.ReadConfigurationInto(status, sections, defs)
311 320
312 def _Cleanup(self, tests): 321 def _Cleanup(self, tests):
313 if not utils.Daemonize(): return 322 if not utils.Daemonize(): return
314 os.execlp('rm', *(['rm', '-rf'] + [t.temp_dir for t in tests])) 323 os.execlp('rm', *(['rm', '-rf'] + [t.temp_dir for t in tests]))
315 raise 324 raise
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