diff options
| -rw-r--r-- | doc/build/changelog/changelog_10.rst | 2 | ||||
| -rw-r--r-- | test/dialect/mysql/test_reflection.py | 7 | ||||
| -rw-r--r-- | test/requirements.py | 4 |
3 files changed, 11 insertions, 2 deletions
diff --git a/doc/build/changelog/changelog_10.rst b/doc/build/changelog/changelog_10.rst index 3e7f6fd29..21a8f3e6f 100644 --- a/doc/build/changelog/changelog_10.rst +++ b/doc/build/changelog/changelog_10.rst @@ -26,7 +26,7 @@ Fixed bug in MySQL reflection where the "fractional sections portion" of the :class:`.mysql.DATETIME`, :class:`.mysql.TIMESTAMP` and :class:`.mysql.TIME` types would be incorrectly placed into the - ``timestamp`` attribute, which is unused by MySQL, instead of the + ``timezone`` attribute, which is unused by MySQL, instead of the ``fsp`` attribute. .. change:: diff --git a/test/dialect/mysql/test_reflection.py b/test/dialect/mysql/test_reflection.py index 351eacea5..a28876262 100644 --- a/test/dialect/mysql/test_reflection.py +++ b/test/dialect/mysql/test_reflection.py @@ -71,11 +71,16 @@ class TypeReflectionTest(fixtures.TestBase): def test_time_types(self): specs = [] + if testing.requires.mysql_fsp.enabled: + fsps = [None, 0, 5] + else: + fsps = [None] + for type_ in (mysql.TIMESTAMP, mysql.DATETIME, mysql.TIME): # MySQL defaults fsp to 0, and if 0 does not report it. # we don't actually render 0 right now in DDL but even if we do, # it comes back blank - for fsp in (None, 0, 5): + for fsp in fsps: if fsp: specs.append((type_(fsp=fsp), type_(fsp=fsp))) else: diff --git a/test/requirements.py b/test/requirements.py index fa69a62f1..ff93a9c3d 100644 --- a/test/requirements.py +++ b/test/requirements.py @@ -866,6 +866,10 @@ class DefaultRequirements(SuiteRequirements): return skip_if(["oracle", "firebird"], "non-standard SELECT scalar syntax") @property + def mysql_fsp(self): + return only_if('mysql >= 5.6.4') + + @property def mysql_fully_case_sensitive(self): return only_if(self._has_mysql_fully_case_sensitive) |