Have -XepPatchChecks: skip disabled checks

Resolves #4943.

Fixes #5102

COPYBARA_INTEGRATE_REVIEW=#5102 from PicnicSupermarket:sschroevers/have-XepPatchChecks-skip-disabled-checks dabd820
PiperOrigin-RevId: 809034699

Author: Stephan202

check_api/src/main/java/com/google/errorprone/scanner/ScannerSupplier.java

@@ -298,7 +298,7 @@ public ScannerSupplier plus(ScannerSupplier other) {
   public ScannerSupplier filter(Predicate<? super BugCheckerInfo> predicate) {
     ImmutableSet<String> disabled =
         getAllChecks().values().stream()
-            .filter(predicate.negate())
+            .filter(bci -> disabled().contains(bci.canonicalName()) || !predicate.test(bci))
             .map(BugCheckerInfo::canonicalName)
             .collect(ImmutableSet.toImmutableSet());
     return new ScannerSupplierImpl(getAllChecks(), severities(), disabled, getFlags());

core/src/test/java/com/google/errorprone/ErrorProneJavaCompilerTest.java

@@ -486,6 +486,33 @@ class StringConstantWrapper {
             """);
   }
 
+  @Test
+  public void patchAllWithDisableAllChecks() throws IOException {
+    JavaFileObject fileObject =
+        createOnDiskFileObject(
+            "StringConstantWrapper.java",
+            """
+            class StringConstantWrapper {
+              String s = "old-value";
+            }
+            """);
+
+    CompilationResult result =
+        doCompile(
+            Collections.singleton(fileObject),
+            ImmutableList.of(
+                "-XepPatchChecks:", "-XepPatchLocation:IN_PLACE", "-XepDisableAllChecks"),
+            ImmutableList.of(AssignmentUpdater.class));
+    assertThat(result.succeeded).isTrue();
+    assertThat(Files.readString(Path.of(fileObject.toUri())))
+        .isEqualTo(
+            """
+            class StringConstantWrapper {
+              String s = "old-value";
+            }
+            """);
+  }
+
   @Test
   public void patchSingleWithCheckDisabled() throws IOException {
     JavaFileObject fileObject =

core/src/test/java/com/google/errorprone/scanner/ScannerSupplierTest.java

@@ -243,7 +243,9 @@ public void filterWorks() {
         ScannerSupplier.fromBugCheckerClasses(
             ArrayEquals.class, BadShiftAmount.class, StaticQualifiedUsingExpression.class);
 
-    assertScanner(ss.filter(input -> input.canonicalName().equals("BadShiftAmount")))
+    ScannerSupplier derivedSupplier = ss.filter(input -> !input.canonicalName().startsWith("S"));
+    assertScanner(derivedSupplier).hasEnabledChecks(ArrayEquals.class, BadShiftAmount.class);
+    assertScanner(derivedSupplier.filter(input -> !input.canonicalName().startsWith("A")))
         .hasEnabledChecks(BadShiftAmount.class);
   }