StreamTracer should be mockable
- Vorherrschende Sprache
- Java
- Sterne
- 12.1k
- Forks
- 4k
- Ø Merge
- 2 T. 17 Std.
- Gemergte PRs (30 T.)
- 37
Beschreibung
#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
Beitragsleitfaden
Rechercherichtung
Beginne mit der durch #3305 hinzugefügten StreamTracer-Einschränkung und vergleiche sie mit Mockitos FAQ zur Thread-Sicherheit, insbesondere den Einschränkungen bei Stubbing und Verifizierung. Prüfe die Dokumentation aus der Arbeit an #4452 und ermittle, ob die Abweichung in Mockito oder in der Verwendung von StreamTracer liegt; abgeschlossen ist die Aufgabe mit entweder einem begründeten Mockito-Bugreport oder der Entfernung der Einschränkung.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- backend, testing-qa
- Issue-Typ
- Bug
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100