Support loading transformer plugins from pub.

sdk/lib/_internal/pub/test/serve/fails_to_load_a_file_that_defines_no_transforms_test.dart

Index: sdk/lib/_internal/pub/test/serve/fails_to_load_a_file_that_defines_no_transforms_test.dart
diff --git a/sdk/lib/_internal/pub/test/serve/fails_to_load_a_file_that_defines_no_transforms_test.dart b/sdk/lib/_internal/pub/test/serve/fails_to_load_a_file_that_defines_no_transforms_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..ae1be2687366b970b157c8e0cce02574f42ece93
--- /dev/null
+++ b/sdk/lib/_internal/pub/test/serve/fails_to_load_a_file_that_defines_no_transforms_test.dart
@@ -0,0 +1,36 @@
+// Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS d.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 pub_tests;
+
+import 'package:scheduled_test/scheduled_test.dart';
+
+import '../descriptor.dart' as d;
+import '../test_pub.dart';
+
+main() {
+  initConfig();
+
+  integration("fails to load a file that defines no transforms", () {
+    d.dir(appPath, [
+      d.pubspec({
+        "name": "myapp",
+        "transformers": ["myapp/src/transformer"]
+      }),
+      d.dir("lib", [d.dir("src", [

[Bob Nystrom, 2013/08/27 22:12:30]

Move d.dir("src") to its own line and indent the next line to make the nesting a
bit clearer please. (Or just put transformer.dart in lib.)

[nweiz, 2013/08/28 20:45:23]

Done.

+        d.file("transformer.dart", "library does_nothing;")
+      ])])
+    ]).create();
+
+    createLockFile('myapp', {}, pkg: ['barback']);
+
+    var pub = startPub(args: ['serve', '--port=0']);
+    expect(pub.nextErrLine(), completion(startsWith('No transformers were '
+       'defined in ')));
+    expect(pub.nextErrLine(), completion(startsWith('required by myapp.')));

[Bob Nystrom, 2013/08/27 22:12:30]

Why not just move these two lines after shouldExit(1) and then do a single test
against remainingStderr()?

[nweiz, 2013/08/28 20:45:23]

The assertions make more sense line-by-line than against a whole string.

+    pub.shouldExit(1);
+    expect(pub.remainingStdout(),
+        completion(isNot(contains('This is an unexpected error'))));
+  });
+}