feat: modify how multipart bytes header is built (no space) on http source (#1018)

* remove unused fixture

* refactor how range header in built

* remove space from range header

Author: lobis

src/uproot/source/http.py

@@ -289,16 +289,14 @@ def multifuture(
         """
         connection = make_connection(source.parsed_url, source.timeout)
 
-        range_strings = []
-        for start, stop in ranges:
-            range_strings.append(f"{start}-{stop - 1}")
-
+        range_header = {
+            "Range": "bytes="
+            + ",".join([f"{start}-{stop - 1}" for start, stop in ranges])
+        }
         connection.request(
             "GET",
             full_path(source.parsed_url),
-            headers=dict(
-                {"Range": "bytes=" + ", ".join(range_strings)}, **source.auth_headers
-            ),
+            headers=dict(**range_header, **source.auth_headers),
         )
 
         def task(resource):
@@ -317,7 +315,7 @@ def task(resource):
                                 "GET",
                                 full_path(redirect_url),
                                 headers={
-                                    "Range": "bytes=" + ", ".join(range_strings),
+                                    **range_header,
                                     **source.auth_headers,
                                 },
                             )

tests/test_0001_source_class.py

@@ -119,7 +119,7 @@ def test_memmap_fail(use_threads, tmp_path):
 
 @pytest.mark.parametrize("use_threads", [True, False])
 @pytest.mark.network
-def test_http(server, use_threads):
+def test_http(use_threads):
     url = "https://example.com"
     with uproot.source.http.HTTPSource(
         url,