Please make `psycopg2` test dependency optional and/or support psycopg-3

vutran1710 / PyrateLimiter

⚔️Python Rate-Limiter using Leaky-Bucket Algorithm Family

https://pyratelimiter.readthedocs.io

MIT License

334 stars 36 forks source link

Please make `psycopg2` test dependency optional and/or support psycopg-3 #163

Closed mgorny closed 5 months ago

mgorny commented 5 months ago

The psycopg2 has been superseded a while ago by psycopg >= 3. However, the most recent release make it impossible to run the test suite without psycopg2 installed:

tests/conftest.py:20: in <module>
    from psycopg2.pool import ThreadedConnectionPool
E   ModuleNotFoundError: No module named 'psycopg2'

Could you please make the dependency optional, so we could deselect the tests requiring it? Or at least move to the modern psycopg package?

vutran1710 commented 5 months ago

Oh my bad, tks for letting me know

vutran1710 commented 5 months ago

@mgorny can you review? #164

vutran1710 commented 5 months ago

Resolved in #164 and #165