fix: prevent illegal negative timeout values into thread sleep() meth…

…od while retrying exceptions in unit tests. * For details on issue see - googleapis#2206
arpan14 · Feb 8, 2023 · 8588fe1 · 8588fe1
1 parent d566613
commit 8588fe1
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/google-cloud-spanner/src/test/java/com/google/cloud/spanner/it/ITClosedSessionTest.java b/google-cloud-spanner/src/test/java/com/google/cloud/spanner/it/ITClosedSessionTest.java
@@ -251,7 +251,10 @@ public void testTransactionManager() throws InterruptedException {
             break;
           }
         } catch (AbortedException e) {
-          Thread.sleep(e.getRetryDelayInMillis());
+          long retryDelayInMillis = e.getRetryDelayInMillis();
+          if (retryDelayInMillis > 0) {
+            Thread.sleep(retryDelayInMillis);
+          }
           txn = manager.resetForRetry();
         }
       }