Added step to remove the semicolons in groovy files

CHANGES.md

@@ -14,6 +14,7 @@ We adhere to the [keepachangelog](https://keepachangelog.com/en/1.0.0/) format (
 * Fix Eclipse JDT on some settings files. ([#1864](https://github.com/diffplug/spotless/pull/1864) fixes [#1638](https://github.com/diffplug/spotless/issues/1638))
 ### Changes
 * Bump default `ktlint` version to latest `1.0.0` -> `1.0.1`. ([#1855](https://github.com/diffplug/spotless/pull/1855))
+* Add Step to remove semicolons from Groovy code. ([#1881])(https://github.com/diffplug/spotless/pull/1881)
 
 ## [2.42.0] - 2023-09-28
 ### Added

lib/src/main/java/com/diffplug/spotless/groovy/RemoveSemiColonsStep.java

@@ -0,0 +1,63 @@
+package com.diffplug.spotless.groovy;
+
+import com.diffplug.spotless.FormatterFunc;
+import com.diffplug.spotless.FormatterStep;
+
+import java.io.BufferedReader;
+import java.io.Serializable;
+import java.io.StringReader;
+
+/**
+ * Removes all semicolons from the end of lines.
+ *
+ * @author Jose Luis Badano
+ */
+public final class RemoveSemiColonsStep {
+
+	private RemoveSemiColonsStep() {
+		// prevent instantiation
+	}
+
+	static final String NAME = "Remove not needed semi colons";
+
+	public static FormatterStep create() {
+		return FormatterStep.createLazy(NAME,
+			State::new,
+			RemoveSemiColonsStep.State::toFormatter);
+	}
+
+
+	private static final class State implements Serializable {
+		private static final long serialVersionUID = 1L;
+
+		FormatterFunc toFormatter() {
+			return raw -> {
+				try (BufferedReader reader = new BufferedReader(new StringReader(raw))) {
+					StringBuilder result = new StringBuilder();
+					String line;
+					while ((line = reader.readLine()) != null) {
+						result.append(removeSemiColon(line));
+						result.append(System.lineSeparator());
+					}
+					return result.toString();
+				}
+			};
+		}
+
+		/**
+		 * Removes the last semicolon in a line if it exists.
+		 *
+		 * @param line the line to remove the semicolon from
+		 * @return the line without the last semicolon
+		 */
+		private String removeSemiColon(String line) {
+			// find last semicolon in a string a remove it
+			int lastSemiColon = line.lastIndexOf(";");
+			if (lastSemiColon != -1 && lastSemiColon == line.length() - 1) {
+				return line.substring(0, lastSemiColon);
+			} else {
+				return line;
+			}
+		}
+	}
+}

plugin-maven/src/main/java/com/diffplug/spotless/maven/groovy/Groovy.java

@@ -52,4 +52,8 @@ public void addGreclipse(GrEclipse greclipse) {
 	public void addImportOrder(ImportOrder importOrder) {
 		addStepFactory(importOrder);
 	}
+
+	public void addRemoveSemiColons(RemoveSemiColons removeSemiColons) {
+		addStepFactory(removeSemiColons);
+	}
 }

plugin-maven/src/main/java/com/diffplug/spotless/maven/groovy/RemoveSemiColons.java

@@ -0,0 +1,32 @@
+/*
+ * Copyright 2016-2023 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.maven.groovy;
+
+
+import com.diffplug.spotless.groovy.RemoveSemiColonsStep;
+
+
+import com.diffplug.spotless.FormatterStep;
+import com.diffplug.spotless.maven.FormatterStepConfig;
+import com.diffplug.spotless.maven.FormatterStepFactory;
+
+public class RemoveSemiColons implements FormatterStepFactory {
+
+	@Override
+	public FormatterStep newFormatterStep(FormatterStepConfig config) {
+		return RemoveSemiColonsStep.create();
+	}
+}

plugin-maven/src/test/java/com/diffplug/spotless/maven/groovy/RemoveSemiColonsTest.java

@@ -0,0 +1,38 @@
+package com.diffplug.spotless.maven.groovy;
+
+import com.diffplug.spotless.maven.MavenIntegrationHarness;
+
+import org.junit.jupiter.api.Test;
+
+class RemoveSemiColonsTest extends MavenIntegrationHarness {
+
+	@Test
+	void testRemoveSemiColonsString() throws Exception {
+		writePomWithGroovySteps("<removeSemiColons/>");
+		runTest("Hello World;", "Hello World");
+	}
+
+	@Test
+	void testNotRemoveSemiColonsString() throws Exception {
+		writePomWithGroovySteps("<removeSemiColons/>");
+		runTest("Hello;World", "Hello;World");
+	}
+
+	@Test
+	void testRemoveSemiColons() throws Exception {
+		writePomWithGroovySteps("<removeSemiColons/>");
+
+		String path = "src/main/groovy/test.groovy";
+		setFile(path).toResource("groovy/removesemicolons/GroovyCodeWithSemiColons.test");
+		mavenRunner().withArguments("spotless:apply").runNoError();
+		assertFile(path).sameAsResource("groovy/removesemicolons/GroovyCodeWithSemiColonsFormatted.test");
+
+	}
+
+	private void runTest(String sourceContent, String targetContent) throws Exception {
+		String path = "src/main/groovy/test.groovy";
+		setFile(path).toContent(sourceContent);
+		mavenRunner().withArguments("spotless:apply").runNoError();
+		assertFile(path).hasContent(targetContent);
+	}
+}

testlib/src/main/resources/groovy/removesemicolons/GroovyCodeWithSemiColons.test

@@ -0,0 +1,10 @@
+import mylib.Unused;
+import mylib.UsedB;
+import mylib.UsedA;
+
+public class SomeClass {
+System.out.println("hello");
+UsedB.someMethod();
+UsedA.someMethod();
+}
+}

testlib/src/main/resources/groovy/removesemicolons/GroovyCodeWithSemiColonsFormatted.test

@@ -0,0 +1,10 @@
+import mylib.Unused
+import mylib.UsedB
+import mylib.UsedA
+
+public class SomeClass {
+System.out.println("hello")
+UsedB.someMethod()
+UsedA.someMethod()
+}
+}