StrictUnusedVariable handles Java 14 records

baseline-error-prone/src/main/java/com/palantir/baseline/errorprone/StrictUnusedVariable.java

@@ -768,6 +768,14 @@ public Void visitLambdaExpression(LambdaExpressionTree node, Void unused) {
 
         @Override
         public Void visitMethod(MethodTree tree, Void unused) {
+            // From the perspective of an errorprone rule there are two standalone trees for a single `record`
+            // definition; A MethodTree which looks like a void function and a ClassTree which has the record fields.
+            //
+            // Its unclear why both trees are emitted, but we can identify and ignore a record's MethodTree by checking
+            // if it does not have any associated source.
+            if (state.getEndPosition(tree) < 0) {
+                return null;
+            }
             return isSuppressed(tree) ? null : super.visitMethod(tree, unused);
         }
     }

baseline-error-prone/src/test/java/com/palantir/baseline/errorprone/StrictUnusedVariableTest.java

@@ -16,10 +16,13 @@
 
 package com.palantir.baseline.errorprone;
 
+import com.google.common.collect.ImmutableList;
 import com.google.errorprone.BugCheckerRefactoringTestHelper.TestMode;
 import com.google.errorprone.CompilationTestHelper;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.condition.DisabledForJreRange;
+import org.junit.jupiter.api.condition.JRE;
 
 public class StrictUnusedVariableTest {
 
@@ -295,6 +298,22 @@ public void fixes_previously_suppressed_variables() {
                 .doTest(TestMode.TEXT_MATCH);
     }
 
+    @Test
+    @DisabledForJreRange(max = JRE.JAVA_13)
+    public void testRecord() {
+        compilationHelper = CompilationTestHelper.newInstance(StrictUnusedVariable.class, getClass())
+                .setArgs(ImmutableList.of("--enable-preview", "--release", "14"));
+
+        compilationHelper
+                .addSourceLines(
+                        "Test.java",
+                        "class Test {",
+                        "@SuppressWarnings(\"StrictUnusedVariable\")",
+                        " record Foo(int foo) {}",
+                        "}")
+                .doTest();
+    }
+
     @Test
     public void testSuppression() {
         refactoringTestHelper