DeprecationWarning: Inheritance class AiohttpSession from ClientSession is discouraged

[RomeoDespres]

Importing aiogoogle raises a DeprecationWarning. Usually we don't see it because Python filters it out, however when running tests with pytest it does show up and kinda messes up the output.

It's not really a big deal but I just thought it was worth flagging. Otherwise the lib is great!

You can see it by running the following:

$ python -Wd -c "import aiogoogle"
C:\Python39\lib\abc.py:85: DeprecationWarning: Inheritance class AiohttpSession from ClientSession is discouraged
  cls = super().__new__(mcls, name, bases, namespace, **kwargs)

aiogoogle: 3.2.1
Python: 3.9.2
OS: Windows 10

[omarryhan]

Hey, thanks for raising the issue!

It's quite an annoying warning. It existed since the time I wrote this lib.

Unfortunately, I don't know how to disable it. Also, I don't think that the way we're doing inheritence should be worthy of this warning.

So, if you can find a way to disable this warning, I will happily merge ur PR.

[elbaro]

The warning is about removing ClientSession from class AiohttpSession(ClientSession, AbstractSession) and using the composition instead:

class AiohttpSession:
    def __init__(self):
        self.session = ClientSession()
    
    async def some_method(self):
        await self.session.some_method()

[omarryhan]

Nice! I’ll happily merge this if you can submit a PR @elbaro