Excavator: Upgrades Baseline to the latest version

.baseline/checkstyle/checkstyle.xml

@@ -84,11 +84,13 @@
         <module name="AvoidStarImport"/> <!-- Java Style Guide: No wildcard imports -->
         <module name="AvoidStaticImport"> <!-- Java Style Guide: No static imports -->
             <property name="excludes" value="
+                com.google.common.base.Preconditions.*,
+                com.palantir.logsafe.Preconditions.*,
                 java.util.Collections.*,
                 java.util.stream.Collectors.*,
-                com.palantir.logsafe.Preconditions.*,
-                com.google.common.base.Preconditions.*,
-                org.apache.commons.lang3.Validate.*"/>
+                org.apache.commons.lang3.Validate.*,
+                org.assertj.core.api.Assertions.*,
+                org.mockito.Mockito.*"/>
         </module>
         <module name="ClassTypeParameterName"> <!-- Java Style Guide: Type variable names -->
             <property name="format" value="(^[A-Z][0-9]?)$|([A-Z][a-zA-Z0-9]*[T]$)"/>

build.gradle

@@ -11,7 +11,7 @@ buildscript {
         classpath 'com.jfrog.bintray.gradle:gradle-bintray-plugin:1.8.4'
         classpath 'com.netflix.nebula:nebula-publishing-plugin:14.0.0'
         classpath 'com.netflix.nebula:gradle-info-plugin:5.1.1'
-        classpath 'com.palantir.baseline:gradle-baseline-java:2.13.0'
+        classpath 'com.palantir.baseline:gradle-baseline-java:2.20.1'
         classpath 'gradle.plugin.org.inferred:gradle-processors:3.1.0'
         classpath 'com.palantir.gradle.gitversion:gradle-git-version:0.12.2'
         classpath 'com.palantir.gradle.consistentversions:gradle-consistent-versions:1.12.4'

idea-plugin/src/main/java/com/palantir/javaformat/intellij/PalantirCodeStyleManager.java

@@ -16,7 +16,6 @@
 
 package com.palantir.javaformat.intellij;
 
-import static com.google.common.base.Preconditions.checkState;
 import static java.util.Comparator.comparing;
 
 import com.github.benmanes.caffeine.cache.Caffeine;
@@ -103,7 +102,8 @@ private static Collection<Range<Integer>> toRanges(Collection<TextRange> textRan
     }
 
     private static TextRange toTextRange(Range<Integer> range) {
-        checkState(range.lowerBoundType().equals(BoundType.CLOSED) && range.upperBoundType().equals(BoundType.OPEN));
+        com.palantir.logsafe.Preconditions.checkState(
+                range.lowerBoundType().equals(BoundType.CLOSED) && range.upperBoundType().equals(BoundType.OPEN));
         return new TextRange(range.lowerEndpoint(), range.upperEndpoint());
     }
 

idea-plugin/src/main/java/com/palantir/javaformat/intellij/PalantirJavaFormatConfigurable.java

@@ -26,6 +26,7 @@
 import com.intellij.uiDesigner.core.Spacer;
 import com.palantir.javaformat.intellij.PalantirJavaFormatSettings.EnabledState;
 import com.palantir.javaformat.java.FormatterService;
+import com.palantir.logsafe.exceptions.SafeRuntimeException;
 import java.awt.Insets;
 import java.io.IOException;
 import java.net.URI;
@@ -125,7 +126,7 @@ private String computeFormatterVersion(List<URI> implementationClassPath) {
                     }
                 })
                 .findFirst()
-                .orElseThrow(() -> new RuntimeException("Couldn't find implementation JAR"));
+                .orElseThrow(() -> new SafeRuntimeException("Couldn't find implementation JAR"));
     }
 
     @Override

palantir-java-format-api/src/main/java/com/palantir/javaformat/java/FormatterDiagnostic.java

@@ -16,8 +16,7 @@
 
 package com.palantir.javaformat.java;
 
-import static com.google.common.base.Preconditions.checkArgument;
-import static com.google.common.base.Preconditions.checkNotNull;
+import com.palantir.logsafe.Preconditions;
 
 /** An error that prevented formatting from succeeding. */
 public class FormatterDiagnostic {
@@ -30,9 +29,9 @@ public static FormatterDiagnostic create(String message) {
     }
 
     public static FormatterDiagnostic create(int lineNumber, int column, String message) {
-        checkArgument(lineNumber >= 0);
-        checkArgument(column >= 0);
-        checkNotNull(message);
+        Preconditions.checkArgument(lineNumber >= 0);
+        Preconditions.checkArgument(column >= 0);
+        Preconditions.checkNotNull(message);
         return new FormatterDiagnostic(lineNumber, column, message);
     }
 
@@ -60,6 +59,7 @@ public String message() {
         return message;
     }
 
+    @Override
     public String toString() {
         StringBuilder sb = new StringBuilder();
         if (lineNumber >= 0) {

palantir-java-format-api/src/main/java/com/palantir/javaformat/java/Replacement.java

@@ -14,27 +14,25 @@
 
 package com.palantir.javaformat.java;
 
-import static com.google.common.base.Preconditions.checkArgument;
-import static com.google.common.base.Preconditions.checkNotNull;
-
 import com.google.common.collect.Range;
+import com.palantir.logsafe.Preconditions;
 import java.util.Objects;
 
 /** Data class representing a range in the original source and replacement text for that range. */
 public final class Replacement {
 
     public static Replacement create(int startPosition, int endPosition, String replaceWith) {
-        checkArgument(startPosition >= 0, "startPosition must be non-negative");
-        checkArgument(startPosition <= endPosition, "startPosition cannot be after endPosition");
+        Preconditions.checkArgument(startPosition >= 0, "startPosition must be non-negative");
+        Preconditions.checkArgument(startPosition <= endPosition, "startPosition cannot be after endPosition");
         return new Replacement(Range.closedOpen(startPosition, endPosition), replaceWith);
     }
 
     private final Range<Integer> replaceRange;
     private final String replacementString;
 
     private Replacement(Range<Integer> replaceRange, String replacementString) {
-        this.replaceRange = checkNotNull(replaceRange, "Null replaceRange");
-        this.replacementString = checkNotNull(replacementString, "Null replacementString");
+        this.replaceRange = Preconditions.checkNotNull(replaceRange, "Null replaceRange");
+        this.replacementString = Preconditions.checkNotNull(replacementString, "Null replacementString");
     }
 
     /** The range of characters in the original source to replace. */

palantir-java-format/src/main/java/com/palantir/javaformat/OpsBuilder.java

@@ -161,7 +161,7 @@ private void add(Op op) {
         } else if (op instanceof CloseOp) {
             depth--;
             if (depth < 0) {
-                throw new AssertionError();
+                throw new IllegalStateException();
             }
         }
         ops.add(op);

palantir-java-format/src/main/java/com/palantir/javaformat/doc/Level.java

@@ -27,6 +27,7 @@
 import com.palantir.javaformat.Indent;
 import com.palantir.javaformat.Output;
 import com.palantir.javaformat.doc.StartsWithBreakVisitor.Result;
+import com.palantir.logsafe.exceptions.SafeIllegalStateException;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Optional;
@@ -167,7 +168,7 @@ public State computeBreaks(CommentsHelper commentsHelper, int maxWidth, State st
                 Level firstLevel = innerLevels.stream()
                         .filter(doc -> StartsWithBreakVisitor.INSTANCE.visit(doc) != Result.EMPTY)
                         .findFirst()
-                        .orElseThrow(() -> new IllegalStateException(
+                        .orElseThrow(() -> new SafeIllegalStateException(
                                 "Levels were broken so expected to find at least a non-empty level"));
 
                 // Add the width of tokens, breaks before the firstLevel. We must always have space for
@@ -239,7 +240,7 @@ private Optional<State> tryBreakLastLevel(
         splitByBreaks(leadingDocs, splits, breaks);
 
         state = tryToLayOutLevelOnOneLine(commentsHelper, maxWidth, state);
-        Preconditions.checkState(
+        com.palantir.logsafe.Preconditions.checkState(
                 !state.mustBreak, "We messed up, it wants to break a bunch of splits that shouldn't be broken");
 
         // manually add the last level to the last split

palantir-java-format/src/main/java/com/palantir/javaformat/java/ImportOrderer.java

@@ -19,7 +19,6 @@
 import static com.palantir.javaformat.java.JavaInput.buildToks;
 
 import com.google.common.base.CharMatcher;
-import com.google.common.base.Preconditions;
 import com.google.common.base.Splitter;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableSet;
@@ -357,7 +356,7 @@ private ImportsAndIndex scanImports(int i) throws FormatterException {
 
     // Produces the sorted output based on the imports we have scanned.
     private String reorderedImportsString(ImmutableSortedSet<Import> imports) {
-        Preconditions.checkArgument(!imports.isEmpty(), "imports");
+        com.palantir.logsafe.Preconditions.checkArgument(!imports.isEmpty(), "imports");
 
         // Pretend that the first import was preceded by another import of the same kind, so we don't
         // insert a newline there.
@@ -402,7 +401,7 @@ private StringAndIndex scanImported(int start) throws FormatterException {
         // At the start of each iteration of this loop, i points to an identifier.
         // On exit from the loop, i points to a token after an identifier or after *.
         while (true) {
-            Preconditions.checkState(isIdentifierToken(i));
+            com.palantir.logsafe.Preconditions.checkState(isIdentifierToken(i));
             imported.append(tokenAt(i));
             i++;
             if (!tokenAt(i).equals(".")) {

palantir-java-format/src/main/java/com/palantir/javaformat/java/JavaInput.java

@@ -14,7 +14,6 @@
 
 package com.palantir.javaformat.java;
 
-import static com.google.common.base.Preconditions.checkNotNull;
 import static com.google.common.collect.Iterables.getLast;
 import static java.nio.charset.StandardCharsets.UTF_8;
 
@@ -33,6 +32,7 @@
 import com.palantir.javaformat.Input;
 import com.palantir.javaformat.Newlines;
 import com.palantir.javaformat.java.JavacTokens.RawTok;
+import com.palantir.logsafe.Preconditions;
 import java.io.IOException;
 import java.net.URI;
 import java.util.ArrayList;
@@ -260,7 +260,7 @@ public String toString() {
      * @throws FormatterException if the input cannot be parsed
      */
     public JavaInput(String text) throws FormatterException {
-        this.text = checkNotNull(text);
+        this.text = Preconditions.checkNotNull(text);
         setLines(ImmutableList.copyOf(Newlines.lineIterator(text)));
         ImmutableList<Tok> toks = buildToks(text);
         positionToColumnMap = makePositionToColumnMap(toks);

palantir-java-format/src/main/java/com/palantir/javaformat/java/JavaInputAstVisitor.java

@@ -53,6 +53,7 @@
 import com.google.common.collect.ImmutableSortedSet;
 import com.google.common.collect.Iterables;
 import com.google.common.collect.Iterators;
+import com.google.common.collect.Lists;
 import com.google.common.collect.Multiset;
 import com.google.common.collect.PeekingIterator;
 import com.google.common.collect.Streams;
@@ -72,6 +73,7 @@
 import com.palantir.javaformat.doc.Token;
 import com.palantir.javaformat.java.DimensionHelpers.SortedDims;
 import com.palantir.javaformat.java.DimensionHelpers.TypeWithDims;
+import com.palantir.logsafe.exceptions.SafeIllegalStateException;
 import java.util.ArrayDeque;
 import java.util.ArrayList;
 import java.util.Collection;
@@ -483,7 +485,7 @@ public boolean visitArrayInitializer(List<? extends ExpressionTree> expressions)
             token("{");
             builder.forcedBreak();
             boolean first = true;
-            for (Iterable<? extends ExpressionTree> row : Iterables.partition(expressions, cols)) {
+            for (Iterable<? extends ExpressionTree> row : Lists.partition(expressions, cols)) {
                 if (!first) {
                     builder.forcedBreak();
                 }
@@ -1953,7 +1955,7 @@ public Void visitTypeParameter(TypeParameterTree node, Void unused) {
 
     @Override
     public Void visitUnionType(UnionTypeTree node, Void unused) {
-        throw new IllegalStateException("expected manual descent into union types");
+        throw new SafeIllegalStateException("expected manual descent into union types");
     }
 
     @Override
@@ -2101,7 +2103,7 @@ void visitAndBreakModifiers(
 
     @Override
     public Void visitModifiers(ModifiersTree node, Void unused) {
-        throw new IllegalStateException("expected manual descent into modifiers");
+        throw new SafeIllegalStateException("expected manual descent into modifiers");
     }
 
     /** Output combined modifiers and annotations and returns the trailing break. */
@@ -2188,7 +2190,7 @@ boolean nextIsModifier() {
 
     @Override
     public Void visitCatch(CatchTree node, Void unused) {
-        throw new IllegalStateException("expected manual descent into catch trees");
+        throw new SafeIllegalStateException("expected manual descent into catch trees");
     }
 
     /** Helper method for {@link CatchTree}s. */

palantir-java-format/src/main/java/com/palantir/javaformat/java/SnippetFormatter.java

@@ -23,6 +23,7 @@
 import com.google.common.collect.Range;
 import com.google.common.collect.RangeSet;
 import com.google.common.collect.TreeRangeSet;
+import com.palantir.logsafe.exceptions.SafeIllegalArgumentException;
 import java.util.ArrayList;
 import java.util.List;
 
@@ -97,7 +98,7 @@ public ImmutableList<Replacement> format(
         }
         if (includeComments) {
             if (kind != SnippetKind.COMPILATION_UNIT) {
-                throw new IllegalArgumentException("comment formatting is only supported for compilation units");
+                throw new SafeIllegalArgumentException("comment formatting is only supported for compilation units");
             }
             return formatter.getFormatReplacements(source, ranges);
         }

palantir-java-format/src/main/java/com/palantir/javaformat/java/Trees.java

@@ -105,7 +105,7 @@ static ClassTree getEnclosingTypeDeclaration(TreePath path) {
                     break;
             }
         }
-        throw new AssertionError();
+        throw new IllegalStateException();
     }
 
     /** Skips a single parenthesized tree. */

palantir-java-format/src/main/java/com/palantir/javaformat/java/TypeNameClassifier.java

@@ -43,7 +43,7 @@ public TyParseState next(JavaCaseFormat n) {
                     case UPPER_CAMEL:
                         return TyParseState.TYPE;
                 }
-                throw new AssertionError();
+                throw new IllegalStateException();
             }
         },
 
@@ -59,7 +59,7 @@ public TyParseState next(JavaCaseFormat n) {
                     case UPPER_CAMEL:
                         return TyParseState.TYPE;
                 }
-                throw new AssertionError();
+                throw new IllegalStateException();
             }
         },
 
@@ -92,7 +92,7 @@ public TyParseState next(JavaCaseFormat n) {
                     case UPPER_CAMEL:
                         return TyParseState.TYPE;
                 }
-                throw new AssertionError();
+                throw new IllegalStateException();
             }
         };
 

palantir-java-format/src/main/java/com/palantir/javaformat/java/UsageException.java

@@ -14,9 +14,8 @@
 
 package com.palantir.javaformat.java;
 
-import static com.google.common.base.Preconditions.checkNotNull;
-
 import com.google.common.base.Joiner;
+import com.palantir.logsafe.Preconditions;
 
 /** Checked exception class for formatter command-line usage errors. */
 final class UsageException extends Exception {
@@ -79,7 +78,7 @@ final class UsageException extends Exception {
     }
 
     UsageException(String message) {
-        super(buildMessage(checkNotNull(message)));
+        super(buildMessage(Preconditions.checkNotNull(message)));
     }
 
     private static String buildMessage(String message) {

palantir-java-format/src/main/java/com/palantir/javaformat/java/javadoc/CharStream.java

@@ -14,9 +14,7 @@
 
 package com.palantir.javaformat.java.javadoc;
 
-import static com.google.common.base.Preconditions.checkArgument;
-import static com.google.common.base.Preconditions.checkNotNull;
-
+import com.palantir.logsafe.Preconditions;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
@@ -31,7 +29,7 @@ final class CharStream {
     int toConsume;
 
     CharStream(String input) {
-        this.remaining = checkNotNull(input);
+        this.remaining = Preconditions.checkNotNull(input);
     }
 
     boolean tryConsume(String expected) {
@@ -50,7 +48,7 @@ boolean tryConsumeRegex(Pattern pattern) {
         if (!matcher.find()) {
             return false;
         }
-        checkArgument(matcher.start() == 0);
+        Preconditions.checkArgument(matcher.start() == 0);
         toConsume = matcher.end();
         return true;
     }

palantir-java-format/src/main/java/com/palantir/javaformat/java/javadoc/JavadocFormatter.java

@@ -138,7 +138,7 @@ private String render(List<Token> input, int blockIndent) {
                     throw new AssertionError(token.getType());
             }
         }
-        throw new AssertionError();
+        throw new IllegalStateException();
     }
 
     /*