apache / apache/servicecomb-java-chassis

TestRestClientInvocation的单元测试无法有效验证功能

Abierto
#3,739 2 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Java
Estrellas
1.9k
Forks
814
Merge medio
8 d 23 h
PR fusionados (30 d)
1

Descripción

问题出在 `TestRestClientInvocation` 的 `testSetCseContext_enable_unicode` 方法和 `testSetCseContext_disable_unicode` 方法中, 两个方法都是开头对`invocation.getContext()`进行mock, 结尾又调用`invocation.getContext()`获取结果进行校验, 获取的结果就是mock返回, 所以达不到测试的预期效果. 1.3.x 和 2.8.x 分支均有此问题, 建议优化一下UT代码.

https://github.com/apache/servicecomb-java-chassis/blob/21d4b4863b71332f151eb63020f4e9ac51583002/transports/transport-rest/transport-rest-client/src/test/java/org/apache/servicecomb/transport/rest/client/http/TestRestClientInvocation.java#L282-L346

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Abre transport-rest/transport-rest-client/src/test/java/org/apache/servicecomb/transport/rest/client/http/TestRestClientInvocation.java e inspecciona testSetCseContext_enable_unicode y testSetCseContext_disable_unicode. Empieza rastreando cómo se mockea invocation.getContext(), después ejecuta estas pruebas y confirma que sus assertions inspeccionan el estado producido por el método en lugar del valor de retorno mockeado. La tarea está terminada cuando ambas pruebas verifican de forma significativa el comportamiento del contexto Unicode en las ramas 1.3.x y 2.8.x.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
java
Área
testing
Tipo de issue
Error
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.