False positive comparison-overlap for nested function and nested class
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 20.6k
- Forks
- 3.3k
- Métricas de merge de PR
- Métricas de PR pendientes
Descripción
Bug Report
Nested functions inside methods incorrectly resolve attributes of their first argument relative to the surrounding class.
To Reproduce
class foo:
a: str = "foo"
class bar:
class a:
pass
def test(self) -> None:
def inner1(f: foo, s: str) -> bool:
return f.a == s
def inner2(s: str, f: foo) -> bool:
return f.a == s
https://mypy-play.net/?mypy=latest&python=3.11&flags=strict&gist=4c27f653074cd1c3a48bb16f49ca2168
Expected Behavior
As far as I can tell this should be valid code, even though this rarely happens in practice due to CamelCase vs snake_case naming conventions.
Actual Behavior
inner1 fails to type-check, because the a attribute is incorrectly assumed to be the a class inside bar
main.py:10: error: Non-overlapping equality check (left operand type: "Type[a]", right operand type: "str") [comparison-overlap]
Interestingly, inner2 is fine.
Your Environment
- Mypy version used: 1.2.0
- Mypy command-line flags:
--strict - Mypy configuration options from
mypy.ini(and other config files): none - Python version used: 3.11
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Reproduce el ejemplo del issue en mypy con Python 3.11 y --strict, y luego sigue cómo las funciones anidadas resuelven el acceso al atributo en la comparación. La corrección está hecha cuando inner1 ya no informa de un error comparison-overlap, mientras inner2 sigue siendo válido y la regresión está cubierta por una prueba.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- devtools
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100