From eef329046e453482e3d19fd556d56ec398bfd341 Mon Sep 17 00:00:00 2001
From: Marvin <velo@users.noreply.github.com>
Date: Wed, 31 Jul 2024 13:57:25 +0000
Subject: [PATCH] fake (#2479)

* fake

* Ignore toolchains plugin when running openrewrite

Signed-off-by: Marvin Froeder <velo.br@gmail.com>

---------

Signed-off-by: Marvin Froeder <velo.br@gmail.com>
---
 .github/workflows/receive-pr.yml       |   2 +-
 core/src/test/java/feign/UtilTest.java |  13 ++-
 pom.xml                                | 154 ++++++++++++++++---------
 3 files changed, 111 insertions(+), 58 deletions(-)

diff --git a/.github/workflows/receive-pr.yml b/.github/workflows/receive-pr.yml
index eeb201ebd..5a709c329 100644
--- a/.github/workflows/receive-pr.yml
+++ b/.github/workflows/receive-pr.yml
@@ -42,7 +42,7 @@ jobs:
 
       # Execute recipes
       - name: Apply OpenRewrite recipes
-        run: mvn --activate-profiles openrewrite org.openrewrite.maven:rewrite-maven-plugin:run
+        run: ./mvnw -Dtoolchain.skip=true -Dlicense.skip=true -DskipTests=true -P openrewrite clean install
 
       # Capture the diff
       - name: Create patch
diff --git a/core/src/test/java/feign/UtilTest.java b/core/src/test/java/feign/UtilTest.java
index cfaeee6f6..7573f8d5a 100644
--- a/core/src/test/java/feign/UtilTest.java
+++ b/core/src/test/java/feign/UtilTest.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2012-2023 The Feign Authors
+ * Copyright 2012-2024 The Feign Authors
  *
  * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except
  * in compliance with the License. You may obtain a copy of the License at
@@ -20,6 +20,7 @@
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 import static org.assertj.core.api.AssertionsForClassTypes.assertThatExceptionOfType;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 import java.io.Reader;
 import java.lang.reflect.Type;
 import java.util.Arrays;
@@ -86,6 +87,16 @@ void resolveLastTypeParameterWhenNotSubtype() throws Exception {
     assertThat(last).isEqualTo(listStringType);
   }
 
+
+  @Test
+  void fake() throws Exception {
+    Type context =
+        LastTypeParameter.class.getDeclaredField("PARAMETERIZED_LIST_STRING").getGenericType();
+    Type listStringType = LastTypeParameter.class.getDeclaredField("LIST_STRING").getGenericType();
+    Type last = resolveLastTypeParameter(context, Parameterized.class);
+    assertEquals(last, listStringType);
+  }
+
   @Test
   void lastTypeFromInstance() throws Exception {
     Parameterized<?> instance = new ParameterizedSubtype();
diff --git a/pom.xml b/pom.xml
index 8f8dee75e..f4ec318ff 100644
--- a/pom.xml
+++ b/pom.xml
@@ -479,9 +479,6 @@
             <redirectTestOutputToFile>true</redirectTestOutputToFile>
             <trimStackTrace>false</trimStackTrace>
             <argLine>${jvm.options}</argLine>
-            <jdkToolchain>
-              <version>${latest.java.version}</version>
-            </jdkToolchain>
           </configuration>
         </plugin>
 
@@ -492,49 +489,12 @@
           <configuration>
             <redirectTestOutputToFile>true</redirectTestOutputToFile>
             <trimStackTrace>false</trimStackTrace>
-            <jdkToolchain>
-              <version>${latest.java.version}</version>
-            </jdkToolchain>
           </configuration>
         </plugin>
       </plugins>
     </pluginManagement>
 
     <plugins>
-      <plugin>
-        <groupId>org.apache.maven.plugins</groupId>
-        <artifactId>maven-toolchains-plugin</artifactId>
-        <version>3.2.0</version>
-        <executions>
-          <execution>
-            <goals>
-              <goal>toolchain</goal>
-            </goals>
-            <configuration>
-              <toolchains>
-                <jdk>
-                  <version>${main.java.version}</version>
-                </jdk>
-              </toolchains>
-            </configuration>
-          </execution>
-          <execution>
-            <id>test</id>
-            <phase>generate-test-sources</phase>
-            <goals>
-              <goal>toolchain</goal>
-            </goals>
-            <configuration>
-              <toolchains>
-                <jdk>
-                  <version>${latest.java.version}</version>
-                </jdk>
-              </toolchains>
-            </configuration>
-          </execution>
-        </executions>
-      </plugin>
-
       <plugin>
         <groupId>org.moditect</groupId>
         <artifactId>moditect-maven-plugin</artifactId>
@@ -667,6 +627,9 @@
                 <exclude>etc/header.txt</exclude>
                 <exclude>**/.idea/**</exclude>
                 <exclude>**/target/**</exclude>
+                <exclude>**/scripts/**</exclude>
+                <exclude>**/src/config/**</exclude>
+                <exclude>**/codequality/**</exclude>
                 <exclude>LICENSE</exclude>
                 <exclude>NOTICE</exclude>
                 <exclude>OSSMETADATA</exclude>
@@ -961,43 +924,122 @@
           <plugin>
             <groupId>org.openrewrite.maven</groupId>
             <artifactId>rewrite-maven-plugin</artifactId>
-            <version>5.35.0</version>
-
-            <configuration>
-              <activeRecipes>
-                <recipe>org.openrewrite.java.testing.hamcrest.MigrateHamcrestToAssertJ</recipe>
-                <recipe>org.openrewrite.java.testing.assertj.JUnitToAssertj</recipe>
-                <recipe>org.openrewrite.java.testing.assertj.Assertj</recipe>
-                <recipe>org.openrewrite.java.migrate.UpgradeToJava21</recipe>
-              </activeRecipes>
-              <exclusions>
-                <exclusion>**/src/main/java/**</exclusion>
-                <exclusion>**/pom.xml</exclusion>
-              </exclusions>
-            </configuration>
+            <version>5.36.0</version>
 
             <dependencies>
               <dependency>
                 <groupId>org.openrewrite.recipe</groupId>
                 <artifactId>rewrite-testing-frameworks</artifactId>
-                <version>2.13.0</version>
+                <version>2.14.0</version>
               </dependency>
               <dependency>
                 <groupId>org.openrewrite.recipe</groupId>
                 <artifactId>rewrite-migrate-java</artifactId>
-                <version>2.19.0</version>
+                <version>2.20.0</version>
               </dependency>
             </dependencies>
 
             <executions>
               <execution>
+                <id>tests</id>
                 <goals>
                   <goal>runNoFork</goal>
                 </goals>
                 <phase>verify</phase>
+                <configuration>
+                  <exportDatatables>true</exportDatatables>
+                  <activeRecipes>
+                    <recipe>org.openrewrite.java.testing.hamcrest.MigrateHamcrestToAssertJ</recipe>
+                    <recipe>org.openrewrite.java.testing.assertj.JUnitToAssertj</recipe>
+                    <recipe>org.openrewrite.java.testing.assertj.Assertj</recipe>
+                    <recipe>org.openrewrite.java.migrate.UpgradeToJava21</recipe>
+                  </activeRecipes>
+                  <exclusions>
+                    <exclusion>**/src/main/java/**</exclusion>
+                    <exclusion>pom.xml</exclusion>
+                  </exclusions>
+                </configuration>
+              </execution>
+              <execution>
+                <id>sources</id>
+                <goals>
+                  <goal>runNoFork</goal>
+                </goals>
+                <phase>verify</phase>
+                <configuration>
+                  <exportDatatables>true</exportDatatables>
+                  <activeRecipes>
+                    <recipe>org.openrewrite.java.migrate.UpgradeToJava8</recipe>
+                  </activeRecipes>
+                </configuration>
+              </execution>
+            </executions>
+          </plugin>
+        </plugins>
+      </build>
+    </profile>
+
+    <profile>
+      <id>toolchain</id>
+      <activation>
+        <property>
+          <name>!toolchain.skip</name>
+        </property>
+      </activation>
+
+      <build>
+        <plugins>
+          <plugin>
+            <groupId>org.apache.maven.plugins</groupId>
+            <artifactId>maven-toolchains-plugin</artifactId>
+            <version>3.2.0</version>
+            <executions>
+              <execution>
+                <goals>
+                  <goal>toolchain</goal>
+                </goals>
+                <configuration>
+                  <toolchains>
+                    <jdk>
+                      <version>${main.java.version}</version>
+                    </jdk>
+                  </toolchains>
+                </configuration>
+              </execution>
+              <execution>
+                <id>test</id>
+                <goals>
+                  <goal>toolchain</goal>
+                </goals>
+                <phase>generate-test-sources</phase>
+                <configuration>
+                  <toolchains>
+                    <jdk>
+                      <version>${latest.java.version}</version>
+                    </jdk>
+                  </toolchains>
+                </configuration>
               </execution>
             </executions>
           </plugin>
+          <plugin>
+            <groupId>org.apache.maven.plugins</groupId>
+            <artifactId>maven-surefire-plugin</artifactId>
+            <configuration>
+              <jdkToolchain>
+                <version>${latest.java.version}</version>
+              </jdkToolchain>
+            </configuration>
+          </plugin>
+          <plugin>
+            <groupId>org.apache.maven.plugins</groupId>
+            <artifactId>maven-failsafe-plugin</artifactId>
+            <configuration>
+              <jdkToolchain>
+                <version>${latest.java.version}</version>
+              </jdkToolchain>
+            </configuration>
+          </plugin>
         </plugins>
       </build>
     </profile>