Adding dependency guards to the Junit 4 -> 5 migration for when the POM or build.gradle have a dependency on org.testng:testng. Relies on openrewrite/rewrite#5725

Author: steve-aom-elliott

src/main/resources/META-INF/rewrite/junit5.yml

@@ -57,6 +57,12 @@ preconditions:
   - org.openrewrite.java.search.DoesNotUseType:
       fullyQualifiedTypeName: org.testng..*
       includeImplicit: true
+  - org.openrewrite.gradle.search.DoesNotIncludeDependency:
+      groupId: org.testng
+      artifactId: testng
+  - org.openrewrite.maven.search.DoesNotIncludeDependency:
+      groupId: org.testng
+      artifactId: testng
 recipeList:
   - org.openrewrite.java.testing.junit5.UseWiremockExtension
   - org.openrewrite.java.testing.junit5.IgnoreToDisabled

src/test/java/org/openrewrite/java/testing/junit5/JUnit5MigrationTest.java

@@ -40,7 +40,7 @@ class JUnit5MigrationTest implements RewriteTest {
     public void defaults(RecipeSpec spec) {
         spec
           .parser(JavaParser.fromJavaVersion()
-            .classpathFromResources(new InMemoryExecutionContext(), "junit-4"))
+            .classpathFromResources(new InMemoryExecutionContext(), "junit-4", "testng"))
           .recipe(Environment.builder()
             .scanRuntimeClasspath("org.openrewrite.java.testing.junit5")
             .build()
@@ -51,11 +51,11 @@ public void defaults(RecipeSpec spec) {
     @Issue("https://github.com/openrewrite/rewrite-testing-frameworks/issues/145")
     @Test
     void assertThatReceiver() {
-        //language=java
         rewriteRun(
           spec -> spec
             .parser(JavaParser.fromJavaVersion()
               .classpathFromResources(new InMemoryExecutionContext(), "junit-4", "hamcrest-3")),
+          //language=java
           java(
             """
               import org.junit.Assert;
@@ -477,6 +477,68 @@ void noJunitDependencyIfApiAlreadyPresent() {
         );
     }
 
+    @Test
+    void noChangesIfTestNgIncluded() {
+        rewriteRun(
+          spec -> spec.beforeRecipe(withToolingApi()),
+          //language=groovy
+          buildGradle(
+            """
+              plugins {
+                  id 'java-library'
+              }
+              repositories {
+                  mavenCentral()
+              }
+              dependencies {
+                  testImplementation 'junit:junit:4.12'
+                  testImplementation 'org.testng:testng:7.8.0'
+              }
+              tasks.withType(Test).configureEach {
+                  useJUnitPlatform()
+              }
+              """
+          ),
+          //language=xml
+          pomXml(
+            """
+              <project>
+                  <modelVersion>4.0.0</modelVersion>
+                  <groupId>dev.ted</groupId>
+                  <artifactId>testcontainer-migrate</artifactId>
+                  <version>0.0.1</version>
+                  <dependencies>
+                      <dependency>
+                          <groupId>junit</groupId>
+                          <artifactId>junit</artifactId>
+                          <version>4.12</version>
+                          <scope>test</scope>
+                      </dependency>
+                      <dependency>
+                          <groupId>org.testng</groupId>
+                          <artifactId>testng</artifactId>
+                          <version>7.8.0</version>
+                      </dependency>
+                  </dependencies>
+              </project>
+              """
+          ),
+          //language=java
+          java(
+            """
+              import org.junit.Ignore;
+              import org.testng.annotations.Test;
+
+              class ExampleClass {
+                  @Ignore
+                  @Test
+                  public void testMethod() {}
+              }
+              """
+          )
+        );
+    }
+
     @Test
     void bumpSurefireOnOlderMavenVersions() {
         rewriteRun(