Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
test(spanner): add additional "implicit begin transaction" failure te…
…sts (#5718) If the initial read from a streaming operation fails when also trying to implicitly start a transaction, ensure the library properly recovers and completes the operation. Test both permanent and transient failures. They behave slightly differently, a permanent failure will cause an explicit `BeginTransaction`, whereas a transient one causes the RPC to be retried with the `begin` selector still set. This was definitely a gap in our test coverage; @thiagotnunes brought a Java customer issue (googleapis/java-spanner#799) to my attention, so I wanted to ensure C++ users were not susceptible.
- Loading branch information
Showing
1 changed file
with
129 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters