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

Unified Diff: dart/tests/compiler/dart2js/warnings_checker.dart

Issue 136753002: Version 1.1.0-dev.5.9 (Closed) Base URL: http://dart.googlecode.com/svn/trunk/
Patch Set: Created 6 years, 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « dart/tests/compiler/dart2js/mixin_language_test.dart ('k') | dart/tests/language/checked_mode_helper.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: dart/tests/compiler/dart2js/warnings_checker.dart
===================================================================
--- dart/tests/compiler/dart2js/warnings_checker.dart (revision 31731)
+++ dart/tests/compiler/dart2js/warnings_checker.dart (working copy)
@@ -16,10 +16,11 @@
import '../../../sdk/lib/_internal/compiler/implementation/util/uri_extras.dart';
import 'dart:convert';
-void checkWarnings(Map<String, dynamic> tests) {
+void checkWarnings(Map<String, dynamic> tests, [List<String> arguments]) {
bool isWindows = Platform.isWindows;
Uri script = currentDirectory.resolveUri(Platform.script);
bool warningsMismatch = false;
+ bool verbose = arguments != null && arguments.contains('-v');
asyncTest(() => Future.forEach(tests.keys, (String test) {
Uri uri = script.resolve('../../$test');
String source = UTF8.decode(readAll(uriPathToNative(uri.path)));
@@ -38,7 +39,7 @@
var compiler = compilerFor(const {},
diagnosticHandler: collector,
options: ['--analyze-only'],
- showDiagnostics: false);
+ showDiagnostics: verbose);
return compiler.run(uri).then((_) {
Map<String, List<int>> statusMap = tests[test];
// Line numbers with known unexpected warnings.
« no previous file with comments | « dart/tests/compiler/dart2js/mixin_language_test.dart ('k') | dart/tests/language/checked_mode_helper.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698