aiohttp mocking does not respect ClientSession base_url or headers passed in constructor

[pandaxtc]

When using pook with aiohttp, the base_url and headers parameters passed to ClientSession are ignored. This causes matching to fail when using those parameters. For example,

#!/usr/bin/env python3

import aiohttp
import pook

BASE_URL = "http://example.com"
URL_SLUG = "/api/v1/get"
URL = f"{BASE_URL}{URL_SLUG}"

HEADERS = {"Authorization": "Token token"}


async def main():
    with pook.use():
        pook.get(URL).headers(HEADERS).reply(200)

        # Passes
        async with (
            aiohttp.ClientSession() as session,
            session.get(URL, headers=HEADERS) as response,
        ):
            print(response.status)

    with pook.use():
        pook.get(URL).headers(HEADERS).reply(200)

        # Fails
        async with (
            aiohttp.ClientSession(
                base_url=BASE_URL,
                headers=HEADERS,
            ) as session,
            session.get(URL_SLUG) as response,
        ):
            print(response.status)


if __name__ == "__main__":
    import asyncio

    asyncio.run(main())

[sarayourfriend]

Thank you for the report and reproduction code! If you're willing to work on a PR for this, I would appreciate it, otherwise I will try to attend to this next week.

[sarayourfriend]

Fixed and release in 2.1.2. Thanks again for the detailed report @pandaxtc!

[pandaxtc]

Thanks for the quick fix!