Pact provider tests in account-service do not verify any interaction
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- java, spring-boot
- Área
- testing
Línea de trabajo
Start with AccountProviderContractTest and Account2ProviderContractTest in account-service, looking at their @State methods and @BeforeEach target setup. Add a @TestTemplate method with @ExtendWith(PactVerificationInvocationContextProvider.class) that calls context.verifyInteraction(), and replace the hard-coded broker on localhost:9092/9292 with @PactFolder or @PactUrl. Done means both provider tests verify real interactions without a running broker and use random ports.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
AccountProviderContractTest and Account2ProviderContractTest (account-service) define @State methods and a @BeforeEach that sets the target, but have no @TestTemplate method calling PactVerificationContext.verifyInteraction() and no @ExtendWith(PactVerificationInvocationContextProvider.class). As a result no Pact interaction is ever verified.
Additionally both tests require a Pact Broker at localhost:9092 / localhost:9292, so once verification is fixed they would fail in CI.
Suggested solution
- Add a
@TestTemplatemethod with@ExtendWith(PactVerificationInvocationContextProvider.class)callingcontext.verifyInteraction(). - Replace the hard-coded broker with
@PactFolder/@PactUrl, pointing to pact files generated by the consumer tests (see the consumer tests issue), or start a Pact Broker in Testcontainers. - Use
RANDOM_PORTinstead of fixed ports8091/DEFINED_PORTwhere possible.
- Lenguaje dominante
- Java
- Estrellas
- 12
- Forks
- 7
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de piomin/sample-spring-cloud-testing
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
question
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
-
enhancement
Dificultad 3/5 1-2 días Aptitud para principiantes 62/100
-
enhancement
Dificultad 3/5 Medio día Aptitud para principiantes 62/100
Todos los issues de piomin/sample-spring-cloud-testing
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
commons-app/apps-android-commons#7275 ·
Los mantenedores suelen responder en 1 día
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
liberation-data/drivine4j#682 ·
Los mantenedores suelen responder en 1 día
-
Document and enforce pull request review guidelinesPosiblemente ocupada @dashorst la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
`Time.at(t, in: nil)` returns UTC instead of the local time (or the time zone of `t`), unlike CRubyAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
hermes-queue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
chrisalunlloyd2-sudo/mindpalace#349 ·
Los mantenedores suelen responder en 1 día