Minor cleanups

README.md

@@ -40,8 +40,8 @@ Don't forget to add the Pitest `test` dependencies to your build file, as they a
 ```java
 repositories = List.of(MAVEN_CENTRAL);
 scope(test)
-    .include(dependency("org.pitest", "pitest", version(1, 16, 1)))
+    .include(dependency("org.pitest", "pitest", version(1, 16, 2)))
-    .include(dependency("org.pitest", "pitest-command-line", version(1, 16, 1)))
+    .include(dependency("org.pitest", "pitest-command-line", version(1, 16, 2)))
     .include(dependency("org.pitest", "pitest-junit5-plugin", version(1, 2, 1)))
     .include(dependency("org.pitest", "pitest-testng-plugin", version(1, 0, 0)));
 ```

src/main/java/rife/bld/extension/PitestOperation.java

@@ -106,8 +106,7 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
      * @see #avoidCallsTo(Collection)
      */
     public PitestOperation avoidCallsTo(String... avoidCallTo) {
-        options_.put("--avoidCallsTo", String.join(",", Arrays.stream(avoidCallTo).filter(this::isNotBlank).toList()));
+        return avoidCallsTo(List.of(avoidCallTo));
-        return this;
     }
 
     /**
@@ -292,9 +291,7 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
      * @see #excludedClasses(Collection)
      */
     public PitestOperation excludedClasses(String... excludedClass) {
-        options_.put("--excludedClasses",
+        return excludedClasses(List.of(excludedClass));
-                String.join(",", Arrays.stream(excludedClass).filter(this::isNotBlank).toList()));
-        return this;
     }
 
     /**
@@ -318,9 +315,7 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
      * @see #excludedGroups(Collection)
      */
     public PitestOperation excludedGroups(String... excludedGroup) {
-        options_.put("--excludedGroups",
+        return excludedGroups(List.of(excludedGroup));
-                String.join(",", Arrays.stream(excludedGroup).filter(this::isNotBlank).toList()));
-        return this;
     }
 
     /**
@@ -344,9 +339,7 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
      * @see #excludedMethods(Collection)
      */
     public PitestOperation excludedMethods(String... excludedMethod) {
-        options_.put("--excludedMethods",
+        return excludedMethods(List.of(excludedMethod));
-                String.join(",", Arrays.stream(excludedMethod).filter(this::isNotBlank).toList()));
-        return this;
     }
 
     /**
@@ -382,8 +375,7 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
      * @see #excludedTestClasses(Collection)
      */
     public PitestOperation excludedTestClasses(String... testClasses) {
-        options_.put("--excludedTestClasses", String.join(",", testClasses));
+        return excludedTestClasses(List.of(testClasses));
-        return this;
     }
 
     /**
@@ -509,8 +501,7 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
      * @see #features(Collection)
      */
     public PitestOperation features(String... feature) {
-        options_.put("--features", String.join(",", Arrays.stream(feature).filter(this::isNotBlank).toList()));
+        return features(List.of(feature));
-        return this;
     }
 
     /**
@@ -625,9 +616,7 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
      * @see #includedGroups(Collection)
      */
     public PitestOperation includedGroups(String... includedGroup) {
-        options_.put("--includedGroups",
+        return includedGroups(List.of(includedGroup));
-                String.join(",", Arrays.stream(includedGroup).filter(this::isNotBlank).toList()));
-        return this;
     }
 
     /**
@@ -685,8 +674,7 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
      * @see #jvmArgs(Collection)
      */
     public PitestOperation jvmArgs(String... args) {
-        options_.put("--jvmArgs", String.join(",", Arrays.stream(args).filter(this::isNotBlank).toList()));
+        return jvmArgs(List.of(args));
-        return this;
     }
 
     /**
@@ -1238,8 +1226,7 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
      * @see #targetClasses(String...)
      */
     public PitestOperation targetClasses(String... targetClass) {
-        options_.put("--targetClasses", String.join(",", Arrays.stream(targetClass).filter(this::isNotBlank).toList()));
+        return targetClasses(List.of(targetClass));
-        return this;
     }
 
     /**
@@ -1255,8 +1242,7 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
      * @see #targetTests(Collection)
      */
     public PitestOperation targetTests(String... test) {
-        options_.put("--targetTests", String.join(",", Arrays.stream(test).filter(this::isNotBlank).toList()));
+        return targetTests(List.of(test));
-        return this;
     }
 
     /**