fix tests and remove added check

Author: azkrishpy

awscrt/http.py

@@ -687,28 +687,6 @@ def activate(self) -> None:
         """
         _awscrt.http_client_stream_activate(self)
 
-    def write_data(self,
-                   data_stream: Union[InputStream, Any],
-                   end_stream: bool = False) -> "concurrent.futures.Future":
-        '''Write data to the HTTP/1.1 request body.
-
-        The stream must have been created with ``manual_write=True`` and
-        :meth:`activate()` must have been called before using this method.
-
-        This method supersedes the deprecated ``write_chunk`` API.
-        Use ``write_data()`` for all manual body writes on both HTTP/1.1
-        and HTTP/2 streams.
-
-        Args:
-            data_stream: Data to write. Wrapped in :class:`~awscrt.io.InputStream` if
-                needed. ``None`` sends zero bytes.
-            end_stream (bool): ``True`` if this is the last write.
-
-        Returns:
-            concurrent.futures.Future: Completes with ``None`` on success.
-        '''
-        return super().write_data(data_stream, end_stream)
-
 
 class Http2ClientStream(HttpClientStreamBase):
     __slots__ = ('_remote_end_stream_future',)

test/test_http_client.py

@@ -1073,12 +1073,12 @@ def test_h1_write_data_with_body_stream_raises(self):
 
             request = HttpRequest('PUT', '/write_data_guard', body_stream=BytesIO(b'body'))
             request.headers.add('host', self.hostname)
-            request.headers.add('Content-Length', '4')
 
-            with self.assertRaises(ValueError) as ctx:
+            try:
                 connection.request(request, manual_write=True)
-
-            self.assertIn('body_stream', str(ctx.exception))
+                self.fail("Expected ValueError from request()")
+            except ValueError as e:
+                self.assertIn('manual data writes', str(e))
 
             connection.close().result(self.timeout)
         finally: