Skip to content

Commit 7ea3958

Browse files
committed
Remove unnecessary NeverUpToDate steps
1 parent 154ca27 commit 7ea3958

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

plugin-gradle/src/test/java/com/diffplug/gradle/spotless/DiffMessageFormatterTest.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121
import java.util.Arrays;
2222
import java.util.Collections;
2323
import java.util.List;
24-
import java.util.regex.Pattern;
2524

2625
import org.assertj.core.api.Assertions;
2726
import org.gradle.api.Project;
@@ -31,11 +30,11 @@
3130

3231
import com.diffplug.common.base.StringPrinter;
3332
import com.diffplug.spotless.FileSignature;
34-
import com.diffplug.spotless.FormatterStep;
3533
import com.diffplug.spotless.LineEnding;
3634
import com.diffplug.spotless.ResourceHarness;
3735
import com.diffplug.spotless.TestProvisioner;
3836
import com.diffplug.spotless.extra.integration.DiffMessageFormatter;
37+
import com.diffplug.spotless.generic.TrimTrailingWhitespaceStep;
3938

4039
class DiffMessageFormatterTest extends ResourceHarness {
4140

@@ -151,10 +150,7 @@ void customRunToFixMessage() throws Exception {
151150
@Test
152151
void whitespaceProblem() throws Exception {
153152
Bundle spotless = create(setFile("testFile").toContent("A \nB\t\nC \n"));
154-
spotless.task.setSteps(List.of(FormatterStep.createNeverUpToDate("trimTrailing", input -> {
155-
Pattern pattern = Pattern.compile("[ \t]+$", Pattern.UNIX_LINES | Pattern.MULTILINE);
156-
return pattern.matcher(input).replaceAll("");
157-
})));
153+
spotless.task.setSteps(List.of(TrimTrailingWhitespaceStep.create()));
158154
assertCheckFailure(spotless,
159155
" testFile",
160156
" @@ -1,3 +1,3 @@",

plugin-gradle/src/test/java/com/diffplug/gradle/spotless/FormatTaskTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,10 @@
2424
import org.junit.jupiter.api.BeforeEach;
2525
import org.junit.jupiter.api.Test;
2626

27-
import com.diffplug.spotless.FormatterStep;
2827
import com.diffplug.spotless.LineEnding;
2928
import com.diffplug.spotless.ResourceHarness;
3029
import com.diffplug.spotless.TestProvisioner;
30+
import com.diffplug.spotless.generic.ReplaceStep;
3131

3232
class FormatTaskTest extends ResourceHarness {
3333
private SpotlessTaskImpl spotlessTask;
@@ -62,7 +62,7 @@ void testStep() throws Exception {
6262
File outputFile = new File(spotlessTask.getOutputDirectory(), "testFile");
6363
spotlessTask.setTarget(Collections.singleton(testFile));
6464

65-
spotlessTask.setSteps(List.of(FormatterStep.createNeverUpToDate("double-p", content -> content.replace("pp", "p"))));
65+
spotlessTask.setSteps(List.of(ReplaceStep.create("double-p", "pp", "p")));
6666
Tasks.execute(spotlessTask);
6767

6868
assertFile(outputFile).hasContent("aple");

0 commit comments

Comments
 (0)