]> git.ipfire.org Git - thirdparty/sqlalchemy/sqlalchemy.git/commit
Add complete coverage and fix lower() for MySQL 88718 workaround
authorMike Bayer <mike_mp@zzzcomputing.com>
Mon, 11 Feb 2019 14:07:03 +0000 (09:07 -0500)
committerMike Bayer <mike_mp@zzzcomputing.com>
Mon, 11 Feb 2019 14:11:37 +0000 (09:11 -0500)
commit49197c7b36573d91b015019d4635071f9da1c216
tree1c23029bfc9532fb3f8655eca74a6db225e758fa
parent833583458c69e24e797c300c934da0ff04348db5
Add complete coverage and fix lower() for MySQL 88718 workaround

Fixed a second regression caused by :ticket:`4344` (the first was
:ticket:`4361`), which works around MySQL issue 88718, where the lower
casing function used was not correct for Python 2 with OSX/Windows casing
conventions, which would then raise ``TypeError``.  Full coverage has been
added to this logic so that every codepath is exercised in a mock style for
all three casing conventions on all versions of Python. MySQL 8.0 has
meanwhile fixed issue 88718 so the workaround is only applies to a
particular span of MySQL 8.0 versions.

Fixes: #4492
Change-Id: I14e7237e0be4a9c21c58c921066304ae99ac4dc6
doc/build/changelog/unreleased_12/4492.rst [new file with mode: 0644]
lib/sqlalchemy/dialects/mysql/base.py
test/dialect/mysql/test_reflection.py