pkg/shelf: change helper method on Request and Response

pkg/shelf/test/response_test.dart

 // Copyright (c) 2014, 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 shelf.response_test;
 
 import 'dart:async';
 import 'dart:convert';
 
-import 'package:shelf/shelf.dart';
+import 'package:shelf/shelf.dart' hide Request;
 import 'package:unittest/unittest.dart';
 
+import 'test_change.dart';
+import 'test_util.dart';
+
 void main() {
-  group("readAsString", () {
-    test("supports a null body", () {
-      var response = new Response(200);
-      expect(response.readAsString(), completion(equals("")));
-    });
-
-    test("supports a String body", () {
+  group("supports a String body", () {
+    test("readAsString", () {
       var response = new Response.ok("hello, world");
       expect(response.readAsString(), completion(equals("hello, world")));
     });
 
-    test("supports a Stream<List<int>> body", () {
-      var controller = new StreamController();
-      var response = new Response.ok(controller.stream);
-      expect(response.readAsString(), completion(equals("hello, world")));
+    test("read", () {
+      var helloWorldBytes = new List.from(HELLO_BYTES)..addAll(WORLD_BYTES);
 
-      controller.add([104, 101, 108, 108, 111, 44]);
-      return new Future(() {
-        controller
-          ..add([32, 119, 111, 114, 108, 100])
-          ..close();
-      });
+      var response = new Response.ok("hello, world");
+      expect(response.read().toList(), completion(equals([helloWorldBytes])));
     });
   });
 
-  group("read", () {
-    test("supports a null body", () {
-      var response = new Response(200);
-      expect(response.read().toList(), completion(isEmpty));
-    });
-
-    test("supports a String body", () {
-      var response = new Response.ok("hello, world");
-      expect(response.read().toList(), completion(equals([[
-        104, 101, 108, 108, 111, 44, 32, 119, 111, 114, 108, 100
-      ]])));
-    });
-
-    test("supports a Stream<List<int>> body", () {
-      var controller = new StreamController();
-      var response = new Response.ok(controller.stream);
-      expect(response.read().toList(), completion(equals([
-        [104, 101, 108, 108, 111, 44],
-        [32, 119, 111, 114, 108, 100]
-      ])));
-
-      controller.add([104, 101, 108, 108, 111, 44]);
-      return new Future(() {
-        controller
-          ..add([32, 119, 111, 114, 108, 100])
-          ..close();
-      });
-    });
-  });
-
   group("new Response", () {
     test("defaults to encoding a String as UTF-8", () {
       expect(new Response.ok("è").read().toList(),
           completion(equals([[195, 168]])));
     });
 
     test("uses the explicit encoding if available", () {
       expect(new Response.ok("è", encoding: LATIN1).read().toList(),
           completion(equals([[232]])));
     });
@@ skipping 71 matching lines @@
     test("is null without a Last-Modified header", () {
       expect(new Response.ok("okay!").lastModified, isNull);
     });
 
     test("comes from the Last-Modified header", () {
       expect(new Response.ok("okay!", headers: {
         'last-modified': 'Sun, 06 Nov 1994 08:49:37 GMT'
       }).lastModified, equals(DateTime.parse("1994-11-06 08:49:37z")));
     });
   });
+
+  group('change', () {
+    test('with no arguments returns instance with equal values', () {

[nweiz, 2014/04/29 19:48:48]

Why isn't this part of [testChange]?

[kevmoo, 2014/04/30 14:27:52]

Because I'm ensuring that all of the non-common fields are copied -- nothing was
forgotten in the custom change impl

+      var controller = new StreamController();
+
+      var request = new Response(345, body: 'hèllo, world', encoding: LATIN1,
+          headers: {'header1': 'header value 1'},
+          context: {'context1': 'context value 1'});
+
+      var copy = request.change();
+
+      expect(copy.statusCode, request.statusCode);
+      expect(copy.readAsString(), completion('hèllo, world'));
+      expect(copy.headers, same(request.headers));
+      expect(copy.encoding, request.encoding);
+      expect(copy.context, same(request.context));
+
+      controller.add(HELLO_BYTES);
+      return new Future(() {
+        controller
+          ..add(WORLD_BYTES)
+          ..close();
+      });
+    });
+
+    testChange(({headers, context}) {
+      return new Response.ok(null, headers: headers,
+          context: context);
+    });
+  });
 }