Add a Server interface.

test/io_server_test.dart

+// Copyright (c) 2015, 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.
+
+@TestOn('vm')
+
+import 'dart:async';
+import 'dart:io';
+
+import 'package:http/http.dart' as http;
+import 'package:shelf/shelf.dart';

[kevmoo, 2015/10/22 00:44:26]

unused import

[nweiz, 2015/10/26 21:32:44]

Done.

+import 'package:shelf/shelf_io.dart';
+import 'package:test/test.dart';
+
+import 'test_util.dart';
+
+void main() {
+  var server;

[kevmoo, 2015/10/22 00:44:26]

type things when there is no way for inference to work

The analyzer can help us find bugs, etc

[nweiz, 2015/10/26 21:32:44]

This is a test—if there are bugs the tests will fail, and it's clear to the user
what the type of this is. Also, typing it would be inconsistent with a huge
swath of other similar test code out.

+  setUp(() async {
+    server = await IOServer.bind(InternetAddress.LOOPBACK_IP_V4, 0);
+  });
+
+  tearDown(() => server.close());
+
+  test("serves HTTP requests with the mounted handler", () async {
+    server.mount(syncHandler);
+    expect(await http.read(server.url), equals('Hello from /'));
+  });
+
+  test("delays HTTP requests until a handler is mounted", () async {
+    expect(http.read(server.url), completion(equals('Hello from /')));
+    await new Future.delayed(Duration.ZERO);
+
+    server.mount(asyncHandler);
+  });
+
+  test("disallows more than one handler from being mounted", () async {
+    server.mount((_) {});
+    expect(() => server.mount((_) {}), throwsStateError);

[kevmoo, 2015/10/22 00:44:26]

actually check for your StateError message – since the stream subscription error
is also a StateError, but with a much less helpful message

[nweiz, 2015/10/26 21:32:44]

Checking for specific messages in tests is generally an anti-pattern; it makes
the tests much more fragile and adds very little value. We shouldn't have to
change a bunch of tests if we change an error message.

+    expect(() => server.mount((_) {}), throwsStateError);
+  });
+}