Remove internal-only methods from testing package
- Langage dominant
- Java
- Étoiles
- 12.1k
- Forks
- 4k
- Merge moyen
- 2 j 17 h
- PR mergées (30 j)
- 37
Description
As mentioned in comments in #3099, there are some issues and limitation to the current unit test tools:
- It's hard to test a user-defined generic interceptor.
- People are abusing `NoopClientCall`, using it as an alternative of `mock(ClientCall.class)`, which is as bad as a mock.
- `TestMethodDescriptors.noopMarshaller` does not gain much benefit. It violates the interface's contract, it's equivalent to a `mock(Marshaller.class)`, and it does not work in InProcess transport or interop test.
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Examinez les utilisations de NoopClientCall et de TestMethodDescriptors.noopMarshaller dans le package de testing, ainsi que la discussion dans #3099. Identifiez les helpers à usage interne uniquement concernés par l’issue et leurs callers affectés ; le travail est terminé lorsque ces helpers n’encouragent plus les patterns de testing listés sans casser les outils de testing.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- grpc, java
- Domaine
- testing
- Type d'issue
- Refactorisation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 35/100