diff options
| -rw-r--r-- | doc/build/changelog/unreleased_20/suite_float_tests.rst | 7 | ||||
| -rw-r--r-- | lib/sqlalchemy/dialects/oracle/types.py | 2 | ||||
| -rw-r--r-- | lib/sqlalchemy/dialects/postgresql/asyncpg.py | 2 | ||||
| -rw-r--r-- | lib/sqlalchemy/dialects/postgresql/pg8000.py | 2 | ||||
| -rw-r--r-- | lib/sqlalchemy/testing/requirements.py | 6 | ||||
| -rw-r--r-- | lib/sqlalchemy/testing/suite/test_types.py | 11 | ||||
| -rw-r--r-- | test/requirements.py | 4 |
7 files changed, 31 insertions, 3 deletions
diff --git a/doc/build/changelog/unreleased_20/suite_float_tests.rst b/doc/build/changelog/unreleased_20/suite_float_tests.rst new file mode 100644 index 000000000..06e7fcd55 --- /dev/null +++ b/doc/build/changelog/unreleased_20/suite_float_tests.rst @@ -0,0 +1,7 @@ +.. change:: + :tags: bug, sql + + Fixed the base class for dialect-specific float/double types; Oracle + :class:`_oracle.BINARY_DOUBLE` now subclasses :class:`_sqltypes.Double`, + and internal types for :class:`_sqltypes.Float` for asyncpg and pg8000 now + correctly subclass :class:`_sqltypes.Float`. diff --git a/lib/sqlalchemy/dialects/oracle/types.py b/lib/sqlalchemy/dialects/oracle/types.py index 1c252ab89..62028c767 100644 --- a/lib/sqlalchemy/dialects/oracle/types.py +++ b/lib/sqlalchemy/dialects/oracle/types.py @@ -97,7 +97,7 @@ class FLOAT(sqltypes.FLOAT): self.binary_precision = binary_precision -class BINARY_DOUBLE(sqltypes.Float): +class BINARY_DOUBLE(sqltypes.Double): __visit_name__ = "BINARY_DOUBLE" diff --git a/lib/sqlalchemy/dialects/postgresql/asyncpg.py b/lib/sqlalchemy/dialects/postgresql/asyncpg.py index c879205e4..a25502b90 100644 --- a/lib/sqlalchemy/dialects/postgresql/asyncpg.py +++ b/lib/sqlalchemy/dialects/postgresql/asyncpg.py @@ -322,7 +322,7 @@ class AsyncpgNumeric(sqltypes.Numeric): ) -class AsyncpgFloat(AsyncpgNumeric): +class AsyncpgFloat(AsyncpgNumeric, sqltypes.Float): __visit_name__ = "float" render_bind_cast = True diff --git a/lib/sqlalchemy/dialects/postgresql/pg8000.py b/lib/sqlalchemy/dialects/postgresql/pg8000.py index 3f01b00e8..a32d375c7 100644 --- a/lib/sqlalchemy/dialects/postgresql/pg8000.py +++ b/lib/sqlalchemy/dialects/postgresql/pg8000.py @@ -148,7 +148,7 @@ class _PGNumeric(sqltypes.Numeric): ) -class _PGFloat(_PGNumeric): +class _PGFloat(_PGNumeric, sqltypes.Float): __visit_name__ = "float" render_bind_cast = True diff --git a/lib/sqlalchemy/testing/requirements.py b/lib/sqlalchemy/testing/requirements.py index b59cce374..c1d6a14aa 100644 --- a/lib/sqlalchemy/testing/requirements.py +++ b/lib/sqlalchemy/testing/requirements.py @@ -1241,6 +1241,12 @@ class SuiteRequirements(Requirements): return exclusions.open() @property + def float_is_numeric(self): + """target backend uses Numeric for Float/Dual""" + + return exclusions.open() + + @property def text_type(self): """Target database must support an unbounded Text() " "type such as TEXT or CLOB""" diff --git a/lib/sqlalchemy/testing/suite/test_types.py b/lib/sqlalchemy/testing/suite/test_types.py index 72f1e8c10..92781cc1b 100644 --- a/lib/sqlalchemy/testing/suite/test_types.py +++ b/lib/sqlalchemy/testing/suite/test_types.py @@ -13,6 +13,7 @@ from .. import fixtures from .. import mock from ..assertions import eq_ from ..assertions import is_ +from ..assertions import ne_ from ..config import requirements from ..schema import Column from ..schema import Table @@ -47,6 +48,7 @@ from ... import UUID from ... import Uuid from ...orm import declarative_base from ...orm import Session +from ...sql import sqltypes from ...sql.sqltypes import LargeBinary from ...sql.sqltypes import PickleType @@ -1090,6 +1092,15 @@ class NumericTest(_LiteralRoundTripFixture, fixtures.TestBase): Numeric(precision=5, scale=3), numbers, numbers, check_scale=True ) + @testing.combinations(sqltypes.Float, sqltypes.Double, argnames="cls_") + @testing.requires.float_is_numeric + def test_float_is_not_numeric(self, connection, cls_): + target_type = cls_().dialect_impl(connection.dialect) + numeric_type = sqltypes.Numeric().dialect_impl(connection.dialect) + + ne_(target_type.__visit_name__, numeric_type.__visit_name__) + ne_(target_type.__class__, numeric_type.__class__) + class BooleanTest(_LiteralRoundTripFixture, fixtures.TablesTest): __backend__ = True diff --git a/test/requirements.py b/test/requirements.py index 3c72cd07d..ae7200238 100644 --- a/test/requirements.py +++ b/test/requirements.py @@ -1447,6 +1447,10 @@ class DefaultRequirements(SuiteRequirements): return skip_if(("mssql+pyodbc", None, None, "crashes due to bug #351")) @property + def float_is_numeric(self): + return exclusions.fails_if(["oracle"]) + + @property def duplicate_key_raises_integrity_error(self): return exclusions.open() |
