Skip to content

Don't accidentally ignore typeguard validity for staticmethods #14953

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
hauntsaninja merged 10 commits into from
Mar 26, 2023
Merged

Don't accidentally ignore typeguard validity for staticmethods #14953

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
bd99419
Fix a slip up I made
A5rocks Mar 24, 2023
9aa5b40
Appease black (?)
A5rocks Mar 24, 2023
96182d2
Appease flake8
A5rocks Mar 24, 2023
e019308
Appease black one more time...
A5rocks Mar 24, 2023
964a6e9
Add a forgotten fixture
A5rocks Mar 24, 2023
a5c48e4
Move error message in test?
A5rocks Mar 24, 2023
b9ee101
Maybe fix the test for good
A5rocks Mar 25, 2023
96fadf7
Update test-data/unit/check-typeguard.test
A5rocks Mar 25, 2023
9270eab
Update test-data/unit/check-typeguard.test
hauntsaninja Mar 25, 2023
93c95f0
Remove some unnecessary fixtures
A5rocks Mar 26, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 13 additions & 3 deletions test-data/unit/check-typeguard.test
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -602,21 +602,31 @@ def func(names: Tuple[str, ...]):
from typing_extensions import TypeGuard

class Z:
def typeguard1(self, *, x: object) -> TypeGuard[int]: # E: TypeGuard functions must have a positional argument
def typeguard1(self, *, x: object) -> TypeGuard[int]: # line 4
...

@staticmethod
def typeguard2(x: object) -> TypeGuard[int]:
...

@staticmethod # E: TypeGuard functions must have a positional argument
@staticmethod # line 11
def typeguard3(*, x: object) -> TypeGuard[int]:
...

def bad_typeguard(*, x: object) -> TypeGuard[int]: # E: TypeGuard functions must have a positional argument
def bad_typeguard(*, x: object) -> TypeGuard[int]: # line 15
...

# seperated outputs are due to a parsing difference (?)
Comment thread
A5rocks marked this conversation as resolved.
Outdated
Comment thread
hauntsaninja marked this conversation as resolved.
Outdated
[builtins fixtures/tuple.pyi]
[builtins fixtures/classmethod.pyi]
Comment thread
A5rocks marked this conversation as resolved.
[out]
main:4: error: TypeGuard functions must have a positional argument
main:11: error: TypeGuard functions must have a positional argument
main:15: error: TypeGuard functions must have a positional argument
[out version>=3.8]
main:4: error: TypeGuard functions must have a positional argument
main:12: error: TypeGuard functions must have a positional argument
main:15: error: TypeGuard functions must have a positional argument

[case testTypeGuardWithKeywordArg]
from typing_extensions import TypeGuard
Expand Down
⚡