This is an automated email from the ASF dual-hosted git repository.
slachiewicz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-invoker.git
The following commit(s) were added to refs/heads/master by this push:
new ce546ac Cleanup tests (#168)
ce546ac is described below
commit ce546ac8aba2cae9c34978b250611ea4f94e603d
Author: Sylwester Lachiewicz <[email protected]>
AuthorDate: Sun Dec 21 15:39:38 2025 +0100
Cleanup tests (#168)
Co-authored-by: Moderne <[email protected]>
---
.../maven/shared/invoker/DefaultInvokerTest.java | 28 ++---
.../invoker/MavenCommandLineBuilderTest.java | 126 ++++++++++-----------
.../maven/shared/invoker/SystemOutHandlerTest.java | 6 +-
.../maven/shared/invoker/SystemOutLoggerTest.java | 62 +++++-----
.../pom.xml | 0
5 files changed, 111 insertions(+), 111 deletions(-)
diff --git
a/src/test/java/org/apache/maven/shared/invoker/DefaultInvokerTest.java
b/src/test/java/org/apache/maven/shared/invoker/DefaultInvokerTest.java
index 8584e56..b6ca6dd 100644
--- a/src/test/java/org/apache/maven/shared/invoker/DefaultInvokerTest.java
+++ b/src/test/java/org/apache/maven/shared/invoker/DefaultInvokerTest.java
@@ -38,11 +38,11 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
class DefaultInvokerTest {
- private Invoker invoker = newInvoker();
- private InvocationRequest request = new DefaultInvocationRequest();
+ private final Invoker invoker = newInvoker();
+ private final InvocationRequest request = new DefaultInvocationRequest();
@BeforeEach
- public void setUp() {
+ void setUp() {
request.setDebug(true);
request.setProperties(getProperties());
request.setBatchMode(true);
@@ -50,7 +50,7 @@ class DefaultInvokerTest {
@Test
@SuppressWarnings("deprecation")
- void testBuildShouldSucceed() throws MavenInvocationException,
URISyntaxException {
+ void buildShouldSucceed() throws Exception {
File basedir = getBasedirForBuild();
request.setBaseDirectory(basedir);
request.setGoals(Arrays.asList("clean", "package"));
@@ -61,7 +61,7 @@ class DefaultInvokerTest {
}
@Test
- void testBuildShouldFail() throws MavenInvocationException,
URISyntaxException {
+ void buildShouldFail() throws Exception {
File basedir = getBasedirForBuild();
request.setBaseDirectory(basedir);
request.addArgs(Arrays.asList("clean", "package"));
@@ -72,7 +72,7 @@ class DefaultInvokerTest {
}
@Test
- void testBuildShouldTimeout() throws MavenInvocationException,
URISyntaxException {
+ void buildShouldTimeout() throws Exception {
File basedir = getBasedirForBuild();
request.setBaseDirectory(basedir);
request.addArgs(Arrays.asList("clean", "package"));
@@ -93,7 +93,7 @@ class DefaultInvokerTest {
}
@Test
- void testSpacePom() throws Exception {
+ void spacePom() throws Exception {
File basedir = getBasedirForBuild();
request.setBaseDirectory(basedir);
request.setPomFileName("pom with spaces.xml");
@@ -105,7 +105,7 @@ class DefaultInvokerTest {
}
@Test
- void testSpaceAndSpecialCharPom() throws Exception {
+ void spaceAndSpecialCharPom() throws Exception {
File basedir = getBasedirForBuild();
request.setBaseDirectory(basedir);
request.setPomFileName("pom with spaces & special char.xml");
@@ -117,7 +117,7 @@ class DefaultInvokerTest {
}
@Test
- void testSpaceSettings() throws Exception {
+ void spaceSettings() throws Exception {
File basedir = getBasedirForBuild();
request.setBaseDirectory(basedir);
request.setUserSettingsFile(new File(basedir, "settings with
spaces.xml"));
@@ -129,7 +129,7 @@ class DefaultInvokerTest {
}
@Test
- void testSpaceLocalRepo() throws Exception {
+ void spaceLocalRepo() throws Exception {
File basedir = getBasedirForBuild();
request.setBaseDirectory(basedir);
request.setLocalRepositoryDirectory(new File(basedir, "repo with
spaces"));
@@ -141,7 +141,7 @@ class DefaultInvokerTest {
}
@Test
- void testSpaceProperties() throws Exception {
+ void spaceProperties() throws Exception {
File basedir = getBasedirForBuild();
request.setBaseDirectory(basedir);
Properties props = getProperties();
@@ -156,7 +156,7 @@ class DefaultInvokerTest {
}
@Test
- void testPomOutsideProject() throws Exception {
+ void pomOutsideProject() throws Exception {
File basedir = getBasedirForBuild();
request.setBaseDirectory(basedir);
File pom = new File(basedir, "temp/pom.xml");
@@ -169,7 +169,7 @@ class DefaultInvokerTest {
}
@Test
- void testMavenWrapperInProject() throws Exception {
+ void mavenWrapperInProject() throws Exception {
File basedir = getBasedirForBuild();
request.setBaseDirectory(basedir);
request.addArg("test-wrapper-goal");
@@ -256,7 +256,7 @@ class DefaultInvokerTest {
StackTraceElement element = new
NullPointerException().getStackTrace()[1];
String methodName = element.getMethodName();
- String dirName = StringUtils.addAndDeHump(methodName);
+ String dirName = "test-" + StringUtils.addAndDeHump(methodName);
ClassLoader cloader = Thread.currentThread().getContextClassLoader();
URL dirResource = cloader.getResource(dirName);
diff --git
a/src/test/java/org/apache/maven/shared/invoker/MavenCommandLineBuilderTest.java
b/src/test/java/org/apache/maven/shared/invoker/MavenCommandLineBuilderTest.java
index 87a28f7..881f8de 100644
---
a/src/test/java/org/apache/maven/shared/invoker/MavenCommandLineBuilderTest.java
+++
b/src/test/java/org/apache/maven/shared/invoker/MavenCommandLineBuilderTest.java
@@ -73,7 +73,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldFailToSetLocalRepoLocationGloballyWhenItIsAFile() {
+ void shouldFailToSetLocalRepoLocationGloballyWhenItIsAFile() {
mclb.setLocalRepositoryDirectory(lrd);
@@ -82,14 +82,14 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldFailToSetLocalRepoLocationFromRequestWhenItIsAFile() {
+ void shouldFailToSetLocalRepoLocationFromRequestWhenItIsAFile() {
InvocationRequest request =
newRequest().setLocalRepositoryDirectory(lrd);
assertThrows(IllegalArgumentException.class, () ->
mclb.setLocalRepository(request, cli));
}
@Test
- void testShouldSetLocalRepoLocationGlobally() throws IOException {
+ void shouldSetLocalRepoLocationGlobally() throws Exception {
File lrd = Files.createDirectory(temporaryFolder.resolve("workdir"))
.toFile()
.getCanonicalFile();
@@ -100,7 +100,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetLocalRepoLocationFromRequest() throws Exception {
+ void shouldSetLocalRepoLocationFromRequest() throws Exception {
File lrd = Files.createDirectory(temporaryFolder.resolve("workdir"))
.toFile()
.getCanonicalFile();
@@ -110,7 +110,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testRequestProvidedLocalRepoLocationShouldOverrideGlobal() throws
Exception {
+ void requestProvidedLocalRepoLocationShouldOverrideGlobal() throws
Exception {
File lrd = Files.createDirectory(temporaryFolder.resolve("workdir"))
.toFile()
.getCanonicalFile();
@@ -125,7 +125,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetWorkingDirectoryGlobally() throws Exception {
+ void shouldSetWorkingDirectoryGlobally() throws Exception {
File wd =
Files.createDirectory(temporaryFolder.resolve("workdir")).toFile();
mclb.setBaseDirectory(wd);
@@ -135,7 +135,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetWorkingDirectoryFromRequest() throws Exception {
+ void shouldSetWorkingDirectoryFromRequest() throws Exception {
File wd =
Files.createDirectory(temporaryFolder.resolve("workdir")).toFile();
InvocationRequest req = newRequest();
@@ -147,7 +147,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testRequestProvidedWorkingDirectoryShouldOverrideGlobal() throws
Exception {
+ void requestProvidedWorkingDirectoryShouldOverrideGlobal() throws
Exception {
File wd =
Files.createDirectory(temporaryFolder.resolve("workdir")).toFile();
File gwd =
Files.createDirectory(temporaryFolder.resolve("global-workdir")).toFile();
@@ -163,7 +163,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldUseSystemOutLoggerWhenNoneSpecified() throws Exception {
+ void shouldUseSystemOutLoggerWhenNoneSpecified() throws Exception {
setupTempMavenHomeIfMissing(false);
mclb.checkRequiredState();
@@ -200,14 +200,14 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldFailIfLoggerSetToNull() {
+ void shouldFailIfLoggerSetToNull() {
mclb.setLogger(null);
assertThrows(IllegalStateException.class, () ->
mclb.checkRequiredState());
}
@Test
- void testShouldFindDummyMavenExecutable() throws Exception {
+ void shouldFindDummyMavenExecutable() throws Exception {
File dummyMavenHomeBin = Files.createDirectories(temporaryFolder
.resolve("invoker-tests")
.resolve("dummy-maven-home")
@@ -229,7 +229,7 @@ class MavenCommandLineBuilderTest {
@Test
@EnabledOnOs(OS.WINDOWS)
- void testShouldFindDummyPS1MavenExecutable() throws Exception {
+ void shouldFindDummyPS1MavenExecutable() throws Exception {
File dummyMavenHomeBin = Files.createDirectories(temporaryFolder
.resolve("invoker-tests")
.resolve("dummy-maven-home")
@@ -244,7 +244,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldFindDummyMavenExecutableWithMavenHomeFromRequest() throws
Exception {
+ void shouldFindDummyMavenExecutableWithMavenHomeFromRequest() throws
Exception {
File dummyMavenHomeBin = Files.createDirectories(temporaryFolder
.resolve("invoker-tests")
.resolve("dummy-maven-home")
@@ -266,7 +266,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetBatchModeFlagFromRequest() {
+ void shouldSetBatchModeFlagFromRequest() {
mclb.setFlags(newRequest().setBatchMode(true), cli);
@@ -274,7 +274,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetOfflineFlagFromRequest() {
+ void shouldSetOfflineFlagFromRequest() {
mclb.setFlags(newRequest().setOffline(true), cli);
@@ -282,7 +282,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetUpdateSnapshotsFlagFromRequest() {
+ void shouldSetUpdateSnapshotsFlagFromRequest() {
mclb.setFlags(newRequest().setUpdateSnapshots(true), cli);
@@ -291,7 +291,7 @@ class MavenCommandLineBuilderTest {
// JUnit5: test methods don't need to be public
@Test
- void testShouldSetUpdateSnapshotsPolicyAlwaysFromRequest() {
+ void shouldSetUpdateSnapshotsPolicyAlwaysFromRequest() {
mclb.setFlags(newRequest().setUpdateSnapshotsPolicy(UpdateSnapshotsPolicy.ALWAYS),
cli);
assertArgumentsPresent(cli, Collections.singleton("-U"));
@@ -299,7 +299,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetUpdateSnapshotsPolicyDefaultFromRequest() {
+ void shouldSetUpdateSnapshotsPolicyDefaultFromRequest() {
mclb.setFlags(newRequest().setUpdateSnapshotsPolicy(UpdateSnapshotsPolicy.DEFAULT),
cli);
Set<String> args = new HashSet<>();
@@ -309,7 +309,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetUpdateSnapshotsPolicyNeverFromRequest() {
+ void shouldSetUpdateSnapshotsPolicyNeverFromRequest() {
mclb.setFlags(newRequest().setUpdateSnapshotsPolicy(UpdateSnapshotsPolicy.NEVER),
cli);
assertArgumentsPresent(cli, Collections.singleton("-nsu"));
@@ -317,7 +317,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetDebugFlagFromRequest() {
+ void shouldSetDebugFlagFromRequest() {
mclb.setFlags(newRequest().setDebug(true), cli);
@@ -325,7 +325,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetErrorFlagFromRequest() {
+ void shouldSetErrorFlagFromRequest() {
mclb.setFlags(newRequest().setShowErrors(true), cli);
@@ -333,7 +333,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetQuietFlagFromRequest() {
+ void shouldSetQuietFlagFromRequest() {
mclb.setFlags(newRequest().setQuiet(true), cli);
@@ -341,21 +341,21 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetNonRecursiveFlagsFromRequest() {
+ void shouldSetNonRecursiveFlagsFromRequest() {
mclb.setFlags(newRequest().setRecursive(false), cli);
assertArgumentsPresent(cli, Collections.singleton("-N"));
}
@Test
- void testShouldSetShowVersionFlagsFromRequest() {
+ void shouldSetShowVersionFlagsFromRequest() {
mclb.setFlags(newRequest().setShowVersion(true), cli);
assertArgumentsPresent(cli, Collections.singleton("-V"));
}
@Test
- void testDebugOptionShouldMaskShowErrorsOption() {
+ void debugOptionShouldMaskShowErrorsOption() {
mclb.setFlags(newRequest().setDebug(true).setShowErrors(true), cli);
@@ -364,14 +364,14 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetBuilderIdOptionsFromRequest() {
+ void shouldSetBuilderIdOptionsFromRequest() {
mclb.setFlags(newRequest().setBuilder("builder-id-123"), cli);
assertArgumentsPresentInOrder(cli, "-b", "builder-id-123");
}
@Test
- void testAlsoMake() {
+ void alsoMake() {
mclb.setReactorBehavior(newRequest().setAlsoMake(true), cli);
@@ -380,7 +380,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testProjectsAndAlsoMake() {
+ void projectsAndAlsoMake() {
mclb.setReactorBehavior(
newRequest().setProjects(Collections.singletonList("proj1")).setAlsoMake(true),
cli);
@@ -389,7 +389,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testAlsoMakeDependents() {
+ void alsoMakeDependents() {
mclb.setReactorBehavior(newRequest().setAlsoMakeDependents(true), cli);
@@ -398,7 +398,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testProjectsAndAlsoMakeDependents() {
+ void projectsAndAlsoMakeDependents() {
mclb.setReactorBehavior(
newRequest().setProjects(Collections.singletonList("proj1")).setAlsoMakeDependents(true),
cli);
@@ -407,7 +407,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testProjectsAndAlsoMakeAndAlsoMakeDependents() {
+ void projectsAndAlsoMakeAndAlsoMakeDependents() {
mclb.setReactorBehavior(
newRequest()
@@ -420,7 +420,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetResumeFrom() {
+ void shouldSetResumeFrom() {
mclb.setReactorBehavior(newRequest().setResumeFrom(":module3"), cli);
@@ -428,7 +428,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetStrictChecksumPolityFlagFromRequest() {
+ void shouldSetStrictChecksumPolityFlagFromRequest() {
mclb.setFlags(newRequest().setGlobalChecksumPolicy(InvocationRequest.CheckSumPolicy.Fail),
cli);
@@ -436,7 +436,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetLaxChecksumPolicyFlagFromRequest() {
+ void shouldSetLaxChecksumPolicyFlagFromRequest() {
mclb.setFlags(newRequest().setGlobalChecksumPolicy(InvocationRequest.CheckSumPolicy.Warn),
cli);
@@ -444,7 +444,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetFailAtEndFlagFromRequest() {
+ void shouldSetFailAtEndFlagFromRequest() {
mclb.setReactorBehavior(
newRequest().setReactorFailureBehavior(InvocationRequest.ReactorFailureBehavior.FailAtEnd),
cli);
@@ -453,7 +453,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetFailNeverFlagFromRequest() {
+ void shouldSetFailNeverFlagFromRequest() {
mclb.setReactorBehavior(
newRequest().setReactorFailureBehavior(InvocationRequest.ReactorFailureBehavior.FailNever),
cli);
@@ -462,7 +462,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldAddArg() throws CommandLineConfigurationException {
+ void shouldAddArg() throws Exception {
InvocationRequest request =
newRequest().addArg("arg1").addArg("arg2").setQuiet(true).setBuilder("bId");
@@ -474,7 +474,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldUseDefaultOfFailFastWhenSpecifiedInRequest() {
+ void shouldUseDefaultOfFailFastWhenSpecifiedInRequest() {
mclb.setReactorBehavior(
newRequest().setReactorFailureBehavior(InvocationRequest.ReactorFailureBehavior.FailFast),
cli);
@@ -487,19 +487,19 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetNoTransferProgressFlagFromRequest() {
+ void shouldSetNoTransferProgressFlagFromRequest() {
mclb.setFlags(newRequest().setNoTransferProgress(true), cli);
assertArgumentsPresent(cli, Collections.singleton("-ntp"));
}
@Test
- void testShouldSetIgnoreTransitiveRepositoriesFromRequest() {
+ void shouldSetIgnoreTransitiveRepositoriesFromRequest() {
mclb.setFlags(newRequest().setIgnoreTransitiveRepositories(true), cli);
assertArgumentsPresent(cli, Collections.singleton("-itr"));
}
@Test
- void testShouldSpecifyFileOptionUsingNonStandardPomFileLocation() throws
Exception {
+ void shouldSpecifyFileOptionUsingNonStandardPomFileLocation() throws
Exception {
File projectDir = Files.createDirectories(
temporaryFolder.resolve("invoker-tests").resolve("file-option-nonstd-pom-file-location"))
.toFile();
@@ -520,7 +520,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldNotSpecifyFileOptionUsingStandardPomFileLocation() throws
Exception {
+ void shouldNotSpecifyFileOptionUsingStandardPomFileLocation() throws
Exception {
File projectDir = Files.createDirectories(
temporaryFolder.resolve("invoker-tests").resolve("std-pom-file-location"))
.toFile();
@@ -541,7 +541,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSetPomForOutsideWorkspace() throws Exception {
+ void shouldSetPomForOutsideWorkspace() throws Exception {
File projectDir = Files.createDirectories(
temporaryFolder.resolve("invoker-tests").resolve("std-pom-file-location"))
.toFile();
@@ -562,7 +562,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldNotSpecifyFileOptionUsingStandardPomInBasedir() throws
Exception {
+ void shouldNotSpecifyFileOptionUsingStandardPomInBasedir() throws
Exception {
File projectDir = Files.createDirectories(
temporaryFolder.resolve("invoker-tests").resolve("std-basedir-is-pom-file"))
.toFile();
@@ -583,7 +583,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldUseDefaultPomFileWhenBasedirSpecifiedWithoutPomFileName()
throws Exception {
+ void shouldUseDefaultPomFileWhenBasedirSpecifiedWithoutPomFileName()
throws Exception {
File projectDir = Files.createDirectories(
temporaryFolder.resolve("invoker-tests").resolve("std-basedir-no-pom-filename"))
.toFile();
@@ -602,7 +602,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSpecifyPomFileWhenBasedirSpecifiedWithPomFileName() throws
Exception {
+ void shouldSpecifyPomFileWhenBasedirSpecifiedWithPomFileName() throws
Exception {
File projectDir = Files.createDirectories(
temporaryFolder.resolve("invoker-tests").resolve("std-basedir-with-pom-filename"))
.toFile();
@@ -621,7 +621,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSpecifyCustomUserSettingsLocationFromRequest() throws
Exception {
+ void shouldSpecifyCustomUserSettingsLocationFromRequest() throws Exception
{
File projectDir = Files.createDirectories(
temporaryFolder.resolve("invoker-tests").resolve("custom-settings"))
.toFile();
@@ -638,7 +638,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSpecifyCustomGlobalSettingsLocationFromRequest() throws
Exception {
+ void shouldSpecifyCustomGlobalSettingsLocationFromRequest() throws
Exception {
File projectDir = Files.createDirectories(
temporaryFolder.resolve("invoker-tests").resolve("custom-settings"))
.toFile()
@@ -656,7 +656,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSpecifyCustomToolchainsLocationFromRequest() throws
Exception {
+ void shouldSpecifyCustomToolchainsLocationFromRequest() throws Exception {
File projectDir = Files.createDirectories(
temporaryFolder.resolve("invoker-tests").resolve("custom-toolchains"))
.toFile();
@@ -673,7 +673,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSpecifyCustomPropertyFromRequest() {
+ void shouldSpecifyCustomPropertyFromRequest() {
Properties properties = new Properties();
properties.setProperty("key", "value");
@@ -684,7 +684,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSpecifyCustomPropertyWithSpacesInValueFromRequest() {
+ void shouldSpecifyCustomPropertyWithSpacesInValueFromRequest() {
Properties properties = new Properties();
properties.setProperty("key", "value with spaces");
@@ -695,7 +695,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSpecifyCustomPropertyWithSpacesInKeyFromRequest() {
+ void shouldSpecifyCustomPropertyWithSpacesInKeyFromRequest() {
Properties properties = new Properties();
properties.setProperty("key with spaces", "value with spaces");
@@ -707,7 +707,7 @@ class MavenCommandLineBuilderTest {
@Test
@SuppressWarnings("deprecation")
- void testShouldSpecifySingleGoalFromRequest() throws
CommandLineConfigurationException {
+ void shouldSpecifySingleGoalFromRequest() throws Exception {
List<String> goals = new ArrayList<>();
goals.add("test");
@@ -718,7 +718,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSpecifySingleGoalFromRequestArg() throws
CommandLineConfigurationException {
+ void shouldSpecifySingleGoalFromRequestArg() throws Exception {
mclb.setArgs(newRequest().addArg("test"), cli);
@@ -727,7 +727,7 @@ class MavenCommandLineBuilderTest {
@Test
@SuppressWarnings("deprecation")
- void testShouldSpecifyTwoGoalsFromRequest() throws
CommandLineConfigurationException {
+ void shouldSpecifyTwoGoalsFromRequest() throws Exception {
List<String> goals = new ArrayList<>();
goals.add("test");
goals.add("clean");
@@ -739,7 +739,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSpecifyTwoGoalsFromRequestArgs() throws
CommandLineConfigurationException {
+ void shouldSpecifyTwoGoalsFromRequestArgs() throws Exception {
List<String> goals = new ArrayList<>();
goals.add("test");
goals.add("clean");
@@ -751,14 +751,14 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldSpecifyThreadsFromRequest() {
+ void shouldSpecifyThreadsFromRequest() {
mclb.setThreads(newRequest().setThreads("2.0C"), cli);
assertArgumentsPresentInOrder(cli, "-T", "2.0C");
}
@Test
- void testBuildTypicalMavenInvocationEndToEnd() throws Exception {
+ void buildTypicalMavenInvocationEndToEnd() throws Exception {
File mavenDir = setupTempMavenHomeIfMissing(false);
InvocationRequest request = newRequest();
@@ -811,7 +811,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testShouldInsertActivatedProfiles() throws Exception {
+ void shouldInsertActivatedProfiles() throws Exception {
setupTempMavenHomeIfMissing(false);
String profile1 = "profile-1";
@@ -831,7 +831,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testMvnExecutableFromInvoker() throws Exception {
+ void mvnExecutableFromInvoker() throws Exception {
assumeTrue(Objects.nonNull(System.getProperty("maven.home")), "Test
only works when maven.home is set");
File mavenExecutable = new File("mvnDebug");
@@ -845,7 +845,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testMvnExecutableFormRequest() throws Exception {
+ void mvnExecutableFormRequest() throws Exception {
assumeTrue(Objects.nonNull(System.getProperty("maven.home")), "Test
only works when maven.home is set");
File mavenExecutable = new File("mvnDebug");
@@ -858,7 +858,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testDefaultMavenCommand() throws Exception {
+ void defaultMavenCommand() throws Exception {
assumeTrue(Objects.nonNull(System.getProperty("maven.home")), "Test
only works when maven.home is set");
mclb.build(newRequest());
@@ -868,7 +868,7 @@ class MavenCommandLineBuilderTest {
}
@Test
- void testAddShellEnvironment() throws Exception {
+ void addShellEnvironment() throws Exception {
setupTempMavenHomeIfMissing(false);
InvocationRequest request = newRequest();
diff --git
a/src/test/java/org/apache/maven/shared/invoker/SystemOutHandlerTest.java
b/src/test/java/org/apache/maven/shared/invoker/SystemOutHandlerTest.java
index cf210f5..b1d896e 100644
--- a/src/test/java/org/apache/maven/shared/invoker/SystemOutHandlerTest.java
+++ b/src/test/java/org/apache/maven/shared/invoker/SystemOutHandlerTest.java
@@ -23,19 +23,19 @@ import org.junit.jupiter.api.Test;
class SystemOutHandlerTest {
@Test
- void testConsumeWithoutAlwaysFlush() {
+ void consumeWithoutAlwaysFlush() {
logTestStart();
new SystemOutHandler(false).consumeLine("This is a test.");
}
@Test
- void testConsumeWithAlwaysFlush() {
+ void consumeWithAlwaysFlush() {
logTestStart();
new SystemOutHandler(true).consumeLine("This is a test.");
}
@Test
- void testConsumeNullLine() {
+ void consumeNullLine() {
logTestStart();
new SystemOutHandler().consumeLine(null);
}
diff --git
a/src/test/java/org/apache/maven/shared/invoker/SystemOutLoggerTest.java
b/src/test/java/org/apache/maven/shared/invoker/SystemOutLoggerTest.java
index 1b6dfff..0e4ae3b 100644
--- a/src/test/java/org/apache/maven/shared/invoker/SystemOutLoggerTest.java
+++ b/src/test/java/org/apache/maven/shared/invoker/SystemOutLoggerTest.java
@@ -34,162 +34,162 @@ class SystemOutLoggerTest {
private static final String MESSAGE = "This is a test message.";
@Test
- void testDebugWithMessageOnly() {
+ void debugWithMessageOnly() {
logTestStart();
new SystemOutLogger().debug(MESSAGE);
}
@Test
- void testDebugWithMessageAndError() {
+ void debugWithMessageAndError() {
logTestStart();
new SystemOutLogger().debug(MESSAGE, EXCEPTION);
}
@Test
- void testDebugWithNullMessageAndNoError() {
+ void debugWithNullMessageAndNoError() {
logTestStart();
new SystemOutLogger().debug(null);
}
@Test
- void testDebugWithNullMessageError() {
+ void debugWithNullMessageError() {
logTestStart();
new SystemOutLogger().debug(null, EXCEPTION);
}
@Test
- void testDebugWithMessageNullError() {
+ void debugWithMessageNullError() {
logTestStart();
new SystemOutLogger().debug(MESSAGE, null);
}
@Test
- void testInfoWithMessageOnly() {
+ void infoWithMessageOnly() {
logTestStart();
new SystemOutLogger().info(MESSAGE);
}
@Test
- void testInfoWithMessageAndError() {
+ void infoWithMessageAndError() {
logTestStart();
new SystemOutLogger().info(MESSAGE, EXCEPTION);
}
@Test
- void testInfoWithNullMessageAndNoError() {
+ void infoWithNullMessageAndNoError() {
logTestStart();
new SystemOutLogger().info(null);
}
@Test
- void testInfoWithNullMessageError() {
+ void infoWithNullMessageError() {
logTestStart();
new SystemOutLogger().info(null, EXCEPTION);
}
@Test
- void testInfoWithMessageNullError() {
+ void infoWithMessageNullError() {
logTestStart();
new SystemOutLogger().info(MESSAGE, null);
}
@Test
- void testWarnWithMessageOnly() {
+ void warnWithMessageOnly() {
logTestStart();
new SystemOutLogger().warn(MESSAGE);
}
@Test
- void testWarnWithMessageAndError() {
+ void warnWithMessageAndError() {
logTestStart();
new SystemOutLogger().warn(MESSAGE, EXCEPTION);
}
@Test
- void testWarnWithNullMessageAndNoError() {
+ void warnWithNullMessageAndNoError() {
logTestStart();
new SystemOutLogger().warn(null);
}
@Test
- void testWarnWithNullMessageError() {
+ void warnWithNullMessageError() {
logTestStart();
new SystemOutLogger().warn(null, EXCEPTION);
}
@Test
- void testWarnWithMessageNullError() {
+ void warnWithMessageNullError() {
logTestStart();
new SystemOutLogger().warn(MESSAGE, null);
}
@Test
- void testErrorWithMessageOnly() {
+ void errorWithMessageOnly() {
logTestStart();
new SystemOutLogger().error(MESSAGE);
}
@Test
- void testErrorWithMessageAndError() {
+ void errorWithMessageAndError() {
logTestStart();
new SystemOutLogger().error(MESSAGE, EXCEPTION);
}
@Test
- void testErrorWithNullMessageAndNoError() {
+ void errorWithNullMessageAndNoError() {
logTestStart();
new SystemOutLogger().error(null);
}
@Test
- void testErrorWithNullMessageError() {
+ void errorWithNullMessageError() {
logTestStart();
new SystemOutLogger().error(null, EXCEPTION);
}
@Test
- void testErrorWithMessageNullError() {
+ void errorWithMessageNullError() {
logTestStart();
new SystemOutLogger().error(MESSAGE, null);
}
@Test
- void testFatalErrorWithMessageOnly() {
+ void fatalErrorWithMessageOnly() {
logTestStart();
new SystemOutLogger().fatalError(MESSAGE);
}
@Test
- void testFatalErrorWithMessageAndError() {
+ void fatalErrorWithMessageAndError() {
logTestStart();
new SystemOutLogger().fatalError(MESSAGE, EXCEPTION);
}
@Test
- void testFatalErrorWithNullMessageAndNoError() {
+ void fatalErrorWithNullMessageAndNoError() {
logTestStart();
new SystemOutLogger().fatalError(null);
}
@Test
- void testFatalErrorWithNullMessageError() {
+ void fatalErrorWithNullMessageError() {
logTestStart();
new SystemOutLogger().fatalError(null, EXCEPTION);
}
@Test
- void testFatalErrorWithMessageNullError() {
+ void fatalErrorWithMessageNullError() {
logTestStart();
new SystemOutLogger().fatalError(MESSAGE, null);
}
@Test
- void testDefaultThresholdInfo() {
+ void defaultThresholdInfo() {
assertEquals(InvokerLogger.INFO, new SystemOutLogger().getThreshold());
}
@Test
- void testThresholdDebug() {
+ void thresholdDebug() {
InvokerLogger logger = new SystemOutLogger();
logger.setThreshold(InvokerLogger.DEBUG);
assertTrue(logger.isDebugEnabled());
@@ -200,7 +200,7 @@ class SystemOutLoggerTest {
}
@Test
- void testThresholdInfo() {
+ void thresholdInfo() {
InvokerLogger logger = new SystemOutLogger();
logger.setThreshold(InvokerLogger.INFO);
assertFalse(logger.isDebugEnabled());
@@ -211,7 +211,7 @@ class SystemOutLoggerTest {
}
@Test
- void testThresholdWarn() {
+ void thresholdWarn() {
InvokerLogger logger = new SystemOutLogger();
logger.setThreshold(InvokerLogger.WARN);
assertFalse(logger.isDebugEnabled());
@@ -222,7 +222,7 @@ class SystemOutLoggerTest {
}
@Test
- void testThresholdError() {
+ void thresholdError() {
InvokerLogger logger = new SystemOutLogger();
logger.setThreshold(InvokerLogger.ERROR);
assertFalse(logger.isDebugEnabled());
@@ -233,7 +233,7 @@ class SystemOutLoggerTest {
}
@Test
- void testThresholdFatal() {
+ void thresholdFatal() {
InvokerLogger logger = new SystemOutLogger();
logger.setThreshold(InvokerLogger.FATAL);
assertFalse(logger.isDebugEnabled());
diff --git a/src/test/resources/not-inherit-env-variables/pom.xml
b/src/test/resources/test-not-inherit-env-variables/pom.xml
similarity index 100%
rename from src/test/resources/not-inherit-env-variables/pom.xml
rename to src/test/resources/test-not-inherit-env-variables/pom.xml
