Merge pull request #820 from solven-eu/cleanthat/headfor-refs_heads_master-2024-07-19

Cleanthat

Author: blacelle

CHANGES.MD

@@ -19,7 +19,6 @@ We adhere to the [keepachangelog](https://keepachangelog.com/en/1.0.0/) format
 
 * `ModifierOrder` shall not prevent further modifiers mutators to operate correctly.
 
-
 ## [2.19]
 
 ### Changes

java/src/main/java/eu/solven/cleanthat/engine/java/refactorer/mutators/UseTextBlocks.java

@@ -80,7 +80,7 @@ protected boolean processNotRecursively(NodeAndSymbolSolver<?> node) {
 			return false;
 		}
 
-		String concat = listString.stream().collect(Collectors.joining());
+		var concat = listString.stream().collect(Collectors.joining());
 
 		// TextBlocks are '\n'-based
 		List<String> rows = Arrays.asList(concat.split("\n"));

java/src/test/java/eu/solven/cleanthat/engine/java/refactorer/javaparser/TestMutatorOnFiles.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2023 Benoit Lacelle - SOLVEN
+ * Copyright 2023-2024 Benoit Lacelle - SOLVEN
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -154,7 +154,7 @@ private void testSourceWithMutator(IWalkingMutator<Node, Node> mutator, String a
 	public void testWithDedicatedMutator() throws IOException {
 		LOGGER.info("Processing: {}", resource);
 
-		if (resource.getFilename().equals("Issue807.java")) {
+		if ("Issue807.java".equals(resource.getFilename())) {
 			LOGGER.warn("We skip {} because JavaParser produces an invalid Java file", resource.getFilename());
 			return;
 		}
@@ -168,7 +168,7 @@ public void testWithDedicatedMutator() throws IOException {
 	public void testWithAllMutators() throws IOException {
 		LOGGER.info("Processing: {}", resource);
 
-		if (resource.getFilename().equals("Issue807.java")) {
+		if ("Issue807.java".equals(resource.getFilename())) {
 			LOGGER.warn("We skip {} because JavaParser produces an invalid Java file", resource.getFilename());
 			return;
 		}