Reflow comment and add explanation for test location, didn't get included in https://github.com/bazelbuild/bazel/commit/8ef29dd57a41efe18b858f725d2143338623953c.
PiperOrigin-RevId: 420318630
diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/SequencedSkyframeExecutor.java b/src/main/java/com/google/devtools/build/lib/skyframe/SequencedSkyframeExecutor.java
index 6b1416b..d5a9d41 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/SequencedSkyframeExecutor.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/SequencedSkyframeExecutor.java
@@ -478,9 +478,8 @@
|| externalFilesKnowledge.anyFilesInExternalReposSeen
|| externalFilesKnowledge.tooManyNonOutputExternalFilesSeen) {
// We freshly compute knowledge of the presence of external files in the skyframe graph. We
- // use
- // a fresh ExternalFilesHelper instance and only set the real instance's knowledge *after* we
- // are done with the graph scan, lest an interrupt during the graph scan causes us to
+ // use a fresh ExternalFilesHelper instance and only set the real instance's knowledge *after*
+ // we are done with the graph scan, lest an interrupt during the graph scan causes us to
// incorrectly think there are no longer any external files.
ExternalFilesHelper tmpExternalFilesHelper =
externalFilesHelper.cloneWithFreshExternalFilesKnowledge();
diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/LocalDiffAwarenessIntegrationTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/LocalDiffAwarenessIntegrationTest.java
index cebba6f..443c177 100644
--- a/src/test/java/com/google/devtools/build/lib/skyframe/LocalDiffAwarenessIntegrationTest.java
+++ b/src/test/java/com/google/devtools/build/lib/skyframe/LocalDiffAwarenessIntegrationTest.java
@@ -127,6 +127,9 @@
assertThat(e.getCause()).hasCauseThat().hasCauseThat().isSameInstanceAs(injectedException);
}
+ // This test doesn't use --watchfs functionality, but if the source filesystem doesn't offer diffs
+ // Bazel must scan the full Skyframe graph anyway, so a bug in checking output files wouldn't be
+ // detected without --watchfs.
@Test
public void ignoreOutputFilesThenCheckAgainDoesCheck() throws Exception {
Path buildFile =
