Cast pg_proc char columns to Text explicitly

[enobayram]

Fixes #125

Seems like char columns can be wrapped as bytes values under certain circumstances. See #125

[enobayram]

Not sure how to properly test these changes, since I don't know what realistic circumstances would cause the char columns to turn to bytes values on the Python side.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.81%. Comparing base (9c82d77) to head (7915f4e).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #126   +/-   ##
=======================================
  Coverage   89.81%   89.81%           
=======================================
  Files          86       86           
  Lines        4056     4056           
  Branches      841      841           
=======================================
  Hits         3643     3643           
  Misses        336      336           
  Partials       77       77           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[DanCardin]

The test would probably be to do all postgres testing on asyncpg (since i believe im only doing psycopg3 right now, if anything?)

but adding a whole extra driver would be a more involved test refactor, so this is fine as-is!

[DanCardin]

Mind bumping the project patch version and adding it to the changelog, and i can release the merged PR directly?

[YaraslauZhylko]

I applied those changes, and it works on my asyncpg setup:

$ poetry run alembic check
[info     ] Context impl PostgresqlImpl.   [alembic.runtime.migration]
[info     ] Will assume transactional DDL. [alembic.runtime.migration]
No new upgrade operations detected.

$ poetry run alembic downgrade -1
[info     ] Context impl PostgresqlImpl.   [alembic.runtime.migration]
[info     ] Will assume transactional DDL. [alembic.runtime.migration]
[info     ] Running downgrade 00226 -> 00225, Remove unique constraint from the 'test' table. [alembic.runtime.migration]

$ poetry run alembic upgrade head
[info     ] Context impl PostgresqlImpl.   [alembic.runtime.migration]
[info     ] Will assume transactional DDL. [alembic.runtime.migration]
[info     ] Running upgrade 00225 -> 00226, Remove unique constraint from the 'test' table. [alembic.runtime.migration]

[enobayram]

@YaraslauZhylko Thanks for confirming.

@DanCardin done!