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

Side by Side Diff: tests/compiler/dart2js/analyze_test_test.dart

Issue 1838903003: Make analyze_test_test less verbose (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Created 4 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
OLDNEW
1 // Copyright (c) 2016, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2016, 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 dart2js.analyze_test.test; 5 library dart2js.analyze_test.test;
6 6
7 import 'dart:io'; 7 import 'dart:io';
8 8
9 import 'package:async_helper/async_helper.dart'; 9 import 'package:async_helper/async_helper.dart';
10 import 'package:compiler/src/apiimpl.dart' show 10 import 'package:compiler/src/apiimpl.dart' show
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 "one_line_dart_program.dart", 49 "one_line_dart_program.dart",
50 "sourcemaps/invokes_test_file.dart", 50 "sourcemaps/invokes_test_file.dart",
51 "cps_ir/input/", 51 "cps_ir/input/",
52 // No longer maintained: 52 // No longer maintained:
53 "backend_dart/", 53 "backend_dart/",
54 // Broken tests: 54 // Broken tests:
55 "http_test.dart", 55 "http_test.dart",
56 ]; 56 ];
57 57
58 main(List<String> arguments) { 58 main(List<String> arguments) {
59 // TODO(johnniwinther): Remove verbose flag when test issues have been 59 List<String> options = <String>[];
60 // resolved.
61 List<String> options = <String>[Flags.verbose];
62 if (arguments.contains('-v') || arguments.contains(Flags.verbose)) { 60 if (arguments.contains('-v') || arguments.contains(Flags.verbose)) {
63 options.add(Flags.verbose); 61 options.add(Flags.verbose);
64 } 62 }
Bill Hesse 2016/03/29 11:56:26 No need to process a --hide-whitelisted argument?
Johnni Winther 2016/03/30 07:02:30 I deliberately med show-whitelisted the default (f
65 asyncTest(() async { 63 asyncTest(() async {
66 List<Uri> uriList = <Uri>[]; 64 List<Uri> uriList = <Uri>[];
67 Directory dir = 65 Directory dir =
68 new Directory.fromUri(Uri.base.resolve('tests/compiler/dart2js/')); 66 new Directory.fromUri(Uri.base.resolve('tests/compiler/dart2js/'));
69 for (FileSystemEntity entity in dir.listSync(recursive: true)) { 67 for (FileSystemEntity entity in dir.listSync(recursive: true)) {
70 if (entity is File && entity.path.endsWith('.dart')) { 68 if (entity is File && entity.path.endsWith('.dart')) {
71 Uri file = Uri.base.resolve(nativeToUriPath(entity.path)); 69 Uri file = Uri.base.resolve(nativeToUriPath(entity.path));
72 if (!SKIP_LIST.any((skip) => file.path.contains(skip))) { 70 if (!SKIP_LIST.any((skip) => file.path.contains(skip))) {
73 uriList.add(file); 71 uriList.add(file);
74 } 72 }
75 } 73 }
76 } 74 }
77 await analyze( 75 await analyze(
78 uriList, WHITE_LIST, mode: AnalysisMode.URI, options: options); 76 uriList, WHITE_LIST, mode: AnalysisMode.URI, options: options);
79 }); 77 });
80 } 78 }
OLDNEW
« tests/compiler/dart2js/analyze_helper.dart ('K') | « tests/compiler/dart2js/analyze_helper.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698