diff options
author | 2017-02-02 20:49:28 +0000 | |
---|---|---|
committer | 2017-02-03 10:25:09 +0000 | |
commit | 53e7a49308d6e03701f595993a98b582579cb0d6 (patch) | |
tree | 85d9e21d926f810dabe606ca1732e1dfb0f8f7ee /src/test/java/com/google/devtools/build/lib/remote/GrpcActionCacheTest.java | |
parent | 97e1acfa0d36d8a0e0f443578a4991e689148124 (diff) |
Fixing reliance on same order for downloading multiple blobs.
RELNOTES: n/a
--
PiperOrigin-RevId: 146396318
MOS_MIGRATED_REVID=146396318
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/remote/GrpcActionCacheTest.java')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/remote/GrpcActionCacheTest.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/remote/GrpcActionCacheTest.java b/src/test/java/com/google/devtools/build/lib/remote/GrpcActionCacheTest.java index d933b8a778..4a2d2058aa 100644 --- a/src/test/java/com/google/devtools/build/lib/remote/GrpcActionCacheTest.java +++ b/src/test/java/com/google/devtools/build/lib/remote/GrpcActionCacheTest.java @@ -17,6 +17,7 @@ import static com.google.common.truth.Truth.assertThat; import static java.nio.charset.StandardCharsets.UTF_8; import com.google.common.collect.ImmutableList; +import com.google.common.collect.Lists; import com.google.common.collect.Maps; import com.google.devtools.build.lib.actions.Root; import com.google.devtools.build.lib.remote.CasServiceGrpc.CasServiceImplBase; @@ -273,7 +274,8 @@ public class GrpcActionCacheTest { observer.onCompleted(); return; } - for (ContentDigest digest : request.getDigestList()) { + // We change the order on purpose, to test for blobs out of order: + for (ContentDigest digest : Lists.reverse(request.getDigestList())) { observer.onNext( CasDownloadReply.newBuilder() .setStatus(CasStatus.newBuilder().setSucceeded(true)) |