fix Starlette TestClient

[alekseik1]

Hi, I'm trying to get Starlette TestClient running with vcrpy.
The problem is that assert not hasattr(resp, "_decoder") fails in _to_serialized_response func, which breaks vcr for Starlette TestClient, see #628.

I see two possible solutions here:

First approach (implemented)

From docs/advanced.rst:

Requests that are ignored by VCR will not be saved in a cassette, nor played back from a cassette. VCR will completely ignore those requests as if it didn't notice them at all, and they will continue to hit the server as if VCR were not there.

Which means it is safe to completely ignore these requests when recording responses.
If we don't record responses, we don't hit this assert.

Second approach

Fix buggy _run_async_function (sorry, it was me who messed it up):

def _run_async_function(sync_func, *args, **kwargs):
    """
    Safely run an asynchronous function from a synchronous context.
    Handles both cases:
    - An event loop is already running.
    - No event loop exists yet.
    """
    try:
        loop = asyncio.get_running_loop()
    except RuntimeError:
        return asyncio.run(sync_func(*args, **kwargs))
    else:
        # If inside a running loop, create a task and wait for it
        return loop.run_in_executor(None, partial(sync_func, *args, **kwargs))  # <<< see here

Because right now it is bugged - we run a task in asyncio but never wait for it to finish.

@jairhenrique what do you think? Should I apply both fixes?

Fixes #628.

[alekseik1]

you know what. I think these asyncio-hacks are bad things to support. It's better to duplicate code than to manage event loops.

[alekseik1]

any updates on this? @kevin1024 @jairhenrique