have 'bazel test' non-test targets depend on --remote_download_output (…

…#17234) This fixes #17190 Closes #17195. PiperOrigin-RevId: 501555041 Change-Id: Ie52f72fb82912dde135eaa56c02d9756eb31365b Co-authored-by: Anthony Ter-Saakov <[email protected]>
bazelbuild · Jan 19, 2023 · 82e3dae · 82e3dae
1 parent c2bfb2e
commit 82e3dae
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/main/java/com/google/devtools/build/lib/remote/ToplevelArtifactsDownloader.java b/src/main/java/com/google/devtools/build/lib/remote/ToplevelArtifactsDownloader.java
@@ -180,9 +180,9 @@ private boolean shouldDownloadToplevelOutputs(ConfiguredTargetKey configuredTarg
           if (configuredTarget instanceof RuleConfiguredTarget) {
             var ruleConfiguredTarget = (RuleConfiguredTarget) configuredTarget;
             var isTestRule = isTestRuleName(ruleConfiguredTarget.getRuleClassString());
-            return !isTestRule;
+            return !isTestRule && downloadToplevel;
           }
-          return true;
+          return downloadToplevel;
         } catch (InterruptedException ignored) {
           Thread.currentThread().interrupt();
           return false;