Adding a check to remote execution to ensure the result from the server has a

non-empty set of output files. This would catch a degenerate case when for some
reaon an empty was returned.

RELNOTES: None.
PiperOrigin-RevId: 202672729

1 files changed, 0 insertions, 75 deletions

diff --git a/src/test/shell/bazel/remote/remote_execution_http_test.sh b/src/test/shell/bazel/remote/remote_execution_http_test.sh
index 2a7c6e76f3..1fe3b1e577 100755
--- a/src/test/shell/bazel/remote/remote_execution_http_test.sh
+++ b/src/test/shell/bazel/remote/remote_execution_http_test.sh
@@ -157,81 +157,6 @@ EOF
     expect_log "dir/l is a symbolic link"
 }
 
-function set_directory_artifact_testfixtures() {
-  mkdir -p a
-  cat > a/BUILD <<'EOF'
-package(default_visibility = ["//visibility:public"])
-genrule(
-  name = "test",
-  srcs = ["dir"],
-  outs = ["qux"],
-  cmd = 'mkdir $@ && paste -d"\n" $(location dir)/foo.txt $(location dir)/sub2/symlinked.txt > $@/out.txt',
-)
-EOF
-
-  # Need this to avoid any leftover
-  rm -rf a/dir
-
-  mkdir -p a/dir
-  cat > a/dir/foo.txt <<EOF
-Hello, world
-EOF
-  mkdir -p a/dir/sub1
-  cat > a/dir/sub1/bar.txt <<EOF
-Shuffle, duffle, muzzle, muff
-EOF
-  mkdir -p a/dir/sub2
-  ln -s ../sub1/bar.txt a/dir/sub2/symlinked.txt
-
-  cat > a/test_expected <<EOF
-Hello, world
-Shuffle, duffle, muzzle, muff
-EOF
-}
-
-function test_directory_artifact_local() {
-  set_directory_artifact_testfixtures
-
-  bazel build //a:test >& $TEST_log \
-    || fail "Failed to build //a:test without remote execution"
-  diff bazel-genfiles/a/qux/out.txt a/test_expected \
-      || fail "Local execution generated different result"
-}
-
-function test_directory_artifact() {
-  set_directory_artifact_testfixtures
-
-  bazel build \
-      --spawn_strategy=remote \
-      --remote_executor=localhost:${worker_port} \
-      //a:test >& $TEST_log \
-      || fail "Failed to build //a:test with remote execution"
-  diff bazel-genfiles/a/qux/out.txt a/test_expected \
-      || fail "Remote execution generated different result"
-}
-
-function test_directory_artifact_grpc_cache() {
-  set_directory_artifact_testfixtures
-
-  bazel build \
-      --remote_cache=localhost:${worker_port} \
-      //a:test >& $TEST_log \
-      || fail "Failed to build //a:test with remote gRPC cache"
-  diff bazel-genfiles/a/qux/out.txt a/test_expected \
-      || fail "Remote cache generated different result"
-}
-
-function test_directory_artifact_rest_cache() {
-  set_directory_artifact_testfixtures
-
-  bazel build \
-      --remote_rest_cache=http://localhost:${hazelcast_port}/hazelcast/rest/maps \
-      //a:test >& $TEST_log \
-      || fail "Failed to build //a:test with remote REST cache"
-  diff bazel-genfiles/a/qux/out.txt a/test_expected \
-      || fail "Remote cache generated different result"
-}
-
 function set_directory_artifact_skylark_testfixtures() {
   mkdir -p a
   cat > a/rule.bzl <<'EOF'