}
/**
- * Fully qualified class name that implements {@code org.testng.ITestObjectFactory} which can be used to create
+ * The list of {@code .class} files or class names implementing {@code ITestRunnerFactory}.
+ *
+ * A fully qualified class name that implements {@code org.testng.ITestObjectFactory} which can be used to create
* test class and listener instances.
*
- * @param objectFactory the object factory
- * @return this operation instance
- */
- public TestNgOperation objectFactory(String objectFactory) {
- options_.put("-objectfactory", objectFactory);
- return this;
- }
-
- /**
- * The list of {@code .class} files or class names implementing {@code ITestRunnerFactory}.
- *
- * @param factory one or more factories
+ * @param factory one or more factory
* @return this operation instance
* @see #objectFactory(Collection) #objectFactory(Collection)
*/
public TestNgOperation objectFactory(String... factory) {
- options_.put("-objectfactory", String.join(",", Arrays.stream(factory).filter(this::isNotBlank).toList()));
- return this;
+ return objectFactory(List.of(factory));
}
/**
* The list of {@code .class} files or class names implementing {@code ITestRunnerFactory}.
+ *
+ * A fully qualified class name that implements {@code org.testng.ITestObjectFactory} which can be used to create
+ * test class and listener instances.
*
* @param factory the list of factories
* @return this operation instance
@@ -570,9 +557,7 @@ public class TestNgOperation extends TestOperation
* @see #overrideIncludedMethods(Collection) #overrideIncludedMethods(Collection)
*/
public TestNgOperation overrideIncludedMethods(String... method) {
- options_.put("-overrideincludedmethods",
- String.join(",", Arrays.stream(method).filter(this::isNotBlank).toList()));
- return this;
+ return overrideIncludedMethods(List.of(method));
}
/**
@@ -600,8 +585,7 @@ public class TestNgOperation extends TestOperation
* @see #packages(Collection) #packages(Collection)
*/
public TestNgOperation packages(String... name) {
- packages_.addAll(Arrays.stream(name).filter(this::isNotBlank).toList());
- return this;
+ return packages(List.of(name));
}
/**
@@ -783,9 +767,7 @@ public class TestNgOperation extends TestOperation
* @see #spiListenersToSkip(Collection) #spiListenersToSkip(Collection)
*/
public TestNgOperation spiListenersToSkip(String... listenerToSkip) {
- options_.put("-spilistenerstoskip",
- String.join(",", Arrays.stream(listenerToSkip).filter(this::isNotBlank).toList()));
- return this;
+ return spiListenersToSkip(List.of(listenerToSkip));
}
/**
@@ -840,8 +822,7 @@ public class TestNgOperation extends TestOperation
* @see #suites(Collection) #suites(Collection)
*/
public TestNgOperation suites(String... suite) {
- suites_.addAll(Arrays.stream(suite).filter(this::isNotBlank).toList());
- return this;
+ return suites(List.of(suite));
}
/**
@@ -888,8 +869,7 @@ public class TestNgOperation extends TestOperation
* @see #testClass(Collection) #testClass(Collection)
*/
public TestNgOperation testClass(String... aClass) {
- options_.put("-testclass", String.join(",", Arrays.stream(aClass).filter(this::isNotBlank).toList()));
- return this;
+ return testClass(List.of(aClass));
}
/**
@@ -914,8 +894,7 @@ public class TestNgOperation extends TestOperation
* @see #testClasspath(String...) #testClasspath(String...)
*/
public TestNgOperation testClasspath(String... entry) {
- testClasspath_.addAll(Arrays.stream(entry).filter(this::isNotBlank).toList());
- return this;
+ return testClasspath(List.of(entry));
}
/**
@@ -976,9 +955,7 @@ public class TestNgOperation extends TestOperation
* @see #testNames(Collection) #testNames(Collection)
*/
public TestNgOperation testNames(String... name) {
- options_.put("-testnames",
- Arrays.stream(name).filter(this::isNotBlank).map(s -> '"' + s + '"').collect(Collectors.joining(",")));
- return this;
+ return testNames(List.of(name));
}
/**
diff --git a/src/test/java/rife/bld/extension/TestNgExample.java b/src/test/java/rife/bld/extension/TestNgExample.java
index bfe36ca..abe0606 100644
--- a/src/test/java/rife/bld/extension/TestNgExample.java
+++ b/src/test/java/rife/bld/extension/TestNgExample.java
@@ -24,6 +24,7 @@ package rife.bld.extension;
*/
@SuppressWarnings({"PMD.TestClassWithoutTestCases", "unused"})
class TestNgExample {
+ @SuppressWarnings("SameReturnValue")
public String getMessage() {
return "Hello World!";
}