-
Notifications
You must be signed in to change notification settings - Fork 0
Narrow is
with final types correctly
#6
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.
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
base: master
Are you sure you want to change the base?
Changes from all commits
99d4465
0076ae8
a0308bc
2fcace1
2372c12
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -1959,6 +1959,16 @@ def covers_at_runtime(item: Type, supertype: Type) -> bool: | |||||||||||||||
item = get_proper_type(item) | ||||||||||||||||
supertype = get_proper_type(supertype) | ||||||||||||||||
|
||||||||||||||||
# Left `Any` or `Type[Any]` type will never be covered at runtime: | ||||||||||||||||
if isinstance(item, AnyType) or ( | ||||||||||||||||
isinstance(item, TypeType) and isinstance(item.item, AnyType) | ||||||||||||||||
): | ||||||||||||||||
return False | ||||||||||||||||
|
||||||||||||||||
if isinstance(item, (CallableType, TypeType)) and item.is_singleton_type(): | ||||||||||||||||
if is_proper_subtype(item, supertype): | ||||||||||||||||
return True | ||||||||||||||||
Comment on lines
+1968
to
+1970
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This condition checks if the item is a singleton type and a proper subtype of the supertype. Can you add a comment explaining what a singleton type is in this context, and why this check is needed?
Suggested change
|
||||||||||||||||
|
||||||||||||||||
# Since runtime type checks will ignore type arguments, erase the types. | ||||||||||||||||
supertype = erase_type(supertype) | ||||||||||||||||
if is_proper_subtype( | ||||||||||||||||
|
Original file line number | Diff line number | Diff line change | ||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -2167,6 +2167,9 @@ def with_unpacked_kwargs(self) -> NormalizedCallableType: | |||||||||||||||||||
) | ||||||||||||||||||||
) | ||||||||||||||||||||
|
||||||||||||||||||||
def is_singleton_type(self) -> bool: | ||||||||||||||||||||
return self.is_type_obj() and self.type_object().is_final | ||||||||||||||||||||
|
||||||||||||||||||||
def with_normalized_var_args(self) -> Self: | ||||||||||||||||||||
var_arg = self.var_arg() | ||||||||||||||||||||
if not var_arg or not isinstance(var_arg.typ, UnpackType): | ||||||||||||||||||||
|
@@ -3111,6 +3114,11 @@ def __eq__(self, other: object) -> bool: | |||||||||||||||||||
return NotImplemented | ||||||||||||||||||||
return self.item == other.item | ||||||||||||||||||||
|
||||||||||||||||||||
def is_singleton_type(self) -> bool: | ||||||||||||||||||||
return (isinstance(self.item, Instance) and self.item.type.is_final) or isinstance( | ||||||||||||||||||||
self.item, NoneType | ||||||||||||||||||||
) | ||||||||||||||||||||
Comment on lines
+3117
to
+3120
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This function determines if a TypeType is a singleton type. Why is
Suggested change
|
||||||||||||||||||||
|
||||||||||||||||||||
def serialize(self) -> JsonDict: | ||||||||||||||||||||
return {".class": "TypeType", "item": self.item.serialize()} | ||||||||||||||||||||
|
||||||||||||||||||||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -1374,7 +1374,102 @@ def g() -> None: | |
foo() | ||
[builtins fixtures/dict.pyi] | ||
|
||
[case testNarrowingIsFinalType] | ||
from typing import Type, Union | ||
from typing_extensions import final | ||
|
||
@final | ||
class Mark: ... | ||
|
||
@final | ||
class Other: ... | ||
|
||
x: Union[Type[Mark], Type[Other], Type[None], int] | ||
|
||
if x is Mark: | ||
reveal_type(x) # N: Revealed type is "Type[__main__.Mark]" | ||
else: | ||
reveal_type(x) # N: Revealed type is "Union[Type[__main__.Other], Type[None], builtins.int]" | ||
|
||
if x is not Mark: | ||
reveal_type(x) # N: Revealed type is "Union[Type[__main__.Other], Type[None], builtins.int]" | ||
else: | ||
reveal_type(x) # N: Revealed type is "Type[__main__.Mark]" | ||
|
||
y: Type[Mark] = Mark | ||
if x is y: | ||
reveal_type(x) # N: Revealed type is "Type[__main__.Mark]" | ||
else: | ||
reveal_type(x) # N: Revealed type is "Union[Type[__main__.Other], Type[None], builtins.int]" | ||
|
||
if x is not y: | ||
reveal_type(x) # N: Revealed type is "Union[Type[__main__.Other], Type[None], builtins.int]" | ||
else: | ||
reveal_type(x) # N: Revealed type is "Type[__main__.Mark]" | ||
|
||
if x is type(None): | ||
reveal_type(x) # N: Revealed type is "Type[None]" | ||
else: | ||
reveal_type(x) # N: Revealed type is "Union[Type[__main__.Mark], Type[__main__.Other], builtins.int]" | ||
|
||
if x is not type(None): | ||
reveal_type(x) # N: Revealed type is "Union[Type[__main__.Mark], Type[__main__.Other], builtins.int]" | ||
else: | ||
reveal_type(x) # N: Revealed type is "Type[None]" | ||
|
||
z: Type[None] | ||
if x is z: | ||
reveal_type(x) # N: Revealed type is "Type[None]" | ||
else: | ||
reveal_type(x) # N: Revealed type is "Union[Type[__main__.Mark], Type[__main__.Other], builtins.int]" | ||
|
||
if x is not z: | ||
reveal_type(x) # N: Revealed type is "Union[Type[__main__.Mark], Type[__main__.Other], builtins.int]" | ||
else: | ||
reveal_type(x) # N: Revealed type is "Type[None]" | ||
[builtins fixtures/isinstancelist.pyi] | ||
|
||
|
||
[case testNarrowingAnyAgainstFinalType] | ||
from typing import Type, Any | ||
from typing_extensions import final | ||
|
||
@final | ||
class Mark: ... | ||
|
||
x: Any | ||
if x is Mark: | ||
reveal_type(x) # N: Revealed type is "def () -> __main__.Mark" | ||
else: | ||
reveal_type(x) # N: Revealed type is "Any" | ||
Comment on lines
+1441
to
+1444
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
|
||
y: Type[Any] | ||
if y is Mark: | ||
reveal_type(y) # N: Revealed type is "Type[__main__.Mark]" | ||
else: | ||
reveal_type(y) # N: Revealed type is "Type[Any]" | ||
|
||
[builtins fixtures/isinstancelist.pyi] | ||
|
||
|
||
[case testNarrowingIsNonFinalType] | ||
from typing import Type, Union | ||
|
||
class Mark: ... | ||
|
||
x: Union[Type[Mark], Type[None], int] | ||
|
||
if x is Mark: | ||
reveal_type(x) # N: Revealed type is "Union[Type[__main__.Mark], Type[None], builtins.int]" | ||
else: | ||
reveal_type(x) # N: Revealed type is "Union[Type[__main__.Mark], Type[None], builtins.int]" | ||
|
||
y: Type[Mark] = Mark | ||
if x is y: | ||
reveal_type(x) # N: Revealed type is "Union[Type[__main__.Mark], Type[None], builtins.int]" | ||
else: | ||
reveal_type(x) # N: Revealed type is "Union[Type[__main__.Mark], Type[None], builtins.int]" | ||
[builtins fixtures/isinstancelist.pyi] | ||
[case testNarrowingOptionalEqualsNone] | ||
from typing import Optional | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This block prevents
Any
orType[Any]
from being covered at runtime. Is this the desired behavior? Should there be a comment explaining why this is necessary?