fix(fetch): serialize primitive multipart values (#3227)

Author: suraj-self

playwright/_impl/_fetch.py

@@ -423,18 +423,10 @@ async def _inner_fetch(
                         await _form_data_field_to_form_field(fd_name, fd_value)
                     )
             else:
-                # Convert file-like values to ServerFilePayload structs.
                 for name, value in multipart.items():
-                    if is_file_payload(value):
-                        payload = cast(FilePayload, value)
-                        assert isinstance(
-                            payload["buffer"], bytes
-                        ), f"Unexpected buffer type of 'data.{name}'"
-                        multipart_data.append(
-                            FormField(name=name, file=file_payload_to_json(payload))
-                        )
-                    elif isinstance(value, str):
-                        multipart_data.append(FormField(name=name, value=value))
+                    multipart_data.append(
+                        await _form_data_field_to_form_field(name, value)
+                    )
         if (
             post_data_buffer is None
             and json_data is None

tests/async/test_fetch_browser_context.py

@@ -345,6 +345,8 @@ async def test_should_support_multipart_form_data(
             multipart={
                 "firstName": "John",
                 "lastName": "Doe",
+                "score": 42.5,
+                "active": True,
                 "file": file,
             },
         ),
@@ -358,6 +360,8 @@ async def test_should_support_multipart_form_data(
     )
     assert request.args[b"firstName"] == [b"John"]
     assert request.args[b"lastName"] == [b"Doe"]
+    assert request.args[b"score"] == [b"42.5"]
+    assert request.args[b"active"] == [b"True"]
     assert request.args[b"file"][0] == file["buffer"]
 
 

tests/sync/test_fetch_browser_context.py

@@ -269,6 +269,8 @@ def test_should_support_multipart_form_data(
             multipart={
                 "firstName": "John",
                 "lastName": "Doe",
+                "score": 42.5,
+                "active": True,
                 "file": file,
             },
         )
@@ -281,6 +283,8 @@ def test_should_support_multipart_form_data(
     )
     assert server_req.value.args[b"firstName"] == [b"John"]
     assert server_req.value.args[b"lastName"] == [b"Doe"]
+    assert server_req.value.args[b"score"] == [b"42.5"]
+    assert server_req.value.args[b"active"] == [b"True"]
     assert server_req.value.args[b"file"][0] == file["buffer"]