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]$)"/>
@@ -127,6 +129,11 @@
             <property name="illegalPkgs" value="junit.framework"/>
             <message key="import.illegal" value="Use JUnit 4-style (org.junit.*) test classes and assertions instead of JUnit 3 (junit.framework.*)."/>
         </module>
+        <module name="IllegalImport"> <!-- Only relevant for pre-Java 11 because javafx is gone completely in Java 11 -->
+            <property name="id" value="BanJavafx"/>
+            <property name="illegalPkgs" value="javafx"/>
+            <message key="import.illegal" value="Must not import javafx classes because some OpenJDK builds do not include javafx."/>
+        </module>
         <module name="IllegalImport"> <!-- Java Coding Guidelines: Import the canonical package -->
             <property name="illegalPkgs" value="org.elasticsearch.common.base, com.clearspring.analytics.util, org.spark_project.guava"/>
             <message key="import.illegal" value="Must not import repackaged classes."/>
@@ -417,6 +424,7 @@
             <property name="allowEmptyMethods" value="true"/>
             <property name="allowEmptyTypes" value="true"/>
             <property name="allowEmptyLoops" value="true"/>
+            <property name="allowEmptyLambdas" value="true"/>
             <property name="ignoreEnhancedForColon" value="false"/>
             <message key="ws.notFollowed" value="WhitespaceAround: ''{0}'' is not followed by whitespace. Empty blocks may only be represented as '{}' when not part of a multi-block statement (4.1.3)"/>
             <message key="ws.notPreceded" value="WhitespaceAround: ''{0}'' is not preceded with whitespace."/>
@@ -430,7 +438,7 @@
         </module>
         <module name="CyclomaticComplexity"/> <!-- Java Coding Guidelines: Reduce Cyclomatic Complexity -->
         <module name="DesignForExtension"> <!-- Java Coding Guidelines: Design for extension -->
-            <property name="ignoredAnnotations" value="Test, Before, BeforeEach, After, AfterEach, BeforeClass, BeforeAll, AfterClass, AfterAll"/>
+            <property name="ignoredAnnotations" value="ParameterizedTest, Test, Before, BeforeEach, After, AfterEach, BeforeClass, BeforeAll, AfterClass, AfterAll"/>
         </module>
         <module name="JavadocMethod"> <!-- Java Style Guide: Where Javadoc is used -->
             <property name="scope" value="public"/>
@@ -463,7 +471,7 @@
         <module name="NestedTryDepth"/> <!-- Java Coding Guide: Try/catch blocks: never nested -->
         <module name="NonEmptyAtclauseDescription"/> <!-- Java Style Guide: At-clauses -->
         <module name="ParameterName"> <!-- Java Style Guide: Parameter names -->
-            <property name="format" value="^[a-z][a-zA-Z0-9]+$"/>
+            <property name="format" value="^_?[a-z][a-zA-Z0-9]+$"/>
             <message key="name.invalidPattern" value="Parameter name ''{0}'' must match pattern ''{1}''."/>
         </module>
         <module name="SingleLineJavadoc"/> <!-- Java Style Guide: General form -->

build.gradle

@@ -29,7 +29,7 @@ buildscript {
         classpath 'com.netflix.nebula:gradle-info-plugin:5.1.1'
         classpath 'com.jfrog.bintray.gradle:gradle-bintray-plugin:1.8.4'
         classpath 'com.netflix.nebula:nebula-publishing-plugin:14.0.0'
-        classpath 'com.palantir.baseline:gradle-baseline-java:0.51.1'
+        classpath 'com.palantir.baseline:gradle-baseline-java:2.24.0'
         classpath 'com.palantir.gradle.conjure:gradle-conjure:4.13.4'
         classpath 'com.palantir.gradle.gitversion:gradle-git-version:0.12.2'
         classpath 'gradle.plugin.org.inferred:gradle-processors:3.1.0'

conjure-core/src/main/java/com/palantir/conjure/defs/ConjureParserUtils.java

@@ -72,6 +72,7 @@
 import com.palantir.conjure.spec.UnionDefinition;
 import com.palantir.conjure.visitor.DealiasingTypeVisitor;
 import com.palantir.conjure.visitor.TypeDefinitionVisitor;
+import com.palantir.logsafe.exceptions.SafeIllegalArgumentException;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.List;
@@ -97,7 +98,7 @@ public static String parsePackageOrElseThrow(
             Optional<String> defaultPackage) {
         String packageName = conjurePackage
                 .map(p -> p.name())
-                .orElseGet(() -> defaultPackage.orElseThrow(() -> new IllegalArgumentException(
+                .orElseGet(() -> defaultPackage.orElseThrow(() -> new SafeIllegalArgumentException(
                         // TODO(rfink): Better errors: Can we provide context on where exactly no package was provided?
                         "Must provide default conjure package or "
                                 + "explicit conjure package for every object and service")));
@@ -373,7 +374,7 @@ private static Optional<AuthType> parseAuthType(
             case NONE:
                 return Optional.empty();
             default:
-                throw new IllegalArgumentException("Unrecognized auth type.");
+                throw new SafeIllegalArgumentException("Unrecognized auth type.");
         }
     }
 
@@ -415,14 +416,14 @@ private static ParameterType parseParameterType(
                     return ParameterType.body(BodyParameterType.of());
                 }
             case HEADER:
-                String headerParamId = argumentDef.paramId().map(id -> id.name()).orElse(argName.get());
+                String headerParamId = argumentDef.paramId().map(id -> id.name()).orElseGet(() -> argName.get());
                 return ParameterType.header(HeaderParameterType.of(ParameterId.of(headerParamId)));
             case PATH:
                 return ParameterType.path(PathParameterType.of());
             case BODY:
                 return ParameterType.body(BodyParameterType.of());
             case QUERY:
-                String queryParamId = argumentDef.paramId().map(id -> id.name()).orElse(argName.get());
+                String queryParamId = argumentDef.paramId().map(id -> id.name()).orElseGet(() -> argName.get());
                 return ParameterType.query(QueryParameterType.of(ParameterId.of(queryParamId)));
             default:
                 throw new IllegalArgumentException("Unknown parameter type: " + argumentDef.paramType());

conjure-core/src/main/java/com/palantir/conjure/defs/ConjureTypeParserVisitor.java

@@ -105,7 +105,7 @@ public ConjureTypeParserVisitor(ReferenceTypeResolver nameResolver) {
     }
 
     @Override
-    public Type visitAny(AnyType type) {
+    public Type visitAny(AnyType _type) {
         return Type.primitive(com.palantir.conjure.spec.PrimitiveType.ANY);
     }
 
@@ -146,12 +146,12 @@ public Type visitSet(SetType type) {
     }
 
     @Override
-    public Type visitBinary(BinaryType type) {
+    public Type visitBinary(BinaryType _type) {
         return Type.primitive(com.palantir.conjure.spec.PrimitiveType.BINARY);
     }
 
     @Override
-    public Type visitDateTime(DateTimeType type) {
+    public Type visitDateTime(DateTimeType _type) {
         return Type.primitive(com.palantir.conjure.spec.PrimitiveType.DATETIME);
     }
 }

conjure-core/src/main/java/com/palantir/conjure/defs/validator/ConjureDefinitionValidator.java

@@ -122,7 +122,7 @@ public void validate(ConjureDefinition definition) {
             // create mapping from object type name -> names of reference types that are fields of that type
             Multimap<TypeName, TypeName> typeToRefFields = HashMultimap.create();
 
-            definition.getTypes().stream().forEach(type ->
+            definition.getTypes().forEach(type ->
                     getReferenceType(type)
                             .ifPresent(entry -> typeToRefFields.put(
                                     type.accept(TypeDefinitionVisitor.TYPE_NAME), entry)));
@@ -181,14 +181,14 @@ public void validate(ConjureDefinition definition) {
             // create mapping for resolving reference types during validation
             Map<TypeName, TypeDefinition> definitionMap = definition.getTypes().stream().collect(
                     Collectors.toMap(entry -> entry.accept(TypeDefinitionVisitor.TYPE_NAME), entry -> entry));
-            definition.getTypes().stream().forEach(def -> validateTypeDefinition(def, definitionMap));
+            definition.getTypes().forEach(def -> validateTypeDefinition(def, definitionMap));
             definition.getErrors().forEach(def -> validateErrorDefinition(def, definitionMap));
             definition.getServices().forEach(def -> validateServiceDefinition(def, definitionMap));
         }
 
         private static void validateServiceDefinition(ServiceDefinition serviceDef,
                 Map<TypeName, TypeDefinition> definitionMap) {
-            serviceDef.getEndpoints().stream().forEach(endpoint -> {
+            serviceDef.getEndpoints().forEach(endpoint -> {
                 endpoint.getArgs().stream()
                         .filter(arg -> recursivelyFindNestedOptionals(arg.getType(), definitionMap, false))
                         .findAny()
@@ -224,7 +224,7 @@ private static void validateTypeDefinition(TypeDefinition typeDef,
 
             typeDef.accept(new TypeDefinition.Visitor<Void>() {
                 @Override
-                public Void visitAlias(AliasDefinition value) {
+                public Void visitAlias(AliasDefinition _value) {
                     AliasDefinition aliasDef = typeDef.accept(TypeDefinitionVisitor.ALIAS);
                     if (recursivelyFindNestedOptionals(aliasDef.getAlias(), definitionMap, false)) {
                         throw new IllegalStateException(
@@ -234,7 +234,7 @@ public Void visitAlias(AliasDefinition value) {
                 }
 
                 @Override
-                public Void visitObject(ObjectDefinition value) {
+                public Void visitObject(ObjectDefinition _value) {
                     ObjectDefinition objectDefinition = typeDef.accept(TypeDefinitionVisitor.OBJECT);
                     objectDefinition.getFields().stream()
                             .filter(fieldDefinition -> recursivelyFindNestedOptionals(
@@ -249,7 +249,7 @@ public Void visitObject(ObjectDefinition value) {
                 }
 
                 @Override
-                public Void visitUnion(UnionDefinition value) {
+                public Void visitUnion(UnionDefinition _value) {
                     UnionDefinition unionDefinition = typeDef.accept(TypeDefinitionVisitor.UNION);
                     unionDefinition.getUnion().stream()
                             .filter(fieldDefinition -> recursivelyFindNestedOptionals(
@@ -264,12 +264,12 @@ public Void visitUnion(UnionDefinition value) {
                 }
 
                 @Override
-                public Void visitEnum(EnumDefinition value) {
+                public Void visitEnum(EnumDefinition _value) {
                     return null;
                 }
 
                 @Override
-                public Void visitUnknown(String unknownType) {
+                public Void visitUnknown(String _unknownType) {
                     return null;
                 }
             });

conjure-core/src/main/java/com/palantir/conjure/defs/validator/EndpointDefinitionValidator.java

@@ -39,6 +39,7 @@
 import com.palantir.conjure.visitor.ParameterTypeVisitor;
 import com.palantir.conjure.visitor.TypeDefinitionVisitor;
 import com.palantir.conjure.visitor.TypeVisitor;
+import com.palantir.logsafe.exceptions.SafeRuntimeException;
 import java.util.Arrays;
 import java.util.HashSet;
 import java.util.List;
@@ -298,25 +299,25 @@ public Boolean visitSet(SetType value) {
                         }
 
                         @Override
-                        public Boolean visitMap(MapType value) {
+                        public Boolean visitMap(MapType _value) {
                             return false;
                         }
 
                         // The cases below should not be handled here, since they implicitly handled by the
                         // DealiasingTypeVisitor above
                         @Override
-                        public Boolean visitReference(TypeName value) {
-                            throw new RuntimeException("Unexpected type when validating query parameters");
+                        public Boolean visitReference(TypeName _value) {
+                            throw new SafeRuntimeException("Unexpected type when validating query parameters");
                         }
 
                         @Override
-                        public Boolean visitExternal(ExternalReference value) {
-                            throw new RuntimeException("Unexpected type when validating query parameters");
+                        public Boolean visitExternal(ExternalReference _value) {
+                            throw new SafeRuntimeException("Unexpected type when validating query parameters");
                         }
 
                         @Override
-                        public Boolean visitUnknown(String unknownType) {
-                            throw new RuntimeException("Unexpected type when validating query parameters");
+                        public Boolean visitUnknown(String _unknownType) {
+                            throw new SafeRuntimeException("Unexpected type when validating query parameters");
                         }
                     }));
         }

conjure-core/src/main/java/com/palantir/conjure/defs/validator/HttpPathValidator.java

@@ -74,7 +74,7 @@ public static void validate(HttpPath httpPath) {
 
         // verify that path template variables are unique
         Set<String> templateVars = new HashSet<>();
-        new UriTemplate(path.toString()).getTemplateVariables().stream().forEach(var -> {
+        new UriTemplate(path.toString()).getTemplateVariables().forEach(var -> {
             Preconditions.checkState(!templateVars.contains(var),
                     "Path parameter %s appears more than once in path %s", var, path);
             templateVars.add(var);

conjure-core/src/main/java/com/palantir/conjure/defs/validator/UnionDefinitionValidator.java

@@ -47,7 +47,7 @@ private static final class NoTrailingUnderscoreValidator implements ConjureValid
 
         @Override
         public void validate(UnionDefinition definition) {
-            definition.getUnion().stream().forEach(fieldDef -> {
+            definition.getUnion().forEach(fieldDef -> {
                 Preconditions.checkArgument(!fieldDef.getFieldName().get().endsWith("_"),
                         "Union member key must not end with an underscore: %s",
                         fieldDef.getFieldName().get());
@@ -73,14 +73,13 @@ private static boolean isValidJavaIdentifier(String key) {
 
         @Override
         public void validate(UnionDefinition definition) {
-            definition.getUnion().stream().forEach(fieldDef -> {
-                        Preconditions.checkArgument(!Strings.isNullOrEmpty(fieldDef.getFieldName().get()),
+            definition.getUnion().forEach(fieldDef -> {
+                        com.palantir.logsafe.Preconditions.checkArgument(!Strings.isNullOrEmpty(fieldDef.getFieldName().get()),
                                 "Union member key must not be empty");
                         Preconditions.checkArgument(isValidJavaIdentifier(fieldDef.getFieldName().get()),
                                 "Union member key must be a valid Java identifier: %s",
                                 fieldDef.getFieldName().get());
-                    }
-            );
+                    });
         }
     }
 }

conjure-core/src/main/java/com/palantir/conjure/parser/services/ArgumentDefinition.java

@@ -99,7 +99,7 @@ class Builder extends ImmutableArgumentDefinition.Builder {}
     class ArgumentDefinitionDeserializer extends JsonDeserializer<ArgumentDefinition> {
         @SuppressWarnings("deprecation")
         @Override
-        public ArgumentDefinition deserialize(JsonParser parser, DeserializationContext context)
+        public ArgumentDefinition deserialize(JsonParser parser, DeserializationContext _context)
                 throws IOException, JsonProcessingException {
 
             String candidate = parser.getValueAsString();

conjure-core/src/main/java/com/palantir/conjure/parser/services/AuthDefinition.java

@@ -16,12 +16,12 @@
 
 package com.palantir.conjure.parser.services;
 
-import static com.google.common.base.Preconditions.checkArgument;
 
 import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonValue;
 import com.fasterxml.jackson.databind.annotation.JsonDeserialize;
 import com.palantir.conjure.defs.ConjureImmutablesStyle;
+import com.palantir.logsafe.Preconditions;
 import java.util.Locale;
 import org.immutables.value.Value;
 
@@ -81,7 +81,7 @@ static AuthDefinition fromString(String value) {
                 id = "NONE";
                 break;
             case COOKIE:
-                checkArgument(parts.length == 2, "Cookie authorization type must include a cookie name");
+                Preconditions.checkArgument(parts.length == 2, "Cookie authorization type must include a cookie name");
                 id = parts[1];
                 break;
             default:

conjure-core/src/main/java/com/palantir/conjure/parser/types/BaseObjectTypeDefinition.java

@@ -29,6 +29,7 @@
 import com.palantir.conjure.parser.types.complex.UnionTypeDefinition;
 import com.palantir.conjure.parser.types.names.ConjurePackage;
 import com.palantir.conjure.parser.types.reference.AliasTypeDefinition;
+import com.palantir.logsafe.exceptions.SafeIllegalArgumentException;
 import java.io.IOException;
 import java.util.Optional;
 
@@ -44,7 +45,7 @@ public interface BaseObjectTypeDefinition {
 
     class BaseObjectTypeDefinitionDeserializer extends JsonDeserializer<BaseObjectTypeDefinition> {
         @Override
-        public BaseObjectTypeDefinition deserialize(JsonParser parser, DeserializationContext ctxt) throws IOException {
+        public BaseObjectTypeDefinition deserialize(JsonParser parser, DeserializationContext _ctxt) throws IOException {
             TreeNode tree = parser.readValueAsTree();
             if (tree.get("fields") != null) {
                 return ObjectTypeDefinition.fromJson(parser, tree);
@@ -57,7 +58,7 @@ public BaseObjectTypeDefinition deserialize(JsonParser parser, DeserializationCo
             } else if (tree.get("namespace") != null) {
                 return ErrorTypeDefinition.fromJson(parser, tree);
             } else {
-                throw new IllegalArgumentException(
+                throw new SafeIllegalArgumentException(
                         "Unrecognized definition, types must have either fields, values or an alias defined.");
             }
         }

conjure-core/src/main/java/com/palantir/conjure/parser/types/TypeParser.java

@@ -212,7 +212,7 @@ public T parse(ParserState input) throws ParseException {
             return instance;
         }
 
-        public static <T> TypeFromString<T> of(String type, T instance, Class<T> metric) {
+        public static <T> TypeFromString<T> of(String type, T instance, Class<T> _metric) {
             return new TypeFromString<>(type, instance);
         }
     }

conjure-core/src/main/java/com/palantir/conjure/parser/types/complex/EnumValueDefinition.java

@@ -44,7 +44,7 @@ class Builder extends ImmutableEnumValueDefinition.Builder {}
     class EnumValueDeserializer extends JsonDeserializer<EnumValueDefinition> {
         @SuppressWarnings("deprecation")
         @Override
-        public EnumValueDefinition deserialize(JsonParser parser, DeserializationContext ctxt)
+        public EnumValueDefinition deserialize(JsonParser parser, DeserializationContext _ctxt)
                 throws IOException {
             String candidate = parser.getValueAsString();
             if (candidate != null) {

conjure-core/src/main/java/com/palantir/conjure/parser/types/complex/ErrorTypeDefinition.java

@@ -46,7 +46,7 @@ public interface ErrorTypeDefinition extends BaseObjectTypeDefinition {
     Map<FieldName, FieldDefinition> unsafeArgs();
 
     @Override
-    default <T> T visit(TypeDefinitionVisitor<T> visitor) {
+    default <T> T visit(TypeDefinitionVisitor<T> _visitor) {
         throw new UnsupportedOperationException();
     }