Flow-sensitive typing ignores dictionary key mutation
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 20.6k
- Forks
- 3.3k
- Métriques de merge des PR
- Métriques de PR en attente
Description
Bug Report
mypy allows runtime type checks for dictionary items. However it ignores subsequent dictionary mutations. This leads to mypy accepting code that fails at runtime.
To Reproduce
from typing import Dict, Optional
foo_dict: Dict[str, Optional[str]] = {'bar': 'baz'}
if foo_dict['bar'] is not None:
foo_dict['bar'] = None
foo_dict['bar'].upper()
Expected Behavior
mypy reports .upper() call as error. This is similar to what already happens with object attributes:
from dataclasses import dataclass
from typing import Optional
@dataclass
class Foo:
bar: Optional[str]
foo = Foo(bar='baz')
if foo.bar is not None:
foo.bar = None
foo.bar.upper()
For such code mypy outputs the following error: "None" has no attribute "upper".
Actual Behavior
mypy reports no error.
Your Environment
- Mypy version used: 0.812
- Mypy command-line flags:
--strict - Mypy configuration options from
mypy.ini(and other config files): none - Python version used: 3.6.8
- Operating system and version: Windows 8.1
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par exécuter la reproduction Python avec mypy --strict et comparez-la avec l’exemple d’attribut de dataclass, en confirmant que le cas du dictionnaire est accepté tandis que le cas de l’attribut est rejeté. Suivez la logique de narrowing sensible au flux pour les expressions d’éléments de dictionnaire et ajoutez une couverture pour la mutation ; c’est terminé lorsque mypy signale l’appel à .upper() comme une erreur.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- compilers, devtools
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 45/100