googleapis / googleapis/google-cloud-cpp

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

Offen
#15,162 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
api: spanner type: cleanup
Vorherrschende Sprache
C++
Sterne
659
Forks
462
Ø Merge
1 T. 2 Std.
Gemergte PRs (30 T.)
89

Beschreibung

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.]

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Find the Spanner Interval integration test file and inspect the existing UsingEmulator() calls nearby. Update the Interval integration tests to use the same pattern, retaining expectations for emulator errors rather than skipping those tests. Run the relevant Spanner integration test target and confirm the emulator behavior remains explicitly checked.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
databases, testing
Issue-Typ
Refactoring
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
55/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.