Remove internal-only methods from testing package
- Lenguaje dominante
- Java
- Estrellas
- 12.1k
- Forks
- 4k
- Merge medio
- 2 d 17 h
- PR fusionados (30 d)
- 37
Descripción
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.
Guía de contribución
Línea de trabajo
Lee los usos de NoopClientCall y TestMethodDescriptors.noopMarshaller en el paquete de testing, junto con el debate en #3099. Identifica los helpers de uso exclusivamente interno cubiertos por el issue y sus callers afectados; se considera terminado cuando esos helpers ya no fomenten los patrones de testing indicados sin romper las herramientas de testing.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- grpc, java
- Área
- testing
- Tipo de issue
- Refactorización
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 35/100