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

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

Issue 1750143005: Move more messages. (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Undefined messages. Created 4 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
« no previous file with comments | « pkg/dart_messages/lib/shared_messages.dart ('k') | tests/compiler/dart2js/resolver_test.dart » ('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) 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
11 CompilerImpl; 11 CompilerImpl;
12 import 'package:compiler/src/commandline_options.dart'; 12 import 'package:compiler/src/commandline_options.dart';
13 import 'package:compiler/src/diagnostics/messages.dart' show 13 import 'package:compiler/src/diagnostics/messages.dart' show
14 MessageKind; 14 MessageKind;
15 import 'package:compiler/src/filenames.dart' show 15 import 'package:compiler/src/filenames.dart' show
16 nativeToUriPath; 16 nativeToUriPath;
17 17
18 import 'analyze_helper.dart'; 18 import 'analyze_helper.dart';
19 import 'memory_compiler.dart'; 19 import 'memory_compiler.dart';
20 20
21 /** 21 /**
22 * Map of white-listed warnings and errors. 22 * Map of white-listed warnings and errors.
23 * 23 *
24 * Use an identifiable suffix of the file uri as key. Use a fixed substring of 24 * Use an identifiable suffix of the file uri as key. Use a fixed substring of
25 * the error/warning message in the list of white-listings for each file. 25 * the error/warning message in the list of white-listings for each file.
26 */ 26 */
27 // TODO(johnniwinther): Support canonical URIs as keys and message kinds as 27 // TODO(johnniwinther): Support canonical URIs as keys and message kinds as
28 // values. 28 // values.
29 const Map<String, List/*<String|MessageKind>*/> WHITE_LIST = const { 29 const Map<String, List/*<String|MessageKind>*/> WHITE_LIST = const {
30 // Several tests import mirrors; any of these might trigger the warning.
31 ".dart": const [
32 MessageKind.IMPORT_EXPERIMENTAL_MIRRORS,
33 ],
Johnni Winther 2016/03/07 10:25:18 Why did this change?
floitsch 2016/03/07 10:26:24 When I ran `tools/test.py dart2js` it complained.
floitsch 2016/03/07 13:52:06 Removed.
34 "/test/src/util/": const [ 30 "/test/src/util/": const [
35 "Library 'package:async/async.dart' doesn't export a " 31 "Library 'package:async/async.dart' doesn't export a "
36 "'ForkableStream' declaration.", 32 "'ForkableStream' declaration.",
37 ], 33 ],
38 "mirrors_test.dart": const [ 34 "mirrors_test.dart": const [
39 MessageKind.INVALID_SYMBOL, 35 MessageKind.INVALID_SYMBOL,
40 MessageKind.PRIVATE_IDENTIFIER, 36 MessageKind.PRIVATE_IDENTIFIER,
41 ], 37 ],
42 }; 38 };
43 39
(...skipping 30 matching lines...) Expand all
74 if (entity is File && entity.path.endsWith('.dart')) { 70 if (entity is File && entity.path.endsWith('.dart')) {
75 Uri file = Uri.base.resolve(nativeToUriPath(entity.path)); 71 Uri file = Uri.base.resolve(nativeToUriPath(entity.path));
76 if (!SKIP_LIST.any((skip) => file.path.contains(skip))) { 72 if (!SKIP_LIST.any((skip) => file.path.contains(skip))) {
77 uriList.add(file); 73 uriList.add(file);
78 } 74 }
79 } 75 }
80 } 76 }
81 await analyze(uriList, WHITE_LIST, mode: AnalysisMode.URI); 77 await analyze(uriList, WHITE_LIST, mode: AnalysisMode.URI);
82 }); 78 });
83 } 79 }
OLDNEW
« no previous file with comments | « pkg/dart_messages/lib/shared_messages.dart ('k') | tests/compiler/dart2js/resolver_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698