You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by mi...@apache.org on 2022/08/07 13:07:09 UTC
[maven-project-info-reports-plugin] 01/01: [MPIR-424] Upgrade Maven Reporting API to 3.1.1/Maven Reporting Impl to 3.2.0
This is an automated email from the ASF dual-hosted git repository.
michaelo pushed a commit to branch MPIR-424
in repository https://gitbox.apache.org/repos/asf/maven-project-info-reports-plugin.git
commit 53561fb452d570a69f35f2d4f3a1e4f2a01b1e9c
Author: Michael Osipov <mi...@apache.org>
AuthorDate: Sun Aug 7 15:06:56 2022 +0200
[MPIR-424] Upgrade Maven Reporting API to 3.1.1/Maven Reporting Impl to 3.2.0
---
pom.xml | 6 +++---
.../projectinfo/AbstractProjectInfoReport.java | 23 ----------------------
.../projectinfo/AbstractProjectInfoTestCase.java | 9 ++++-----
.../report/projectinfo/CiManagementReportTest.java | 3 ++-
.../report/projectinfo/DependenciesReportTest.java | 3 ++-
.../DependencyConvergenceReportTest.java | 3 ++-
.../DependencyManagementReportTest.java | 3 ++-
.../maven/report/projectinfo/IndexReportTest.java | 3 ++-
.../projectinfo/IssueManagementReportTest.java | 3 ++-
.../report/projectinfo/LicensesReportTest.java | 6 ++++--
.../report/projectinfo/MailingListsReportTest.java | 3 ++-
.../report/projectinfo/ModulesReportTest.java | 3 ++-
.../projectinfo/PluginManagementReportTest.java | 6 ++++--
.../maven/report/projectinfo/ScmReportTest.java | 3 ++-
.../report/projectinfo/SummaryReportTest.java | 3 ++-
.../maven/report/projectinfo/TeamReportTest.java | 3 ++-
16 files changed, 37 insertions(+), 46 deletions(-)
diff --git a/pom.xml b/pom.xml
index 0220143..6ea3735 100644
--- a/pom.xml
+++ b/pom.xml
@@ -124,7 +124,7 @@ under the License.
<mavenVersion>3.2.5</mavenVersion>
<scmVersion>1.13.0</scmVersion>
<aetherVersion>1.0.0.v20140518</aetherVersion>
- <sitePluginVersion>3.12.0</sitePluginVersion>
+ <sitePluginVersion>3.12.1</sitePluginVersion>
<fluidoSkinVersion>1.11.1</fluidoSkinVersion>
<surefire.version>2.22.2</surefire.version>
<checkstyle.violation.ignore>ParameterNumber,MethodLength</checkstyle.violation.ignore>
@@ -135,12 +135,12 @@ under the License.
<dependency>
<groupId>org.apache.maven.reporting</groupId>
<artifactId>maven-reporting-api</artifactId>
- <version>3.1.0</version>
+ <version>3.1.1</version>
</dependency>
<dependency>
<groupId>org.apache.maven.reporting</groupId>
<artifactId>maven-reporting-impl</artifactId>
- <version>3.1.0</version>
+ <version>3.2.0</version>
<exclusions>
<exclusion>
<groupId>org.apache.maven</groupId>
diff --git a/src/main/java/org/apache/maven/report/projectinfo/AbstractProjectInfoReport.java b/src/main/java/org/apache/maven/report/projectinfo/AbstractProjectInfoReport.java
index dd3d591..4d6f9c9 100644
--- a/src/main/java/org/apache/maven/report/projectinfo/AbstractProjectInfoReport.java
+++ b/src/main/java/org/apache/maven/report/projectinfo/AbstractProjectInfoReport.java
@@ -34,7 +34,6 @@ import java.util.MissingResourceException;
import java.util.ResourceBundle;
import org.apache.maven.artifact.repository.ArtifactRepository;
-import org.apache.maven.doxia.tools.SiteTool;
import org.apache.maven.execution.MavenSession;
import org.apache.maven.model.Plugin;
import org.apache.maven.plugins.annotations.Component;
@@ -67,14 +66,6 @@ public abstract class AbstractProjectInfoReport
// Mojo components
// ----------------------------------------------------------------------
- /**
- * SiteTool component.
- *
- * @since 2.1
- */
- @Component
- protected SiteTool siteTool;
-
/**
* Artifact Resolver component.
*/
@@ -103,20 +94,6 @@ public abstract class AbstractProjectInfoReport
@Parameter( defaultValue = "${session}", readonly = true, required = true )
private MavenSession session;
- /**
- * Local Repository.
- */
- @Parameter( property = "localRepository", required = true, readonly = true )
- protected ArtifactRepository localRepository;
-
- /**
- * Remote repositories used for the project.
- *
- * @since 2.1
- */
- @Parameter( property = "project.remoteArtifactRepositories" )
- protected List<ArtifactRepository> remoteRepositories;
-
/**
* Plugin repositories used for the project.
*
diff --git a/src/test/java/org/apache/maven/report/projectinfo/AbstractProjectInfoTestCase.java b/src/test/java/org/apache/maven/report/projectinfo/AbstractProjectInfoTestCase.java
index d445b70..4ae02ca 100644
--- a/src/test/java/org/apache/maven/report/projectinfo/AbstractProjectInfoTestCase.java
+++ b/src/test/java/org/apache/maven/report/projectinfo/AbstractProjectInfoTestCase.java
@@ -49,7 +49,7 @@ public abstract class AbstractProjectInfoTestCase
extends AbstractMojoTestCase
{
private ArtifactStubFactory artifactStubFactory;
-
+
/**
* The default locale is English.
*/
@@ -77,7 +77,7 @@ public abstract class AbstractProjectInfoTestCase
artifactStubFactory = new DependencyArtifactStubFactory( getTestFile( "target" ), true, false );
artifactStubFactory.getWorkingDir().mkdirs();
-
+
// Set the default Locale
Locale.setDefault( DEFAULT_LOCALE );
}
@@ -180,7 +180,7 @@ public abstract class AbstractProjectInfoTestCase
{
AbstractProjectInfoReport mojo = (AbstractProjectInfoReport) lookupMojo( goal, pluginXmlFile );
assertNotNull( "Mojo found.", mojo );
-
+
LegacySupport legacySupport = lookup( LegacySupport.class );
legacySupport.setSession( newMavenSession( new MavenProjectStub() ) );
DefaultRepositorySystemSession repoSession =
@@ -199,11 +199,10 @@ public abstract class AbstractProjectInfoTestCase
mojo.execute();
ProjectBuilder builder = lookup( ProjectBuilder.class );
-
+
ProjectBuildingRequest buildingRequest = new DefaultProjectBuildingRequest();
buildingRequest.setRepositorySession( lookup( LegacySupport.class ).getRepositorySession() );
- assertNotNull( "Local repository", mojo.localRepository );
testMavenProject = builder.build( pluginXmlFile, buildingRequest ).getProject();
File outputDir = mojo.getReportOutputDirectory();
diff --git a/src/test/java/org/apache/maven/report/projectinfo/CiManagementReportTest.java b/src/test/java/org/apache/maven/report/projectinfo/CiManagementReportTest.java
index 7ba4c92..027c328 100644
--- a/src/test/java/org/apache/maven/report/projectinfo/CiManagementReportTest.java
+++ b/src/test/java/org/apache/maven/report/projectinfo/CiManagementReportTest.java
@@ -64,7 +64,8 @@ public class CiManagementReportTest
assertTrue( response.getContentLength() > 0 );
// Test the Page title
- String expectedTitle = getString( "report.ci-management.title" );
+ String expectedTitle = prepareTitle( "ci mangement project info",
+ getString( "report.ci-management.title" ) );
assertEquals( expectedTitle, response.getTitle() );
// Test the texts
diff --git a/src/test/java/org/apache/maven/report/projectinfo/DependenciesReportTest.java b/src/test/java/org/apache/maven/report/projectinfo/DependenciesReportTest.java
index edb2350..9f2f0ad 100644
--- a/src/test/java/org/apache/maven/report/projectinfo/DependenciesReportTest.java
+++ b/src/test/java/org/apache/maven/report/projectinfo/DependenciesReportTest.java
@@ -64,7 +64,8 @@ public class DependenciesReportTest
assertTrue( response.getContentLength() > 0 );
// Test the Page title
- String expectedTitle = getString( "report.dependencies.title" );
+ String expectedTitle = prepareTitle( "dependencies project info",
+ getString( "report.dependencies.title" ) );
assertEquals( expectedTitle, response.getTitle() );
// Test the tables
diff --git a/src/test/java/org/apache/maven/report/projectinfo/DependencyConvergenceReportTest.java b/src/test/java/org/apache/maven/report/projectinfo/DependencyConvergenceReportTest.java
index 4d9c0d4..91768f2 100644
--- a/src/test/java/org/apache/maven/report/projectinfo/DependencyConvergenceReportTest.java
+++ b/src/test/java/org/apache/maven/report/projectinfo/DependencyConvergenceReportTest.java
@@ -63,7 +63,8 @@ public class DependencyConvergenceReportTest
assertTrue( response.getContentLength() > 0 );
// Test the Page title
- String expectedTitle = getString( "report.dependency-convergence.reactor.title" );
+ String expectedTitle = prepareTitle( "dependency convergence project info",
+ getString( "report.dependency-convergence.reactor.title" ) );
assertEquals( expectedTitle, response.getTitle() );
// Test the texts
diff --git a/src/test/java/org/apache/maven/report/projectinfo/DependencyManagementReportTest.java b/src/test/java/org/apache/maven/report/projectinfo/DependencyManagementReportTest.java
index 8870f88..143cb2d 100644
--- a/src/test/java/org/apache/maven/report/projectinfo/DependencyManagementReportTest.java
+++ b/src/test/java/org/apache/maven/report/projectinfo/DependencyManagementReportTest.java
@@ -64,7 +64,8 @@ public class DependencyManagementReportTest
assertTrue( response.getContentLength() > 0 );
// Test the Page title
- String expectedTitle = getString( "report.dependency-management.title" );
+ String expectedTitle = prepareTitle( "dependency management project info",
+ getString( "report.dependency-management.title" ) );
assertEquals( expectedTitle, response.getTitle() );
// Test the tables
diff --git a/src/test/java/org/apache/maven/report/projectinfo/IndexReportTest.java b/src/test/java/org/apache/maven/report/projectinfo/IndexReportTest.java
index d202be8..ccd6e37 100644
--- a/src/test/java/org/apache/maven/report/projectinfo/IndexReportTest.java
+++ b/src/test/java/org/apache/maven/report/projectinfo/IndexReportTest.java
@@ -64,7 +64,8 @@ public class IndexReportTest
// Test the Page title
// Index does not have a 'name' but 'title' only
- String expectedTitle = getString( "report.index.title" );
+ String expectedTitle = prepareTitle( "index project info",
+ getString( "report.index.title" ) );
assertEquals( expectedTitle, response.getTitle() );
// Test the texts
diff --git a/src/test/java/org/apache/maven/report/projectinfo/IssueManagementReportTest.java b/src/test/java/org/apache/maven/report/projectinfo/IssueManagementReportTest.java
index 3d61cf5..0f235bf 100644
--- a/src/test/java/org/apache/maven/report/projectinfo/IssueManagementReportTest.java
+++ b/src/test/java/org/apache/maven/report/projectinfo/IssueManagementReportTest.java
@@ -64,7 +64,8 @@ public class IssueManagementReportTest
assertTrue( response.getContentLength() > 0 );
// Test the Page title
- String expectedTitle = getString( "report.issue-management.title" );
+ String expectedTitle = prepareTitle( "issue management project info",
+ getString( "report.issue-management.title" ) );
assertEquals( expectedTitle, response.getTitle() );
// Test the links
diff --git a/src/test/java/org/apache/maven/report/projectinfo/LicensesReportTest.java b/src/test/java/org/apache/maven/report/projectinfo/LicensesReportTest.java
index 6facb28..95fdf52 100644
--- a/src/test/java/org/apache/maven/report/projectinfo/LicensesReportTest.java
+++ b/src/test/java/org/apache/maven/report/projectinfo/LicensesReportTest.java
@@ -64,7 +64,8 @@ public class LicensesReportTest
assertTrue( response.getContentLength() > 0 );
// Test the Page title
- String expectedTitle = getString( "report.licenses.title" );
+ String expectedTitle = prepareTitle( "licenses project info",
+ getString( "report.licenses.title" ) );
assertEquals( expectedTitle, response.getTitle() );
// Test the texts
@@ -98,7 +99,8 @@ public class LicensesReportTest
assertTrue( response.getContentLength() > 0 );
// Test the Page title
- String expectedTitle = getString( "report.licenses.title" );
+ String expectedTitle = prepareTitle( "licenses project info",
+ getString( "report.licenses.title" ) );
assertEquals( expectedTitle, response.getTitle() );
// Test the texts
diff --git a/src/test/java/org/apache/maven/report/projectinfo/MailingListsReportTest.java b/src/test/java/org/apache/maven/report/projectinfo/MailingListsReportTest.java
index aba655b..2a96e15 100644
--- a/src/test/java/org/apache/maven/report/projectinfo/MailingListsReportTest.java
+++ b/src/test/java/org/apache/maven/report/projectinfo/MailingListsReportTest.java
@@ -65,7 +65,8 @@ public class MailingListsReportTest
assertTrue( response.getContentLength() > 0 );
// Test the Page title
- String expectedTitle = getString( "report.mailing-lists.title" );
+ String expectedTitle = prepareTitle( "mailing lists project info",
+ getString( "report.mailing-lists.title" ) );
assertEquals( expectedTitle, response.getTitle() );
// Test the texts
diff --git a/src/test/java/org/apache/maven/report/projectinfo/ModulesReportTest.java b/src/test/java/org/apache/maven/report/projectinfo/ModulesReportTest.java
index ceba692..7a1463a 100644
--- a/src/test/java/org/apache/maven/report/projectinfo/ModulesReportTest.java
+++ b/src/test/java/org/apache/maven/report/projectinfo/ModulesReportTest.java
@@ -81,7 +81,8 @@ public class ModulesReportTest
assertTrue( response.getContentLength() > 0 );
// Test the Page title
- String expectedTitle = getString( "report.modules.title" );
+ String expectedTitle = prepareTitle( "modules project info",
+ getString( "report.modules.title" ) );
assertEquals( expectedTitle, response.getTitle() );
// Test the texts
diff --git a/src/test/java/org/apache/maven/report/projectinfo/PluginManagementReportTest.java b/src/test/java/org/apache/maven/report/projectinfo/PluginManagementReportTest.java
index 362b8bb..68cd90e 100644
--- a/src/test/java/org/apache/maven/report/projectinfo/PluginManagementReportTest.java
+++ b/src/test/java/org/apache/maven/report/projectinfo/PluginManagementReportTest.java
@@ -101,7 +101,8 @@ public class PluginManagementReportTest
assertTrue( response.getContentLength() > 0 );
// Test the Page title
- String expectedTitle = getString( "report.plugin-management.title" );
+ String expectedTitle = prepareTitle( "plugin management project info",
+ getString( "report.plugin-management.title" ) );
assertEquals( expectedTitle, response.getTitle() );
// Test the tables
@@ -139,7 +140,8 @@ public class PluginManagementReportTest
assertTrue( response.getContentLength() > 0 );
// Test the Page title
- String expectedTitle = getString( "report.plugin-management.title" );
+ String expectedTitle = prepareTitle( "plugin management project info",
+ getString( "report.plugin-management.title" ) );
assertEquals( expectedTitle, response.getTitle() );
// Test the tables
diff --git a/src/test/java/org/apache/maven/report/projectinfo/ScmReportTest.java b/src/test/java/org/apache/maven/report/projectinfo/ScmReportTest.java
index 170d027..6f23220 100644
--- a/src/test/java/org/apache/maven/report/projectinfo/ScmReportTest.java
+++ b/src/test/java/org/apache/maven/report/projectinfo/ScmReportTest.java
@@ -66,7 +66,8 @@ public class ScmReportTest
assertTrue( response.getContentLength() > 0 );
// Test the Page title
- String expectedTitle = getString( "report.scm.title" );
+ String expectedTitle = prepareTitle( "scm project info",
+ getString( "report.scm.title" ) );
assertEquals( expectedTitle, response.getTitle() );
// Test the texts
diff --git a/src/test/java/org/apache/maven/report/projectinfo/SummaryReportTest.java b/src/test/java/org/apache/maven/report/projectinfo/SummaryReportTest.java
index 36e51ff..a716488 100644
--- a/src/test/java/org/apache/maven/report/projectinfo/SummaryReportTest.java
+++ b/src/test/java/org/apache/maven/report/projectinfo/SummaryReportTest.java
@@ -63,7 +63,8 @@ public class SummaryReportTest
assertTrue( response.getContentLength() > 0 );
// Test the Page title
- String expectedTitle = getString( "report.summary.title" );
+ String expectedTitle = prepareTitle( "summary project info",
+ getString( "report.summary.title" ) );
assertEquals( expectedTitle, response.getTitle() );
// Test the texts
diff --git a/src/test/java/org/apache/maven/report/projectinfo/TeamReportTest.java b/src/test/java/org/apache/maven/report/projectinfo/TeamReportTest.java
index 21bb1a5..82d2c4d 100644
--- a/src/test/java/org/apache/maven/report/projectinfo/TeamReportTest.java
+++ b/src/test/java/org/apache/maven/report/projectinfo/TeamReportTest.java
@@ -70,7 +70,8 @@ public class TeamReportTest
assertTrue( response.getContentLength() > 0 );
// Test the Page title
- String expectedTitle = getString( "report.team.title" );
+ String expectedTitle = prepareTitle( "team project info",
+ getString( "report.team.title" ) );
assertEquals( expectedTitle, response.getTitle() );
assertTrue( response.getText().contains( "gravatar" ));