Cleaned up API to match bld operations aand options APIs

.idea/bld.xml

@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<project version="4">
+  <component name="BldConfiguration">
+    <events />
+  </component>
+</project>

examples/lib/bld/bld-wrapper.properties

@@ -1,7 +1,7 @@
 bld.downloadExtensionJavadoc=false
 bld.downloadExtensionSources=true
 bld.downloadLocation=
-bld.extension-pitest=com.uwyn.rife2:bld-pitest:0.9.9
+bld.extension-pitest=com.uwyn.rife2:bld-pitest:1.0.0-SNAPSHOT
 bld.repositories=MAVEN_CENTRAL,RIFE2_RELEASES,MAVEN_LOCAL,RIFE2_SNAPSHOTS
 bld.sourceDirectories=
 bld.version=2.0.1

src/bld/java/rife/bld/extension/PitestOperationBuild.java

@@ -33,7 +33,7 @@ public class PitestOperationBuild extends Project {
     public PitestOperationBuild() {
         pkg = "rife.bld.extension";
         name = "PitestExtension";
-        version = version(0, 9, 9);
+        version = version(1, 0, 0, "SNAPSHOT");
 
         javaRelease = 17;
 

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

@@ -22,6 +22,7 @@ import rife.bld.operations.exceptions.ExitStatusException;
 
 import java.io.File;
 import java.io.IOException;
+import java.nio.file.Path;
 import java.util.*;
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.logging.Level;
@@ -60,6 +61,7 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
         return this;
     }
 
+
     /**
      * List of packages and classes which are to be considered outside the scope of mutation. Any lines of code
      * containing calls to these classes will not be mutated.
@@ -129,8 +131,31 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
      * @see #classPath(Collection)
      */
     public PitestOperation classPath(String... path) {
-        options_.put("--classPath", String.join(",", Arrays.stream(path).filter(this::isNotBlank).toList()));
+        return classPath(List.of(path));
-        return this;
+    }
+
+    /**
+     * List of packages and classes which are to be considered outside the scope of mutation. Any lines of code
+     * containing calls to these classes will not be mutated.
+     * <p>
+     * If a list is not explicitly supplied then PIT will default to a list of common logging packages as follows
+     * <p>
+     * <ul>
+     * <li>java.util.logging</li>
+     * <li>org.apache.log4j</li>
+     * <li>org.slf4j</li>
+     * <li>org.apache.commons.logging</li>
+     * </ul>
+     * <p>
+     * If the feature {@code FLOGCALL} is disabled, this parameter is ignored and logging calls are also mutated.
+     * Additional classpath entries to use when looking for tests and mutable code.
+     *
+     * @param path one or more paths
+     * @return this operation instance
+     * @see #classPathPaths(Collection)
+     */
+    public PitestOperation classPath(Path... path) {
+        return classPathPaths(List.of(path));
     }
 
     /**
@@ -145,6 +170,30 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
         return this;
     }
 
+    /**
+     * List of packages and classes which are to be considered outside the scope of mutation. Any lines of code
+     * containing calls to these classes will not be mutated.
+     * <p>
+     * If a list is not explicitly supplied then PIT will default to a list of common logging packages as follows
+     * <p>
+     * <ul>
+     * <li>java.util.logging</li>
+     * <li>org.apache.log4j</li>
+     * <li>org.slf4j</li>
+     * <li>org.apache.commons.logging</li>
+     * </ul>
+     * <p>
+     * If the feature {@code FLOGCALL} is disabled, this parameter is ignored and logging calls are also mutated.
+     * Additional classpath entries to use when looking for tests and mutable code.
+     *
+     * @param path one or more paths
+     * @return this operation instance
+     * @see #classPathFiles(Collection)
+     */
+    public PitestOperation classPath(File... path) {
+        return classPathFiles(List.of(path));
+    }
+
     /**
      * File with a list of additional classpath elements (one per line).
      *
@@ -158,6 +207,28 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
         return this;
     }
 
+    /**
+     * Additional classpath entries to use when looking for tests and mutable code.
+     *
+     * @param path the list of paths
+     * @return this operation instance
+     * @see #classPath(File...)
+     */
+    public PitestOperation classPathFiles(Collection<File> path) {
+        return classPath(path.stream().map(File::getAbsolutePath).toList());
+    }
+
+    /**
+     * Additional classpath entries to use when looking for tests and mutable code.
+     *
+     * @param path the list of paths
+     * @return this operation instance
+     * @see #classPath(Path...)
+     */
+    public PitestOperation classPathPaths(Collection<Path> path) {
+        return classPath(path.stream().map(Path::toFile).map(File::getAbsolutePath).toList());
+    }
+
     /**
      * Line coverage threshold below which the build will fail. This is an integer percent (0-100) that represents the
      * fraction of the project covered by the tests.
@@ -470,6 +541,26 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
         return this;
     }
 
+    /**
+     * Path to a file containing history information for incremental analysis.
+     *
+     * @param path the path
+     * @return this operation instance
+     */
+    public PitestOperation historyInputLocation(File path) {
+        return historyInputLocation(path.getAbsolutePath());
+    }
+
+    /**
+     * Path to a file containing history information for incremental analysis.
+     *
+     * @param path the path
+     * @return this operation instance
+     */
+    public PitestOperation historyInputLocation(Path path) {
+        return historyInputLocation(path.toFile());
+    }
+
     /**
      * Path to write history information for incremental analysis. May be the same as
      * {@link #historyInputLocation(String)
@@ -484,6 +575,28 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
         return this;
     }
 
+    /**
+     * Path to write history information for incremental analysis. May be the same as
+     * {@link #historyInputLocation(String)
+     *
+     * @param path the path
+     * @return this operation instance
+     */
+    public PitestOperation historyOutputLocation(File path) {
+        return historyOutputLocation(path.getAbsolutePath());
+    }
+
+    /**
+     * Path to write history information for incremental analysis. May be the same as
+     * {@link #historyInputLocation(String)
+     *
+     * @param path the path
+     * @return this operation instance
+     */
+    public PitestOperation historyOutputLocation(Path path) {
+        return historyOutputLocation(path.toFile());
+    }
+
     /**
      * Indicates if the PIT should try to mutate classes on the classpath with which it was launched. If not supplied
      * this flag defaults to {@code true}. If set to {@code false} only classes found on the paths specified by the
@@ -603,6 +716,28 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
         return this;
     }
 
+    /**
+     * The path to the java executable to be used to launch test with. If none is supplied defaults to the one
+     * pointed to by {@code JAVA_HOME}.
+     *
+     * @param path the path
+     * @return this operation instance
+     */
+    public PitestOperation jvmPath(File path) {
+        return jvmPath(path.getAbsolutePath());
+    }
+
+    /**
+     * The path to the java executable to be used to launch test with. If none is supplied defaults to the one
+     * pointed to by {@code JAVA_HOME}.
+     *
+     * @param path the path
+     * @return this operation instance
+     */
+    public PitestOperation jvmPath(Path path) {
+        return jvmPath(path.toFile());
+    }
+
     /**
      * Maximum number of surviving mutants to allow without throwing an error.
      *
@@ -640,8 +775,43 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
      * @see #mutableCodePaths(Collection)
      */
     public PitestOperation mutableCodePaths(String... path) {
-        options_.put("--mutableCodePaths", String.join(",", Arrays.stream(path).filter(this::isNotBlank).toList()));
+        return mutableCodePaths(List.of(path));
-        return this;
+    }
+
+    /**
+     * List of classpaths which should be considered to contain mutable code. If your build maintains separate output
+     * directories for tests and production classes this parameter should be set to your code output directory in order
+     * to avoid mutating test helper classes etc.
+     * <p>
+     * If no mutableCodePath is supplied PIT will default to considering anything not defined within a jar or zip file
+     * as being a candidate for mutation.
+     * <p>
+     * PIT will always attempt not to mutate test classes even if they are defined on a mutable path.
+     *
+     * @param path one or one paths
+     * @return this operation instance
+     * @see #mutableCodePathsPaths(Collection)
+     */
+    public PitestOperation mutableCodePaths(Path... path) {
+        return mutableCodePathsPaths(List.of(path));
+    }
+
+    /**
+     * List of classpaths which should be considered to contain mutable code. If your build maintains separate output
+     * directories for tests and production classes this parameter should be set to your code output directory in order
+     * to avoid mutating test helper classes etc.
+     * <p>
+     * If no mutableCodePath is supplied PIT will default to considering anything not defined within a jar or zip file
+     * as being a candidate for mutation.
+     * <p>
+     * PIT will always attempt not to mutate test classes even if they are defined on a mutable path.
+     *
+     * @param path one or one paths
+     * @return this operation instance
+     * @see #mutableCodePathsFiles(Collection)
+     */
+    public PitestOperation mutableCodePaths(File... path) {
+        return mutableCodePathsFiles(List.of(path));
     }
 
     /**
@@ -663,6 +833,42 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
         return this;
     }
 
+    /**
+     * List of classpaths which should be considered to contain mutable code. If your build maintains separate output
+     * directories for tests and production classes this parameter should be set to your code output directory in order
+     * to avoid mutating test helper classes etc.
+     * <p>
+     * If no mutableCodePath is supplied PIT will default to considering anything not defined within a jar or zip file
+     * as being a candidate for mutation.
+     * <p>
+     * PIT will always attempt not to mutate test classes even if they are defined on a mutable path.
+     *
+     * @param paths the list of paths
+     * @return this operation instance
+     * @see #mutableCodePaths(File...)
+     */
+    public PitestOperation mutableCodePathsFiles(Collection<File> paths) {
+        return mutableCodePaths(paths.stream().map(File::getAbsolutePath).toList());
+    }
+
+    /**
+     * List of classpaths which should be considered to contain mutable code. If your build maintains separate output
+     * directories for tests and production classes this parameter should be set to your code output directory in order
+     * to avoid mutating test helper classes etc.
+     * <p>
+     * If no mutableCodePath is supplied PIT will default to considering anything not defined within a jar or zip file
+     * as being a candidate for mutation.
+     * <p>
+     * PIT will always attempt not to mutate test classes even if they are defined on a mutable path.
+     *
+     * @param paths the list of paths
+     * @return this operation instance
+     * @see #mutableCodePaths(Path...)
+     */
+    public PitestOperation mutableCodePathsPaths(Collection<Path> paths) {
+        return mutableCodePaths(paths.stream().map(Path::toFile).map(File::getAbsolutePath).toList());
+    }
+
     /**
      * Mutation engine to use.
      * <p>
@@ -752,6 +958,34 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
         return this;
     }
 
+    /**
+     * A list of formats in which to write mutation results as the mutations are analysed.
+     * Supported formats are {@code HTML}, {@code XML}, {@code CSV}.
+     * <p>
+     * Defaults to {@code HTML}.
+     *
+     * @param outputFormat one or more output formats
+     * @return this operation instance
+     * @see #outputFormatsFiles(Collection)
+     */
+    public PitestOperation outputFormats(File... outputFormat) {
+        return outputFormatsFiles(List.of(outputFormat));
+    }
+
+    /**
+     * A list of formats in which to write mutation results as the mutations are analysed.
+     * Supported formats are {@code HTML}, {@code XML}, {@code CSV}.
+     * <p>
+     * Defaults to {@code HTML}.
+     *
+     * @param outputFormat one or more output formats
+     * @return this operation instance
+     * @see #outputFormatsPaths(Collection)
+     */
+    public PitestOperation outputFormats(Path... outputFormat) {
+        return outputFormatsPaths(List.of(outputFormat));
+    }
+
     /**
      * A list of formats in which to write mutation results as the mutations are analysed.
      * Supported formats are {@code HTML}, {@code XML}, {@code CSV}.
@@ -763,9 +997,7 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
      * @see #outputFormats(Collection)
      */
     public PitestOperation outputFormats(String... outputFormat) {
-        options_.put("--outputFormats",
+        return outputFormats(List.of(outputFormat));
-                String.join(",", Arrays.stream(outputFormat).filter(this::isNotBlank).toList()));
-        return this;
     }
 
     /**
@@ -783,6 +1015,35 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
         return this;
     }
 
+    /**
+     * A list of formats in which to write mutation results as the mutations are analysed.
+     * Supported formats are {@code HTML}, {@code XML}, {@code CSV}.
+     * <p>
+     * Defaults to {@code HTML}.
+     *
+     * @param outputFormats the list of output formats
+     * @return this operation instance
+     * @see #outputFormats(File...)
+     */
+    public PitestOperation outputFormatsFiles(Collection<File> outputFormats) {
+        return outputFormats(outputFormats.stream().map(File::getAbsolutePath).toList());
+    }
+
+    /**
+     * A list of formats in which to write mutation results as the mutations are analysed.
+     * Supported formats are {@code HTML}, {@code XML}, {@code CSV}.
+     * <p>
+     * Defaults to {@code HTML}.
+     *
+     * @param outputFormats the list of output formats
+     * @return this operation instance
+     * @see #outputFormats(Path...)
+     */
+    public PitestOperation outputFormatsPaths(Collection<Path> outputFormats) {
+        return outputFormats(outputFormats.stream().map(Path::toFile).map(File::getAbsolutePath).toList());
+
+    }
+
     /**
      * Custom plugin properties.
      *
@@ -806,6 +1067,27 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
         return this;
     }
 
+    /**
+     * Project base.
+     *
+     * @param file the file
+     * @return this operations instance
+     */
+    public PitestOperation projectBase(File file) {
+        return projectBase(file.getAbsolutePath());
+    }
+
+    /**
+     * Project base.
+     *
+     * @param file the file
+     * @return this operations instance
+     */
+    public PitestOperation projectBase(Path file) {
+        return projectBase(file.toFile());
+    }
+
+
     /**
      * Output directory for the reports.
      *
@@ -819,6 +1101,26 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
         return this;
     }
 
+    /**
+     * Output directory for the reports.
+     *
+     * @param dir the directory
+     * @return this operation instance
+     */
+    public PitestOperation reportDir(File dir) {
+        return reportDir(dir.getAbsolutePath());
+    }
+
+    /**
+     * Output directory for the reports.
+     *
+     * @param dir the directory
+     * @return this operation instance
+     */
+    public PitestOperation reportDir(Path dir) {
+        return reportDir(dir.toFile());
+    }
+
     /**
      * whether to ignore failing tests when computing coverage.
      * <p>
@@ -844,8 +1146,29 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
      * @see #sourceDirs(Collection)
      */
     public PitestOperation sourceDirs(String... dir) {
-        options_.put(SOURCE_DIRS, String.join(",", Arrays.stream(dir).filter(this::isNotBlank).toList()));
+        return sourceDirs(List.of(dir));
-        return this;
+    }
+
+    /**
+     * The folder(s) containing the source code.
+     *
+     * @param dir one or more directories
+     * @return this operation instance
+     * @see #sourceDirsFiles(Collection)
+     */
+    public PitestOperation sourceDirs(File... dir) {
+        return sourceDirsFiles(List.of(dir));
+    }
+
+    /**
+     * The folder(s) containing the source code.
+     *
+     * @param dir one or more directories
+     * @return this operation instance
+     * @see #sourceDirsPaths(Collection)
+     */
+    public PitestOperation sourceDirs(Path... dir) {
+        return sourceDirsPaths(List.of(dir));
     }
 
     /**
@@ -860,6 +1183,29 @@ public class PitestOperation extends AbstractProcessOperation<PitestOperation> {
         return this;
     }
 
+    /**
+     * The folder(s) containing the source code.
+     *
+     * @param dirs the list of directories
+     * @return this operation instance
+     * @see #sourceDirs(File...)
+     */
+    public PitestOperation sourceDirsFiles(Collection<File> dirs) {
+        return sourceDirs(dirs.stream().map(File::getAbsolutePath).toList());
+    }
+
+    /**
+     * The folder(s) containing the source code.
+     *
+     * @param dirs the list of directories
+     * @return this operation instance
+     * @see #sourceDirs(Path...)
+     */
+    public PitestOperation sourceDirsPaths(Collection<Path> dirs) {
+        return sourceDirs(dirs.stream().map(Path::toFile).map(File::getAbsolutePath).toList());
+
+    }
+
     /**
      * The classes to be mutated. This is expressed as a list of globs.
      * <p>

src/test/java/rife/bld/extension/PitestOperationTest.java

@@ -22,6 +22,7 @@ import rife.bld.Project;
 import rife.bld.WebProject;
 import rife.bld.operations.exceptions.ExitStatusException;
 
+import java.io.File;
 import java.io.IOException;
 import java.nio.file.Files;
 import java.nio.file.Paths;
@@ -33,6 +34,7 @@ import static org.assertj.core.api.Assertions.assertThatCode;
 import static rife.bld.extension.PitestOperation.FALSE;
 import static rife.bld.extension.PitestOperation.TRUE;
 
+@SuppressWarnings("PMD.AvoidDuplicateLiterals")
 class PitestOperationTest {
     private static final String AS_LIST = "as list";
     private final static String BAR = "bar";
@@ -589,6 +591,175 @@ class PitestOperationTest {
         assertThat(op.options().get("--targetTests")).as(AS_LIST).isEqualTo(FOOBAR);
     }
 
+    @Test
+    void testClassPath() {
+        var foo = new File(FOO);
+        var bar = new File(BAR);
+
+        var foobar = String.format("%s,%s", FOO, BAR);
+        var op = new PitestOperation().classPath(FOO, BAR);
+        assertThat(op.options().get("--classPath")).as("String...").isEqualTo(foobar);
+
+        op = new PitestOperation().classPath(List.of(FOO, BAR));
+        assertThat(op.options().get("--classPath")).as("List(String...)").isEqualTo(foobar);
+
+        foobar = String.format("%s,%s", foo.getAbsolutePath(), bar.getAbsolutePath());
+        op = new PitestOperation().classPath(foo, bar);
+        assertThat(op.options().get("--classPath")).as("File...").isEqualTo(foobar);
+
+        op = new PitestOperation().classPathFiles(List.of(foo, bar));
+        assertThat(op.options().get("--classPath")).as("List(String...)").isEqualTo(foobar);
+
+        op = new PitestOperation().classPath(foo.toPath(), bar.toPath());
+        assertThat(op.options().get("--classPath")).as("Path...").isEqualTo(foobar);
+
+        op = new PitestOperation().classPathPaths(List.of(foo.toPath(), bar.toPath()));
+        assertThat(op.options().get("--classPath")).as("List(Path...)").isEqualTo(foobar);
+    }
+
+    @Test
+    void testHistoryInputLocation() {
+        var foo = new File(FOO);
+        var op = new PitestOperation().historyInputLocation(FOO);
+        assertThat(op.options().get("--historyInputLocation")).as("as string").isEqualTo(FOO);
+
+        op = new PitestOperation().historyInputLocation(foo);
+        assertThat(op.options().get("--historyInputLocation")).as("as file").isEqualTo(foo.getAbsolutePath());
+
+        op = new PitestOperation().historyInputLocation(foo.toPath());
+        assertThat(op.options().get("--historyInputLocation")).as("as path").isEqualTo(foo.getAbsolutePath());
+    }
+
+    @Test
+    void testHistoryOutputLocation() {
+        var foo = new File(FOO);
+        var op = new PitestOperation().historyOutputLocation(FOO);
+        assertThat(op.options().get("--historyOutputLocation")).as("as string").isEqualTo(FOO);
+
+        op = new PitestOperation().historyOutputLocation(foo);
+        assertThat(op.options().get("--historyOutputLocation")).as("as file").isEqualTo(foo.getAbsolutePath());
+
+        op = new PitestOperation().historyOutputLocation(foo.toPath());
+        assertThat(op.options().get("--historyOutputLocation")).as("as path").isEqualTo(foo.getAbsolutePath());
+    }
+
+    @Test
+    void testJvmPath() {
+        var foo = new File(FOO);
+        var op = new PitestOperation().jvmPath(FOO);
+        assertThat(op.options().get("--jvmPath")).as("as string").isEqualTo(FOO);
+
+        op = new PitestOperation().jvmPath(foo);
+        assertThat(op.options().get("--jvmPath")).as("as file").isEqualTo(foo.getAbsolutePath());
+
+        op = new PitestOperation().jvmPath(foo.toPath());
+        assertThat(op.options().get("--jvmPath")).as("as path").isEqualTo(foo.getAbsolutePath());
+    }
+
+    @Test
+    void testMutableCodePaths() {
+        var foo = new File(FOO);
+        var bar = new File(BAR);
+
+        var foobar = String.format("%s,%s", FOO, BAR);
+        var op = new PitestOperation().mutableCodePaths(FOO, BAR);
+        assertThat(op.options().get("--mutableCodePaths")).as("String...").isEqualTo(foobar);
+
+        op = new PitestOperation().mutableCodePaths(List.of(FOO, BAR));
+        assertThat(op.options().get("--mutableCodePaths")).as("List(String...)").isEqualTo(foobar);
+
+        foobar = String.format("%s,%s", foo.getAbsolutePath(), bar.getAbsolutePath());
+        op = new PitestOperation().mutableCodePaths(foo, bar);
+        assertThat(op.options().get("--mutableCodePaths")).as("File...").isEqualTo(foobar);
+
+        op = new PitestOperation().mutableCodePathsFiles(List.of(foo, bar));
+        assertThat(op.options().get("--mutableCodePaths")).as("List(String...)").isEqualTo(foobar);
+
+        op = new PitestOperation().mutableCodePaths(foo.toPath(), bar.toPath());
+        assertThat(op.options().get("--mutableCodePaths")).as("Path...").isEqualTo(foobar);
+
+        op = new PitestOperation().mutableCodePathsPaths(List.of(foo.toPath(), bar.toPath()));
+        assertThat(op.options().get("--mutableCodePaths")).as("List(Path...)").isEqualTo(foobar);
+    }
+
+    @Test
+    void testOutputFormats() {
+        var foo = new File(FOO);
+        var bar = new File(BAR);
+
+        var foobar = String.format("%s,%s", FOO, BAR);
+        var op = new PitestOperation().outputFormats(FOO, BAR);
+        assertThat(op.options().get("--outputFormats")).as("String...").isEqualTo(foobar);
+
+        op = new PitestOperation().outputFormats(List.of(FOO, BAR));
+        assertThat(op.options().get("--outputFormats")).as("List(String...)").isEqualTo(foobar);
+
+        foobar = String.format("%s,%s", foo.getAbsolutePath(), bar.getAbsolutePath());
+        op = new PitestOperation().outputFormats(foo, bar);
+        assertThat(op.options().get("--outputFormats")).as("File...").isEqualTo(foobar);
+
+        op = new PitestOperation().outputFormatsFiles(List.of(foo, bar));
+        assertThat(op.options().get("--outputFormats")).as("List(String...)").isEqualTo(foobar);
+
+        op = new PitestOperation().outputFormats(foo.toPath(), bar.toPath());
+        assertThat(op.options().get("--outputFormats")).as("Path...").isEqualTo(foobar);
+
+        op = new PitestOperation().outputFormatsPaths(List.of(foo.toPath(), bar.toPath()));
+        assertThat(op.options().get("--outputFormats")).as("List(Path...)").isEqualTo(foobar);
+    }
+
+    @Test
+    void testProjectBase() {
+        var foo = new File(FOO);
+        var op = new PitestOperation().projectBase(FOO);
+        assertThat(op.options().get("--projectBase")).as("as string").isEqualTo(FOO);
+
+        op = new PitestOperation().projectBase(foo);
+        assertThat(op.options().get("--projectBase")).as("as file").isEqualTo(foo.getAbsolutePath());
+
+        op = new PitestOperation().projectBase(foo.toPath());
+        assertThat(op.options().get("--projectBase")).as("as path").isEqualTo(foo.getAbsolutePath());
+    }
+
+    @Test
+    void testReportDir() {
+        var foo = new File(FOO);
+        var op = new PitestOperation().reportDir(FOO);
+        assertThat(op.options().get("--reportDir")).as("as string").isEqualTo(FOO);
+
+        op = new PitestOperation().reportDir(foo);
+        assertThat(op.options().get("--reportDir")).as("as file").isEqualTo(foo.getAbsolutePath());
+
+        op = new PitestOperation().reportDir(foo.toPath());
+        assertThat(op.options().get("--reportDir")).as("as path").isEqualTo(foo.getAbsolutePath());
+    }
+
+    @Test
+    void testSourceDirs() {
+        var foo = new File(FOO);
+        var bar = new File(BAR);
+
+        var foobar = String.format("%s,%s", FOO, BAR);
+        var op = new PitestOperation().sourceDirs(FOO, BAR);
+        assertThat(op.options().get("--sourceDirs")).as("String...").isEqualTo(foobar);
+
+        op = new PitestOperation().sourceDirs(List.of(FOO, BAR));
+        assertThat(op.options().get("--sourceDirs")).as("List(String...)").isEqualTo(foobar);
+
+        foobar = String.format("%s,%s", foo.getAbsolutePath(), bar.getAbsolutePath());
+        op = new PitestOperation().sourceDirs(foo, bar);
+        assertThat(op.options().get("--sourceDirs")).as("File...").isEqualTo(foobar);
+
+        op = new PitestOperation().sourceDirsFiles(List.of(foo, bar));
+        assertThat(op.options().get("--sourceDirs")).as("List(String...)").isEqualTo(foobar);
+
+        op = new PitestOperation().sourceDirs(foo.toPath(), bar.toPath());
+        assertThat(op.options().get("--sourceDirs")).as("Path...").isEqualTo(foobar);
+
+        op = new PitestOperation().sourceDirsPaths(List.of(foo.toPath(), bar.toPath()));
+        assertThat(op.options().get("--sourceDirs")).as("List(Path...)").isEqualTo(foobar);
+    }
+
     @Test
     void testStrengthThreshold() {
         var op = new PitestOperation()