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

Side by Side Diff: tools/testing/dart/test_suite.dart

Issue 9361040: Reduce flakiness of browser tests. (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 | « tools/testing/dart/test_runner.dart ('k') | tools/testing/perf_testing/create_graph.py » ('j') | 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) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, 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 #library("test_suite"); 5 #library("test_suite");
6 6
7 #import("dart:io"); 7 #import("dart:io");
8 #import("status_file_parser.dart"); 8 #import("status_file_parser.dart");
9 #import("test_runner.dart"); 9 #import("test_runner.dart");
10 #import("multitest.dart"); 10 #import("multitest.dart");
(...skipping 423 matching lines...) Expand 10 before | Expand all | Expand 10 after
434 if (new Platform().operatingSystem() == 'windows') { 434 if (new Platform().operatingSystem() == 'windows') {
435 // Firefox on Windows does not like absolute file path names that start 435 // Firefox on Windows does not like absolute file path names that start
436 // with 'C:' adding 'file:///' solves the problem. 436 // with 'C:' adding 'file:///' solves the problem.
437 filePrefix = 'file:///'; 437 filePrefix = 'file:///';
438 } 438 }
439 htmlTest.writeStringSync(GetHtmlContents( 439 htmlTest.writeStringSync(GetHtmlContents(
440 filename, 440 filename,
441 '$filePrefix$dartDir/client/testing/unittest/test_controller.js', 441 '$filePrefix$dartDir/client/testing/unittest/test_controller.js',
442 scriptType, 442 scriptType,
443 filePrefix + scriptPath)); 443 filePrefix + scriptPath));
444 htmlTest.flushSync();
444 htmlTest.closeSync(); 445 htmlTest.closeSync();
445 446
446 List<String> compilerArgs = TestUtils.standardOptions(configuration); 447 List<String> compilerArgs = TestUtils.standardOptions(configuration);
447 String compilerExecutable = TestUtils.compilerPath(configuration); 448 String compilerExecutable = TestUtils.compilerPath(configuration);
448 switch (component) { 449 switch (component) {
449 case 'chromium': 450 case 'chromium':
450 compilerArgs.addAll(['--work', tempDir.path]); 451 compilerArgs.addAll(['--work', tempDir.path]);
451 compilerArgs.addAll(vmOptions); 452 compilerArgs.addAll(vmOptions);
452 compilerArgs.add('--ignore-unrecognized-flags'); 453 compilerArgs.add('--ignore-unrecognized-flags');
453 // TODO(zundel): remove assumption of generated code from dartc 454 // TODO(zundel): remove assumption of generated code from dartc
(...skipping 602 matching lines...) Expand 10 before | Expand all | Expand 10 after
1056 * $noCrash tests are expected to be flaky but not crash 1057 * $noCrash tests are expected to be flaky but not crash
1057 * $pass tests are expected to pass 1058 * $pass tests are expected to pass
1058 * $failOk tests are expected to fail that we won't fix 1059 * $failOk tests are expected to fail that we won't fix
1059 * $fail tests are expected to fail that we should fix 1060 * $fail tests are expected to fail that we should fix
1060 * $crash tests are expected to crash that we should fix 1061 * $crash tests are expected to crash that we should fix
1061 * $timeout tests are allowed to timeout 1062 * $timeout tests are allowed to timeout
1062 """; 1063 """;
1063 print(report); 1064 print(report);
1064 } 1065 }
1065 } 1066 }
OLDNEW
« no previous file with comments | « tools/testing/dart/test_runner.dart ('k') | tools/testing/perf_testing/create_graph.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698