diff options
author | Federico Caselli <cfederico87@gmail.com> | 2023-04-18 19:38:18 +0000 |
---|---|---|
committer | Gerrit Code Review <gerrit@bbpush.zzzcomputing.com> | 2023-04-18 19:38:18 +0000 |
commit | 280322af927848f51aeb8cfe2f43de91960052ba (patch) | |
tree | dfa718b1b68c62c4a65c24f043089c95a91ec7e3 | |
parent | 756501b65a9c88790c1947d7c39956bfc374b8e8 (diff) | |
parent | ceec98e1d4d24389b10b3a26a693e94ddb70d65e (diff) | |
download | sqlalchemy-280322af927848f51aeb8cfe2f43de91960052ba.tar.gz |
Merge "Define type hints for remaining column operators" into main
-rw-r--r-- | doc/build/changelog/unreleased_20/9650.rst | 13 | ||||
-rw-r--r-- | lib/sqlalchemy/sql/elements.py | 43 | ||||
-rw-r--r-- | test/ext/mypy/plain_files/sql_operations.py | 65 |
3 files changed, 104 insertions, 17 deletions
diff --git a/doc/build/changelog/unreleased_20/9650.rst b/doc/build/changelog/unreleased_20/9650.rst new file mode 100644 index 000000000..88dc8d03d --- /dev/null +++ b/doc/build/changelog/unreleased_20/9650.rst @@ -0,0 +1,13 @@ +.. change:: + :tags: bug, typing + :tickets: 9650 + + Added typing information for recently added operators + :meth:`.ColumnOperators.icontains`, :meth:`.ColumnOperators.istartswith`, + :meth:`.ColumnOperators.iendswith`, and bitwise operators + :meth:`.ColumnOperators.bitwise_and`, :meth:`.ColumnOperators.bitwise_or`, + :meth:`.ColumnOperators.bitwise_xor`, :meth:`.ColumnOperators.bitwise_not`, + :meth:`.ColumnOperators.bitwise_lshift` + :meth:`.ColumnOperators.bitwise_rshift`. Pull request courtesy Martijn + Pieters. + diff --git a/lib/sqlalchemy/sql/elements.py b/lib/sqlalchemy/sql/elements.py index 0f356ae27..b32201c7f 100644 --- a/lib/sqlalchemy/sql/elements.py +++ b/lib/sqlalchemy/sql/elements.py @@ -833,6 +833,12 @@ class SQLCoreOperations(Generic[_T], ColumnOperators, TypingOnly): def __getitem__(self, index: Any) -> ColumnElement[Any]: ... + def __lshift__(self, other: Any) -> ColumnElement[Any]: + ... + + def __rshift__(self, other: Any) -> ColumnElement[Any]: + ... + @overload def concat(self: _SQO[str], other: Any) -> ColumnElement[str]: ... @@ -854,6 +860,24 @@ class SQLCoreOperations(Generic[_T], ColumnOperators, TypingOnly): ) -> BinaryExpression[bool]: ... + def bitwise_xor(self, other: Any) -> BinaryExpression[Any]: + ... + + def bitwise_or(self, other: Any) -> BinaryExpression[Any]: + ... + + def bitwise_and(self, other: Any) -> BinaryExpression[Any]: + ... + + def bitwise_not(self) -> UnaryExpression[_T]: + ... + + def bitwise_lshift(self, other: Any) -> BinaryExpression[Any]: + ... + + def bitwise_rshift(self, other: Any) -> BinaryExpression[Any]: + ... + def in_( self, other: Union[ @@ -915,6 +939,14 @@ class SQLCoreOperations(Generic[_T], ColumnOperators, TypingOnly): ) -> ColumnElement[bool]: ... + def istartswith( + self, + other: Any, + escape: Optional[str] = None, + autoescape: bool = False, + ) -> ColumnElement[bool]: + ... + def endswith( self, other: Any, @@ -923,9 +955,20 @@ class SQLCoreOperations(Generic[_T], ColumnOperators, TypingOnly): ) -> ColumnElement[bool]: ... + def iendswith( + self, + other: Any, + escape: Optional[str] = None, + autoescape: bool = False, + ) -> ColumnElement[bool]: + ... + def contains(self, other: Any, **kw: Any) -> ColumnElement[bool]: ... + def icontains(self, other: Any, **kw: Any) -> ColumnElement[bool]: + ... + def match(self, other: Any, **kwargs: Any) -> ColumnElement[bool]: ... diff --git a/test/ext/mypy/plain_files/sql_operations.py b/test/ext/mypy/plain_files/sql_operations.py index d658f3d50..95993f628 100644 --- a/test/ext/mypy/plain_files/sql_operations.py +++ b/test/ext/mypy/plain_files/sql_operations.py @@ -67,23 +67,54 @@ stmt = select(column("q")).where(lambda: column("g") > 5).where(c2 == 5) expr9 = c1.bool_op("@@")(func.to_tsquery("some & query")) -# add tests for #9148 -and_(c1.is_(q)) -and_(c1.is_not(q)) -and_(c1.isnot(q)) -and_(c1.not_in(["x"])) -and_(c1.notin_(["x"])) -and_(c1.not_like("x")) -and_(c1.notlike("x")) -and_(c1.not_ilike("x")) -and_(c1.notilike("x")) - -# issue #9451 -s1 = c1.cast(Integer) -s2 = c1.cast(Float) -s3 = c1.op("foobar")("operand").cast(DateTime) -s4 = cast(c1, Float) -s5 = cast(c1.op("foobar")("operand"), DateTime) + +def test_issue_9418() -> None: + and_(c1.is_(q)) + and_(c1.is_not(q)) + and_(c1.isnot(q)) + and_(c1.not_in(["x"])) + and_(c1.notin_(["x"])) + and_(c1.not_like("x")) + and_(c1.notlike("x")) + and_(c1.not_ilike("x")) + and_(c1.notilike("x")) + + +def test_issue_9451() -> None: + # issue #9451 + c1.cast(Integer) + c1.cast(Float) + c1.op("foobar")("operand").cast(DateTime) + cast(c1, Float) + cast(c1.op("foobar")("operand"), DateTime) + + +def test_issue_9650_char() -> None: + and_(c1.contains("x")) + and_(c1.startswith("x")) + and_(c1.endswith("x")) + and_(c1.icontains("x")) + and_(c1.istartswith("x")) + and_(c1.iendswith("x")) + + +def test_issue_9650_bitwise() -> None: + # EXPECTED_TYPE: BinaryExpression[Any] + reveal_type(c2.bitwise_and(5)) + # EXPECTED_TYPE: BinaryExpression[Any] + reveal_type(c2.bitwise_or(5)) + # EXPECTED_TYPE: BinaryExpression[Any] + reveal_type(c2.bitwise_xor(5)) + # EXPECTED_TYPE: UnaryExpression[int] + reveal_type(c2.bitwise_not()) + # EXPECTED_TYPE: BinaryExpression[Any] + reveal_type(c2.bitwise_lshift(5)) + # EXPECTED_TYPE: BinaryExpression[Any] + reveal_type(c2.bitwise_rshift(5)) + # EXPECTED_TYPE: ColumnElement[Any] + reveal_type(c2 << 5) + # EXPECTED_TYPE: ColumnElement[Any] + reveal_type(c2 >> 5) if typing.TYPE_CHECKING: |