abstract class shouldn't be considered Callable
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 20.6k
- Fork
- 3.3k
- Metriche di merge delle PR
- Metriche PR in attesa
Descrizione
Bug Report
An abstract class fits the typing for Callable.
To Reproduce
import abc
from typing import Callable
class A(abc.ABC):
@abc.abstractmethod
def m(self) -> None:
...
class C(A):
def m(self) -> None:
pass
def f() -> A:
return C()
def t(x: Callable[[], A]) -> None:
x()
def main() -> None:
t(f)
t(C)
t(A) # should give typing error - A is not callable because it can't be instantiated
A() # mypy correctly reports that A can't be instantiated
if __name__ == "__main__":
main()
Expected Behavior
mypy should report a typing error if an abstract class is put where a Callable should go
Actual Behavior
mypy doesn't report any error if an abstract class is put where a Callable should go
Your Environment
- Mypy version used: 0.910
- Python version used: 3.8.10
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia eseguendo l’esempio Python fornito con la versione di mypy indicata e conferma l’assenza della diagnostica per t(A). Traccia quindi il modo in cui mypy verifica la compatibilità di Callable e l’istanza di classi astratte, poi aggiungi una copertura che mostri che una classe astratta viene rifiutata, mentre f e C continuano a essere accettati. Il lavoro è completato quando viene segnalato l’errore di typing previsto per A.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- compilers, devtools
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100