Add test for sending file-like objects.

Author: Lukasa

hyper/http20/connection.py

@@ -14,6 +14,7 @@
 from ..common.bufsocket import BufferedSocket
 from ..common.headers import HTTPHeaderMap
 from ..common.util import to_host_port_tuple, to_native_string, to_bytestring
+from ..compat import unicode, bytes
 from .stream import Stream
 from .response import HTTP20Response, HTTP20Push
 from .window import FlowControlManager
@@ -164,7 +165,7 @@ def request(self, method, url, body=None, headers={}):
             self.putheader(name, value, stream_id, replace=is_default)
 
         # Convert the body to bytes if needed.
-        if body:
+        if body and isinstance(body, (unicode, bytes)):
             body = to_bytestring(body)
 
         self.endheaders(message_body=body, final=True, stream_id=stream_id)

test/test_hyper.py

@@ -411,6 +411,28 @@ def consume_single_frame():
 
         assert mutable['counter'] == 10
 
+    def test_sending_file(self, frame_buffer):
+        # Prepare a socket so we can open a stream.
+        sock = DummySocket()
+        c = HTTP20Connection('www.google.com')
+        c._sock = sock
+
+        # Send a request that involves uploading a file handle.
+        with open(__file__, 'rb') as f:
+            c.request('GET', '/', body=f)
+
+        # Get all the frames
+        frame_buffer.add_data(b''.join(sock.queue))
+        frames = list(frame_buffer)
+
+        # Reconstruct the file from the sent data.
+        sent_data = b''.join(
+            f.data for f in frames if isinstance(f, DataFrame)
+        )
+
+        with open(__file__, 'rb') as f:
+            assert f.read() == sent_data
+
 
 class TestServerPush(object):
     def setup_method(self, method):