Skip to content

Commit

Permalink
[red-knot] Fix Boolean flags in mdtests (#14654)
Browse files Browse the repository at this point in the history
## Summary

Similar to #14652, but now with conditions that are `Literal[True]`
(instead of `Literal[False]`), where we want them to be `bool`.
  • Loading branch information
sharkdp authored Nov 28, 2024
1 parent d8bca0d commit a378ff3
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -21,10 +21,11 @@ reveal_type(Identity[0]) # revealed: str
## Class getitem union

```py
flag = True
def bool_instance() -> bool:
return True

class UnionClassGetItem:
if flag:
if bool_instance():

def __class_getitem__(cls, item: int) -> str:
return item
Expand Down Expand Up @@ -59,9 +60,10 @@ reveal_type(x[0]) # revealed: str | int
## Class getitem with unbound method union

```py
flag = True
def bool_instance() -> bool:
return True

if flag:
if bool_instance():
class Spam:
def __class_getitem__(self, x: int) -> str:
return "foo"
Expand All @@ -77,9 +79,10 @@ reveal_type(Spam[42])
## TODO: Class getitem non-class union

```py
flag = True
def bool_instance() -> bool:
return True

if flag:
if bool_instance():
class Eggs:
def __class_getitem__(self, x: int) -> str:
return "foo"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,10 +30,11 @@ reveal_type(Identity()[0]) # revealed: int
## Getitem union

```py
flag = True
def bool_instance() -> bool:
return True

class Identity:
if flag:
if bool_instance():

def __getitem__(self, index: int) -> int:
return index
Expand Down

0 comments on commit a378ff3

Please sign in to comment.