Excavator: Upgrades Baseline to the latest version

.baseline/checkstyle/checkstyle.xml

@@ -127,6 +127,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 +422,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 +436,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 +469,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.0'
         classpath 'com.jfrog.bintray.gradle:gradle-bintray-plugin:1.8.4'
         classpath 'com.netflix.nebula:nebula-publishing-plugin:13.6.1'
-        classpath 'com.palantir.baseline:gradle-baseline-java:0.51.1'
+        classpath 'com.palantir.baseline:gradle-baseline-java:2.9.1'
         classpath 'com.palantir.gradle.conjure:gradle-conjure:4.13.1'
         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

@@ -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/UnionDefinitionValidator.java

@@ -74,7 +74,7 @@ private static boolean isValidJavaIdentifier(String key) {
         @Override
         public void validate(UnionDefinition definition) {
             definition.getUnion().stream().forEach(fieldDef -> {
-                        Preconditions.checkArgument(!Strings.isNullOrEmpty(fieldDef.getFieldName().get()),
+                        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",

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();
     }
 

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

@@ -45,7 +45,7 @@ static FieldDefinition of(ConjureType type) {
     class FieldDefinitionDeserializer extends JsonDeserializer<FieldDefinition> {
         @SuppressWarnings("deprecation")
         @Override
-        public FieldDefinition deserialize(JsonParser parser, DeserializationContext context) throws IOException {
+        public FieldDefinition deserialize(JsonParser parser, DeserializationContext _context) throws IOException {
 
             String candidate = parser.getValueAsString();
             if (candidate != null) {

conjure-core/src/test/java/com/palantir/conjure/defs/ConjureDefTest.java

@@ -38,7 +38,6 @@ public void resolvesImportedAliases() {
     @Test
     @Ignore
     public void handlesNonJavaExternalType() {
-        ConjureDefinition conjureDefinition = ConjureParserUtils.parseConjureDef(
-                ImmutableList.of(ConjureParser.parse(new File("src/test/resources/example-external-types.yml"))));
+
     }
 }

conjure-core/src/test/java/com/palantir/conjure/defs/ConjureSpecTest.java

@@ -69,7 +69,7 @@ public ConjureSpecTest(TestCaseDefinition testCaseDef) {
     @Test
     public void testConjureSpec() {
         // test positive cases
-        testCaseDef.positive().orElse(new TreeMap<>()).entrySet().stream().forEach(entry -> {
+        testCaseDef.positive().orElseGet(() -> new TreeMap<>()).entrySet().stream().forEach(entry -> {
             String testName = String.format("positive case %s", entry.getKey());
             String yml = getYmlAsString(testName, entry.getValue().conjure());
             try {
@@ -80,7 +80,7 @@ public void testConjureSpec() {
         });
 
         // test negative cases
-        testCaseDef.negative().orElse(new TreeMap<>()).entrySet().stream().forEach(entry -> {
+        testCaseDef.negative().orElseGet(() -> new TreeMap<>()).entrySet().stream().forEach(entry -> {
             String testName = String.format("negative case %s", entry.getKey());
             String yml = getYmlAsString(testName, entry.getValue().conjure());
             try {

conjure-generator-common/src/main/java/com/palantir/conjure/either/Either.java

@@ -39,7 +39,7 @@ abstract class Left<L, R> implements Either<L, R> {
         abstract L value();
 
         @Override
-        public <T> T fold(Function<? super L, ? extends T> mapLeft, Function<? super R, ? extends T> mapRight) {
+        public <T> T fold(Function<? super L, ? extends T> mapLeft, Function<? super R, ? extends T> _mapRight) {
             return mapLeft.apply(value());
         }
     }
@@ -50,7 +50,7 @@ abstract class Right<L, R> implements Either<L, R> {
         abstract R value();
 
         @Override
-        public <T> T fold(Function<? super L, ? extends T> mapLeft, Function<? super R, ? extends T> mapRight) {
+        public <T> T fold(Function<? super L, ? extends T> _mapLeft, Function<? super R, ? extends T> mapRight) {
             return mapRight.apply(value());
         }
     }

conjure-generator-common/src/main/java/com/palantir/conjure/visitor/AuthTypeVisitor.java

@@ -43,31 +43,31 @@ public CookieAuthType visitCookie(CookieAuthType value) {
 
     private static class DefaultIsAuthTypeVisitor implements AuthType.Visitor<Boolean> {
         @Override
-        public Boolean visitHeader(HeaderAuthType value) {
+        public Boolean visitHeader(HeaderAuthType _value) {
             return false;
         }
 
         @Override
-        public Boolean visitCookie(CookieAuthType value) {
+        public Boolean visitCookie(CookieAuthType _value) {
             return false;
         }
 
         @Override
-        public Boolean visitUnknown(String unknownType) {
+        public Boolean visitUnknown(String _unknownType) {
             return false;
         }
     }
 
     private static class IsHeaderAuthTypeVisitor extends DefaultIsAuthTypeVisitor {
         @Override
-        public Boolean visitHeader(HeaderAuthType value) {
+        public Boolean visitHeader(HeaderAuthType _value) {
             return true;
         }
     }
 
     private static class IsCookieAuthTypeVisitor extends DefaultIsAuthTypeVisitor {
         @Override
-        public Boolean visitCookie(CookieAuthType value) {
+        public Boolean visitCookie(CookieAuthType _value) {
             return true;
         }
     }