chore: work on remove unsed imports

Author: simschla

app/src/main/java/com/diffplug/spotless/cli/steps/RemoveUnusedImports.java

@@ -15,20 +15,19 @@
  */
 package com.diffplug.spotless.cli.steps;
 
+import java.util.Collections;
+import java.util.List;
+
+import org.jetbrains.annotations.NotNull;
+
 import com.diffplug.spotless.FormatterStep;
 import com.diffplug.spotless.cli.core.SpotlessActionContext;
 import com.diffplug.spotless.cli.help.AdditionalInfoLinks;
 import com.diffplug.spotless.cli.help.OptionConstants;
 import com.diffplug.spotless.cli.help.SupportedFileTypes;
-import com.diffplug.spotless.java.CleanthatJavaStep;
 import com.diffplug.spotless.java.RemoveUnusedImportsStep;
-import org.jetbrains.annotations.NotNull;
-import picocli.CommandLine;
 
-import java.util.ArrayList;
-import java.util.Collections;
-import java.util.List;
-import java.util.Objects;
+import picocli.CommandLine;
 
 @CommandLine.Command(name = "remove-unused-imports", description = "Removes unused imports from Java files.")
 @SupportedFileTypes("Java")
@@ -38,16 +37,15 @@ public class RemoveUnusedImports extends SpotlessFormatterStep {
     @CommandLine.Option(
             names = {"--engine", "-e"},
             defaultValue = "GOOGLE_JAVA_FORMAT",
-            description =
-                    "The backing engine to use for detecting and removing unused imports." + OptionConstants.VALID_AND_DEFAULT_VALUES_SUFFIX)
+            description = "The backing engine to use for detecting and removing unused imports."
+                    + OptionConstants.VALID_AND_DEFAULT_VALUES_SUFFIX)
     Engine engine;
 
     public enum Engine {
         GOOGLE_JAVA_FORMAT {
             @Override
             String formatterName() {
                 return RemoveUnusedImportsStep.defaultFormatter();
-
             }
         },
         CLEAN_THAT {
@@ -57,7 +55,6 @@ String formatterName() {
             }
         };
 
-
         abstract String formatterName();
     }
 

app/src/test/java/com/diffplug/spotless/cli/steps/RemoveUnusedImportsTest.java

@@ -1,9 +1,25 @@
+/*
+ * Copyright 2025 DiffPlug
+ *
+ * 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
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 package com.diffplug.spotless.cli.steps;
 
+import org.junit.jupiter.api.Test;
+
 import com.diffplug.spotless.cli.CLIIntegrationHarness;
 import com.diffplug.spotless.tag.CliNativeTest;
 import com.diffplug.spotless.tag.CliProcessTest;
-import org.junit.jupiter.api.Test;
 
 import static org.junit.jupiter.api.Assertions.*;
 
@@ -15,16 +31,12 @@ class RemoveUnusedImportsTest extends CLIIntegrationHarness {
     void itRemovesUnusedImportsWithDefaultEngine() {
         setFile("Java.java").toResource("java/removeunusedimports/JavaCodeWithLicensePackageUnformatted.test");
 
-        cliRunner()
-                .withTargets("Java.java")
-                .withStep(RemoveUnusedImports.class)
-                .run();
+        cliRunner().withTargets("Java.java").withStep(RemoveUnusedImports.class).run();
 
         assertFile("Java.java")
                 .notSameSasResource("java/removeunusedimports/JavaCodeWithLicensePackageUnformatted.test")
                 .hasNotContent("Unused");
 
-
         selfie().expectResource("Java.java").toMatchDisk();
     }