Skip to content

Commit 9e404b5

Browse files
authored
[MPLUGINTESTING-89] Upgrade Maven from 4.0.0-alpha-4 to 4.0.0-alpha-8 (#33)
1 parent 6861f06 commit 9e404b5

File tree

4 files changed

+28
-10
lines changed

4 files changed

+28
-10
lines changed

maven-plugin-testing-harness/src/main/java/org/apache/maven/api/plugin/testing/stubs/ProjectStub.java

Lines changed: 24 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,9 @@ public class ProjectStub implements Project {
4141
private Model model = Model.newInstance();
4242
private Path basedir;
4343
private File pomPath;
44-
private boolean executionRoot;
44+
private boolean topProject;
4545
private Artifact artifact;
46+
private Path rootDirectory;
4647

4748
public void setModel(Model model) {
4849
this.model = model;
@@ -117,7 +118,7 @@ public void setBasedir(Path basedir) {
117118

118119
@Override
119120
public boolean isExecutionRoot() {
120-
return executionRoot;
121+
return isTopProject();
121122
}
122123

123124
@Override
@@ -135,6 +136,21 @@ public List<RemoteRepository> getRemotePluginRepositories() {
135136
return Collections.emptyList();
136137
}
137138

139+
@Override
140+
public boolean isTopProject() {
141+
return topProject;
142+
}
143+
144+
@Override
145+
public boolean isRootProject() {
146+
return model.isRoot();
147+
}
148+
149+
@Override
150+
public Path getRootDirectory() {
151+
return rootDirectory;
152+
}
153+
138154
public void setGroupId(String groupId) {
139155
model = model.withGroupId(groupId);
140156
}
@@ -163,11 +179,15 @@ public void setPomPath(File pomPath) {
163179
this.pomPath = pomPath;
164180
}
165181

166-
public void setExecutionRoot(boolean executionRoot) {
167-
this.executionRoot = executionRoot;
182+
public void setTopProject(boolean topProject) {
183+
this.topProject = topProject;
168184
}
169185

170186
public void setMavenModel(org.apache.maven.model.Model model) {
171187
this.model = model.getDelegate();
172188
}
189+
190+
public void setRootDirectory(Path rootDirectory) {
191+
this.rootDirectory = rootDirectory;
192+
}
173193
}

maven-plugin-testing-harness/src/main/java/org/apache/maven/api/plugin/testing/stubs/SessionStub.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@
5050
import org.apache.maven.api.services.RepositoryFactory;
5151
import org.apache.maven.api.services.xml.ModelXmlFactory;
5252
import org.apache.maven.internal.impl.DefaultModelXmlFactory;
53-
import org.apache.maven.model.io.xpp3.MavenXpp3Reader;
53+
import org.apache.maven.model.v4.MavenStaxReader;
5454
import org.mockito.ArgumentMatchers;
5555

5656
import static org.mockito.ArgumentMatchers.any;
@@ -185,9 +185,7 @@ public static Session getMockSession(LocalRepository localRepository) {
185185
when(projectBuilder.build(any(ProjectBuilderRequest.class))).then(iom -> {
186186
ProjectBuilderRequest request = iom.getArgument(0, ProjectBuilderRequest.class);
187187
ProjectBuilderResult result = mock(ProjectBuilderResult.class);
188-
Model model = new MavenXpp3Reader()
189-
.read(request.getSource().get().getInputStream())
190-
.getDelegate();
188+
Model model = new MavenStaxReader().read(request.getSource().get().openStream());
191189
ProjectStub projectStub = new ProjectStub();
192190
projectStub.setModel(model);
193191
ArtifactStub artifactStub = new ArtifactStub(

maven-plugin-testing-harness/src/test/java/org/apache/maven/api/plugin/testing/ExpressionEvaluatorTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ Session session() {
8989
Session session = SessionStub.getMockSession(LOCAL_REPO);
9090
doReturn(new Properties()).when(session).getSystemProperties();
9191
doReturn(new Properties()).when(session).getUserProperties();
92-
doAnswer(iom -> Paths.get(MojoExtension.getBasedir())).when(session).getExecutionRootDirectory();
92+
doAnswer(iom -> Paths.get(MojoExtension.getBasedir())).when(session).getRootDirectory();
9393
return session;
9494
}
9595
}

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ under the License.
6565

6666
<properties>
6767
<surefire.version>3.2.1</surefire.version>
68-
<mavenVersion>4.0.0-alpha-4</mavenVersion>
68+
<mavenVersion>4.0.0-alpha-8</mavenVersion>
6969
<maven.site.path>plugin-testing-archives/LATEST</maven.site.path>
7070
<javaVersion>8</javaVersion>
7171
<project.build.outputTimestamp>2023-02-15T20:04:02Z</project.build.outputTimestamp>

0 commit comments

Comments
 (0)