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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: pkg/polymer/test/build/code_extractor.dart

Issue 180933002: combine script extractor and import inliner (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
« pkg/polymer/lib/src/build/import_inliner.dart ('K') | « pkg/polymer/test/build/all_phases_test.dart ('k') | pkg/polymer/test/build/code_extractor_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/polymer/test/build/code_extractor.dart
diff --git a/pkg/polymer/test/build/code_extractor.dart b/pkg/polymer/test/build/code_extractor.dart
new file mode 100644
index 0000000000000000000000000000000000000000..b7fec379a17affaface1178c911a038d535f3e75
--- /dev/null
+++ b/pkg/polymer/test/build/code_extractor.dart
@@ -0,0 +1,202 @@
+// Copyright (c) 2013, 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.
+
+// TODO(jmesserly): merge into import_inliner_test.dart.
+// Keeping here for now so it's easier to see the diff.
+part of polymer.test.build.import_inliner_test;
+
+void codeExtractorTests() {
+ testPhases('no changes', phases, {
+ 'a|web/test.html': '<!DOCTYPE html><html></html>',
+ }, {
+ 'a|web/test.html': '<!DOCTYPE html><html></html>',
+ });
+
+ testPhases('single script, no library in script', phases, {
+ 'a|web/test.html':
+ '<!DOCTYPE html><html><head>'
+ '<script type="application/dart">main() { }</script>',
+ }, {
+ 'a|web/test.html':
+ '<!DOCTYPE html><html><head></head><body>'
+ '<script type="application/dart" src="test.html.dart"></script>'
+ '</body></html>',
+
+ 'a|web/test.html.dart':
+ 'library web_test_html;\nmain() { }',
+ });
+
+ testPhases('single script, with library', phases, {
+ 'a|web/test.html':
+ '<!DOCTYPE html><html><head>'
+ '<script type="application/dart">library f;\nmain() { }</script>',
+ }, {
+ 'a|web/test.html':
+ '<!DOCTYPE html><html><head></head><body>'
+ '<script type="application/dart" src="test.html.dart"></script>'
+ '</body></html>',
+
+ 'a|web/test.html.dart':
+ 'library f;\nmain() { }',
+ });
+
+ testPhases('under lib/ directory not transformed', phases, {
+ 'a|lib/test.html':
+ '<!DOCTYPE html><html><head>'
+ '<script type="application/dart">library f;\nmain() { }</script>',
+ }, {
+ 'a|lib/test.html':
+ '<!DOCTYPE html><html><head>'
+ '<script type="application/dart">library f;\nmain() { }</script>'
+ });
+
+ testPhases('multiple scripts - only emit first', phases, {
+ 'a|web/test.html':
+ '<!DOCTYPE html><html><head>'
+ '<script type="application/dart">library a1;\nmain1() { }</script>'
+ '<script type="application/dart">library a2;\nmain2() { }</script>',
+ }, {
+ 'a|web/test.html':
+ '<!DOCTYPE html><html><head></head><body>'
+ '<script type="application/dart" src="test.html.dart"></script>'
+ '</body></html>',
+
+ 'a|web/test.html.dart':
+ 'library a1;\nmain1() { }',
+ });
+
+ testPhases('multiple deeper scripts', phases, {
+ 'a|web/test.html':
+ '<!DOCTYPE html><html><head>'
+ '<script type="application/dart">main1() { }</script>'
+ '</head><body><div>'
+ '<script type="application/dart">main2() { }</script>'
+ '</div><div><div>'
+ '<script type="application/dart">main3() { }</script>'
+ '</div></div>'
+ }, {
+ 'a|web/test.html':
+ '<!DOCTYPE html><html><head>'
+ '</head><body>'
+ '<script type="application/dart" src="test.html.dart"></script>'
+ '<div></div><div><div>'
+ '</div></div></body></html>',
+
+ 'a|web/test.html.dart':
+ 'library web_test_html;\nmain1() { }',
+ });
Siggi Cherem (dart-lang) 2014/02/28 01:26:45 to check the issue I broke with the .$counter thin
Jennifer Messerly 2014/02/28 02:21:49 done.
+
+
+ group('fixes import/export/part URIs', dartUriTests);
+}
+
+dartUriTests() {
+
+ testPhases('from web folder', phases, {
+ 'a|web/test.html':
+ '<!DOCTYPE html><html><head>'
+ '<link rel="import" href="test2/foo.html">'
+ '</head><body></body></html>',
+ 'a|web/test2/foo.html':
+ '<!DOCTYPE html><html><head></head><body>'
+ '<script type="application/dart">'
+ "import 'package:qux/qux.dart';"
+ "import 'foo.dart';"
+ "export 'bar.dart';"
+ "part 'baz.dart';"
+ '</script>'
+ '</body></html>',
+ }, {
+ 'a|web/test.html':
+ '<!DOCTYPE html><html><head></head><body></body></html>',
+ 'a|web/test.html.scriptUrls': '[["a","web/test.html.dart"]]',
+ 'a|web/test.html.dart':
+ "library web_test_html;\n"
+ "import 'package:qux/qux.dart';"
+ "import 'test2/foo.dart';"
+ "export 'test2/bar.dart';"
+ "part 'test2/baz.dart';",
+ 'a|web/test2/foo.html':
+ '<!DOCTYPE html><html><head></head><body>'
+ '<script type="application/dart" src="foo.html.dart"></script>'
+ '</body></html>',
+ 'a|web/test2/foo.html.scriptUrls': '[]',
+ 'a|web/test2/foo.html.dart':
+ "library web_test2_foo_html;\n"
+ "import 'package:qux/qux.dart';"
+ "import 'foo.dart';"
+ "export 'bar.dart';"
+ "part 'baz.dart';",
+ });
+
+ testPhases('from lib folder', phases, {
+ 'a|web/test.html':
+ '<!DOCTYPE html><html><head>'
+ '<link rel="import" href="packages/a/test2/foo.html">'
+ '</head><body></body></html>',
+ 'a|lib/test2/foo.html':
+ '<!DOCTYPE html><html><head></head><body>'
+ '<script type="application/dart">'
+ "import 'package:qux/qux.dart';"
+ "import 'foo.dart';"
+ "export 'bar.dart';"
+ "part 'baz.dart';"
+ '</script>'
+ '</body></html>',
+ }, {
+ 'a|web/test.html':
+ '<!DOCTYPE html><html><head></head><body></body></html>',
+ 'a|web/test.html.scriptUrls': '[["a","web/test.html.dart"]]',
+ 'a|web/test.html.dart':
+ "library web_test_html;\n"
+ "import 'package:qux/qux.dart';"
+ "import 'package:a/test2/foo.dart';"
+ "export 'package:a/test2/bar.dart';"
+ "part 'package:a/test2/baz.dart';",
+ 'a|lib/test2/foo.html':
+ '<!DOCTYPE html><html><head></head><body>'
+ '<script type="application/dart">'
+ "import 'package:qux/qux.dart';"
+ "import 'foo.dart';"
+ "export 'bar.dart';"
+ "part 'baz.dart';"
+ '</script>'
+ '</body></html>',
+ });
+
+ testPhases('from another pkg', phases, {
+ 'a|web/test.html':
+ '<!DOCTYPE html><html><head>'
+ '<link rel="import" href="packages/b/test2/foo.html">'
+ '</head><body></body></html>',
+ 'b|lib/test2/foo.html':
+ '<!DOCTYPE html><html><head></head><body>'
+ '<script type="application/dart">'
+ "import 'package:qux/qux.dart';"
+ "import 'foo.dart';"
+ "export 'bar.dart';"
+ "part 'baz.dart';"
+ '</script>'
+ '</body></html>',
+ }, {
+ 'a|web/test.html':
+ '<!DOCTYPE html><html><head></head><body></body></html>',
+ 'a|web/test.html.scriptUrls': '[["a","web/test.html.dart"]]',
+ 'a|web/test.html.dart':
+ "library web_test_html;\n"
+ "import 'package:qux/qux.dart';"
+ "import 'package:b/test2/foo.dart';"
+ "export 'package:b/test2/bar.dart';"
+ "part 'package:b/test2/baz.dart';",
+ 'b|lib/test2/foo.html':
+ '<!DOCTYPE html><html><head></head><body>'
+ '<script type="application/dart">'
+ "import 'package:qux/qux.dart';"
+ "import 'foo.dart';"
+ "export 'bar.dart';"
+ "part 'baz.dart';"
+ '</script>'
+ '</body></html>',
+ });
+}
« pkg/polymer/lib/src/build/import_inliner.dart ('K') | « pkg/polymer/test/build/all_phases_test.dart ('k') | pkg/polymer/test/build/code_extractor_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698