fix: retry transactions that fail with expired transaction IDs

[schmidt-sebastian]

Port of googleapis/nodejs-firestore#1347

Once a transaction expires, the backend returns INVALID_ARGUMENT. While INVALID_ARGUMENT is not generally retryable, we want to retry INVALID_ARGUMENT for expired transactions only. We should only retry if the error message contains "transaction has expired" and if we have previously received a transaction ID.

This PR aligns the Java SDK a little more with the Node SDK:

• Rollback RPC is the first step in transaction processing, which allows it to use the same ApiFutures.catchingAsync for all error handling.

There are some test updates:

• I had to drop the Map from the ResponseStubber since one of the tests I created has two Rollback RPCs, which ended up overwriting each other in the Map implementation (based on key equality). Instead, I am using an ArrayList.
• I centralized some of the request validation in ResponseStubber.

[yoshi-code-bot]

Some suggestions could not be made:

• google-cloud-firestore/src/main/java/com/google/cloud/firestore/TransactionRunner.java
  • lines 295-295

[codecov]

Codecov Report

Merging #447 into master will decrease coverage by 0.01%.
The diff coverage is 97.14%.

@@             Coverage Diff              @@
##             master     #447      +/-   ##
============================================
- Coverage     73.39%   73.38%   -0.02%     
- Complexity     1090     1094       +4     
============================================
  Files            65       65              
  Lines          5785     5786       +1     
  Branches        721      721              
============================================
  Hits           4246     4246              
  Misses         1313     1313              
- Partials        226      227       +1     

Impacted Files	Coverage Δ	Complexity Δ
.../com/google/cloud/firestore/TransactionRunner.java	99.03% <96.77%> (-0.97%)	7.00 <2.00> (+2.00)	⬇️
...n/java/com/google/cloud/firestore/Transaction.java	85.41% <100.00%> (-0.86%)	13.00 <2.00> (+2.00)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e241f8e...a604178. Read the comment docs.

[yoshi-code-bot]

Some suggestions could not be made:

• google-cloud-firestore/src/main/java/com/google/cloud/firestore/TransactionRunner.java
  • lines 295-295