googleapis / googleapis/google-cloud-cpp

Update Spanner Interval integration tests to call `UsingEmulator()`

Abierto
#15,162 0 comentarios 0 reacciones 0 asignados Ver en GitHub
api: spanner type: cleanup
Lenguaje dominante
C++
Estrellas
659
Forks
462
Merge medio
1 d 2 h
PR fusionados (30 d)
89

Descripción

The Spanner integrations tests have been written to expect particular behavior from the emulator, even it is an error, rather than just skipping the test altogether. See all the other UsingEmulator() calls in this file, for example. This allows us to learn when the emulator changes behavior and react accordingly, rather than just forgetting about the whole issue forever more.

[It also helped the emulator team, who (at least previously) ran our tests to check their implementation. Rather than just seeing successes no matter what they did, expecting the old (error) behavior allowed them to verify that things had indeed changed, and also to compare their new behavior to that of the service.]

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Busca el archivo de pruebas de integración de Spanner Interval e inspecciona las llamadas existentes a UsingEmulator() cercanas. Actualiza las pruebas de integración de Interval para que usen el mismo patrón, manteniendo las expectativas para los errores del emulador en lugar de omitir esas pruebas. Ejecuta el objetivo de pruebas de integración de Spanner correspondiente y confirma que el comportamiento del emulador se sigue comprobando explícitamente.

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

Evaluación

Stack tecnológico
cpp
Área
databases, testing
Tipo de issue
Refactorización
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
55/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.