fix: patch bug in locals_to_params

[danphenderson]

Calls to locals_to_params now handle nested params properly.

See comment in issue: 1269

[A1exKH]

Code LGTM.

[A1exKH]

@danphenderson please sync your branch with base branch.

[mxschmitt]

Looks like generally speaking we can do something like that but we need to skip the logic for json_data like we do in .NET: https://github.com/search?q=repo%3Amicrosoft%2Fplaywright-dotnet+DefaultJsonSerializerOptionsKeepNulls&type=code

Edit: Seems like we don't need to skip it, since we serialise it differently:

playwright-python/playwright/_impl/_fetch.py

Line 373 in 04b7851

json_data = json.dumps(data)

Would it be possible to add a test for it?

[danphenderson]

I updated my branch. Sorry for the delay.

Regarding tests, can you provide me with more precise instructions on where you would like them to live?

[mxschmitt]

Somewhere https://github.com/microsoft/playwright-python/blob/04b78516a96fb18d0df912652698d0d5b6a55bb6/tests/async/test_browsercontext_proxy.py and

playwright-python/tests/async/test_proxy.py

Line 49 in 04b7851

async def test_should_use_proxy_for_second_page(

[danphenderson]

Thanks, the test coverage has been added - please let me know if any additional changes are needed.