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

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

Issue 9649015: Change string used to annotate static type issues to 'static type warning' in multitest (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Merged up to tip Created 8 years, 9 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') | 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) 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 /** 5 /**
6 * Classes and methods for enumerating and preparing tests. 6 * Classes and methods for enumerating and preparing tests.
7 * 7 *
8 * This library includes: 8 * This library includes:
9 * 9 *
10 * - Creating tests by listing all the Dart files in certain directories, 10 * - Creating tests by listing all the Dart files in certain directories,
(...skipping 321 matching lines...) Expand 10 before | Expand all | Expand 10 after
332 case 'legium': 332 case 'legium':
333 case 'webdriver': 333 case 'webdriver':
334 enqueueBrowserTest(filename, testName, optionsFromFile, 334 enqueueBrowserTest(filename, testName, optionsFromFile,
335 expectations, isNegative); 335 expectations, isNegative);
336 break; 336 break;
337 default: 337 default:
338 isNegative = isNegative || 338 isNegative = isNegative ||
339 (configuration['checked'] && info.isNegativeIfChecked); 339 (configuration['checked'] && info.isNegativeIfChecked);
340 340
341 if (configuration['component'] == 'dartc') { 341 if (configuration['component'] == 'dartc') {
342 // dartc can detect static type errors by the 342 // dartc can detect static type warnings by the
343 // format of the error line 343 // format of the error line
344 if (info.hasFatalTypeErrors) { 344 if (info.hasFatalTypeErrors) {
345 isNegative = true; 345 isNegative = true;
346 } else if (info.hasRuntimeErrors) { 346 } else if (info.hasRuntimeErrors) {
347 isNegative = false; 347 isNegative = false;
348 } 348 }
349 } 349 }
350 350
351 var argumentLists = argumentListsFromFile(filename, 351 var argumentLists = argumentListsFromFile(filename,
352 optionsFromFile); 352 optionsFromFile);
(...skipping 405 matching lines...) Expand 10 before | Expand all | Expand 10 after
758 758
759 return result; 759 return result;
760 } 760 }
761 761
762 Map readOptionsFromFile(String filename) { 762 Map readOptionsFromFile(String filename) {
763 RegExp testOptionsRegExp = const RegExp(@"// VMOptions=(.*)"); 763 RegExp testOptionsRegExp = const RegExp(@"// VMOptions=(.*)");
764 RegExp dartOptionsRegExp = const RegExp(@"// DartOptions=(.*)"); 764 RegExp dartOptionsRegExp = const RegExp(@"// DartOptions=(.*)");
765 RegExp otherScriptsRegExp = const RegExp(@"// OtherScripts=(.*)"); 765 RegExp otherScriptsRegExp = const RegExp(@"// OtherScripts=(.*)");
766 RegExp multiTestRegExp = const RegExp(@"/// [0-9][0-9]:(.*)"); 766 RegExp multiTestRegExp = const RegExp(@"/// [0-9][0-9]:(.*)");
767 RegExp staticTypeRegExp = 767 RegExp staticTypeRegExp =
768 const RegExp(@"/// ([0-9][0-9]:){0,1}\s*static type error"); 768 const RegExp(@"/// ([0-9][0-9]:){0,1}\s*static type warning");
769 RegExp compileTimeRegExp = 769 RegExp compileTimeRegExp =
770 const RegExp(@"/// ([0-9][0-9]:){0,1}\s*compile-time error"); 770 const RegExp(@"/// ([0-9][0-9]:){0,1}\s*compile-time error");
771 RegExp staticCleanRegExp = const RegExp(@"// @static-clean"); 771 RegExp staticCleanRegExp = const RegExp(@"// @static-clean");
772 RegExp leadingHashRegExp = const RegExp(@"^#", multiLine: true); 772 RegExp leadingHashRegExp = const RegExp(@"^#", multiLine: true);
773 RegExp isolateStubsRegExp = const RegExp(@"// IsolateStubs=(.*)"); 773 RegExp isolateStubsRegExp = const RegExp(@"// IsolateStubs=(.*)");
774 RegExp domImportRegExp = 774 RegExp domImportRegExp =
775 const RegExp(@"^#import.*(dart:(dom|html)|html\.dart).*\)", 775 const RegExp(@"^#import.*(dart:(dom|html)|html\.dart).*\)",
776 multiLine: true); 776 multiLine: true);
777 RegExp libraryDefinitionRegExp = 777 RegExp libraryDefinitionRegExp =
778 const RegExp(@"^#library\(", multiLine: true); 778 const RegExp(@"^#library\(", multiLine: true);
(...skipping 437 matching lines...) Expand 10 before | Expand all | Expand 10 after
1216 * $noCrash tests are expected to be flaky but not crash 1216 * $noCrash tests are expected to be flaky but not crash
1217 * $pass tests are expected to pass 1217 * $pass tests are expected to pass
1218 * $failOk tests are expected to fail that we won't fix 1218 * $failOk tests are expected to fail that we won't fix
1219 * $fail tests are expected to fail that we should fix 1219 * $fail tests are expected to fail that we should fix
1220 * $crash tests are expected to crash that we should fix 1220 * $crash tests are expected to crash that we should fix
1221 * $timeout tests are allowed to timeout 1221 * $timeout tests are allowed to timeout
1222 """; 1222 """;
1223 print(report); 1223 print(report);
1224 } 1224 }
1225 } 1225 }
OLDNEW
« no previous file with comments | « tools/testing/dart/test_runner.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698