Skip to content

Commit b3efec3

Browse files
committed
added comments
1 parent 77da0c5 commit b3efec3

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

  • java-showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it

java-showcase/gapic-showcase/src/test/java/com/google/showcase/v1beta1/it/ITRetries.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -539,10 +539,15 @@ private static void assertAttemptSequence(SequenceReport report, Code... expecte
539539
List<SequenceReport.Attempt> attempts = report.getAttemptsList();
540540
assertThat(attempts).hasSize(expectedCodes.length);
541541
for (int i = 0; i < expectedCodes.length; i++) {
542+
// expectedCodes[i].getNumber() maps the com.google.rpc.Code enum to its standard gRPC integer
543+
// value (1-15)
542544
assertThat(attempts.get(i).getStatus().getCode()).isEqualTo(expectedCodes[i].getNumber());
543545
if (i == 0) {
546+
// Initial attempt executes immediately (0ms or more delay)
544547
assertThat(Durations.toMillis(attempts.get(i).getAttemptDelay())).isAtLeast(0L);
545548
} else {
549+
// Retry attempts check for at least 1ms delay (isAtLeast(1L)) rather than checking strict
550+
// backoff thresholds to prevent test flakiness across execution environments.
546551
assertThat(Durations.toMillis(attempts.get(i).getAttemptDelay())).isAtLeast(1L);
547552
}
548553
}

0 commit comments

Comments
 (0)