add tests for new early exit cases, restore coverage, surpisingly reduce time of headers_dict_to_raw to 1.7883 seconds in benchmark

Author: albertedwardson

tests/test_http.py

@@ -35,6 +35,10 @@ def test_headers_raw_dict_none(self):
         assert headers_raw_to_dict(None) is None
         assert headers_dict_to_raw(None) is None
 
+    def test_headers_raw_dict_empty(self):
+        assert headers_raw_to_dict(b"") == {}
+        assert headers_dict_to_raw({}) == b""
+
     def test_headers_raw_to_dict(self):
         raw = b"Content-type: text/html\n\rAccept: gzip\n\r\
                 Cache-Control: no-cache\n\rCache-Control: no-store\n\n"

w3lib/http.py

@@ -100,21 +100,21 @@ def headers_dict_to_raw(headers_dict: HeadersDictInput | None) -> bytes | None:
     if not headers_dict:
         return b""
 
-    parts = bytearray()
+    parts = b""
 
     for key, value in headers_dict.items():
         if isinstance(value, bytes):
             if parts:
-                parts.extend(b"\r\n")
-            parts.extend(key + b": " + value)
+                parts += b"\r\n"
+            parts += key + b": " + value
 
         elif isinstance(value, (list, tuple)):
             for v in value:
                 if parts:
-                    parts.extend(b"\r\n")
-                parts.extend(key + b": " + v)
+                    parts += b"\r\n"
+                parts += key + b": " + v
 
-    return bytes(parts)
+    return parts
 
 
 def basic_auth_header(