grpc / grpc/grpc-java

StreamTracer should be mockable

Aperta
#4,658 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub
code health
Lingua principale
Java
Stelle
12.1k
Fork
4k
Merge medio
2g 17h
PR unite (30g)
37

Descrizione

#3305 put DoNotMock on StreamTracer saying that the mocks are not thread-safe. But [mockito claims they are](https://github.com/mockito/mockito/wiki/FAQ#is-mockito-thread-safe), except for stubbing and verify methods. We need to figure out where the discrepancy is and either file a bug against Mockito or let the StreamTracer be mocked.

I discovered this as part of removing DoNotMock for #4452. For my changes, I left things as-is, but I found the documentation I wrote to be questionable.

CC @carl-mastrangelo

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia con la restrizione di StreamTracer aggiunta da #3305 e confrontala con le FAQ di Mockito sulla thread-safety, in particolare con i limiti relativi a stubbing e verifica. Esamina la documentazione del lavoro su #4452 e determina se la discrepanza è in Mockito o nell’uso di StreamTracer; il lavoro è completato con una segnalazione di bug di Mockito motivata oppure con la rimozione della restrizione.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
java
Ambito
backend, testing-qa
Tipo di issue
Bug
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.