This is an automated email from the ASF dual-hosted git repository.
slachiewicz pushed a commit to branch MSHARED-1346
in repository https://gitbox.apache.org/repos/asf/maven-jarsigner.git
commit c7275cc00e2989faafc03f7c88d3dda80db81199
Author: Sylwester Lachiewicz <slachiew...@gmail.com>
AuthorDate: Mon Dec 25 21:25:43 2023 +0100
[MSHARED-1346] Convert to JSR330
---
pom.xml | 36 ++++++++++------------
.../maven/shared/jarsigner/DefaultJarSigner.java | 5 +--
.../shared/jarsigner/AbstractJarSignerTest.java | 4 +--
.../maven/shared/jarsigner/JarSignerUtilTest.java | 3 ++
.../maven/shared/jarsigner/SimpleJarSignTest.java | 14 +++++----
5 files changed, 33 insertions(+), 29 deletions(-)
diff --git a/pom.xml b/pom.xml
index 5668b06..fc6de69 100644
--- a/pom.xml
+++ b/pom.xml
@@ -67,18 +67,25 @@
<version>3.4.2</version>
</dependency>
+ <!-- This is a Sisu Component, DI container should be provided by consumer
-->
<dependency>
- <groupId>org.codehaus.plexus</groupId>
- <artifactId>plexus-component-annotations</artifactId>
- <version>2.1.1</version>
+ <groupId>javax.inject</groupId>
+ <artifactId>javax.inject</artifactId>
+ <version>1</version>
+ <scope>provided</scope>
</dependency>
-
<dependency>
- <groupId>org.apache.maven</groupId>
- <artifactId>maven-core</artifactId>
- <version>${mavenVersion}</version>
+ <groupId>org.eclipse.sisu</groupId>
+ <artifactId>org.eclipse.sisu.inject</artifactId>
+ <version>0.9.0.M2</version>
+ <scope>provided</scope>
+ </dependency>
+ <dependency>
+ <groupId>com.google.inject</groupId>
+ <artifactId>guice</artifactId>
+ <version>6.0.0</version>
+ <scope>test</scope>
</dependency>
-
<dependency>
<groupId>junit</groupId>
<artifactId>junit</artifactId>
@@ -115,17 +122,8 @@
</pluginManagement>
<plugins>
<plugin>
- <groupId>org.codehaus.plexus</groupId>
- <artifactId>plexus-component-metadata</artifactId>
- <version>2.1.1</version>
- <executions>
- <execution>
- <id>create-component-descriptor</id>
- <goals>
- <goal>generate-metadata</goal>
- </goals>
- </execution>
- </executions>
+ <groupId>org.eclipse.sisu</groupId>
+ <artifactId>sisu-maven-plugin</artifactId>
</plugin>
</plugins>
</build>
diff --git
a/src/main/java/org/apache/maven/shared/jarsigner/DefaultJarSigner.java
b/src/main/java/org/apache/maven/shared/jarsigner/DefaultJarSigner.java
index 991e03c..fdefc83 100644
--- a/src/main/java/org/apache/maven/shared/jarsigner/DefaultJarSigner.java
+++ b/src/main/java/org/apache/maven/shared/jarsigner/DefaultJarSigner.java
@@ -18,11 +18,12 @@
*/
package org.apache.maven.shared.jarsigner;
+import javax.inject.Named;
+
import org.apache.maven.shared.utils.cli.Commandline;
import org.apache.maven.shared.utils.cli.StreamConsumer;
import org.apache.maven.shared.utils.cli.javatool.AbstractJavaTool;
import org.apache.maven.shared.utils.cli.javatool.JavaToolException;
-import org.codehaus.plexus.component.annotations.Component;
import org.slf4j.LoggerFactory;
/**
@@ -31,7 +32,7 @@ import org.slf4j.LoggerFactory;
* @author Tony Chemit
* @since 1.0
*/
-@Component(role = JarSigner.class, hint = "default")
+@Named
public class DefaultJarSigner extends AbstractJavaTool<JarSignerRequest>
implements JarSigner {
public DefaultJarSigner() {
diff --git
a/src/test/java/org/apache/maven/shared/jarsigner/AbstractJarSignerTest.java
b/src/test/java/org/apache/maven/shared/jarsigner/AbstractJarSignerTest.java
index c8032a9..412afcb 100644
--- a/src/test/java/org/apache/maven/shared/jarsigner/AbstractJarSignerTest.java
+++ b/src/test/java/org/apache/maven/shared/jarsigner/AbstractJarSignerTest.java
@@ -22,9 +22,9 @@ import java.io.File;
import java.io.IOException;
import org.apache.maven.shared.utils.io.FileUtils;
-import org.codehaus.plexus.PlexusTestCase;
+import org.eclipse.sisu.launch.InjectedTest;
-public abstract class AbstractJarSignerTest extends PlexusTestCase {
+public abstract class AbstractJarSignerTest extends InjectedTest {
protected File prepareTestJar(String filename) throws IOException {
File file = new File("src/test", filename);
diff --git
a/src/test/java/org/apache/maven/shared/jarsigner/JarSignerUtilTest.java
b/src/test/java/org/apache/maven/shared/jarsigner/JarSignerUtilTest.java
index 2004c46..4df793f 100644
--- a/src/test/java/org/apache/maven/shared/jarsigner/JarSignerUtilTest.java
+++ b/src/test/java/org/apache/maven/shared/jarsigner/JarSignerUtilTest.java
@@ -25,6 +25,9 @@ import java.util.jar.Attributes;
import java.util.jar.JarFile;
import java.util.jar.Manifest;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+
/**
* Created on 11/8/13.
*
diff --git
a/src/test/java/org/apache/maven/shared/jarsigner/SimpleJarSignTest.java
b/src/test/java/org/apache/maven/shared/jarsigner/SimpleJarSignTest.java
index 122837d..bca1f01 100644
--- a/src/test/java/org/apache/maven/shared/jarsigner/SimpleJarSignTest.java
+++ b/src/test/java/org/apache/maven/shared/jarsigner/SimpleJarSignTest.java
@@ -18,22 +18,23 @@
*/
package org.apache.maven.shared.jarsigner;
+import javax.inject.Inject;
+
import java.io.File;
import org.apache.maven.shared.utils.cli.javatool.JavaToolResult;
+import org.junit.Test;
+
+import static org.junit.Assert.assertEquals;
/**
* @author Olivier Lamy
*/
public class SimpleJarSignTest extends AbstractJarSignerTest {
+ @Inject
private JarSigner jarSigner;
- public void setUp() throws Exception {
- super.setUp();
-
- jarSigner = (JarSigner) lookup(JarSigner.class.getName());
- }
-
+ @Test
public void testSimpleSign() throws Exception {
File target = prepareTestJar("simple.jar");
@@ -51,6 +52,7 @@ public class SimpleJarSignTest extends AbstractJarSignerTest {
assertEquals("not exit code 0 " +
jarSignerResult.getExecutionException(), 0, jarSignerResult.getExitCode());
}
+ @Test
public void testSimpleSignAndVerify() throws Exception {
testSimpleSign();
