This is an automated email from the ASF dual-hosted git repository. michaelo pushed a commit to branch doxia-2.0.0 in repository https://gitbox.apache.org/repos/asf/maven-surefire.git
commit db4458aa365fb0fd94796f176c93505cccfd8b60 Author: Michael Osipov <micha...@apache.org> AuthorDate: Tue Jan 2 15:23:55 2024 +0100 [SUREFIRE-2161] Align Mojo class names and output names --- .../plugin/surefire/AbstractSurefireMojo.java | 1 - .../{SurefirePlugin.java => SurefireMojo.java} | 2 +- ...refirePluginTest.java => SurefireMojoTest.java} | 52 +++++++++--------- ...ReportMojo.java => AbstractSurefireReport.java} | 2 +- ...safeReportMojo.java => FailsafeOnlyReport.java} | 4 +- ...ReportOnlyMojo.java => SurefireOnlyReport.java} | 2 +- ...SurefireReportMojo.java => SurefireReport.java} | 4 +- .../site/apt/examples/changing-report-name.apt.vm | 2 +- .../plugins/surefire/report/JUnit4SuiteTest.java | 2 +- ...ReportMojoTest.java => SurefireReportTest.java} | 61 ++++++++++------------ ...ub.java => SurefireReportMavenProjectStub.java} | 2 +- ...2.java => SurefireReportMavenProjectStub2.java} | 2 +- .../plugin-config.xml | 4 +- .../plugin-config.xml | 4 +- .../plugin-config.xml | 4 +- .../plugin-config.xml | 4 +- .../basic-surefire-report-test/plugin-config.xml | 4 +- .../resources/unit/surefire-1183/plugin-config.xml | 4 +- .../plugin-config.xml | 4 +- .../surefire-report-enclosed/plugin-config.xml | 4 +- .../plugin-config.xml | 4 +- .../surefire-report-nestedClass/plugin-config.xml | 4 +- .../surefire-report-single-error/plugin-config.xml | 4 +- .../maven/surefire/its/AggregateReportIT.java | 2 +- .../surefire/its/CheckTestNgReportTestIT.java | 2 +- .../Surefire1490ReportTitleDescriptionIT.java | 8 +-- .../jiras/Surefire260TestWithIdenticalNamesIT.java | 2 +- .../Surefire570MultipleReportDirectoriesIT.java | 4 +- .../its/jiras/Surefire740TruncatedCommaIT.java | 2 +- .../its/jiras/Surefire772BothReportsIT.java | 12 ++--- .../its/jiras/Surefire772NoFailsafeReportsIT.java | 6 +-- .../its/jiras/Surefire772NoSurefireReportsIT.java | 16 +++--- .../its/jiras/Surefire772SpecifiedReportsIT.java | 12 ++--- 33 files changed, 120 insertions(+), 126 deletions(-) diff --git a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java index f158c68d1..3c1e4de79 100644 --- a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java +++ b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java @@ -159,7 +159,6 @@ import static org.apache.maven.surefire.shared.utils.cli.ShutdownHookUtils.remov * Abstract base class for running tests using Surefire. * * @author Stephen Connolly - * @version $Id: SurefirePlugin.java 945065 2010-05-17 10:26:22Z stephenc $ */ public abstract class AbstractSurefireMojo extends AbstractMojo implements SurefireExecutionParameters { private static final Map<String, String> JAVA_9_MATCHER_OLD_NOTATION = singletonMap("version", "[1.9,)"); diff --git a/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java b/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefireMojo.java similarity index 99% rename from maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java rename to maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefireMojo.java index addc8834a..c757fd66b 100644 --- a/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java +++ b/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefireMojo.java @@ -45,7 +45,7 @@ import static org.apache.maven.plugin.surefire.SurefireHelper.reportExecution; defaultPhase = LifecyclePhase.TEST, threadSafe = true, requiresDependencyResolution = ResolutionScope.TEST) -public class SurefirePlugin extends AbstractSurefireMojo implements SurefireReportParameters { +public class SurefireMojo extends AbstractSurefireMojo implements SurefireReportParameters { /** * The directory containing generated classes of the project being tested. This will be included after the test diff --git a/maven-surefire-plugin/src/test/java/org/apache/maven/plugin/surefire/SurefirePluginTest.java b/maven-surefire-plugin/src/test/java/org/apache/maven/plugin/surefire/SurefireMojoTest.java similarity index 68% rename from maven-surefire-plugin/src/test/java/org/apache/maven/plugin/surefire/SurefirePluginTest.java rename to maven-surefire-plugin/src/test/java/org/apache/maven/plugin/surefire/SurefireMojoTest.java index c5a0c3e63..74cc03802 100644 --- a/maven-surefire-plugin/src/test/java/org/apache/maven/plugin/surefire/SurefirePluginTest.java +++ b/maven-surefire-plugin/src/test/java/org/apache/maven/plugin/surefire/SurefireMojoTest.java @@ -31,26 +31,26 @@ import static org.assertj.core.api.Assertions.assertThat; /** * */ -public class SurefirePluginTest extends TestCase { +public class SurefireMojoTest extends TestCase { @Rule public final ExpectedException e = ExpectedException.none(); public void testDefaultIncludes() { - assertThat(new SurefirePlugin().getDefaultIncludes()) + assertThat(new SurefireMojo().getDefaultIncludes()) .containsOnly("**/Test*.java", "**/*Test.java", "**/*Tests.java", "**/*TestCase.java"); } public void testReportSchemaLocation() { - assertThat(new SurefirePlugin().getReportSchemaLocation()) + assertThat(new SurefireMojo().getReportSchemaLocation()) .isEqualTo("https://maven.apache.org/surefire/maven-surefire-plugin/xsd/surefire-test-report.xsd"); } public void testFailIfNoTests() throws Exception { RunResult runResult = new RunResult(0, 0, 0, 0); try { - SurefirePlugin plugin = new SurefirePlugin(); - plugin.setFailIfNoTests(true); - plugin.handleSummary(runResult, null); + SurefireMojo mojo = new SurefireMojo(); + mojo.setFailIfNoTests(true); + mojo.handleSummary(runResult, null); } catch (MojoFailureException e) { assertThat(e.getLocalizedMessage()) .isEqualTo("No tests were executed! (Set -DfailIfNoTests=false to ignore this error.)"); @@ -63,8 +63,8 @@ public class SurefirePluginTest extends TestCase { public void testTestFailure() throws Exception { RunResult runResult = new RunResult(1, 0, 1, 0); try { - SurefirePlugin plugin = new SurefirePlugin(); - plugin.handleSummary(runResult, null); + SurefireMojo mojo = new SurefireMojo(); + mojo.handleSummary(runResult, null); } catch (MojoFailureException e) { assertThat(e.getLocalizedMessage()) .isEqualTo("There are test failures.\n\nPlease refer to null " @@ -79,47 +79,47 @@ public class SurefirePluginTest extends TestCase { } public void testPluginName() { - assertThat(new SurefirePlugin().getPluginName()).isEqualTo("surefire"); + assertThat(new SurefireMojo().getPluginName()).isEqualTo("surefire"); } public void testShouldGetNullEnv() { - SurefirePlugin plugin = new SurefirePlugin(); - assertThat(plugin.getExcludedEnvironmentVariables()).hasSize(0); + SurefireMojo mojo = new SurefireMojo(); + assertThat(mojo.getExcludedEnvironmentVariables()).hasSize(0); } public void testShouldGetEnv() { - SurefirePlugin plugin = new SurefirePlugin(); - plugin.setExcludedEnvironmentVariables(new String[] {"ABC", "KLM"}); - assertThat(plugin.getExcludedEnvironmentVariables()).hasSize(2).contains("ABC", "KLM"); + SurefireMojo mojo = new SurefireMojo(); + mojo.setExcludedEnvironmentVariables(new String[] {"ABC", "KLM"}); + assertThat(mojo.getExcludedEnvironmentVariables()).hasSize(2).contains("ABC", "KLM"); } public void testShouldGetPropertyFile() { - SurefirePlugin plugin = new SurefirePlugin(); - plugin.setSystemPropertiesFile(new File("testShouldGetPropertyFile")); - assertThat(plugin.getSystemPropertiesFile()).isEqualTo(new File("testShouldGetPropertyFile")); + SurefireMojo mojo = new SurefireMojo(); + mojo.setSystemPropertiesFile(new File("testShouldGetPropertyFile")); + assertThat(mojo.getSystemPropertiesFile()).isEqualTo(new File("testShouldGetPropertyFile")); } public void testNegativeFailOnFlakeCount() { - SurefirePlugin plugin = new SurefirePlugin(); - plugin.setFailOnFlakeCount(-1); + SurefireMojo mojo = new SurefireMojo(); + mojo.setFailOnFlakeCount(-1); e.expect(MojoFailureException.class); e.expectMessage("Parameter \"failOnFlakeCount\" should not be negative."); } public void testFailOnFlakeCountWithoutRerun() { - SurefirePlugin plugin = new SurefirePlugin(); - plugin.setFailOnFlakeCount(1); + SurefireMojo mojo = new SurefireMojo(); + mojo.setFailOnFlakeCount(1); e.expect(MojoFailureException.class); e.expectMessage("\"failOnFlakeCount\" requires rerunFailingTestsCount to be at least 1."); } public void testShouldHaveJUnit5EnginesFilter() { - SurefirePlugin plugin = new SurefirePlugin(); + SurefireMojo mojo = new SurefireMojo(); - plugin.setIncludeJUnit5Engines(new String[] {"e1", "e2"}); - assertThat(plugin.getIncludeJUnit5Engines()).isEqualTo(new String[] {"e1", "e2"}); + mojo.setIncludeJUnit5Engines(new String[] {"e1", "e2"}); + assertThat(mojo.getIncludeJUnit5Engines()).isEqualTo(new String[] {"e1", "e2"}); - plugin.setExcludeJUnit5Engines(new String[] {"e1", "e2"}); - assertThat(plugin.getExcludeJUnit5Engines()).isEqualTo(new String[] {"e1", "e2"}); + mojo.setExcludeJUnit5Engines(new String[] {"e1", "e2"}); + assertThat(mojo.getExcludeJUnit5Engines()).isEqualTo(new String[] {"e1", "e2"}); } } diff --git a/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReportMojo.java b/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReport.java similarity index 99% rename from maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReportMojo.java rename to maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReport.java index 3ec4928f9..847c29cb2 100644 --- a/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReportMojo.java +++ b/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReport.java @@ -55,7 +55,7 @@ import static org.apache.maven.plugins.surefire.report.SurefireReportParser.hasR * * @author Stephen Connolly */ -public abstract class AbstractSurefireReportMojo extends AbstractMavenReport { +public abstract class AbstractSurefireReport extends AbstractMavenReport { /** * If set to false, only failures are shown. diff --git a/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/FailsafeReportMojo.java b/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/FailsafeOnlyReport.java similarity index 94% rename from maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/FailsafeReportMojo.java rename to maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/FailsafeOnlyReport.java index 9e889cd66..7e6b20840 100644 --- a/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/FailsafeReportMojo.java +++ b/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/FailsafeOnlyReport.java @@ -35,12 +35,12 @@ import org.apache.maven.project.MavenProject; */ @Mojo(name = "failsafe-report-only") @SuppressWarnings("unused") -public class FailsafeReportMojo extends AbstractSurefireReportMojo { +public class FailsafeOnlyReport extends AbstractSurefireReport { /** * The filename to use for the report. */ - @Parameter(defaultValue = "failsafe-report", property = "outputName", required = true) + @Parameter(defaultValue = "failsafe", property = "outputName", required = true) private String outputName; /** diff --git a/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportOnlyMojo.java b/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireOnlyReport.java similarity index 95% rename from maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportOnlyMojo.java rename to maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireOnlyReport.java index ec60196e3..d523e9769 100644 --- a/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportOnlyMojo.java +++ b/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireOnlyReport.java @@ -34,4 +34,4 @@ import org.apache.maven.plugins.annotations.Mojo; @Mojo(name = "report-only") @Execute(phase = LifecyclePhase.NONE) @SuppressWarnings("unused") -public class SurefireReportOnlyMojo extends SurefireReportMojo {} +public class SurefireOnlyReport extends SurefireReport {} diff --git a/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportMojo.java b/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReport.java similarity index 94% rename from maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportMojo.java rename to maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReport.java index 78a096fd7..1405ba9c9 100644 --- a/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportMojo.java +++ b/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReport.java @@ -34,12 +34,12 @@ import org.apache.maven.project.MavenProject; @Mojo(name = "report", inheritByDefault = false) @Execute(lifecycle = "surefire", phase = LifecyclePhase.TEST) @SuppressWarnings("unused") -public class SurefireReportMojo extends AbstractSurefireReportMojo { +public class SurefireReport extends AbstractSurefireReport { /** * The filename to use for the report. */ - @Parameter(defaultValue = "surefire-report", property = "outputName", required = true) + @Parameter(defaultValue = "surefire", property = "outputName", required = true) private String outputName; /** diff --git a/maven-surefire-report-plugin/src/site/apt/examples/changing-report-name.apt.vm b/maven-surefire-report-plugin/src/site/apt/examples/changing-report-name.apt.vm index de9e8b5e2..59b404d23 100644 --- a/maven-surefire-report-plugin/src/site/apt/examples/changing-report-name.apt.vm +++ b/maven-surefire-report-plugin/src/site/apt/examples/changing-report-name.apt.vm @@ -26,7 +26,7 @@ Changing the Report Name In order to configure the file name of the generated report (which is - "<surefire-report>" by default), the <<outputName>> property should + "<<<surefire>>>" by default), the <<<outputName>>> property should be set to the desired name. +----+ diff --git a/maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/JUnit4SuiteTest.java b/maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/JUnit4SuiteTest.java index 74df21fec..fc1eaefd3 100644 --- a/maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/JUnit4SuiteTest.java +++ b/maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/JUnit4SuiteTest.java @@ -34,7 +34,7 @@ public class JUnit4SuiteTest extends TestCase { TestSuite suite = new TestSuite(); suite.addTest(new JUnit4TestAdapter(Surefire597Test.class)); suite.addTest(new JUnit4TestAdapter(SurefireSchemaValidationTest.class)); - suite.addTestSuite(SurefireReportMojoTest.class); + suite.addTestSuite(SurefireReportTest.class); return suite; } } diff --git a/maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/SurefireReportMojoTest.java b/maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/SurefireReportTest.java similarity index 94% rename from maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/SurefireReportMojoTest.java rename to maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/SurefireReportTest.java index 8dab21e86..3da193a92 100644 --- a/maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/SurefireReportMojoTest.java +++ b/maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/SurefireReportTest.java @@ -42,7 +42,7 @@ import static org.hamcrest.MatcherAssert.assertThat; * @author <a href="mailto:arami...@apache.org">Allan Ramirez</a> */ @SuppressWarnings("checkstyle:linelength") -public class SurefireReportMojoTest extends AbstractMojoTestCase { +public class SurefireReportTest extends AbstractMojoTestCase { private ArtifactStubFactory artifactStubFactory; // Can be removed with Doxia 2.0.0 @@ -64,8 +64,8 @@ public class SurefireReportMojoTest extends AbstractMojoTestCase { super.tearDown(); } - protected SurefireReportMojo createReportMojo(File pluginXmlFile) throws Exception { - SurefireReportMojo mojo = (SurefireReportMojo) lookupMojo("report", pluginXmlFile); + protected SurefireReport createReportMojo(File pluginXmlFile) throws Exception { + SurefireReport mojo = (SurefireReport) lookupMojo("report", pluginXmlFile); assertNotNull("Mojo found.", mojo); LegacySupport legacySupport = lookup(LegacySupport.class); @@ -82,7 +82,7 @@ public class SurefireReportMojoTest extends AbstractMojoTestCase { public void testBasicSurefireReport() throws Exception { File testPom = new File(getUnitBaseDir(), "basic-surefire-report-test/plugin-config.xml"); - SurefireReportMojo mojo = createReportMojo(testPom); + SurefireReport mojo = createReportMojo(testPom); File outputDir = (File) getVariableValueFromObject(mojo, "outputDirectory"); boolean showSuccess = (Boolean) getVariableValueFromObject(mojo, "showSuccess"); File reportsDir = (File) getVariableValueFromObject(mojo, "reportsDirectory"); @@ -96,14 +96,14 @@ public class SurefireReportMojoTest extends AbstractMojoTestCase { new File(getBasedir() + "/src/test/resources/unit/basic-surefire-report-test/surefire-reports") .getAbsolutePath(), reportsDir.getAbsolutePath()); - assertEquals("surefire-report", outputName); + assertEquals("surefire", outputName); assertEquals( new File(getBasedir() + "/target/site/unit/basic-surefire-report-test/xref-test").getAbsolutePath(), xrefLocation.getAbsolutePath()); assertTrue(linkXRef); mojo.execute(); - File report = new File(getBasedir(), "target/site/unit/basic-surefire-report-test/surefire-report.html"); + File report = new File(getBasedir(), "target/site/unit/basic-surefire-report-test/surefire.html"); assertTrue(report.exists()); String htmlContent = FileUtils.fileRead(report); @@ -119,12 +119,11 @@ public class SurefireReportMojoTest extends AbstractMojoTestCase { public void testBasicSurefireReportIfShowSuccessIsFalse() throws Exception { File testPom = new File(getUnitBaseDir(), "basic-surefire-report-success-false/plugin-config.xml"); - SurefireReportMojo mojo = createReportMojo(testPom); + SurefireReport mojo = createReportMojo(testPom); boolean showSuccess = (Boolean) getVariableValueFromObject(mojo, "showSuccess"); assertFalse(showSuccess); mojo.execute(); - File report = - new File(getBasedir(), "target/site/unit/basic-surefire-report-success-false/surefire-report.html"); + File report = new File(getBasedir(), "target/site/unit/basic-surefire-report-success-false/surefire.html"); assertTrue(report.exists()); String htmlContent = FileUtils.fileRead(report); @@ -134,12 +133,11 @@ public class SurefireReportMojoTest extends AbstractMojoTestCase { public void testBasicSurefireReportIfLinkXrefIsFalse() throws Exception { File testPom = new File(getUnitBaseDir(), "basic-surefire-report-linkxref-false/plugin-config.xml"); - SurefireReportMojo mojo = createReportMojo(testPom); + SurefireReport mojo = createReportMojo(testPom); boolean linkXRef = (Boolean) getVariableValueFromObject(mojo, "linkXRef"); assertFalse(linkXRef); mojo.execute(); - File report = - new File(getBasedir(), "target/site/unit/basic-surefire-report-linkxref-false/surefire-report.html"); + File report = new File(getBasedir(), "target/site/unit/basic-surefire-report-linkxref-false/surefire.html"); assertTrue(report.exists()); String htmlContent = FileUtils.fileRead(report); @@ -149,10 +147,9 @@ public class SurefireReportMojoTest extends AbstractMojoTestCase { public void testBasicSurefireReportIfReportingIsNull() throws Exception { File testPom = new File(getUnitBaseDir(), "basic-surefire-report-reporting-null/plugin-config.xml"); - SurefireReportMojo mojo = createReportMojo(testPom); + SurefireReport mojo = createReportMojo(testPom); mojo.execute(); - File report = - new File(getBasedir(), "target/site/unit/basic-surefire-report-reporting-null/surefire-report.html"); + File report = new File(getBasedir(), "target/site/unit/basic-surefire-report-reporting-null/surefire.html"); assertTrue(report.exists()); String htmlContent = FileUtils.fileRead(report); @@ -163,10 +160,9 @@ public class SurefireReportMojoTest extends AbstractMojoTestCase { @SuppressWarnings("checkstyle:methodname") public void testBasicSurefireReport_AnchorTestCases() throws Exception { File testPom = new File(getUnitBaseDir(), "basic-surefire-report-anchor-test-cases/plugin-config.xml"); - SurefireReportMojo mojo = createReportMojo(testPom); + SurefireReport mojo = createReportMojo(testPom); mojo.execute(); - File report = - new File(getBasedir(), "target/site/unit/basic-surefire-report-anchor-test-cases/surefire-report.html"); + File report = new File(getBasedir(), "target/site/unit/basic-surefire-report-anchor-test-cases/surefire.html"); assertTrue(report.exists()); String htmlContent = FileUtils.fileRead(report); @@ -180,9 +176,9 @@ public class SurefireReportMojoTest extends AbstractMojoTestCase { public void testSurefireReportSingleError() throws Exception { File testPom = new File(getUnitBaseDir(), "surefire-report-single-error/plugin-config.xml"); - SurefireReportMojo mojo = createReportMojo(testPom); + SurefireReport mojo = createReportMojo(testPom); mojo.execute(); - File report = new File(getBasedir(), "target/site/unit/surefire-report-single-error/surefire-report.html"); + File report = new File(getBasedir(), "target/site/unit/surefire-report-single-error/surefire.html"); assertTrue(report.exists()); String htmlContent = FileUtils.fileRead(report); @@ -267,10 +263,10 @@ public class SurefireReportMojoTest extends AbstractMojoTestCase { public void testSurefireReportNestedClassTrimStackTrace() throws Exception { File testPom = new File(getUnitBaseDir(), "surefire-report-nestedClass-trimStackTrace/plugin-config.xml"); - SurefireReportMojo mojo = createReportMojo(testPom); + SurefireReport mojo = createReportMojo(testPom); mojo.execute(); - File report = new File( - getBasedir(), "target/site/unit/surefire-report-nestedClass-trimStackTrace/surefire-report.html"); + File report = + new File(getBasedir(), "target/site/unit/surefire-report-nestedClass-trimStackTrace/surefire.html"); assertTrue(report.exists()); String htmlContent = FileUtils.fileRead(report); @@ -331,9 +327,9 @@ public class SurefireReportMojoTest extends AbstractMojoTestCase { public void testSurefireReportNestedClass() throws Exception { File testPom = new File(getUnitBaseDir(), "surefire-report-nestedClass/plugin-config.xml"); - SurefireReportMojo mojo = createReportMojo(testPom); + SurefireReport mojo = createReportMojo(testPom); mojo.execute(); - File report = new File(getBasedir(), "target/site/unit/surefire-report-nestedClass/surefire-report.html"); + File report = new File(getBasedir(), "target/site/unit/surefire-report-nestedClass/surefire.html"); assertTrue(report.exists()); String htmlContent = FileUtils.fileRead(report); @@ -418,10 +414,9 @@ public class SurefireReportMojoTest extends AbstractMojoTestCase { public void testSurefireReportEnclosedTrimStackTrace() throws Exception { File testPom = new File(getUnitBaseDir(), "surefire-report-enclosed-trimStackTrace/plugin-config.xml"); - SurefireReportMojo mojo = createReportMojo(testPom); + SurefireReport mojo = createReportMojo(testPom); mojo.execute(); - File report = - new File(getBasedir(), "target/site/unit/surefire-report-enclosed-trimStackTrace/surefire-report.html"); + File report = new File(getBasedir(), "target/site/unit/surefire-report-enclosed-trimStackTrace/surefire.html"); assertTrue(report.exists()); String htmlContent = FileUtils.fileRead(report); @@ -482,9 +477,9 @@ public class SurefireReportMojoTest extends AbstractMojoTestCase { public void testSurefireReportEnclosed() throws Exception { File testPom = new File(getUnitBaseDir(), "surefire-report-enclosed/plugin-config.xml"); - SurefireReportMojo mojo = createReportMojo(testPom); + SurefireReport mojo = createReportMojo(testPom); mojo.execute(); - File report = new File(getBasedir(), "target/site/unit/surefire-report-enclosed/surefire-report.html"); + File report = new File(getBasedir(), "target/site/unit/surefire-report-enclosed/surefire.html"); assertTrue(report.exists()); String htmlContent = FileUtils.fileRead(report); @@ -579,7 +574,7 @@ public class SurefireReportMojoTest extends AbstractMojoTestCase { public void testCustomTitleAndDescriptionReport() throws Exception { File testPom = new File(getUnitBaseDir(), "surefire-1183/plugin-config.xml"); - SurefireReportMojo mojo = createReportMojo(testPom); + SurefireReport mojo = createReportMojo(testPom); File outputDir = (File) getVariableValueFromObject(mojo, "outputDirectory"); String outputName = (String) getVariableValueFromObject(mojo, "outputName"); @@ -590,11 +585,11 @@ public class SurefireReportMojoTest extends AbstractMojoTestCase { new File(getBasedir() + "/src/test/resources/unit/surefire-1183/acceptancetest-reports") .getAbsolutePath(), reportsDir.getAbsolutePath()); - assertEquals("acceptance-test-report", outputName); + assertEquals("acceptance-test", outputName); mojo.execute(); - File report = new File(getBasedir(), "target/site/unit/surefire-1183/acceptance-test-report.html"); + File report = new File(getBasedir(), "target/site/unit/surefire-1183/acceptance-test.html"); assertTrue(report.exists()); diff --git a/maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/stubs/SurefireRepMavenProjectStub.java b/maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/stubs/SurefireReportMavenProjectStub.java similarity index 97% rename from maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/stubs/SurefireRepMavenProjectStub.java rename to maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/stubs/SurefireReportMavenProjectStub.java index bd4497c24..de27f52f8 100644 --- a/maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/stubs/SurefireRepMavenProjectStub.java +++ b/maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/stubs/SurefireReportMavenProjectStub.java @@ -33,7 +33,7 @@ import org.apache.maven.plugin.testing.stubs.MavenProjectStub; /** * @author <a href="mailto:arami...@apache.org">Allan Ramirez</a> */ -public class SurefireRepMavenProjectStub extends MavenProjectStub { +public class SurefireReportMavenProjectStub extends MavenProjectStub { /** * {@inheritDoc} */ diff --git a/maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/stubs/SurefireRepMavenProjectStub2.java b/maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/stubs/SurefireReportMavenProjectStub2.java similarity index 94% rename from maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/stubs/SurefireRepMavenProjectStub2.java rename to maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/stubs/SurefireReportMavenProjectStub2.java index 9365879e4..1e4a58d6e 100644 --- a/maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/stubs/SurefireRepMavenProjectStub2.java +++ b/maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/stubs/SurefireReportMavenProjectStub2.java @@ -27,7 +27,7 @@ import org.apache.maven.plugin.testing.stubs.MavenProjectStub; /** * @author <a href="mailto:arami...@apache.org">Allan Ramirez</a> */ -public class SurefireRepMavenProjectStub2 extends MavenProjectStub { +public class SurefireReportMavenProjectStub2 extends MavenProjectStub { /** * {@inheritDoc} */ diff --git a/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-anchor-test-cases/plugin-config.xml b/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-anchor-test-cases/plugin-config.xml index 3045dce6d..e94305364 100644 --- a/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-anchor-test-cases/plugin-config.xml +++ b/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-anchor-test-cases/plugin-config.xml @@ -25,11 +25,11 @@ <configuration> <outputDirectory>${basedir}/target/site/unit/basic-surefire-report-anchor-test-cases</outputDirectory> <localRepository>${localRepository}</localRepository> - <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireRepMavenProjectStub"/> + <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireReportMavenProjectStub"/> <showSuccess>true</showSuccess> <reportsDirectory>${basedir}/src/test/resources/unit/basic-surefire-report-anchor-test-cases/surefire-reports </reportsDirectory> - <outputName>surefire-report</outputName> + <outputName>surefire</outputName> <xrefLocation>${basedir}/target/site/unit/basic-surefire-report-anchor-test-cases/xref-test</xrefLocation> </configuration> </plugin> diff --git a/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-linkxref-false/plugin-config.xml b/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-linkxref-false/plugin-config.xml index e0586aad1..d175839c6 100644 --- a/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-linkxref-false/plugin-config.xml +++ b/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-linkxref-false/plugin-config.xml @@ -25,11 +25,11 @@ <configuration> <outputDirectory>${basedir}/target/site/unit/basic-surefire-report-linkxref-false</outputDirectory> <localRepository>${localRepository}</localRepository> - <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireRepMavenProjectStub"/> + <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireReportMavenProjectStub"/> <showSuccess>true</showSuccess> <reportsDirectory>${basedir}/src/test/resources/unit/basic-surefire-report-linkxref-false/surefire-reports </reportsDirectory> - <outputName>surefire-report</outputName> + <outputName>surefire</outputName> <xrefLocation>${basedir}/target/site/unit/basic-surefire-report-linkxref-false/xref-test</xrefLocation> <linkXRef>false</linkXRef> </configuration> diff --git a/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-reporting-null/plugin-config.xml b/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-reporting-null/plugin-config.xml index 8aca877bd..dfe6b6767 100644 --- a/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-reporting-null/plugin-config.xml +++ b/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-reporting-null/plugin-config.xml @@ -25,11 +25,11 @@ <configuration> <outputDirectory>${basedir}/target/site/unit/basic-surefire-report-reporting-null</outputDirectory> <localRepository>${localRepository}</localRepository> - <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireRepMavenProjectStub2"/> + <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireReportMavenProjectStub2"/> <showSuccess>true</showSuccess> <reportsDirectory>${basedir}/src/test/resources/unit/basic-surefire-report-reporting-null/surefire-reports </reportsDirectory> - <outputName>surefire-report</outputName> + <outputName>surefire</outputName> <xrefLocation>${basedir}/target/site/unit/basic-surefire-report-test/xref-test</xrefLocation> <linkXRef>true</linkXRef> </configuration> diff --git a/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-success-false/plugin-config.xml b/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-success-false/plugin-config.xml index c79859112..eae36a008 100644 --- a/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-success-false/plugin-config.xml +++ b/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-success-false/plugin-config.xml @@ -25,11 +25,11 @@ <configuration> <outputDirectory>${basedir}/target/site/unit/basic-surefire-report-success-false</outputDirectory> <localRepository>${localRepository}</localRepository> - <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireRepMavenProjectStub"/> + <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireReportMavenProjectStub"/> <showSuccess>false</showSuccess> <reportsDirectory>${basedir}/src/test/resources/unit/basic-surefire-report-success-false/surefire-reports </reportsDirectory> - <outputName>surefire-report</outputName> + <outputName>surefire</outputName> <xrefLocation>${basedir}/target/site/unit/basic-surefire-report-success-false/xref-test</xrefLocation> <linkXRef>true</linkXRef> </configuration> diff --git a/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-test/plugin-config.xml b/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-test/plugin-config.xml index 81e59caf1..08cc7a39e 100644 --- a/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-test/plugin-config.xml +++ b/maven-surefire-report-plugin/src/test/resources/unit/basic-surefire-report-test/plugin-config.xml @@ -25,11 +25,11 @@ <configuration> <outputDirectory>${basedir}/target/site/unit/basic-surefire-report-test</outputDirectory> <localRepository>${localRepository}</localRepository> - <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireRepMavenProjectStub"/> + <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireReportMavenProjectStub"/> <showSuccess>true</showSuccess> <reportsDirectory>${basedir}/src/test/resources/unit/basic-surefire-report-test/surefire-reports </reportsDirectory> - <outputName>surefire-report</outputName> + <outputName>surefire</outputName> <xrefLocation>${basedir}/target/site/unit/basic-surefire-report-test/xref-test</xrefLocation> <linkXRef>true</linkXRef> </configuration> diff --git a/maven-surefire-report-plugin/src/test/resources/unit/surefire-1183/plugin-config.xml b/maven-surefire-report-plugin/src/test/resources/unit/surefire-1183/plugin-config.xml index 108f58350..2a4c04a0b 100644 --- a/maven-surefire-report-plugin/src/test/resources/unit/surefire-1183/plugin-config.xml +++ b/maven-surefire-report-plugin/src/test/resources/unit/surefire-1183/plugin-config.xml @@ -25,8 +25,8 @@ <artifactId>maven-surefire-report-plugin</artifactId> <configuration> <project - implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireRepMavenProjectStub"/> - <outputName>acceptance-test-report</outputName> + implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireReportMavenProjectStub"/> + <outputName>acceptance-test</outputName> <localRepository>${localRepository}</localRepository> <customBundle>${basedir}/src/test/resources/unit/surefire-1183/src/site/custom/surefire-report.properties</customBundle> <outputDirectory>${basedir}/target/site/unit/surefire-1183</outputDirectory> diff --git a/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-enclosed-trimStackTrace/plugin-config.xml b/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-enclosed-trimStackTrace/plugin-config.xml index 981bb1882..492c62a3b 100644 --- a/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-enclosed-trimStackTrace/plugin-config.xml +++ b/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-enclosed-trimStackTrace/plugin-config.xml @@ -25,11 +25,11 @@ <configuration> <outputDirectory>${basedir}/target/site/unit/surefire-report-enclosed-trimStackTrace</outputDirectory> <localRepository>${localRepository}</localRepository> - <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireRepMavenProjectStub"/> + <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireReportMavenProjectStub"/> <showSuccess>true</showSuccess> <reportsDirectory>${basedir}/src/test/resources/unit/surefire-report-enclosed-trimStackTrace/surefire-reports </reportsDirectory> - <outputName>surefire-report</outputName> + <outputName>surefire</outputName> <xrefLocation>${basedir}/target/site/unit/surefire-report-enclosed-trimStackTrace/xref-test</xrefLocation> <linkXRef>true</linkXRef> </configuration> diff --git a/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-enclosed/plugin-config.xml b/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-enclosed/plugin-config.xml index 26bae34ab..25527c370 100644 --- a/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-enclosed/plugin-config.xml +++ b/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-enclosed/plugin-config.xml @@ -25,11 +25,11 @@ <configuration> <outputDirectory>${basedir}/target/site/unit/surefire-report-enclosed</outputDirectory> <localRepository>${localRepository}</localRepository> - <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireRepMavenProjectStub"/> + <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireReportMavenProjectStub"/> <showSuccess>true</showSuccess> <reportsDirectory>${basedir}/src/test/resources/unit/surefire-report-enclosed/surefire-reports </reportsDirectory> - <outputName>surefire-report</outputName> + <outputName>surefire</outputName> <xrefLocation>${basedir}/target/site/unit/surefire-report-enclosed/xref-test</xrefLocation> <linkXRef>true</linkXRef> </configuration> diff --git a/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-nestedClass-trimStackTrace/plugin-config.xml b/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-nestedClass-trimStackTrace/plugin-config.xml index dcf6a9a72..ce9bc4971 100644 --- a/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-nestedClass-trimStackTrace/plugin-config.xml +++ b/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-nestedClass-trimStackTrace/plugin-config.xml @@ -25,11 +25,11 @@ <configuration> <outputDirectory>${basedir}/target/site/unit/surefire-report-nestedClass-trimStackTrace</outputDirectory> <localRepository>${localRepository}</localRepository> - <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireRepMavenProjectStub"/> + <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireReportMavenProjectStub"/> <showSuccess>true</showSuccess> <reportsDirectory>${basedir}/src/test/resources/unit/surefire-report-nestedClass-trimStackTrace/surefire-reports </reportsDirectory> - <outputName>surefire-report</outputName> + <outputName>surefire</outputName> <xrefLocation>${basedir}/target/site/unit/surefire-report-nestedClass-trimStackTrace/xref-test</xrefLocation> <linkXRef>true</linkXRef> </configuration> diff --git a/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-nestedClass/plugin-config.xml b/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-nestedClass/plugin-config.xml index 86ff4639b..27302b0a9 100644 --- a/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-nestedClass/plugin-config.xml +++ b/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-nestedClass/plugin-config.xml @@ -25,11 +25,11 @@ <configuration> <outputDirectory>${basedir}/target/site/unit/surefire-report-nestedClass</outputDirectory> <localRepository>${localRepository}</localRepository> - <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireRepMavenProjectStub"/> + <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireReportMavenProjectStub"/> <showSuccess>true</showSuccess> <reportsDirectory>${basedir}/src/test/resources/unit/surefire-report-nestedClass/surefire-reports </reportsDirectory> - <outputName>surefire-report</outputName> + <outputName>surefire</outputName> <xrefLocation>${basedir}/target/site/unit/surefire-report-nestedClass/xref-test</xrefLocation> <linkXRef>true</linkXRef> </configuration> diff --git a/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-single-error/plugin-config.xml b/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-single-error/plugin-config.xml index e8a3f5889..70767d4b4 100644 --- a/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-single-error/plugin-config.xml +++ b/maven-surefire-report-plugin/src/test/resources/unit/surefire-report-single-error/plugin-config.xml @@ -25,11 +25,11 @@ <configuration> <outputDirectory>${basedir}/target/site/unit/surefire-report-single-error</outputDirectory> <localRepository>${localRepository}</localRepository> - <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireRepMavenProjectStub"/> + <project implementation="org.apache.maven.plugins.surefire.report.stubs.SurefireReportMavenProjectStub"/> <showSuccess>true</showSuccess> <reportsDirectory>${basedir}/src/test/resources/unit/surefire-report-single-error/surefire-reports </reportsDirectory> - <outputName>surefire-report</outputName> + <outputName>surefire</outputName> <xrefLocation>${basedir}/target/site/unit/surefire-report-single-error/xref-test</xrefLocation> <linkXRef>true</linkXRef> </configuration> diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/AggregateReportIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/AggregateReportIT.java index 68676a807..8a5293faa 100644 --- a/surefire-its/src/test/java/org/apache/maven/surefire/its/AggregateReportIT.java +++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/AggregateReportIT.java @@ -42,7 +42,7 @@ public class AggregateReportIT extends SurefireJUnit4IntegrationTestCase { public void aggregateReport() { OutputValidator outputValidator = unpack("/aggregate-report").addSurefireReportGoal().executeCurrentGoals(); - TestFile surefireReportHtml = outputValidator.getSiteFile("surefire-report.html"); + TestFile surefireReportHtml = outputValidator.getSiteFile("surefire.html"); assertTrue("surefire report missing: " + surefireReportHtml.getAbsolutePath(), surefireReportHtml.exists()); // TODO HtmlUnit tests on the surefire report diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgReportTestIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgReportTestIT.java index ddf01f500..30aaefcf3 100644 --- a/surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgReportTestIT.java +++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/CheckTestNgReportTestIT.java @@ -38,7 +38,7 @@ public class CheckTestNgReportTestIT extends SurefireJUnit4IntegrationTestCase { .addSurefireReportGoal() .executeCurrentGoals() .verifyErrorFree(3) - .getSiteFile("surefire-report.html") + .getSiteFile("surefire.html") .assertFileExists(); } diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire1490ReportTitleDescriptionIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire1490ReportTitleDescriptionIT.java index 903a2e55f..e49aa4624 100644 --- a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire1490ReportTitleDescriptionIT.java +++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire1490ReportTitleDescriptionIT.java @@ -42,12 +42,12 @@ public class Surefire1490ReportTitleDescriptionIT extends SurefireJUnit4Integrat .assertContainsText("Report on the integration test results of the project."); validator - .getSiteFile("failsafe-report.html") + .getSiteFile("failsafe.html") .assertContainsText("Failsafe Report") .assertContainsText("Surefire1490IT"); validator - .getSiteFile("surefire-report.html") + .getSiteFile("surefire.html") .assertContainsText("Surefire Report") .assertContainsText("Surefire1490Test"); } @@ -65,12 +65,12 @@ public class Surefire1490ReportTitleDescriptionIT extends SurefireJUnit4Integrat .assertContainsText("failsafe desc"); validator - .getSiteFile("failsafe-report.html") + .getSiteFile("failsafe.html") .assertContainsText("failsafe title") .assertContainsText("Surefire1490IT"); validator - .getSiteFile("surefire-report.html") + .getSiteFile("surefire.html") .assertContainsText("surefire title") .assertContainsText("Surefire1490Test"); } diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire260TestWithIdenticalNamesIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire260TestWithIdenticalNamesIT.java index 981bfd02c..07e6da39c 100644 --- a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire260TestWithIdenticalNamesIT.java +++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire260TestWithIdenticalNamesIT.java @@ -50,7 +50,7 @@ public class Surefire260TestWithIdenticalNamesIT extends SurefireJUnit4Integrati .addSurefireReportGoal() .executeCurrentGoals(); - TestFile siteFile = validator.getSiteFile("surefire-report.html"); + TestFile siteFile = validator.getSiteFile("surefire.html"); final URI uri = siteFile.toURI(); try (WebClient webClient = new WebClient()) { diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire570MultipleReportDirectoriesIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire570MultipleReportDirectoriesIT.java index 25b76e2ca..95a980930 100644 --- a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire570MultipleReportDirectoriesIT.java +++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire570MultipleReportDirectoriesIT.java @@ -38,7 +38,7 @@ public class Surefire570MultipleReportDirectoriesIT extends SurefireJUnit4Integr surefireLauncher.executeTest(); surefireLauncher.addGoal("-Daggregate=true"); OutputValidator validator = surefireLauncher.executeSurefireReport(); - TestFile siteFile = validator.getSiteFile("surefire-report.html"); + TestFile siteFile = validator.getSiteFile("surefire.html"); siteFile.assertContainsText("MyModule1ClassTest"); siteFile.assertContainsText("MyModule2ClassTest"); siteFile.assertContainsText("MyDummyClassM1Test"); @@ -51,7 +51,7 @@ public class Surefire570MultipleReportDirectoriesIT extends SurefireJUnit4Integr surefireLauncher.reset(); surefireLauncher.executeSurefireReport(); OutputValidator module1 = surefireLauncher.getSubProjectValidator("module1"); - TestFile siteFile = module1.getSiteFile("surefire-report.html"); + TestFile siteFile = module1.getSiteFile("surefire.html"); siteFile.assertContainsText("MyModule1ClassTest"); siteFile.assertContainsText("MyDummyClassM1Test"); } diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire740TruncatedCommaIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire740TruncatedCommaIT.java index a74ed027e..78340ce3b 100644 --- a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire740TruncatedCommaIT.java +++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire740TruncatedCommaIT.java @@ -39,7 +39,7 @@ public class Surefire740TruncatedCommaIT extends SurefireJUnit4IntegrationTestCa .addSurefireReportGoal() .executeCurrentGoals(); - TestFile siteFile = validator.getSiteFile("surefire-report.html"); + TestFile siteFile = validator.getSiteFile("surefire.html"); assertTrue("Expecting file", siteFile.exists()); siteFile.assertContainsText("027"); // Avoid asserting with the "," or "." ;) } diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire772BothReportsIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire772BothReportsIT.java index 10bf88dce..5042988ad 100644 --- a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire772BothReportsIT.java +++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire772BothReportsIT.java @@ -45,10 +45,10 @@ public class Surefire772BothReportsIT extends SurefireJUnit4IntegrationTestCase OutputValidator outputValidator = unpack().addFailsafeReportOnlyGoal().addSurefireReportOnlyGoal().executeCurrentGoals(); - TestFile siteFile = outputValidator.getSiteFile("surefire-report.html"); + TestFile siteFile = outputValidator.getSiteFile("surefire.html"); assertTrue("Expecting surefire report file", siteFile.isFile()); - siteFile = outputValidator.getSiteFile("failsafe-report.html"); + siteFile = outputValidator.getSiteFile("failsafe.html"); assertTrue("Expecting failsafe report file", siteFile.isFile()); } @@ -59,10 +59,10 @@ public class Surefire772BothReportsIT extends SurefireJUnit4IntegrationTestCase .addSurefireReportOnlyGoal() .executeCurrentGoals(); - TestFile siteFile = validator.getSiteFile("surefire-report.html"); + TestFile siteFile = validator.getSiteFile("surefire.html"); assertTrue("Expecting surefire report file", siteFile.isFile()); - siteFile = validator.getSiteFile("failsafe-report.html"); + siteFile = validator.getSiteFile("failsafe.html"); assertFalse("Expecting no failsafe report file", siteFile.isFile()); } @@ -74,10 +74,10 @@ public class Surefire772BothReportsIT extends SurefireJUnit4IntegrationTestCase .addSurefireReportOnlyGoal() .executeCurrentGoals(); - TestFile siteFile = validator.getSiteFile("surefire-report.html"); + TestFile siteFile = validator.getSiteFile("surefire.html"); assertFalse("Expecting no surefire report file", siteFile.isFile()); - siteFile = validator.getSiteFile("failsafe-report.html"); + siteFile = validator.getSiteFile("failsafe.html"); assertTrue("Expecting failsafe report file", siteFile.isFile()); } } diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire772NoFailsafeReportsIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire772NoFailsafeReportsIT.java index adce8ef2f..aa4bd21b6 100644 --- a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire772NoFailsafeReportsIT.java +++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire772NoFailsafeReportsIT.java @@ -77,17 +77,17 @@ public class Surefire772NoFailsafeReportsIT extends SurefireJUnit4IntegrationTes } private void assertNoFailsefeReport(OutputValidator site) { - TestFile siteFile = site.getSiteFile("failsafe-report.html"); + TestFile siteFile = site.getSiteFile("failsafe.html"); assertFalse("Expecting no failsafe report file", siteFile.isFile()); } private void assertFailsafeReport(OutputValidator site) { - TestFile siteFile = site.getSiteFile("failsafe-report.html"); + TestFile siteFile = site.getSiteFile("failsafe.html"); assertTrue("Expecting no failsafe report file", siteFile.isFile()); } private void assertSurefireReportPresent(OutputValidator site) { - TestFile siteFile = site.getSiteFile("surefire-report.html"); + TestFile siteFile = site.getSiteFile("surefire.html"); assertTrue("Expecting surefire report file", siteFile.isFile()); } diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire772NoSurefireReportsIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire772NoSurefireReportsIT.java index a34bd5112..845909210 100644 --- a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire772NoSurefireReportsIT.java +++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire772NoSurefireReportsIT.java @@ -38,10 +38,10 @@ public class Surefire772NoSurefireReportsIT extends SurefireJUnit4IntegrationTes OutputValidator validator = unpack().addFailsafeReportOnlyGoal().addSurefireReportOnlyGoal().executeCurrentGoals(); - TestFile siteFile = validator.getSiteFile("surefire-report.html"); + TestFile siteFile = validator.getSiteFile("surefire.html"); assertTrue("Expecting surefire report file", siteFile.isFile()); - siteFile = validator.getSiteFile("failsafe-report.html"); + siteFile = validator.getSiteFile("failsafe.html"); assertTrue("Expecting failsafe report file", siteFile.isFile()); } @@ -52,10 +52,10 @@ public class Surefire772NoSurefireReportsIT extends SurefireJUnit4IntegrationTes .addSurefireReportOnlyGoal() .executeCurrentGoals(); - TestFile siteFile = validator.getSiteFile("surefire-report.html"); + TestFile siteFile = validator.getSiteFile("surefire.html"); assertFalse("Expecting no surefire report file", siteFile.isFile()); - siteFile = validator.getSiteFile("failsafe-report.html"); + siteFile = validator.getSiteFile("failsafe.html"); assertTrue("Expecting failsafe report file", siteFile.isFile()); } @@ -66,10 +66,10 @@ public class Surefire772NoSurefireReportsIT extends SurefireJUnit4IntegrationTes .addSurefireReportOnlyGoal() .executeCurrentGoals(); - TestFile siteFile = validator.getSiteFile("surefire-report.html"); + TestFile siteFile = validator.getSiteFile("surefire.html"); assertFalse("Expecting no surefire report file", siteFile.isFile()); - siteFile = validator.getSiteFile("failsafe-report.html"); + siteFile = validator.getSiteFile("failsafe.html"); assertTrue("Expecting failsafe report file", siteFile.isFile()); } @@ -81,10 +81,10 @@ public class Surefire772NoSurefireReportsIT extends SurefireJUnit4IntegrationTes .addSurefireReportOnlyGoal() .executeCurrentGoals(); - TestFile siteFile = validator.getSiteFile("surefire-report.html"); + TestFile siteFile = validator.getSiteFile("surefire.html"); assertFalse("Expecting no surefire report file", siteFile.isFile()); - siteFile = validator.getSiteFile("failsafe-report.html"); + siteFile = validator.getSiteFile("failsafe.html"); assertTrue("Expecting failsafe report file", siteFile.isFile()); } diff --git a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire772SpecifiedReportsIT.java b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire772SpecifiedReportsIT.java index 6c57ae8c2..fc37d7b6f 100644 --- a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire772SpecifiedReportsIT.java +++ b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire772SpecifiedReportsIT.java @@ -38,10 +38,10 @@ public class Surefire772SpecifiedReportsIT extends SurefireJUnit4IntegrationTest OutputValidator validator = unpack().addFailsafeReportOnlyGoal().addSurefireReportOnlyGoal().executeCurrentGoals(); - TestFile siteFile = validator.getSiteFile("surefire-report.html"); + TestFile siteFile = validator.getSiteFile("surefire.html"); assertTrue("Expecting surefire report file", siteFile.isFile()); - siteFile = validator.getSiteFile("failsafe-report.html"); + siteFile = validator.getSiteFile("failsafe.html"); assertTrue("Expecting failsafe report file", siteFile.isFile()); } @@ -52,10 +52,10 @@ public class Surefire772SpecifiedReportsIT extends SurefireJUnit4IntegrationTest .addSurefireReportOnlyGoal() .executeCurrentGoals(); - TestFile siteFile = validator.getSiteFile("surefire-report.html"); + TestFile siteFile = validator.getSiteFile("surefire.html"); assertTrue("Expecting surefire report file", siteFile.isFile()); - siteFile = validator.getSiteFile("failsafe-report.html"); + siteFile = validator.getSiteFile("failsafe.html"); assertFalse("Expecting no failsafe report file", siteFile.isFile()); } @@ -66,10 +66,10 @@ public class Surefire772SpecifiedReportsIT extends SurefireJUnit4IntegrationTest .addSurefireReportOnlyGoal() .executeCurrentGoals(); - TestFile siteFile = validator.getSiteFile("surefire-report.html"); + TestFile siteFile = validator.getSiteFile("surefire.html"); assertFalse("Expecting no surefire report file", siteFile.isFile()); - siteFile = validator.getSiteFile("failsafe-report.html"); + siteFile = validator.getSiteFile("failsafe.html"); assertTrue("Expecting failsafe report file", siteFile.isFile()); }