Support directories other than "web" in pub build.

sdk/lib/_internal/pub/test/build/compiles_entrypoints_in_benchmark_test.dart

 // 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.
 
 import 'package:scheduled_test/scheduled_test.dart';
 
 import '../descriptor.dart' as d;
 import '../test_pub.dart';
 
 main() {
   initConfig();
 
-  integration("compiles Dart entrypoints to JS", () {
+  integration("compiles Dart entrypoints in benchmark/ to JS", () {
     // Dart2js can take a long time to compile dart code, so we increase the
     // timeout to cope with that.
     currentSchedule.timeout *= 3;
 
     d.dir(appPath, [
       d.appPubspec(),
-      d.dir('web', [
+      d.dir('benchmark', [
         d.file('file.dart', 'void main() => print("hello");'),
         d.file('lib.dart', 'void foo() => print("hello");'),
         d.dir('subdir', [
           d.file('subfile.dart', 'void main() => print("ping");')
         ])
       ])
     ]).create();
 
-    // TODO(rnystrom): If we flesh out the command-line output, validate that
-    // here.
-    schedulePub(args: ["build"],
+    schedulePub(args: ["build", "*"],
         output: new RegExp(r"Built 6 files!"),
         exitCode: 0);
 
     d.dir(appPath, [
       d.dir('build', [
-        d.matcherFile('file.dart.js', isNot(isEmpty)),
-        d.matcherFile('file.dart.precompiled.js', isNot(isEmpty)),
-        d.matcherFile('file.dart.js.map', isNot(isEmpty)),
-        d.nothing('file.dart'),
-        d.nothing('lib.dart'),
-        d.dir('subdir', [
-          d.matcherFile('subfile.dart.js', isNot(isEmpty)),
-          d.matcherFile('subfile.dart.precompiled.js', isNot(isEmpty)),
-          d.matcherFile('subfile.dart.js.map', isNot(isEmpty)),
-          d.nothing('subfile.dart')
+        d.dir('benchmark', [
+          d.matcherFile('file.dart.js', isNot(isEmpty)),
+          d.matcherFile('file.dart.precompiled.js', isNot(isEmpty)),
+          d.matcherFile('file.dart.js.map', isNot(isEmpty)),
+          d.nothing('file.dart'),
+          d.nothing('lib.dart'),
+          d.dir('subdir', [
+            d.matcherFile('subfile.dart.js', isNot(isEmpty)),
+            d.matcherFile('subfile.dart.precompiled.js', isNot(isEmpty)),
+            d.matcherFile('subfile.dart.js.map', isNot(isEmpty)),
+            d.nothing('subfile.dart')
+          ])
         ])
       ])
     ]).validate();
   });
 }

[nweiz, 2014/01/28 03:21:51]

This test is confusing. If you're testing that "benchmark/" is allowed, this
should run "pub build benchmark". There should be a separate test to validate
that "pub build *" picks up all the correct directories.

[Bob Nystrom, 2014/01/28 23:02:04]

Done.