Skip to content

Commit d27d8dc

Browse files
gnodetgnodet
committed
Upgrade to Maven 4.0.0-alpha-9
1 parent 347d946 commit d27d8dc

File tree

22 files changed

+351
-243
lines changed

22 files changed

+351
-243
lines changed

‎pom.xml‎

Lines changed: 5 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ under the License.
6969

7070
<properties>
7171
<javaVersion>8</javaVersion>
72-
<mavenVersion>4.0.0-alpha-8</mavenVersion>
72+
<mavenVersion>4.0.0-alpha-9-SNAPSHOT</mavenVersion>
7373
<!-- Keep in sync with resolver used in maven above -->
7474
<slf4jVersion>1.7.5</slf4jVersion>
7575
<!-- Keep in sync with resolver used in maven above -->
@@ -84,7 +84,8 @@ under the License.
8484
<mavenInstallPluginVersion>3.1.0</mavenInstallPluginVersion>
8585
<mavenJarPluginVersion>3.3.0</mavenJarPluginVersion>
8686
<mavenJavadocPluginVersion>3.4.1</mavenJavadocPluginVersion>
87-
<mavenPluginToolsVersion>3.9.0</mavenPluginToolsVersion>
87+
<mavenPluginPluginVersion>3.10.3-SNAPSHOT</mavenPluginPluginVersion>
88+
<mavenPluginTestingHarnessVersion>4.0.0-alpha-3-SNAPSHOT</mavenPluginTestingHarnessVersion>
8889
<mavenResourcesPluginVersion>3.3.0</mavenResourcesPluginVersion>
8990
<mavenSourcePluginVersion>3.2.1</mavenSourcePluginVersion>
9091
<mavenSurefirePluginVersion>${surefire.version}</mavenSurefirePluginVersion>
@@ -107,7 +108,7 @@ under the License.
107108
<dependency>
108109
<groupId>org.apache.maven.plugin-testing</groupId>
109110
<artifactId>maven-plugin-testing-harness</artifactId>
110-
<version>4.0.0-alpha-2</version>
111+
<version>${mavenPluginTestingHarnessVersion}</version>
111112
<scope>test</scope>
112113
</dependency>
113114
<dependency>
@@ -123,30 +124,6 @@ under the License.
123124
<version>${mavenVersion}</version>
124125
<scope>test</scope>
125126
</dependency>
126-
<!-- <dependency>-->
127-
<!-- <groupId>org.apache.maven.resolver</groupId>-->
128-
<!-- <artifactId>maven-resolver-connector-basic</artifactId>-->
129-
<!-- <version>${resolverVersion}</version>-->
130-
<!-- <scope>test</scope>-->
131-
<!-- </dependency>-->
132-
<!-- <dependency>-->
133-
<!-- <groupId>org.apache.maven.resolver</groupId>-->
134-
<!-- <artifactId>maven-resolver-transport-file</artifactId>-->
135-
<!-- <version>${resolverVersion}</version>-->
136-
<!-- <scope>test</scope>-->
137-
<!-- </dependency>-->
138-
<!-- <dependency>-->
139-
<!-- <groupId>org.apache.maven.resolver</groupId>-->
140-
<!-- <artifactId>maven-resolver-transport-http</artifactId>-->
141-
<!-- <version>${resolverVersion}</version>-->
142-
<!-- <scope>test</scope>-->
143-
<!-- </dependency>-->
144-
<!-- <dependency>-->
145-
<!-- <groupId>org.mockito</groupId>-->
146-
<!-- <artifactId>mockito-core</artifactId>-->
147-
<!-- <version>2.28.2</version>-->
148-
<!-- <scope>test</scope>-->
149-
<!-- </dependency>-->
150127
<dependency>
151128
<groupId>org.mockito</groupId>
152129
<artifactId>mockito-junit-jupiter</artifactId>
@@ -167,7 +144,7 @@ under the License.
167144
<plugin>
168145
<groupId>org.apache.maven.plugins</groupId>
169146
<artifactId>maven-plugin-plugin</artifactId>
170-
<version>${mavenPluginToolsVersion}</version>
147+
<version>${mavenPluginPluginVersion}</version>
171148
</plugin>
172149
<plugin>
173150
<groupId>com.diffplug.spotless</groupId>

‎src/it/deploy-attached-sources/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ under the License.
7777
<version>@project.version@</version>
7878
<configuration>
7979
<!-- Deployments will be written to ${basedir}/target -->
80-
<altDeploymentRepository>mine::default::file://${basedir}/target</altDeploymentRepository>
80+
<altDeploymentRepository>mine::default::file://${project.basedir}/target</altDeploymentRepository>
8181
</configuration>
8282
</plugin>
8383
</plugins>

‎src/it/deploy-attached-sources/test.properties‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,10 @@
1616
# under the License.
1717

1818
# Properties passed to invocations of the deploy plugin
19-
pomFile = ${basedir}/pom.xml
19+
pomFile = ${project.basedir}/pom.xml
2020

21-
file = ${basedir}/target/${project.artifactId}-${project.version}.jar
22-
sources = ${basedir}/target/${project.artifactId}-${project.version}-sources.jar
23-
javadoc = ${basedir}/target/${project.artifactId}-${project.version}-javadoc.jar
21+
file = ${project.basedir}/target/${project.artifactId}-${project.version}.jar
22+
sources = ${project.basedir}/target/${project.artifactId}-${project.version}-sources.jar
23+
javadoc = ${project.basedir}/target/${project.artifactId}-${project.version}-javadoc.jar
2424

25-
url = file://${basedir}/target/repo
25+
url = file://${project.basedir}/target/repo

‎src/it/deploy-default-packaging/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ under the License.
7777
<version>@project.version@</version>
7878
<configuration>
7979
<!-- Deployments will be written to ${basedir}/target -->
80-
<altDeploymentRepository>mine::default::file://${basedir}/target</altDeploymentRepository>
80+
<altDeploymentRepository>mine::default::file://${project.basedir}/target</altDeploymentRepository>
8181
</configuration>
8282
</plugin>
8383
</plugins>

‎src/it/deploy-default-packaging/test.properties‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,5 +19,5 @@
1919
groupId = org.apache.maven.test
2020
artifactId = test
2121
version = 1.1
22-
file = ${basedir}/lib/test-1.1.jar
23-
url = file://${basedir}/target/repo
22+
file = ${project.basedir}/lib/test-1.1.jar
23+
url = file://${project.basedir}/target/repo

‎src/it/setup-mock-phase-maven-plugin/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ under the License.
4545
<plugin>
4646
<groupId>org.apache.maven.plugins</groupId>
4747
<artifactId>maven-plugin-plugin</artifactId>
48-
<version>@mavenPluginToolsVersion@</version>
48+
<version>@mavenPluginPluginVersion@</version>
4949
<configuration>
5050
<skipErrorNoDescriptorsFound>true</skipErrorNoDescriptorsFound>
5151
</configuration>

‎src/main/java/org/apache/maven/plugins/deploy/AbstractDeployMojo.java‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,13 +18,13 @@
1818
*/
1919
package org.apache.maven.plugins.deploy;
2020

21+
import jakarta.inject.Inject;
2122
import org.apache.maven.api.RemoteRepository;
2223
import org.apache.maven.api.Session;
2324
import org.apache.maven.api.Version;
2425
import org.apache.maven.api.plugin.Log;
2526
import org.apache.maven.api.plugin.Mojo;
2627
import org.apache.maven.api.plugin.MojoException;
27-
import org.apache.maven.api.plugin.annotations.Component;
2828
import org.apache.maven.api.plugin.annotations.Parameter;
2929
import org.apache.maven.api.services.VersionParser;
3030

@@ -36,10 +36,10 @@ public abstract class AbstractDeployMojo implements Mojo {
3636

3737
private static final String FIXED_MAVEN_VERSION = "3.9.0";
3838

39-
@Component
39+
@Inject
4040
protected Log logger;
4141

42-
@Component
42+
@Inject
4343
protected Session session;
4444

4545
/**

‎src/main/java/org/apache/maven/plugins/deploy/DeployFileMojo.java‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@
5252
*
5353
* @author <a href="mailto:aramirez@apache.org">Allan Ramirez</a>
5454
*/
55-
@Mojo(name = "deploy-file", requiresProject = false)
55+
@Mojo(name = "deploy-file", projectRequired = false)
5656
@SuppressWarnings("unused")
5757
public class DeployFileMojo extends AbstractDeployMojo {
5858
private static final String TAR = "tar.";

‎src/main/java/org/apache/maven/plugins/deploy/DeployMojo.java‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
import java.util.regex.Matcher;
2626
import java.util.regex.Pattern;
2727

28+
import jakarta.inject.Inject;
2829
import org.apache.maven.api.Artifact;
2930
import org.apache.maven.api.MojoExecution;
3031
import org.apache.maven.api.Project;
@@ -33,7 +34,6 @@
3334
import org.apache.maven.api.model.Plugin;
3435
import org.apache.maven.api.model.PluginExecution;
3536
import org.apache.maven.api.plugin.MojoException;
36-
import org.apache.maven.api.plugin.annotations.Component;
3737
import org.apache.maven.api.plugin.annotations.LifecyclePhase;
3838
import org.apache.maven.api.plugin.annotations.Mojo;
3939
import org.apache.maven.api.plugin.annotations.Parameter;
@@ -55,10 +55,10 @@ public class DeployMojo extends AbstractDeployMojo {
5555

5656
private static final Pattern ALT_REPO_SYNTAX_PATTERN = Pattern.compile("(.+?)::(.+)");
5757

58-
@Component
58+
@Inject
5959
private Project project;
6060

61-
@Component
61+
@Inject
6262
private MojoExecution mojoExecution;
6363

6464
/**
@@ -194,7 +194,7 @@ private boolean allProjectsMarked() {
194194
}
195195

196196
private boolean hasDeployExecution(Project p) {
197-
String key = mojoExecution.getPlugin().getKey();
197+
String key = mojoExecution.getPlugin().getModel().getKey();
198198
Plugin plugin = p.getBuild().getPluginsAsMap().get(key);
199199
if (plugin != null) {
200200
for (PluginExecution execution : plugin.getExecutions()) {

‎src/test/java/org/apache/maven/plugins/deploy/DeployFileMojoTest.java‎

Lines changed: 51 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
import java.io.File;
2222
import java.nio.file.Files;
2323
import java.nio.file.Path;
24+
import java.nio.file.Paths;
2425
import java.util.ArrayList;
2526
import java.util.List;
2627
import java.util.concurrent.atomic.AtomicReference;
@@ -33,6 +34,8 @@
3334
import org.apache.maven.api.Session;
3435
import org.apache.maven.api.model.Model;
3536
import org.apache.maven.api.plugin.testing.InjectMojo;
37+
import org.apache.maven.api.plugin.testing.MojoExtension;
38+
import org.apache.maven.api.plugin.testing.MojoParameter;
3639
import org.apache.maven.api.plugin.testing.MojoTest;
3740
import org.apache.maven.api.plugin.testing.stubs.ArtifactStub;
3841
import org.apache.maven.api.plugin.testing.stubs.SessionStub;
@@ -41,6 +44,7 @@
4144
import org.apache.maven.api.services.ArtifactManager;
4245
import org.apache.maven.api.services.ProjectBuilder;
4346
import org.apache.maven.api.services.ProjectManager;
47+
import org.apache.maven.internal.impl.InternalSession;
4448
import org.junit.jupiter.api.Test;
4549

4650
import static org.apache.maven.api.plugin.testing.MojoExtension.getVariableValueFromObject;
@@ -50,6 +54,7 @@
5054
import static org.junit.jupiter.api.Assertions.assertTrue;
5155
import static org.mockito.ArgumentMatchers.any;
5256
import static org.mockito.Mockito.doAnswer;
57+
import static org.mockito.Mockito.when;
5358

5459
/**
5560
* @author <a href="mailto:aramirez@apache.org">Allan Ramirez</a>
@@ -71,13 +76,25 @@ public class DeployFileMojoTest {
7176
private ArtifactManager artifactManager;
7277

7378
@Test
74-
@InjectMojo(goal = "deploy-file", pom = "classpath:/unit/deploy-file/plugin-config-test.xml")
79+
@InjectMojo(goal = "deploy-file")
7580
public void testDeployTestEnvironment(DeployFileMojo mojo) {
7681
assertNotNull(mojo);
7782
}
7883

7984
@Test
80-
@InjectMojo(goal = "deploy-file", pom = "classpath:/unit/deploy-file/plugin-config-test.xml")
85+
@InjectMojo(goal = "deploy-file")
86+
@MojoParameter(name = "groupId", value = "org.apache.maven.test")
87+
@MojoParameter(name = "artifactId", value = "maven-deploy-file-test")
88+
@MojoParameter(name = "version", value = "1.0")
89+
@MojoParameter(name = "packaging", value = "jar")
90+
@MojoParameter(
91+
name = "file",
92+
value = "${session.topDirectory}/src/test/resources/unit/maven-deploy-test-1.0-SNAPSHOT.jar")
93+
@MojoParameter(name = "repositoryId", value = "deploy-test")
94+
@MojoParameter(name = "url", value = "file://${session.topDirectory}/target/remote-repo/deploy-file")
95+
@MojoParameter(name = "description", value = "POM was created from deploy:deploy-file")
96+
@MojoParameter(name = "generatePom", value = "true")
97+
@MojoParameter(name = "skip", value = "snapshots")
8198
public void testBasicDeployFile(DeployFileMojo mojo) throws Exception {
8299
assertNotNull(mojo);
83100

@@ -93,9 +110,9 @@ public void testBasicDeployFile(DeployFileMojo mojo) throws Exception {
93110
assertEquals("maven-deploy-file-test", artifactId);
94111
assertEquals("1.0", version);
95112
assertEquals("jar", packaging);
96-
assertTrue(Files.exists(file));
113+
assertTrue(Files.exists(file), file.toString());
97114
assertEquals("deploy-test", repositoryId);
98-
assertEquals("file://" + getBasedir() + "/target/remote-repo/deploy-file-test", url);
115+
assertEquals("file://" + getBasedir() + "/target/remote-repo/deploy-file", url);
99116

100117
execute(mojo, request -> {
101118
assertNotNull(request);
@@ -128,7 +145,18 @@ public void testBasicDeployFile(DeployFileMojo mojo) throws Exception {
128145
}
129146

130147
@Test
131-
@InjectMojo(goal = "deploy-file", pom = "classpath:/unit/deploy-file/plugin-config-classifier.xml")
148+
@InjectMojo(goal = "deploy-file")
149+
@MojoParameter(name = "groupId", value = "org.apache.maven.test")
150+
@MojoParameter(name = "artifactId", value = "maven-deploy-file-test")
151+
@MojoParameter(name = "version", value = "1.0")
152+
@MojoParameter(name = "packaging", value = "jar")
153+
@MojoParameter(
154+
name = "file",
155+
value = "${session.topDirectory}/src/test/resources/unit/maven-deploy-test-1.0-SNAPSHOT.jar")
156+
@MojoParameter(name = "repositoryId", value = "deploy-test")
157+
@MojoParameter(name = "url", value = "file://${session.topDirectory}/target/remote-repo/deploy-file")
158+
@MojoParameter(name = "classifier", value = "bin")
159+
@MojoParameter(name = "generatePom", value = "true")
132160
public void testDeployIfClassifierIsSet(DeployFileMojo mojo) throws Exception {
133161
assertNotNull(mojo);
134162

@@ -149,7 +177,7 @@ public void testDeployIfClassifierIsSet(DeployFileMojo mojo) throws Exception {
149177
assertEquals(new ArtifactStub(groupId, artifactId, "bin", version, "jar"), a1);
150178
Path p1 = artifactManager.getPath(a1).orElse(null);
151179
assertNotNull(p1);
152-
assertTrue(p1.toString().endsWith("deploy-test-file-1.0-SNAPSHOT.jar"));
180+
assertTrue(p1.toString().endsWith("maven-deploy-test-1.0-SNAPSHOT.jar"));
153181
// second artifact
154182
Artifact a2 = artifacts.get(1);
155183
assertEquals(new ArtifactStub(groupId, artifactId, "", version, "pom"), a2);
@@ -162,7 +190,14 @@ public void testDeployIfClassifierIsSet(DeployFileMojo mojo) throws Exception {
162190
}
163191

164192
@Test
165-
@InjectMojo(goal = "deploy-file", pom = "classpath:/unit/deploy-file/plugin-config-artifact-not-jar.xml")
193+
@InjectMojo(goal = "deploy-file")
194+
@MojoParameter(name = "groupId", value = "org.apache.maven.test")
195+
@MojoParameter(name = "artifactId", value = "maven-deploy-file-test")
196+
@MojoParameter(name = "version", value = "1.0")
197+
@MojoParameter(name = "file", value = "${session.topDirectory}/src/test/resources/unit/maven-deploy-test.zip")
198+
@MojoParameter(name = "repositoryId", value = "deploy-test")
199+
@MojoParameter(name = "url", value = "file://${session.topDirectory}/target/remote-repo/deploy-file")
200+
@MojoParameter(name = "generatePom", value = "true")
166201
public void testDeployIfArtifactIsNotJar(DeployFileMojo mojo) throws Exception {
167202
assertNotNull(mojo);
168203

@@ -183,7 +218,7 @@ public void testDeployIfArtifactIsNotJar(DeployFileMojo mojo) throws Exception {
183218
Path p1 = artifactManager.getPath(a1).orElse(null);
184219
Path p2 = artifactManager.getPath(a2).orElse(null);
185220
assertNotNull(p1);
186-
assertTrue(p1.toString().endsWith("deploy-test-file.zip"));
221+
assertTrue(p1.toString().endsWith("maven-deploy-test.zip"));
187222
assertNotNull(p2);
188223
assertTrue(p2.toString().endsWith(".pom"));
189224

@@ -213,31 +248,33 @@ private void execute(DeployFileMojo mojo, Consumer<ArtifactDeployerRequest> cons
213248
@Provides
214249
@Singleton
215250
@SuppressWarnings("unused")
216-
private Session getMockSession() {
217-
return SessionStub.getMockSession(LOCAL_REPO);
251+
private InternalSession createSession() {
252+
InternalSession session = SessionStub.getMockSession(LOCAL_REPO);
253+
when(session.getTopDirectory()).thenReturn(Paths.get(MojoExtension.getBasedir()));
254+
return session;
218255
}
219256

220257
@Provides
221258
@SuppressWarnings("unused")
222-
private ArtifactDeployer getMockArtifactDeployer(Session session) {
259+
private ArtifactDeployer createArtifactDeployer(Session session) {
223260
return session.getService(ArtifactDeployer.class);
224261
}
225262

226263
@Provides
227264
@SuppressWarnings("unused")
228-
private ArtifactManager getMockArtifactManager(Session session) {
265+
private ArtifactManager createArtifactManager(Session session) {
229266
return session.getService(ArtifactManager.class);
230267
}
231268

232269
@Provides
233270
@SuppressWarnings("unused")
234-
private ProjectManager getMockProjectManager(Session session) {
271+
private ProjectManager createProjectManager(Session session) {
235272
return session.getService(ProjectManager.class);
236273
}
237274

238275
@Provides
239276
@SuppressWarnings("unused")
240-
private ProjectBuilder getMockProjectBuilder(Session session) {
277+
private ProjectBuilder createProjectBuilder(Session session) {
241278
return session.getService(ProjectBuilder.class);
242279
}
243280
}

0 commit comments

Comments
 (0)