diff --git a/src/main/java/org/apache/maven/plugins/dependency/AddDependencyMojo.java b/src/main/java/org/apache/maven/plugins/dependency/AddDependencyMojo.java
index 0aec3fcbb..396445fca 100644
--- a/src/main/java/org/apache/maven/plugins/dependency/AddDependencyMojo.java
+++ b/src/main/java/org/apache/maven/plugins/dependency/AddDependencyMojo.java
@@ -631,7 +631,7 @@ private static long getDependencyCount(PomEditor editor, boolean managed) {
/**
* Detects the project's dependency management conventions by analyzing existing dependencies.
*/
- private Conventions detectConventions(MavenProject project) throws MojoExecutionException {
+ private Conventions detectConventions(MavenProject project) {
Conventions conv = new Conventions();
File pomFile = project.getFile();
diff --git a/src/main/java/org/apache/maven/plugins/dependency/RemoveDependencyMojo.java b/src/main/java/org/apache/maven/plugins/dependency/RemoveDependencyMojo.java
index c6171793a..d525b8bc7 100644
--- a/src/main/java/org/apache/maven/plugins/dependency/RemoveDependencyMojo.java
+++ b/src/main/java/org/apache/maven/plugins/dependency/RemoveDependencyMojo.java
@@ -271,8 +271,7 @@ private DependencyEntry resolveCoordinates() throws MojoFailureException {
return coords;
}
- private void checkChildModuleDependencies(MavenProject parentProject, String depGroupId, String depArtifactId)
- throws MojoExecutionException {
+ private void checkChildModuleDependencies(MavenProject parentProject, String depGroupId, String depArtifactId) {
if (parentProject.getBasedir() == null) {
getLog().debug("Parent project basedir is null, skipping child module dependency check");
return;
diff --git a/src/main/java/org/apache/maven/plugins/dependency/tree/TreeMojo.java b/src/main/java/org/apache/maven/plugins/dependency/tree/TreeMojo.java
index a2e7d8e84..f6c2df382 100644
--- a/src/main/java/org/apache/maven/plugins/dependency/tree/TreeMojo.java
+++ b/src/main/java/org/apache/maven/plugins/dependency/tree/TreeMojo.java
@@ -33,7 +33,6 @@
import org.apache.maven.execution.MavenSession;
import org.apache.maven.plugin.AbstractMojo;
import org.apache.maven.plugin.MojoExecutionException;
-import org.apache.maven.plugin.MojoFailureException;
import org.apache.maven.plugins.annotations.Mojo;
import org.apache.maven.plugins.annotations.Parameter;
import org.apache.maven.plugins.annotations.ResolutionScope;
@@ -220,7 +219,7 @@ public TreeMojo(
* @see org.apache.maven.plugin.Mojo#execute()
*/
@Override
- public void execute() throws MojoExecutionException, MojoFailureException {
+ public void execute() throws MojoExecutionException {
if (isSkip()) {
getLog().info("Skipping plugin execution");
return;
diff --git a/src/main/java/org/apache/maven/plugins/dependency/utils/markers/SourcesFileMarkerHandler.java b/src/main/java/org/apache/maven/plugins/dependency/utils/markers/SourcesFileMarkerHandler.java
index 7cd9cef93..e55551009 100644
--- a/src/main/java/org/apache/maven/plugins/dependency/utils/markers/SourcesFileMarkerHandler.java
+++ b/src/main/java/org/apache/maven/plugins/dependency/utils/markers/SourcesFileMarkerHandler.java
@@ -80,12 +80,9 @@ protected File getMarkerFile(boolean res) {
*
* @return true if and only if the file or directory denoted by this abstract pathname exists;
* false otherwise
- * @throws MojoExecutionException if a security manager exists and its {@link
- * java.lang.SecurityManager#checkRead(java.lang.String)} method denies read access to the file or
- * directory
*/
@Override
- public boolean isMarkerSet() throws MojoExecutionException {
+ public boolean isMarkerSet() {
File marker = getMarkerFile();
File marker2 = getMarkerFile(!this.resolved);
@@ -94,7 +91,7 @@ public boolean isMarkerSet() throws MojoExecutionException {
}
@Override
- public boolean isMarkerOlder(Artifact theArtifact) throws MojoExecutionException {
+ public boolean isMarkerOlder(Artifact theArtifact) {
File marker = getMarkerFile();
if (marker.exists()) {
return theArtifact.getFile().lastModified() > marker.lastModified();
@@ -142,11 +139,9 @@ public void setMarker() throws MojoExecutionException {
*
* @return true if and only if the file or directory is successfully deleted; false
* otherwise
- * @throws SecurityException if a security manager exists and its {@link
- * java.lang.SecurityManager#checkDelete} method denies delete access to the file
*/
@Override
- public boolean clearMarker() throws MojoExecutionException {
+ public boolean clearMarker() {
File marker = getMarkerFile();
File marker2 = getMarkerFile(!this.resolved);
boolean markResult = marker.delete();
diff --git a/src/test/java/org/apache/maven/plugins/dependency/AddDependencyMojoTest.java b/src/test/java/org/apache/maven/plugins/dependency/AddDependencyMojoTest.java
index e66f38bfa..ad7a7ce10 100644
--- a/src/test/java/org/apache/maven/plugins/dependency/AddDependencyMojoTest.java
+++ b/src/test/java/org/apache/maven/plugins/dependency/AddDependencyMojoTest.java
@@ -60,7 +60,7 @@ class AddDependencyMojoTest {
private AddDependencyMojo mojo;
@BeforeEach
- void setUp() throws Exception {
+ void setUp() {
session = mock(MavenSession.class);
buildContext = mock(BuildContext.class);
project = mock(MavenProject.class);
diff --git a/src/test/java/org/apache/maven/plugins/dependency/RemoveDependencyMojoTest.java b/src/test/java/org/apache/maven/plugins/dependency/RemoveDependencyMojoTest.java
index 449493636..000d49a15 100644
--- a/src/test/java/org/apache/maven/plugins/dependency/RemoveDependencyMojoTest.java
+++ b/src/test/java/org/apache/maven/plugins/dependency/RemoveDependencyMojoTest.java
@@ -58,7 +58,7 @@ class RemoveDependencyMojoTest {
private RemoveDependencyMojo mojo;
@BeforeEach
- void setUp() throws Exception {
+ void setUp() {
session = mock(MavenSession.class);
buildContext = mock(BuildContext.class);
project = mock(MavenProject.class);
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 62098d61d..33e45d497 100644
--- a/src/test/java/org/apache/maven/plugins/dependency/TestCollectMojo.java
+++ b/src/test/java/org/apache/maven/plugins/dependency/TestCollectMojo.java
@@ -43,7 +43,7 @@ class TestCollectMojo {
private DependencyArtifactStubFactory stubFactory;
@BeforeEach
- void setUp() throws Exception {
+ void setUp() {
stubFactory = new DependencyArtifactStubFactory(null, false);
}
@@ -101,7 +101,7 @@ void testCollectTestEnvironmentExcludeTransitive(CollectDependenciesMojo mojo) t
@Test
@InjectMojo(goal = "collect")
- void testSilent(CollectDependenciesMojo mojo) throws Exception {
+ void testSilent(CollectDependenciesMojo mojo) {
assertFalse(mojo.getLog() instanceof DependencySilentLog);
mojo.setSilent(true);
diff --git a/src/test/java/org/apache/maven/plugins/dependency/exclusion/ExclusionCheckerTest.java b/src/test/java/org/apache/maven/plugins/dependency/exclusion/ExclusionCheckerTest.java
index 2663595df..a0e03d84b 100644
--- a/src/test/java/org/apache/maven/plugins/dependency/exclusion/ExclusionCheckerTest.java
+++ b/src/test/java/org/apache/maven/plugins/dependency/exclusion/ExclusionCheckerTest.java
@@ -34,7 +34,7 @@ class ExclusionCheckerTest {
private ExclusionChecker checker;
@BeforeEach
- void setUp() throws Exception {
+ void setUp() {
checker = new ExclusionChecker();
}
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 f29043cd8..aca0d235d 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
@@ -18,8 +18,6 @@
*/
package org.apache.maven.plugins.dependency.fromConfiguration;
-import java.io.IOException;
-
import org.apache.maven.artifact.Artifact;
import org.apache.maven.artifact.DefaultArtifact;
import org.junit.jupiter.api.Test;
@@ -29,7 +27,7 @@
class TestArtifactItem {
@Test
- void testArtifactItemConstructor() throws IOException {
+ void testArtifactItemConstructor() {
Artifact artifact = new DefaultArtifact("g", "a", "1.0", Artifact.SCOPE_COMPILE, "jar", "one", null);
ArtifactItem item = new ArtifactItem(artifact);
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 06a53c126..ac4669361 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
@@ -72,7 +72,7 @@ class TestUnpackMojo {
private ArchiverManager archiverManager;
@BeforeEach
- void setUp() throws Exception {
+ void setUp() {
stubFactory = new DependencyArtifactStubFactory(tempDir, true, false);
stubFactory.setUnpackableFile(archiverManager);
stubFactory.setSrcFile(MojoExtension.getTestFile("test.txt"));
diff --git a/src/test/java/org/apache/maven/plugins/dependency/utils/filters/TestDestFileFilter.java b/src/test/java/org/apache/maven/plugins/dependency/utils/filters/TestDestFileFilter.java
index e721bbdc1..8891184d7 100644
--- a/src/test/java/org/apache/maven/plugins/dependency/utils/filters/TestDestFileFilter.java
+++ b/src/test/java/org/apache/maven/plugins/dependency/utils/filters/TestDestFileFilter.java
@@ -44,7 +44,7 @@ class TestDestFileFilter {
private DependencyArtifactStubFactory fact;
@BeforeEach
- void setUp() throws Exception {
+ void setUp() {
this.fact = new DependencyArtifactStubFactory(outputFolder, false);
}