This is an automated email from the ASF dual-hosted git repository.

elharo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-doap-plugin.git


The following commit(s) were added to refs/heads/master by this push:
     new cdf79a3  wip
cdf79a3 is described below

commit cdf79a307166fe004a444118aff23c162e3747f5
Author: Elliotte Rusty Harold <elh...@ibiblio.org>
AuthorDate: Mon Dec 16 07:37:56 2024 -0500

    wip
---
 pom.xml                                            | 11 ++++++
 .../org/apache/maven/plugin/doap/DoapMojo.java     | 42 ++++++++++++++--------
 .../org/apache/maven/plugin/doap/DoapMojoTest.java | 10 ------
 3 files changed, 38 insertions(+), 25 deletions(-)

diff --git a/pom.xml b/pom.xml
index e4f5559..0d4c0cd 100644
--- a/pom.xml
+++ b/pom.xml
@@ -111,6 +111,17 @@ under the License.
     </dependency>
 
     <!-- misc -->
+    <dependency>
+      <groupId>javax.inject</groupId>
+      <artifactId>javax.inject</artifactId>
+      <version>1</version>
+      <scope>provided</scope>
+    </dependency>
+    <dependency>
+      <groupId>org.eclipse.sisu</groupId>
+      <artifactId>org.eclipse.sisu.plexus</artifactId>
+      <version>0.9.0.M2</version>
+    </dependency>
     <dependency>
       <groupId>org.codehaus.plexus</groupId>
       <artifactId>plexus-utils</artifactId>
diff --git a/src/main/java/org/apache/maven/plugin/doap/DoapMojo.java 
b/src/main/java/org/apache/maven/plugin/doap/DoapMojo.java
index c03f1b0..4846257 100644
--- a/src/main/java/org/apache/maven/plugin/doap/DoapMojo.java
+++ b/src/main/java/org/apache/maven/plugin/doap/DoapMojo.java
@@ -18,6 +18,8 @@
  */
 package org.apache.maven.plugin.doap;
 
+import javax.inject.Inject;
+
 import java.io.File;
 import java.io.IOException;
 import java.io.Writer;
@@ -59,7 +61,6 @@ import org.apache.maven.plugin.doap.options.DoapArtifact;
 import org.apache.maven.plugin.doap.options.DoapOptions;
 import org.apache.maven.plugin.doap.options.ExtOptions;
 import org.apache.maven.plugin.doap.options.Standard;
-import org.apache.maven.plugins.annotations.Component;
 import org.apache.maven.plugins.annotations.Mojo;
 import org.apache.maven.plugins.annotations.Parameter;
 import org.apache.maven.project.MavenProject;
@@ -124,32 +125,28 @@ public class DoapMojo extends AbstractMojo {
      *
      * @since 1.0
      */
-    @Component
-    private ScmManager scmManager;
+    private final ScmManager scmManager;
 
     /**
      * Artifact factory.
      *
      * @since 1.0
      */
-    @Component
-    private ArtifactFactory artifactFactory;
+    private final ArtifactFactory artifactFactory;
 
     /**
      * Used to resolve artifacts.
      *
      * @since 1.0
      */
-    @Component
-    private RepositoryMetadataManager repositoryMetadataManager;
+    private final RepositoryMetadataManager repositoryMetadataManager;
 
     /**
      * Internationalization component.
      *
      * @since 1.0
      */
-    @Component
-    private I18N i18n;
+    private final I18N i18n;
 
     // ----------------------------------------------------------------------
     // Mojo parameters
@@ -196,24 +193,21 @@ public class DoapMojo extends AbstractMojo {
      *
      * @since 1.1
      */
-    @Component
-    private ArtifactFactory factory;
+    private final ArtifactFactory factory;
 
     /**
      * Project builder
      *
      * @since 1.1
      */
-    @Component
-    private MavenProjectBuilder mavenProjectBuilder;
+    private final MavenProjectBuilder mavenProjectBuilder;
 
     /**
      * Used for resolving artifacts
      *
      * @since 1.1
      */
-    @Component
-    private ArtifactResolver resolver;
+    private final ArtifactResolver resolver;
 
     /**
      * The current user system settings for use in Maven.
@@ -388,6 +382,24 @@ public class DoapMojo extends AbstractMojo {
      */
     private UserMessages messages = new UserMessages();
 
+    @Inject
+    public DoapMojo(
+            ScmManager scmManager,
+            ArtifactFactory artifactFactory,
+            RepositoryMetadataManager repositoryMetadataManager,
+            I18N i18n,
+            ArtifactFactory factory,
+            MavenProjectBuilder mavenProjectBuilder,
+            ArtifactResolver resolver) {
+        this.scmManager = scmManager;
+        this.artifactFactory = artifactFactory;
+        this.repositoryMetadataManager = repositoryMetadataManager;
+        this.i18n = i18n;
+        this.factory = factory;
+        this.mavenProjectBuilder = mavenProjectBuilder;
+        this.resolver = resolver;
+    }
+
     // ----------------------------------------------------------------------
     // Public methods
     // ----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/maven/plugin/doap/DoapMojoTest.java 
b/src/test/java/org/apache/maven/plugin/doap/DoapMojoTest.java
index 31c673c..da51465 100644
--- a/src/test/java/org/apache/maven/plugin/doap/DoapMojoTest.java
+++ b/src/test/java/org/apache/maven/plugin/doap/DoapMojoTest.java
@@ -35,16 +35,6 @@ import org.codehaus.plexus.util.StringUtils;
  * @author <a href="mailto:vincent.sive...@gmail.com";>Vincent Siveton</a>
  */
 public class DoapMojoTest extends AbstractMojoTestCase {
-    @Override
-    protected void setUp() throws Exception {
-        super.setUp();
-    }
-
-    @Override
-    protected void tearDown() throws Exception {
-        super.tearDown();
-    }
-
     /**
      * Verify the generation of a pure DOAP file.
      *

Reply via email to