Fix pub after changing the HttpServer.bind signature

pkg/scheduled_test/lib/src/utils.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.
 
 library utils;
 
 import 'dart:async';
 
 import 'package:stack_trace/stack_trace.dart';
 
@@ skipping 17 matching lines @@
 /// Configures [future] so that its result (success or exception) is passed on
 /// to [completer].
 void chainToCompleter(Future future, Completer completer) {
   future.then((value) => completer.complete(value),
       onError: (e) => completer.completeError(e));
 }
 
 /// Prepends each line in [text] with [prefix]. If [firstPrefix] is passed, the
 /// first line is prefixed with that instead.
 String prefixLines(String text, {String prefix: '| ', String firstPrefix}) {
+  if (text == null) return '';

[Bob Nystrom, 2013/05/01 16:03:17]

Why is this change needed?

[Søren Gjesse, 2013/05/01 17:38:35]

I got an exception that text was null, which was hiding the actual error
backtrace/printout. With this change I got the real error output and was able to
find the actual error.

[nweiz, 2013/05/06 17:19:47]

[prefixLines] shouldn't do manual null-checking (any more than other methods
that take strings do). It's the caller's responsibility to make the string
null-safe. Where was this being called with a null input?

[Søren Gjesse, 2013/05/13 13:10:57]

If you undo this change and the change to
sdk/lib/_internal/pub/lib/src/safe_http_server.dart from this cl, and then run
the pub tests you get errors like below.

FAILED: none-vm release_ia32 pub/test/install/hosted/install_test
Expected: pass 
Actual: fail

stdout:
✗ installs a package from a pub server
  Caught The null object does not have a method 'split'.
  
  NoSuchMethodError : method not found: 'split'
  Receiver: null
  Arguments: ["\n"]

✗ URL encodes the package name
  Caught Bad state: There seems to be another scheduled test still running.



stderr:
Unhandled exception:
FormatException: Couldn't parse stack trace line '_setUpScheduledTest.<anonymous
closure>   package:scheduled_test/scheduled_test.dart 319:9'.
#0      _StackFrame._StackFrame
(file:///usr/local/prj/dart/git/dart/sdk/lib/_internal/pub/test/command_line_config.dart:157:9)
#1      CommandLineConfiguration._printStackTrace
(file:///usr/local/prj/dart/git/dart/sdk/lib/_internal/pub/test/command_line_config.dart:89:17)
#2      CommandLineConfiguration.onTestResult
(file:///usr/local/prj/dart/git/dart/sdk/lib/_internal/pub/test/command_line_config.dart:52:21)
#3      TestCase._setResult (package:unittest/src/test_case.dart:155:27)
#4      TestCase._complete (package:unittest/src/test_case.dart:174:15)
#5      TestCase.fail (package:unittest/src/test_case.dart:211:16)
#6      _registerException (package:unittest/unittest.dart:757:18)
#7      _guardAsync (package:unittest/unittest.dart:737:23)
#8      _nextBatch (package:unittest/unittest.dart:775:24)
#9      _nextTestCase.<anonymous closure>
(package:unittest/unittest.dart:669:15)
#10     _asyncRunCallback._asyncRunCallback (dart:async/event_loop.dart:15:17)
#11     _asyncRunCallback._asyncRunCallback (dart:async/event_loop.dart:25:9)
#12     Timer.run.<anonymous closure> (dart:async/timer.dart:17:21)
#13     Timer.run.<anonymous closure> (dart:async/timer.dart:25:13)
#14     Timer.Timer.<anonymous closure> (dart:async-patch/timer_patch.dart:9:15)
#15     _Timer._createTimerHandler._handleTimeout (dart:io:7177:28)
#16     _Timer._createTimerHandler._handleTimeout (dart:io:7185:7)
#17     _Timer._createTimerHandler.<anonymous closure> (dart:io:7193:23)
#18     _ReceivePortImpl._handleMessage
(dart:isolate-patch/isolate_patch.dart:81:92)


Command line: out/ReleaseIA32/dart-sdk/bin/dart "--ignore-unrecognized-flags"
"--package-root=out/ReleaseIA32/packages/"
"/usr/local/prj/dart/git/dart/sdk/lib/_internal/pub/test/install/hosted/install_test.dart"

On 2013/05/06 17:19:47, nweiz wrote:

   var lines = text.split('\n');
   if (firstPrefix == null) {
     return lines.map((line) => '$prefix$line').join('\n');
   }
 
   var firstLine = "$firstPrefix${lines.first}";
   lines = lines.skip(1).map((line) => '$prefix$line').toList();
   lines.insert(0, firstLine);
   return lines.join('\n');
 }
@@ skipping 139 matching lines @@
 }
 
 /// Returns a string representation of [trace] that has the core and test frames
 /// folded together.
 String terseTraceString(StackTrace trace) {
   return new Trace.from(trace).terse.foldFrames((frame) {
     return frame.package == 'scheduled_test' || frame.package == 'unittest' ||
         frame.isCore;
   }).toString().trim();
 }