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

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

Issue 1247773002: Add access the Message in CompilerDiagnostics.report. (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Updated cf. comment. Created 5 years, 5 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 | « tests/compiler/dart2js/missing_file_test.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/compiler/dart2js/package_root_test.dart
diff --git a/tests/compiler/dart2js/package_root_test.dart b/tests/compiler/dart2js/package_root_test.dart
index f1e6ee6eb89f99212f1fe7cb8ee1248399154e99..122170ceae305030ac8e931e358b165e6d1f13f0 100644
--- a/tests/compiler/dart2js/package_root_test.dart
+++ b/tests/compiler/dart2js/package_root_test.dart
@@ -12,6 +12,8 @@ import 'package:async_helper/async_helper.dart';
import 'package:expect/expect.dart';
import 'package:compiler/compiler.dart'
show DiagnosticHandler, Diagnostic, PackagesDiscoveryProvider;
+import 'package:compiler/src/dart2jslib.dart'
+ show MessageKind;
import 'package:package_config/packages.dart';
import 'memory_compiler.dart';
@@ -30,60 +32,58 @@ main() {}
final Uri PACKAGE_CONFIG_URI = Uri.parse('memory:package.config');
-void runCompiler(Uri main,
- bool checkError(DiagnosticMessage message),
- {Uri packageRoot,
- Uri packageConfig,
- PackagesDiscoveryProvider packagesDiscoveryProvider}) {
+Future runTest(Uri main,
+ MessageKind expectedMessageKind,
+ {Uri packageRoot,
+ Uri packageConfig,
+ PackagesDiscoveryProvider packagesDiscoveryProvider}) async {
DiagnosticCollector collector = new DiagnosticCollector();
- Compiler compiler = compilerFor(
- MEMORY_SOURCE_FILES,
+ await runCompiler(
+ entryPoint: main,
+ memorySourceFiles: MEMORY_SOURCE_FILES,
diagnosticHandler: collector,
packageRoot: packageRoot,
packageConfig: packageConfig,
packagesDiscoveryProvider: packagesDiscoveryProvider);
-
- asyncTest(() => compiler.run(main).then((_) {
- Expect.equals(1, collector.errors.length,
- "Unexpected errors: ${collector.errors}");
- Expect.isTrue(checkError(collector.errors.first),
- "Unexpected error: ${collector.errors.first}");
- }));
+ Expect.equals(1, collector.errors.length,
+ "Unexpected errors: ${collector.errors}");
+ Expect.equals(expectedMessageKind, collector.errors.first.message.kind,
+ "Unexpected error: ${collector.errors.first}");
}
void main() {
- Uri script = currentDirectory.resolveUri(Platform.script);
- Uri packageRoot = script.resolve('./packages/');
-
- PackagesDiscoveryProvider noPackagesDiscovery = (Uri uri) {
- return new Future.value(Packages.noPackages);
- };
-
- bool containsErrorReading(DiagnosticMessage message) {
- return message.message.contains("Error reading ");
- }
-
- bool isLibraryNotFound(DiagnosticMessage message) {
- return message.message.startsWith("Library not found ");
- }
-
- runCompiler(Uri.parse('memory:main.dart'),
- containsErrorReading,
- packageRoot: packageRoot);
- runCompiler(Uri.parse('memory:main.dart'),
- isLibraryNotFound,
- packageConfig: PACKAGE_CONFIG_URI);
- runCompiler(Uri.parse('memory:main.dart'),
- isLibraryNotFound,
- packagesDiscoveryProvider: noPackagesDiscovery);
-
- runCompiler(Uri.parse('package:foo/foo.dart'),
- containsErrorReading,
- packageRoot: packageRoot);
- runCompiler(Uri.parse('package:foo/foo.dart'),
- isLibraryNotFound,
- packageConfig: PACKAGE_CONFIG_URI);
- runCompiler(Uri.parse('package:foo/foo.dart'),
- isLibraryNotFound,
- packagesDiscoveryProvider: noPackagesDiscovery);
+ asyncTest(() async {
+ Uri script = currentDirectory.resolveUri(Platform.script);
+ Uri packageRoot = script.resolve('./packages/');
+
+ PackagesDiscoveryProvider noPackagesDiscovery = (Uri uri) {
+ return new Future.value(Packages.noPackages);
+ };
+
+ await runTest(
+ Uri.parse('memory:main.dart'),
+ MessageKind.READ_SCRIPT_ERROR,
+ packageRoot: packageRoot);
+ await runTest(
+ Uri.parse('memory:main.dart'),
+ MessageKind.LIBRARY_NOT_FOUND,
+ packageConfig: PACKAGE_CONFIG_URI);
+ await runTest(
+ Uri.parse('memory:main.dart'),
+ MessageKind.LIBRARY_NOT_FOUND,
+ packagesDiscoveryProvider: noPackagesDiscovery);
+
+ await runTest(
+ Uri.parse('package:foo/foo.dart'),
+ MessageKind.READ_SELF_ERROR,
+ packageRoot: packageRoot);
+ await runTest(
+ Uri.parse('package:foo/foo.dart'),
+ MessageKind.LIBRARY_NOT_FOUND,
+ packageConfig: PACKAGE_CONFIG_URI);
+ await runTest(
+ Uri.parse('package:foo/foo.dart'),
+ MessageKind.LIBRARY_NOT_FOUND,
+ packagesDiscoveryProvider: noPackagesDiscovery);
+ });
}
« no previous file with comments | « tests/compiler/dart2js/missing_file_test.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698