diff --git a/check_api/src/main/java/com/google/errorprone/ErrorProneOptions.java b/check_api/src/main/java/com/google/errorprone/ErrorProneOptions.java index a0c63f0f68b..eb0f58f0cbf 100644 --- a/check_api/src/main/java/com/google/errorprone/ErrorProneOptions.java +++ b/check_api/src/main/java/com/google/errorprone/ErrorProneOptions.java @@ -140,21 +140,7 @@ abstract static class Builder { abstract Builder importOrganizer(ImportOrganizer importOrganizer); - abstract PatchingOptions autoBuild(); - - final PatchingOptions build() { - - PatchingOptions patchingOptions = autoBuild(); - - // If anything is specified, then (checkers or refaster) and output must be set. - if ((!patchingOptions.namedCheckers().isEmpty() - || patchingOptions.customRefactorer().isPresent()) - ^ patchingOptions.doRefactor()) { - throw new InvalidCommandLineOptionException( - "-XepPatchChecks and -XepPatchLocation must be specified together"); - } - return patchingOptions; - } + abstract PatchingOptions build(); } } @@ -419,6 +405,8 @@ public static ErrorProneOptions processArgs(Iterable args) { * You can pass the IGNORE_UNKNOWN_CHECKS_FLAG to opt-out of that checking. This allows you to * use command lines from different versions of error-prone interchangeably. */ + boolean patchLocationSet = false; + boolean patchCheckSet = false; Builder builder = new Builder(); for (String arg : args) { switch (arg) { @@ -458,6 +446,7 @@ public static ErrorProneOptions processArgs(Iterable args) { } else if (arg.startsWith(ErrorProneFlags.PREFIX)) { builder.parseFlag(arg); } else if (arg.startsWith(PATCH_OUTPUT_LOCATION)) { + patchLocationSet = true; String remaining = arg.substring(PATCH_OUTPUT_LOCATION.length()); if (remaining.equals("IN_PLACE")) { builder.patchingOptionsBuilder().inPlace(true); @@ -468,6 +457,7 @@ public static ErrorProneOptions processArgs(Iterable args) { builder.patchingOptionsBuilder().baseDirectory(remaining); } } else if (arg.startsWith(PATCH_CHECKS_PREFIX)) { + patchCheckSet = true; String remaining = arg.substring(PATCH_CHECKS_PREFIX.length()); if (remaining.startsWith("refaster:")) { // Refaster rule, load from InputStream at file @@ -485,7 +475,8 @@ public static ErrorProneOptions processArgs(Iterable args) { } }); } else { - Iterable checks = Splitter.on(',').trimResults().split(remaining); + Iterable checks = + Splitter.on(',').trimResults().omitEmptyStrings().split(remaining); builder.patchingOptionsBuilder().namedCheckers(ImmutableSet.copyOf(checks)); } } else if (arg.startsWith(PATCH_IMPORT_ORDER_PREFIX)) { @@ -505,6 +496,11 @@ public static ErrorProneOptions processArgs(Iterable args) { } } + if (patchCheckSet && !patchLocationSet) { + throw new InvalidCommandLineOptionException( + "-XepPatchLocation must be specified when -XepPatchChecks is"); + } + return builder.build(remainingArgs.build()); } diff --git a/check_api/src/test/java/com/google/errorprone/ErrorProneOptionsTest.java b/check_api/src/test/java/com/google/errorprone/ErrorProneOptionsTest.java index 1bd1ed419c7..6465bd2d57b 100644 --- a/check_api/src/test/java/com/google/errorprone/ErrorProneOptionsTest.java +++ b/check_api/src/test/java/com/google/errorprone/ErrorProneOptionsTest.java @@ -238,9 +238,6 @@ public void recognizesPatch() { @Test public void throwsExceptionWithBadPatchArgs() { - assertThrows( - InvalidCommandLineOptionException.class, - () -> ErrorProneOptions.processArgs(new String[] {"-XepPatchLocation:IN_PLACE"})); assertThrows( InvalidCommandLineOptionException.class, () -> @@ -257,6 +254,24 @@ public void recognizesRefaster() { assertThat(options.patchingOptions().customRefactorer()).isPresent(); } + @Test + public void understandsEmptySetOfNamedCheckers() { + ErrorProneOptions options = + ErrorProneOptions.processArgs(new String[] {"-XepPatchLocation:IN_PLACE"}); + assertThat(options.patchingOptions().doRefactor()).isTrue(); + assertThat(options.patchingOptions().inPlace()).isTrue(); + assertThat(options.patchingOptions().namedCheckers()).isEmpty(); + assertThat(options.patchingOptions().customRefactorer()).isAbsent(); + + options = + ErrorProneOptions.processArgs( + new String[] {"-XepPatchLocation:IN_PLACE", "-XepPatchChecks:"}); + assertThat(options.patchingOptions().doRefactor()).isTrue(); + assertThat(options.patchingOptions().inPlace()).isTrue(); + assertThat(options.patchingOptions().namedCheckers()).isEmpty(); + assertThat(options.patchingOptions().customRefactorer()).isAbsent(); + } + @Test public void importOrder_staticFirst() { ErrorProneOptions options =