Print a warning when running "pub build" and "pub serve" (#1832)

Closes #1823

Author: nex3

lib/src/command/barback.dart

@@ -83,6 +83,11 @@ abstract class BarbackCommand extends PubCommand {
     log.json.enabled =
         argResults.options.contains("format") && argResults["format"] == "json";
 
+    log.warning(
+        log.yellow("Dart 2 will have a new build system. Learn how to migrate "
+            "from ${log.bold('pub build')} and\n"
+            "${log.bold('pub serve')}: https://webdev.dartlang.org/dart-2\n"));
+
     _parseSourceDirectories();
     return onRunTransformerCommand();
   }

lib/src/log.dart

@@ -491,33 +491,46 @@ String gray(text) {
 /// that supports that.
 ///
 /// Use this to highlight something interesting but neither good nor bad.
-String cyan(text) => sparkle ? "$text" : "$_cyan$text$_noColor";
+String cyan(text) => _addColor(text, _cyan);
 
 /// Wraps [text] in the ANSI escape codes to color it green when on a platform
 /// that supports that.
 ///
 /// Use this to highlight something successful or otherwise positive.
-String green(text) => sparkle ? "$text" : "$_green$text$_noColor";
+String green(text) => _addColor(text, _green);
 
 /// Wraps [text] in the ANSI escape codes to color it magenta when on a
 /// platform that supports that.
 ///
 /// Use this to highlight something risky that the user should be aware of but
 /// may intend to do.
-String magenta(text) => sparkle ? "$text" : "$_magenta$text$_noColor";
+String magenta(text) => _addColor(text, _magenta);
 
 /// Wraps [text] in the ANSI escape codes to color it red when on a platform
 /// that supports that.
 ///
 /// Use this to highlight unequivocal errors, problems, or failures.
-String red(text) => sparkle ? "$text" : "$_red$text$_noColor";
+String red(text) => _addColor(text, _red);
 
 /// Wraps [text] in the ANSI escape codes to color it yellow when on a platform
 /// that supports that.
 ///
 /// Use this to highlight warnings, cautions or other things that are bad but
 /// do not prevent the user's goal from being reached.
-String yellow(text) => sparkle ? "$text" : "$_yellow$text$_noColor";
+String yellow(text) => _addColor(text, _yellow);
+
+/// Returns [text] colored using the given [colorCode].
+///
+/// This is resilient to the text containing other colors or bold text.
+String _addColor(Object text, String colorCode) {
+  if (sparkle) return text.toString();
+  return colorCode +
+      text
+          .toString()
+          .replaceAll(_none, _none + colorCode)
+          .replaceAll(_noColor, _none + colorCode) +
+      _noColor;
+}
 
 /// Log function that prints the message to stdout.
 void _logToStdout(Entry entry) {

test/barback/utils.dart

@@ -37,9 +37,12 @@ void pubBuildAndServeShouldFail(String description,
   Object serveExpectation = serveError;
   if (exitCode == exit_codes.USAGE) {
     buildExpectation =
-        allOf(startsWith(buildExpectation), contains("Usage: pub build"));
+        allOf(contains(buildExpectation), contains("Usage: pub build"));
     serveExpectation =
-        allOf(startsWith(serveExpectation), contains("Usage: pub serve"));
+        allOf(contains(serveExpectation), contains("Usage: pub serve"));
+  } else {
+    buildExpectation = contains(buildExpectation);
+    serveExpectation = contains(serveExpectation);
   }
 
   test("build fails $description", () {

test/compiler/does_not_support_invalid_command_line_options_type_test.dart

@@ -30,10 +30,10 @@ main() {
     await requestShould404("main.dart.js");
     expect(
         server.stderr,
-        emitsLines('Build error:\n'
+        emitsThrough(emitsLines('Build error:\n'
             'Transform Dart2JS on myapp|web/main.dart threw error: '
             'Invalid value for \$dart2js.commandLineOptions: '
-            '"foo" (expected list of strings).'));
+            '"foo" (expected list of strings).')));
     await endPubServe();
   });
 }

test/compiler/does_not_support_invalid_environment_type_test.dart

@@ -30,10 +30,10 @@ main() {
     await requestShould404("main.dart.js");
     expect(
         server.stderr,
-        emitsLines('Build error:\n'
+        emitsThrough(emitsLines('Build error:\n'
             'Transform Dart2JS on myapp|web/main.dart threw error: '
             'Invalid value for \$dart2js.environment: "foo" '
-            '(expected map from strings to strings).'));
+            '(expected map from strings to strings).')));
     await endPubServe();
   });
 }

test/compiler/does_not_support_invalid_option_test.dart

@@ -28,8 +28,8 @@ main() {
     // TODO(nweiz): This should provide more context about how the option got
     // passed to dart2js. See issue 16008.
     var pub = await startPubServe();
-    await expect(
-        pub.stderr, emits('Unrecognized dart2js option "invalidOption".'));
+    await expect(pub.stderr,
+        emitsThrough('Unrecognized dart2js option "invalidOption".'));
     await pub.shouldExit(exit_codes.DATA);
   });
 }

test/compiler/doesnt_support_invalid_type_for_boolean_option_test.dart

@@ -29,10 +29,10 @@ main() {
     await requestShould404("main.dart.js");
     expect(
         server.stderr,
-        emitsLines('Build error:\n'
+        emitsThrough(emitsLines('Build error:\n'
             'Transform Dart2JS on myapp|web/main.dart threw error: '
             'Invalid value for \$dart2js.checked: "foo" '
-            '(expected true or false).'));
+            '(expected true or false).')));
     await endPubServe();
   });
 }

test/compiler/reports_dart_parse_errors_test.dart

@@ -107,5 +107,6 @@ Future _expectErrors(PubProcess pub, Compiler compiler,
 
   // It's nondeterministic what order the dart2js transformers start running,
   // so we allow the error messages to be emitted in either order.
-  await expectLater(pub.stderr, emitsInAnyOrder([consumeFile, consumeSubfile]));
+  await expectLater(
+      pub.stderr, emitsThrough(emitsInAnyOrder([consumeFile, consumeSubfile])));
 }

test/transformer/a_transformer_rejects_its_config_test.dart

@@ -46,7 +46,7 @@ main() {
     var pub = await startPubServe();
     expect(
         pub.stderr,
-        emits(endsWith('Error loading transformer: I hate these '
+        emitsThrough(endsWith('Error loading transformer: I hate these '
             'settings!')));
     await pub.shouldExit(1);
   });

test/transformer/asset_not_found_exceptions_are_detectable_test.dart

@@ -59,6 +59,6 @@ main() {
 
     // Since the AssetNotFoundException was caught and handled, the server
     // shouldn't print any error information for it.
-    expect(server.stderr, emitsDone);
+    expect(server.stderr, neverEmits(contains("nonexistent")));
   });
 }