grpc / grpc/grpc-java

Remove internal-only methods from testing package

Abierto
#3,105 1 comentario 0 reacciones 0 asignados Ver en GitHub
code health
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

Abrir la 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.