Correct test grpc response

The execute service in retryUploadAndExecuteOnMissingInputs should not complete the same handled request multiple times. Closes #7384. PiperOrigin-RevId: 236084104
bazelbuild · Feb 28, 2019 · c210d2f · c210d2f
1 parent 046d253
commit c210d2f
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/src/test/java/com/google/devtools/build/lib/remote/GrpcRemoteExecutionClientTest.java b/src/test/java/com/google/devtools/build/lib/remote/GrpcRemoteExecutionClientTest.java
@@ -1177,15 +1177,16 @@ public void execute(ExecuteRequest request, StreamObserver<Operation> responseOb
                       .setResponse(Any.pack(ExecuteResponse.newBuilder().setStatus(status).build()))
                       .build());
               responseObserver.onCompleted();
+            } else {
+              assertThat(request.getSkipCacheLookup()).isFalse();
+              responseObserver.onNext(
+                  Operation.newBuilder()
+                      .setDone(true)
+                      .setResponse(
+                          Any.pack(ExecuteResponse.newBuilder().setResult(actionResult).build()))
+                      .build());
+              responseObserver.onCompleted();
             }
-            assertThat(request.getSkipCacheLookup()).isFalse();
-            responseObserver.onNext(
-                Operation.newBuilder()
-                    .setDone(true)
-                    .setResponse(
-                        Any.pack(ExecuteResponse.newBuilder().setResult(actionResult).build()))
-                    .build());
-            responseObserver.onCompleted();
           }
         });
     AtomicInteger numCacheUploads = new AtomicInteger();