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

Unified Diff: pkg/code_transformers/test/resolver_test.dart

Issue 140203007: Adding package:code_transformers for unifying common transformers code (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 10 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
Index: pkg/code_transformers/test/resolver_test.dart
diff --git a/pkg/code_transformers/test/resolver_test.dart b/pkg/code_transformers/test/resolver_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..b8a5dcc1149c0463a6bb1ee6707f3774901848d0
--- /dev/null
+++ b/pkg/code_transformers/test/resolver_test.dart
@@ -0,0 +1,283 @@
+// Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+library code_transformers.test.resolver_test;
+
+import 'package:barback/barback.dart';
+import 'package:code_transformers/resolver.dart';
+import 'package:code_transformers/tests.dart';
+import 'package:unittest/compact_vm_config.dart';
+import 'package:unittest/unittest.dart';
+
+main() {
+ useCompactVMConfiguration();
+
+ var entryPoint = new AssetId('a', 'web/main.dart');
+ var transformer = new ResolverTransformer(dartSdkDirectory,
+ (asset) => asset.id == entryPoint);
+
+ var phases = [[transformer]];
+
+ group('Resolver', () {
+
+ test('should handle empty files', () {
+ return updateTransformers(phases,
+ inputs: {
+ 'a|web/main.dart': '',
+ }).then((_) {
+ var resolver = transformer.getResolver(entryPoint);
+ var source = resolver.sources[entryPoint];
+ expect(source.modificationStamp, 1);
+
+ var lib = resolver.entryLibrary;
+ expect(lib, isNotNull);
+ expect(lib.entryPoint, isNull);
+ });
+ });
+
+ test('should update when sources change', () {
+ return updateTransformers(phases,
+ inputs: {
+ 'a|web/main.dart': ''' main() {} ''',
+ }).then((_) {
+ var resolver = transformer.getResolver(entryPoint);
+ var source = resolver.sources[entryPoint];
+ expect(source.modificationStamp, 2);
+
+ var lib = resolver.entryLibrary;
+ expect(lib, isNotNull);
+ expect(lib.entryPoint, isNotNull);
+ });
+ });
+
+ test('should follow imports', () {
+ return updateTransformers(phases,
+ inputs: {
+ 'a|web/main.dart': '''
+ import 'a.dart';
+
+ main() {
+ } ''',
+ 'a|web/a.dart': '''
Siggi Cherem (dart-lang) 2014/02/14 02:13:37 unindent (here and in other tests below)
blois 2014/02/19 21:42:58 Ack. Not sure how that happened!
+ library a;
+ ''',
+ }).then((_) {
+ var resolver = transformer.getResolver(entryPoint);
+ var lib = resolver.entryLibrary;
+ expect(lib.importedLibraries.length, 2);
+ var libA = lib.importedLibraries.where((l) => l.name == 'a').single;
+ expect(libA.getType('Foo'), isNull);
+ });
+ });
+
+ test('should update changed imports', () {
+ return updateTransformers(phases,
+ inputs: {
+ 'a|web/main.dart': '''
+ import 'a.dart';
+
+ main() {
+ } ''',
+ 'a|web/a.dart': '''
+ library a;
+ class Foo {}
+ ''',
+ }).then((_) {
+ var lib = transformer.getResolver(entryPoint).entryLibrary;
+ expect(lib.importedLibraries.length, 2);
+ var libA = lib.importedLibraries.where((l) => l.name == 'a').single;
+ expect(libA.getType('Foo'), isNotNull);
+ });
+ });
+
+ test('should follow package imports', () {
+ return updateTransformers(phases,
+ inputs: {
+ 'a|web/main.dart': '''
+ import 'package:b/b.dart';
+
+ main() {
+ } ''',
+ 'b|lib/b.dart': '''
+ library b;
+ ''',
+ }).then((_) {
+ var lib = transformer.getResolver(entryPoint).entryLibrary;
+ expect(lib.importedLibraries.length, 2);
+ var libB = lib.importedLibraries.where((l) => l.name == 'b').single;
+ expect(libB.getType('Foo'), isNull);
+ });
+ });
+
+ test('should update on changed package imports', () {
+ return updateTransformers(phases,
+ inputs: {
+ 'a|web/main.dart': '''
+ import 'package:b/b.dart';
+
+ main() {
+ } ''',
+ 'b|lib/b.dart': '''
+ library b;
+ class Bar {}
+ ''',
+ }).then((_) {
+ var lib = transformer.getResolver(entryPoint).entryLibrary;
+ expect(lib.importedLibraries.length, 2);
+ var libB = lib.importedLibraries.where((l) => l.name == 'b').single;
+ expect(libB.getType('Bar'), isNotNull);
+ });
+ });
+
+ test('should handle deleted files', () {
+ return updateTransformers(phases,
+ inputs: {
+ 'a|web/main.dart': '''
+ import 'package:b/b.dart';
+
+ main() {
+ } ''',
+ },
+ messages: [
+ 'error: Unable to find asset for "package:b/b.dart"',
+ 'error: Unable to find asset for "package:b/b.dart"',
+ ]).then((_) {
+ var lib = transformer.getResolver(entryPoint).entryLibrary;
+ expect(lib.importedLibraries.length, 1);
+ });
+ });
+
+ test('should fail on absolute URIs', () {
+ return updateTransformers(phases,
+ inputs: {
+ 'a|web/main.dart': '''
+ import '/b.dart';
+
+ main() {
+ } ''',
+ },
+ messages: [
+ // First from the AST walker
+ 'error: absolute paths not allowed: "/b.dart" (web/main.dart 0 14)',
+ // Then two from the resolver.
+ 'error: absolute paths not allowed: "/b.dart"',
+ 'error: absolute paths not allowed: "/b.dart"',
+ ]).then((_) {
+ var lib = transformer.getResolver(entryPoint).entryLibrary;
+ expect(lib.importedLibraries.length, 1);
+ });
+ });
+
+ test('should list all libraries', () {
+ return updateTransformers(phases,
+ inputs: {
+ 'a|web/main.dart': '''
+ library a.main;
+ import 'package:a/a.dart';
+ import 'package:a/b.dart';
+ ''',
+ 'a|lib/a.dart': 'library a.a;\n import "package:a/c.dart";',
+ 'a|lib/b.dart': 'library a.b;\n import "c.dart";',
+ 'a|lib/c.dart': 'library a.c;'
+ }).then((_) {
+ var resolver = transformer.getResolver(entryPoint);
+ var libs = resolver.libraries.where((l) => !l.isInSdk);
+ expect(libs.map((l) => l.name), unorderedEquals([
+ 'a.main',
+ 'a.a',
+ 'a.b',
+ 'a.c',
+ ]));
+ });
+ });
+
+ test('should resolve types and library uris', () {
+ return updateTransformers(phases,
+ inputs: {
+ 'a|web/main.dart': '''
+ import 'dart:core';
+ import 'package:a/a.dart';
+ import 'package:a/b.dart';
+ import 'sub_dir/d.dart';
+ class Foo {}
+ ''',
+ 'a|lib/a.dart': 'library a.a;\n import "package:a/c.dart";',
+ 'a|lib/b.dart': 'library a.b;\n import "c.dart";',
+ 'a|lib/c.dart': '''
+ library a.c;
+ class Bar {}
+ ''',
+ 'a|web/sub_dir/d.dart': '''
+ library a.web.sub_dir.d;
+ class Baz{}
+ ''',
+ }).then((_) {
+ var resolver = transformer.getResolver(entryPoint);
+
+ var a = resolver.getLibraryByName('a.a');
+ expect(a, isNotNull);
+ expect(resolver.getImportUri(a).toString(),
+ 'package:a/a.dart');
+ expect(resolver.getLibraryByUri(Uri.parse('package:a/a.dart')), a);
+
+ var main = resolver.getLibraryByName('');
+ expect(main, isNotNull);
+ expect(resolver.getImportUri(main), isNull);
+
+ var fooType = resolver.getType('Foo');
+ expect(fooType, isNotNull);
+ expect(fooType.library, main);
+
+ var barType = resolver.getType('a.c.Bar');
+ expect(barType, isNotNull);
+ expect(resolver.getImportUri(barType.library).toString(),
+ 'package:a/c.dart');
+ expect(resolver.getSourceAssetId(barType),
+ new AssetId('a', 'lib/c.dart'));
+
+ var bazType = resolver.getType('a.web.sub_dir.d.Baz');
+ expect(bazType, isNotNull);
+ expect(resolver.getImportUri(bazType.library), isNull);
+ expect(resolver
+ .getImportUri(bazType.library, from: entryPoint).toString(),
+ 'sub_dir/d.dart');
+
+ var hashMap = resolver.getType('dart.collection.HashMap');
+ expect(resolver.getImportUri(hashMap.library).toString(),
+ 'dart:collection');
+ expect(resolver.getLibraryByUri(Uri.parse('dart:collection')),
+ hashMap.library);
+
+ });
+ });
+
+ test('deleted files should be removed', () {
+ return updateTransformers(phases,
+ inputs: {
+ 'a|web/main.dart': '''import 'package:a/a.dart';''',
+ 'a|lib/a.dart': '''import 'package:a/b.dart';''',
+ 'a|lib/b.dart': '''class Engine{}''',
+ }).then((_) {
+ var resolver = transformer.getResolver(entryPoint);
+ var engine = resolver.getType('Engine');
+ var uri = resolver.getImportUri(engine.library);
+ expect(uri.toString(), 'package:a/b.dart');
+ }).then((_) {
+ return updateTransformers(phases,
+ inputs: {
+ 'a|web/main.dart': '''import 'package:a/a.dart';''',
+ 'a|lib/a.dart': '''lib a;\n class Engine{}'''
+ });
+ }).then((_) {
+ var resolver = transformer.getResolver(entryPoint);
+ var engine = resolver.getType('Engine');
+ var uri = resolver.getImportUri(engine.library);
+ expect(uri.toString(), 'package:a/a.dart');
+
+ // Make sure that we haven't leaked any sources.
+ expect(resolver.sources.length, 2);
+ });
+ });
+ });
+}

Powered by Google App Engine
This is Rietveld 408576698