From 89f578938b7b900e35812df5b5b0167e102bf26b Mon Sep 17 00:00:00 2001
From: Niels de Bruin <nielsdebruin@gmail.com>
Date: Tue, 31 Dec 2024 14:11:29 +0100
Subject: [PATCH] `ParameterizedLogging` support for logger fields added
 through Lombok annotations (#205)

* Add test for lombok generated logger

* Update lombok initialization

* Drop missing marker reference

* Only run select tests with Lombok support enabled

---------

Co-authored-by: Tim te Beek <tim@moderne.io>
---
 .../logging/ParameterizedLoggingTest.java     | 49 ++++++++++++++++++-
 1 file changed, 48 insertions(+), 1 deletion(-)

diff --git a/src/test/java/org/openrewrite/java/logging/ParameterizedLoggingTest.java b/src/test/java/org/openrewrite/java/logging/ParameterizedLoggingTest.java
index 8bfc38b..ba9299d 100644
--- a/src/test/java/org/openrewrite/java/logging/ParameterizedLoggingTest.java
+++ b/src/test/java/org/openrewrite/java/logging/ParameterizedLoggingTest.java
@@ -15,6 +15,9 @@
  */
 package org.openrewrite.java.logging;
 
+import org.junit.jupiter.api.AfterAll;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Nested;
 import org.junit.jupiter.api.Test;
 import org.openrewrite.DocumentExample;
 import org.openrewrite.InMemoryExecutionContext;
@@ -37,7 +40,7 @@ class ParameterizedLoggingTest implements RewriteTest {
     @Override
     public void defaults(RecipeSpec spec) {
         spec.parser(JavaParser.fromJavaVersion()
-          .classpathFromResources(new InMemoryExecutionContext(), "slf4j-api-2.1", "log4j-api-2.23", "log4j-core-2.23"));
+          .classpathFromResources(new InMemoryExecutionContext(), "slf4j-api-2.1", "log4j-api-2.23", "log4j-core-2.23", "lombok"));
     }
 
     @DocumentExample
@@ -697,4 +700,48 @@ void method(int cnt, String name) {
         );
     }
 
+    @Nested
+    class LombokSupport {
+
+        @BeforeAll
+        static void setUp() {
+            System.setProperty("rewrite.lombok", "true");
+        }
+
+        @AfterAll
+        static void tearDown() {
+            System.clearProperty("rewrite.lombok");
+        }
+
+        @Test
+        void lombokLoggingAnnotation() {
+            rewriteRun(
+              spec -> spec.recipe(new ParameterizedLogging("org.slf4j.Logger info(..)", false)),
+              // language=java
+              java(
+                """
+                  import lombok.extern.slf4j.Slf4j;
+
+                  @Slf4j
+                  class Test {
+                      static void method(String name) {
+                          log.info("Hello " + name + ", nice to meet you " + name);
+                      }
+                  }
+                  """,
+                """
+                  import lombok.extern.slf4j.Slf4j;
+
+                  @Slf4j
+                  class Test {
+                      static void method(String name) {
+                          log.info("Hello {}, nice to meet you {}", name, name);
+                      }
+                  }
+                  """
+              )
+            );
+        }
+
+    }
 }