Remove some now obsolete code

Author: shanman190

rewrite-gradle/src/main/java/org/openrewrite/gradle/AddDependencyVisitor.java

@@ -73,9 +73,6 @@ public class AddDependencyVisitor extends JavaIsoVisitor<ExecutionContext> {
     public @Nullable J visit(@Nullable Tree tree, ExecutionContext ctx) {
         if (tree instanceof JavaSourceFile) {
             JavaSourceFile sourceFile = (JavaSourceFile) tree;
-            if (sourceFile == null) {
-                return null;
-            }
             gradleProject = sourceFile.getMarkers().findFirst(GradleProject.class).orElse(null);
             if (gradleProject == null) {
                 return sourceFile;
@@ -99,12 +96,6 @@ public class AddDependencyVisitor extends JavaIsoVisitor<ExecutionContext> {
                     }
                 }
             }
-        }
-
-        J j = super.visit(tree, ctx);
-
-        if (j instanceof JavaSourceFile) {
-            JavaSourceFile sourceFile = (JavaSourceFile) j;
 
             sourceFile = (JavaSourceFile) new org.openrewrite.gradle.internal.AddDependencyVisitor(configuration, groupId, artifactId, resolvedVersion, classifier, extension, insertPredicate, dependencyModifier, isKotlinDsl)
                     .visitNonNull(sourceFile, ctx);
@@ -121,7 +112,7 @@ public class AddDependencyVisitor extends JavaIsoVisitor<ExecutionContext> {
 
             return sourceFile;
         }
-        return j;
+        return (J) tree;
     }
 
     public enum DependencyModifier {

rewrite-gradle/src/main/java/org/openrewrite/gradle/search/FindJVMTestSuites.java

@@ -38,11 +38,6 @@ public class FindJVMTestSuites extends Recipe {
 
     transient JVMTestSuitesDefined jvmTestSuitesDefined = new JVMTestSuitesDefined(this);
 
-    @Option(displayName = "Requires dependencies",
-            description = "Whether the test suite configuration defines dependencies to be resolved. Defaults to false.")
-    @Nullable
-    Boolean definesDependencies;
-
     @Option(displayName = "Insert rows",
             description = "Whether to insert rows into the table. Defaults to true.")
     @Nullable
@@ -60,7 +55,6 @@ public String getDescription() {
 
     @Override
     public TreeVisitor<?, ExecutionContext> getVisitor() {
-        boolean requireDependencies = definesDependencies != null && definesDependencies;
         boolean tableAvailable = this.insertRows == null || this.insertRows;
         return Preconditions.check(new IsBuildGradle<>(), jvmTestSuite().asVisitor((suite, ctx) -> {
             if (tableAvailable) {
@@ -70,8 +64,8 @@ public TreeVisitor<?, ExecutionContext> getVisitor() {
         }));
     }
 
-    public static Set<String> jvmTestSuiteNames(Tree tree, boolean definesDependencies) {
-        return TreeVisitor.collect(new FindJVMTestSuites(definesDependencies, false).getVisitor(), tree, new HashSet<>())
+    public static Set<String> jvmTestSuiteNames(Tree tree) {
+        return TreeVisitor.collect(new FindJVMTestSuites(false).getVisitor(), tree, new HashSet<>())
                 .stream()
                 .filter(J.MethodInvocation.class::isInstance)
                 .map(J.MethodInvocation.class::cast)

rewrite-gradle/src/test/java/org/openrewrite/gradle/search/FindJVMTestSuitesTest.java

@@ -54,14 +54,6 @@ void configFalse() {
               )
             );
         }
-
-        @Test
-        void configTrue() {
-            rewriteRun(
-              spec -> spec.recipe(new FindJVMTestSuites(true, null)),
-              buildGradle(withoutDependency)
-            );
-        }
     }
 
     @Nested