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-dependency-plugin.git
The following commit(s) were added to refs/heads/master by this push:
new 9bdfecf8 [MDEP-979] Clean up AbstractDependencyMojoTestCase setUp
9bdfecf8 is described below
commit 9bdfecf8ebf91183e396c0cd80506e1bc56116f9
Author: Ndacyayisenga-droid <[email protected]>
AuthorDate: Thu Jun 26 09:07:26 2025 +0300
[MDEP-979] Clean up AbstractDependencyMojoTestCase setUp
---
.../dependency/AbstractDependencyMojoTestCase.java | 55 ++++++++++++++++++----
.../maven/plugins/dependency/TestCollectMojo.java | 13 ++++-
.../maven/plugins/dependency/TestGetMojo.java | 13 ++++-
.../plugins/dependency/TestListClassesMojo.java | 13 ++++-
.../plugins/dependency/TestPropertiesMojo.java | 14 +++++-
.../exclusion/AnalyzeExclusionsMojoTest.java | 20 +++++++-
.../fromConfiguration/TestArtifactItem.java | 14 +++++-
.../dependency/fromConfiguration/TestCopyMojo.java | 19 +++++++-
.../TestIncludeExcludeUnpackMojo.java | 18 ++++++-
.../fromConfiguration/TestUnpackMojo.java | 19 +++++++-
.../fromDependencies/TestBuildClasspathMojo.java | 13 ++++-
.../fromDependencies/TestCopyDependenciesMojo.java | 17 ++++++-
.../TestCopyDependenciesMojo2.java | 12 ++++-
.../TestIncludeExcludeUnpackDependenciesMojo.java | 13 ++++-
.../TestUnpackDependenciesMojo.java | 18 ++++++-
.../TestUnpackDependenciesMojo2.java | 13 ++++-
.../dependency/resolvers/GoOfflineMojoTest.java | 13 ++++-
.../resolvers/ResolveDependenciesMojoTest.java | 13 ++++-
.../dependency/resolvers/TestResolveMojo.java | 12 ++++-
.../plugins/dependency/tree/TestTreeMojo.java | 12 ++++-
.../dependency/utils/TestDependencyStatusSets.java | 13 ++++-
.../translators/TestClassifierTypeTranslator.java | 12 ++++-
22 files changed, 328 insertions(+), 31 deletions(-)
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/AbstractDependencyMojoTestCase.java
b/src/test/java/org/apache/maven/plugins/dependency/AbstractDependencyMojoTestCase.java
index 9681f242..7e100677 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/AbstractDependencyMojoTestCase.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/AbstractDependencyMojoTestCase.java
@@ -43,25 +43,62 @@ public abstract class AbstractDependencyMojoTestCase
extends AbstractMojoTestCas
protected DependencyArtifactStubFactory stubFactory;
- protected void setUp(String testDirectoryName, boolean createFiles) throws
Exception {
- setUp(testDirectoryName, createFiles, true);
- }
-
- protected void setUp(String testDirectoryName, boolean createFiles,
boolean flattenedPath) throws Exception {
- // required for mojo lookups to work
+ /**
+ * Initializes the test environment by creating a temporary directory and
setting up the stub factory.
+ * Subclasses must call super.setUp() in their own setUp method to ensure
proper initialization.
+ * To customize the test directory name, file creation, or path structure,
override getTestDirectoryName(),
+ * shouldCreateFiles(), and shouldUseFlattenedPath() respectively.
+ *
+ * @throws Exception if setup fails
+ */
+ protected void setUp() throws Exception {
+ // Required for mojo lookups to work
super.setUp();
- testDir = Files.createTempDirectory(testDirectoryName).toFile();
+ testDir = Files.createTempDirectory(getTestDirectoryName()).toFile();
testDir.deleteOnExit();
- stubFactory = new DependencyArtifactStubFactory(this.testDir,
createFiles, flattenedPath);
+ stubFactory = new DependencyArtifactStubFactory(testDir,
shouldCreateFiles(), shouldUseFlattenedPath());
+ }
+
+ /**
+ * Returns the name of the temporary test directory. Subclasses can
override to customize.
+ *
+ * @return the test directory name
+ */
+ protected String getTestDirectoryName() {
+ return "test-dir";
+ }
+
+ /**
+ * Determines whether files should be created by the stub factory.
Subclasses can override to customize.
+ *
+ * @return true if files should be created, false otherwise
+ */
+ protected boolean shouldCreateFiles() {
+ return true;
+ }
+
+ /**
+ * Determines whether the stub factory should use flattened paths.
Subclasses can override to customize.
+ *
+ * @return true if flattened paths should be used, false otherwise
+ */
+ protected boolean shouldUseFlattenedPath() {
+ return true;
}
+ /**
+ * Cleans up the test environment by deleting the temporary directory.
+ * Subclasses must call super.tearDown() in their own tearDown method to
ensure proper cleanup.
+ *
+ * @throws Exception if cleanup fails
+ */
@Override
protected void tearDown() throws Exception {
if (testDir != null) {
FileUtils.deleteDirectory(testDir);
- assertFalse(testDir.exists());
+ assertFalse("Test directory should not exist after cleanup",
testDir.exists());
}
super.tearDown();
}
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/TestCollectMojo.java
b/src/test/java/org/apache/maven/plugins/dependency/TestCollectMojo.java
index 71fb6a03..f93ee2ce 100644
--- a/src/test/java/org/apache/maven/plugins/dependency/TestCollectMojo.java
+++ b/src/test/java/org/apache/maven/plugins/dependency/TestCollectMojo.java
@@ -32,10 +32,21 @@ import org.apache.maven.project.MavenProject;
public class TestCollectMojo extends AbstractDependencyMojoTestCase {
+ @Override
+ protected String getTestDirectoryName() {
+ return "markers";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return false;
+ }
+
@Override
protected void setUp() throws Exception {
// required for mojo lookups to work
- super.setUp("markers", false);
+ super.setUp();
+
MavenProject project = new DependencyProjectStub();
getContainer().addComponent(project, MavenProject.class.getName());
diff --git a/src/test/java/org/apache/maven/plugins/dependency/TestGetMojo.java
b/src/test/java/org/apache/maven/plugins/dependency/TestGetMojo.java
index fd820791..ae793392 100644
--- a/src/test/java/org/apache/maven/plugins/dependency/TestGetMojo.java
+++ b/src/test/java/org/apache/maven/plugins/dependency/TestGetMojo.java
@@ -47,9 +47,20 @@ import org.eclipse.jetty.util.security.Constraint;
public class TestGetMojo extends AbstractDependencyMojoTestCase {
private GetMojo mojo;
+ @Override
+ protected String getTestDirectoryName() {
+ return "markers";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return false;
+ }
+
+ @Override
protected void setUp() throws Exception {
// required for mojo lookups to work
- super.setUp("markers", false);
+ super.setUp();
MavenProject project = new DependencyProjectStub();
getContainer().addComponent(project, MavenProject.class.getName());
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/TestListClassesMojo.java
b/src/test/java/org/apache/maven/plugins/dependency/TestListClassesMojo.java
index 0d79086f..1569b917 100644
--- a/src/test/java/org/apache/maven/plugins/dependency/TestListClassesMojo.java
+++ b/src/test/java/org/apache/maven/plugins/dependency/TestListClassesMojo.java
@@ -38,8 +38,19 @@ import org.mockito.Mockito;
public class TestListClassesMojo extends AbstractDependencyMojoTestCase {
private ListClassesMojo mojo;
+ @Override
+ protected String getTestDirectoryName() {
+ return "markers";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return false;
+ }
+
+ @Override
protected void setUp() throws Exception {
- super.setUp("markers", false);
+ super.setUp();
MavenProject project = new DependencyProjectStub();
getContainer().addComponent(project, MavenProject.class.getName());
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/TestPropertiesMojo.java
b/src/test/java/org/apache/maven/plugins/dependency/TestPropertiesMojo.java
index 75ae7cab..2e3db085 100644
--- a/src/test/java/org/apache/maven/plugins/dependency/TestPropertiesMojo.java
+++ b/src/test/java/org/apache/maven/plugins/dependency/TestPropertiesMojo.java
@@ -27,9 +27,21 @@ import
org.apache.maven.plugins.dependency.testUtils.stubs.DependencyProjectStub
import org.apache.maven.project.MavenProject;
public class TestPropertiesMojo extends AbstractDependencyMojoTestCase {
+
+ @Override
+ protected String getTestDirectoryName() {
+ return "markers";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return true;
+ }
+
+ @Override
protected void setUp() throws Exception {
// required for mojo lookups to work
- super.setUp("markers", true);
+ super.setUp();
MavenProject project = new DependencyProjectStub();
getContainer().addComponent(project, MavenProject.class.getName());
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/exclusion/AnalyzeExclusionsMojoTest.java
b/src/test/java/org/apache/maven/plugins/dependency/exclusion/AnalyzeExclusionsMojoTest.java
index f4e37d42..4c69bfd1 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/exclusion/AnalyzeExclusionsMojoTest.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/exclusion/AnalyzeExclusionsMojoTest.java
@@ -60,8 +60,24 @@ public class AnalyzeExclusionsMojoTest extends
AbstractDependencyMojoTestCase {
private ResolverUtil resolverUtil;
@Override
- public void setUp() throws Exception {
- super.setUp("analyze-exclusions", true, false);
+ protected String getTestDirectoryName() {
+ return "analyze-exclusions";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return true;
+ }
+
+ @Override
+ protected boolean shouldUseFlattenedPath() {
+ return false;
+ }
+
+ @Override
+ protected void setUp() throws Exception {
+ // required for mojo lookups to work
+ super.setUp();
project = new DependencyProjectStub();
project.setName("projectName");
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestArtifactItem.java
b/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestArtifactItem.java
index 9aa993f8..63fefb1d 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestArtifactItem.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestArtifactItem.java
@@ -25,8 +25,20 @@ import
org.apache.maven.plugins.dependency.AbstractDependencyMojoTestCase;
public class TestArtifactItem extends AbstractDependencyMojoTestCase {
+ @Override
+ protected String getTestDirectoryName() {
+ return "artifactItems";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return false;
+ }
+
+ @Override
protected void setUp() throws Exception {
- setUp("artifactItems", false);
+ // required for mojo lookups to work
+ super.setUp();
}
public void testArtifactItemConstructor() throws IOException {
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestCopyMojo.java
b/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestCopyMojo.java
index 1836b1d3..ef001335 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestCopyMojo.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestCopyMojo.java
@@ -38,9 +38,26 @@ import org.apache.maven.project.MavenProject;
public class TestCopyMojo extends AbstractDependencyMojoTestCase {
private CopyMojo mojo;
+ @Override
+ protected String getTestDirectoryName() {
+ return "copy";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return false;
+ }
+
+ @Override
+ protected boolean shouldUseFlattenedPath() {
+ return false;
+ }
+
@Override
protected void setUp() throws Exception {
- super.setUp("copy", false, false);
+ // required for mojo lookups to work
+ super.setUp();
+
MavenProject project = new DependencyProjectStub();
getContainer().addComponent(project, MavenProject.class.getName());
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestIncludeExcludeUnpackMojo.java
b/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestIncludeExcludeUnpackMojo.java
index e72e30a1..f42e42ae 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestIncludeExcludeUnpackMojo.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestIncludeExcludeUnpackMojo.java
@@ -43,9 +43,25 @@ public class TestIncludeExcludeUnpackMojo extends
AbstractDependencyMojoTestCase
private UnpackMojo mojo;
+ @Override
+ protected String getTestDirectoryName() {
+ return "unpack";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return true;
+ }
+
+ @Override
+ protected boolean shouldUseFlattenedPath() {
+ return false;
+ }
+
+ @Override
protected void setUp() throws Exception {
// required for mojo lookups to work
- super.setUp("unpack", true, false);
+ super.setUp();
MavenProject project = new DependencyProjectStub();
getContainer().addComponent(project, MavenProject.class.getName());
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestUnpackMojo.java
b/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestUnpackMojo.java
index f18776fb..0b8d4444 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestUnpackMojo.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/fromConfiguration/TestUnpackMojo.java
@@ -45,8 +45,25 @@ public class TestUnpackMojo extends
AbstractDependencyMojoTestCase {
UnpackMojo mojo;
+ @Override
+ protected String getTestDirectoryName() {
+ return "unpack";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return true;
+ }
+
+ @Override
+ protected boolean shouldUseFlattenedPath() {
+ return false;
+ }
+
+ @Override
protected void setUp() throws Exception {
- super.setUp("unpack", true, false);
+ // required for mojo lookups to work
+ super.setUp();
MavenProject project = new DependencyProjectStub();
getContainer().addComponent(project, MavenProject.class.getName());
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestBuildClasspathMojo.java
b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestBuildClasspathMojo.java
index f310ec0d..86856c63 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestBuildClasspathMojo.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestBuildClasspathMojo.java
@@ -33,9 +33,20 @@ public class TestBuildClasspathMojo extends
AbstractDependencyMojoTestCase {
private BuildClasspathMojo mojo;
+ @Override
+ protected String getTestDirectoryName() {
+ return "build-classpath";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return true;
+ }
+
+ @Override
protected void setUp() throws Exception {
// required for mojo lookups to work
- super.setUp("build-classpath", true);
+ super.setUp();
MavenProject project = new DependencyProjectStub();
getContainer().addComponent(project, MavenProject.class.getName());
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo.java
b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo.java
index 36e85399..bd5043bd 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo.java
@@ -43,10 +43,25 @@ public class TestCopyDependenciesMojo extends
AbstractDependencyMojoTestCase {
CopyDependenciesMojo mojo;
+ @Override
+ protected String getTestDirectoryName() {
+ return "copy-dependencies";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return true;
+ }
+
+ @Override
+ protected boolean shouldUseFlattenedPath() {
+ return false;
+ }
+
@Override
protected void setUp() throws Exception {
// required for mojo lookups to work
- super.setUp("copy-dependencies", true, false);
+ super.setUp();
MavenProject project = new DependencyProjectStub();
getContainer().addComponent(project, MavenProject.class.getName());
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo2.java
b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo2.java
index bb554d33..ae53378f 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo2.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo2.java
@@ -51,10 +51,20 @@ public class TestCopyDependenciesMojo2 extends
AbstractDependencyMojoTestCase {
private CopyDependenciesMojo mojo;
+ @Override
+ protected String getTestDirectoryName() {
+ return "copy-dependencies";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return true;
+ }
+
@Override
protected void setUp() throws Exception {
// required for mojo lookups to work
- super.setUp("copy-dependencies", true);
+ super.setUp();
MavenProject project = new DependencyProjectStub();
getContainer().addComponent(project, MavenProject.class.getName());
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestIncludeExcludeUnpackDependenciesMojo.java
b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestIncludeExcludeUnpackDependenciesMojo.java
index 675e48e3..eeb09381 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestIncludeExcludeUnpackDependenciesMojo.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestIncludeExcludeUnpackDependenciesMojo.java
@@ -38,9 +38,20 @@ public class TestIncludeExcludeUnpackDependenciesMojo
extends AbstractDependency
private UnpackDependenciesMojo mojo;
+ @Override
+ protected String getTestDirectoryName() {
+ return "unpack-dependencies";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return true;
+ }
+
+ @Override
protected void setUp() throws Exception {
// required for mojo lookups to work
- super.setUp("unpack-dependencies", true);
+ super.setUp();
MavenProject project = new DependencyProjectStub();
getContainer().addComponent(project, MavenProject.class.getName());
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestUnpackDependenciesMojo.java
b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestUnpackDependenciesMojo.java
index 4f66e6d6..98ddc813 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestUnpackDependenciesMojo.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestUnpackDependenciesMojo.java
@@ -50,9 +50,25 @@ public class TestUnpackDependenciesMojo extends
AbstractDependencyMojoTestCase {
UnpackDependenciesMojo mojo;
+ @Override
+ protected String getTestDirectoryName() {
+ return "unpack-dependencies";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return true;
+ }
+
+ @Override
+ protected boolean shouldUseFlattenedPath() {
+ return false;
+ }
+
+ @Override
protected void setUp() throws Exception {
// required for mojo lookups to work
- super.setUp("unpack-dependencies", true, false);
+ super.setUp();
MavenProject project = new DependencyProjectStub();
getContainer().addComponent(project, MavenProject.class.getName());
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestUnpackDependenciesMojo2.java
b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestUnpackDependenciesMojo2.java
index 8f576da4..932dbb56 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestUnpackDependenciesMojo2.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestUnpackDependenciesMojo2.java
@@ -45,9 +45,20 @@ public class TestUnpackDependenciesMojo2 extends
AbstractDependencyMojoTestCase
private UnpackDependenciesMojo mojo;
+ @Override
+ protected String getTestDirectoryName() {
+ return "unpack-dependencies";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return true;
+ }
+
+ @Override
protected void setUp() throws Exception {
// required for mojo lookups to work
- super.setUp("unpack-dependencies", true);
+ super.setUp();
MavenProject project = new DependencyProjectStub();
getContainer().addComponent(project, MavenProject.class.getName());
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/resolvers/GoOfflineMojoTest.java
b/src/test/java/org/apache/maven/plugins/dependency/resolvers/GoOfflineMojoTest.java
index af393858..9c48d72c 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/resolvers/GoOfflineMojoTest.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/resolvers/GoOfflineMojoTest.java
@@ -34,9 +34,20 @@ import org.junit.jupiter.api.Disabled;
public class GoOfflineMojoTest extends AbstractDependencyMojoTestCase {
private GoOfflineMojo subject;
+ @Override
+ protected String getTestDirectoryName() {
+ return "go-offline";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return true;
+ }
+
+ @Override
protected void setUp() throws Exception {
// required for mojo lookups to work
- super.setUp("go-offline", true);
+ super.setUp();
MavenProject project = new DependencyProjectStub();
getContainer().addComponent(project, MavenProject.class.getName());
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/resolvers/ResolveDependenciesMojoTest.java
b/src/test/java/org/apache/maven/plugins/dependency/resolvers/ResolveDependenciesMojoTest.java
index c59b0254..da3b9f8f 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/resolvers/ResolveDependenciesMojoTest.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/resolvers/ResolveDependenciesMojoTest.java
@@ -28,10 +28,21 @@ import
org.apache.maven.plugins.dependency.AbstractDependencyMojoTestCase;
import org.apache.maven.plugins.dependency.utils.DependencyStatusSets;
public class ResolveDependenciesMojoTest extends
AbstractDependencyMojoTestCase {
+
+ @Override
+ protected String getTestDirectoryName() {
+ return "dss";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return true;
+ }
+
@Override
protected void setUp() throws Exception {
// required for mojo lookups to work
- super.setUp("dss", true);
+ super.setUp();
}
public void testDependencyStatusLog() throws IOException {
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/resolvers/TestResolveMojo.java
b/src/test/java/org/apache/maven/plugins/dependency/resolvers/TestResolveMojo.java
index 2bdcb6fc..2a5d2d84 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/resolvers/TestResolveMojo.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/resolvers/TestResolveMojo.java
@@ -31,10 +31,20 @@ import org.apache.maven.project.MavenProject;
public class TestResolveMojo extends AbstractDependencyMojoTestCase {
+ @Override
+ protected String getTestDirectoryName() {
+ return "markers";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return false;
+ }
+
@Override
protected void setUp() throws Exception {
// required for mojo lookups to work
- super.setUp("markers", false);
+ super.setUp();
MavenProject project = new DependencyProjectStub();
getContainer().addComponent(project, MavenProject.class.getName());
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/tree/TestTreeMojo.java
b/src/test/java/org/apache/maven/plugins/dependency/tree/TestTreeMojo.java
index 6cfa0ef9..b34fa55b 100644
--- a/src/test/java/org/apache/maven/plugins/dependency/tree/TestTreeMojo.java
+++ b/src/test/java/org/apache/maven/plugins/dependency/tree/TestTreeMojo.java
@@ -56,13 +56,23 @@ import
org.apache.maven.shared.dependency.graph.internal.DefaultDependencyNode;
public class TestTreeMojo extends AbstractDependencyMojoTestCase {
// TestCase methods -------------------------------------------------------
+ @Override
+ protected String getTestDirectoryName() {
+ return "tree";
+ }
+
+ @Override
+ protected boolean shouldUseFlattenedPath() {
+ return false;
+ }
+
/*
* @see org.apache.maven.plugin.testing.AbstractMojoTestCase#setUp()
*/
@Override
protected void setUp() throws Exception {
// required for mojo lookups to work
- super.setUp("tree", false);
+ super.setUp();
MavenProject project = new MavenProjectStub();
getContainer().addComponent(project, MavenProject.class.getName());
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/utils/TestDependencyStatusSets.java
b/src/test/java/org/apache/maven/plugins/dependency/utils/TestDependencyStatusSets.java
index 7da3810f..58b33a84 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/utils/TestDependencyStatusSets.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/utils/TestDependencyStatusSets.java
@@ -22,9 +22,20 @@ import
org.apache.maven.plugins.dependency.AbstractDependencyMojoTestCase;
public class TestDependencyStatusSets extends AbstractDependencyMojoTestCase {
+ @Override
+ protected String getTestDirectoryName() {
+ return "dss";
+ }
+
+ @Override
+ protected boolean shouldCreateFiles() {
+ return true;
+ }
+
+ @Override
protected void setUp() throws Exception {
// required for mojo lookups to work
- super.setUp("dss", true);
+ super.setUp();
}
public void testDependencyStatusSettersGetters() {
diff --git
a/src/test/java/org/apache/maven/plugins/dependency/utils/translators/TestClassifierTypeTranslator.java
b/src/test/java/org/apache/maven/plugins/dependency/utils/translators/TestClassifierTypeTranslator.java
index 86b35142..39c59a7d 100644
---
a/src/test/java/org/apache/maven/plugins/dependency/utils/translators/TestClassifierTypeTranslator.java
+++
b/src/test/java/org/apache/maven/plugins/dependency/utils/translators/TestClassifierTypeTranslator.java
@@ -48,9 +48,19 @@ public class TestClassifierTypeTranslator extends
AbstractDependencyMojoTestCase
private ArtifactHandlerManager artifactHandlerManager;
+ @Override
+ protected String getTestDirectoryName() {
+ return "classifiertype-translator";
+ }
+
+ @Override
+ protected boolean shouldUseFlattenedPath() {
+ return false;
+ }
+
@Override
protected void setUp() throws Exception {
- super.setUp("classifiertype-translator", false);
+ super.setUp();
artifactHandlerManager = new DefaultArtifactHandlerManager();
this.setVariableValueToObject(artifactHandlerManager,
"artifactHandlers", new HashMap<>());
