diff --git a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractEditModeRewritingReleasePhaseTestCase.java b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractEditModeRewritingReleasePhaseTestCase.java index ecaf3a1a5..53a7e60d1 100644 --- a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractEditModeRewritingReleasePhaseTestCase.java +++ b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractEditModeRewritingReleasePhaseTestCase.java @@ -63,7 +63,7 @@ void testRewriteBasicPomWithEditMode() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -130,7 +130,7 @@ void testRewritePomPluginDependencies() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -161,7 +161,7 @@ void testRewritePomProfile() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test diff --git a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractReleaseTestCase.java b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractReleaseTestCase.java index ed04e707f..01d5451a9 100644 --- a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractReleaseTestCase.java +++ b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractReleaseTestCase.java @@ -243,16 +243,13 @@ protected static Map getProjectsAsMap(List r return map; } - protected boolean comparePomFiles(List reactorProjects) throws IOException { - return comparePomFiles(reactorProjects, true); + protected void comparePomFiles(List reactorProjects) throws IOException { + comparePomFiles(reactorProjects, true); } - protected boolean comparePomFiles(List reactorProjects, boolean normalizeLineEndings) + protected void comparePomFiles(List reactorProjects, boolean normalizeLineEndings) throws IOException { comparePomFiles(reactorProjects, "", normalizeLineEndings); - - // TODO: return void since this is redundant - return true; } protected void comparePomFiles(List reactorProjects, String expectedFileSuffix) throws IOException { @@ -279,10 +276,6 @@ protected void comparePomFiles(MavenProject project, String expectedFileSuffix, comparePomFiles(expectedFile, actualFile, normalizeLineEndings, false); } - protected void comparePomFiles(File expectedFile, File actualFile) throws IOException { - comparePomFiles(expectedFile, actualFile, true, false); - } - protected void comparePomFiles( File expectedFile, File actualFile, boolean normalizeLineEndings, boolean ignoreComments) throws IOException { diff --git a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractRewritingReleasePhaseTestCase.java b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractRewritingReleasePhaseTestCase.java index 10a48903a..efa316db4 100644 --- a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractRewritingReleasePhaseTestCase.java +++ b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractRewritingReleasePhaseTestCase.java @@ -73,7 +73,7 @@ public void testRewriteBasicPom() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -86,7 +86,7 @@ public void testRewriteBasicPomEntities() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -99,7 +99,7 @@ public void testRewriteBasicPomNamespace() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -112,7 +112,7 @@ public void testRewriteBasicPomWithEncoding() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -125,7 +125,7 @@ public void testRewritePomWithParent() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -167,7 +167,7 @@ public void testRewritePomWithReleasedParent() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } protected abstract void mapAlternateNextVersion(ReleaseDescriptorBuilder config, String projectId); @@ -182,7 +182,7 @@ public void testRewritePomWithInheritedVersion() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -213,7 +213,7 @@ public void testRewritePomDependencies() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -245,7 +245,7 @@ public void testRewritePomDependenciesDifferentVersion() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -258,7 +258,7 @@ public void testRewriteManagedPomDependencies() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -289,7 +289,7 @@ public void testRewritePomPlugins() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -320,7 +320,7 @@ public void testRewritePomPluginsDifferentVersion() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -333,7 +333,7 @@ public void testRewriteManagedPomPlugins() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -365,7 +365,7 @@ public void testRewritePomReportPlugins() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -397,7 +397,7 @@ public void testRewritePomReportPluginsDifferentVersion() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -410,7 +410,7 @@ public void testRewritePomExtension() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -443,7 +443,7 @@ public void testRewritePomExtensionDifferentVersion() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -456,7 +456,7 @@ public void testRewritePomExtensionUndefinedVersion() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -584,7 +584,7 @@ public void testRewriteWhitespaceAroundValues() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -598,7 +598,7 @@ public void testRewriteCommentsAroundValues() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -612,7 +612,7 @@ public void testRewriteCDataAroundValues() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -684,7 +684,7 @@ public void testRewritePomDependenciesWithNamespace() throws Exception { .execute( ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } protected abstract List prepareReactorProjects(String path) throws Exception; diff --git a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/GenerateReleasePomsPhaseTest.java b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/GenerateReleasePomsPhaseTest.java index 833dea869..b65197579 100644 --- a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/GenerateReleasePomsPhaseTest.java +++ b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/GenerateReleasePomsPhaseTest.java @@ -43,7 +43,6 @@ import org.junit.jupiter.api.Test; import static org.codehaus.plexus.testing.PlexusExtension.getTestFile; -import static org.junit.jupiter.api.Assertions.assertTrue; import static org.mockito.ArgumentMatchers.argThat; import static org.mockito.ArgumentMatchers.isA; import static org.mockito.Mockito.mock; @@ -139,7 +138,7 @@ void testFinalName() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } /* diff --git a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForBranchPhaseTest.java b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForBranchPhaseTest.java index 81b5b3b3f..62f583013 100644 --- a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForBranchPhaseTest.java +++ b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForBranchPhaseTest.java @@ -159,7 +159,7 @@ void testScmOverridden() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Override @@ -211,7 +211,7 @@ void testRewriteBasicPomWithGit() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -224,7 +224,7 @@ void testRewriteBasicPomWithScmExpression() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -237,7 +237,7 @@ void testRewriteBasicPomWithTagBase() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -250,7 +250,7 @@ void testRewriteBasicPomWithTagBaseAndVaryingScmUrls() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -266,7 +266,7 @@ void testRewriteBasicPomWithGitFromTag() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -277,7 +277,7 @@ void testRewriteBasicPomWithEmptyScm() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -287,7 +287,7 @@ void testRewriteInterpolatedVersions() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -319,7 +319,7 @@ void testRewriteBasicPomWithInheritedScm() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -334,7 +334,7 @@ void testRewritePomWithParentAndProperties() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } // MRELEASE-311 @@ -350,7 +350,7 @@ void testRewritePomWithDependencyPropertyCoordinate() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } // MRELEASE-305 @@ -365,7 +365,7 @@ void testRewritePomWithScmOfParentEndingWithASlash() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -380,7 +380,7 @@ void testRewritePomWithDeepSubprojects() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -392,7 +392,7 @@ void testRewritePomForFlatMultiModule() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } // MRELEASE-383 @@ -409,7 +409,7 @@ void testRewritePomWithCDATASectionOnWindows() throws Exception { phase.execute(builderReleaseDescriptor, new DefaultReleaseEnvironment(), reactorProjects); // compare POMS without line ending normalization - assertTrue(comparePomFiles(reactorProjects, false)); + comparePomFiles(reactorProjects, false); } protected ReleaseDescriptorBuilder createDescriptorFromProjects( @@ -428,7 +428,7 @@ void testRewritePomWithImportedDependencyManagementInReactor() throws Exception phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -440,6 +440,6 @@ void testRewritePomWithDifferentVersionsAcrossModules() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } } diff --git a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForDevelopmentPhaseTest.java b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForDevelopmentPhaseTest.java index ae7ee2a4b..a86ba6755 100644 --- a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForDevelopmentPhaseTest.java +++ b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForDevelopmentPhaseTest.java @@ -250,7 +250,7 @@ void testRewriteBasicPomWithGit() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -269,7 +269,7 @@ void testRewriteBasicPomWithGitFromTag() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -287,7 +287,7 @@ void testRewriteBasicPomWithSvnFromTag() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -312,7 +312,7 @@ void testRewriteBasicPomWithInheritedScm() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -347,7 +347,7 @@ private void performTestRewritePomWithParentAndProperties(String path) throws Ex phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -395,7 +395,7 @@ void testRewritePomWithDependencyPropertyCoordinate() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -409,7 +409,7 @@ void testRewritePomDependenciesWithoutDependenciesVersionUpdate() throws Excepti phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -427,7 +427,7 @@ void testRewritePomWithCiFriendlyReactor() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -442,6 +442,6 @@ void testRewritePomWithCiFriendlyReactorWithOnlyRevision() throws Exception { builder.addDevelopmentVersion("groupId:subproject1", NEXT_VERSION); phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } } diff --git a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForReleasePhaseTest.java b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForReleasePhaseTest.java index 482af75b6..aa386c6bc 100644 --- a/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForReleasePhaseTest.java +++ b/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForReleasePhaseTest.java @@ -159,7 +159,7 @@ void testScmOverridden() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Override @@ -211,7 +211,7 @@ void testRewriteBasicPomWithGit() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -224,7 +224,7 @@ void testRewriteBasicPomWithScmExpression() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -237,7 +237,7 @@ void testRewriteBasicPomWithTagBase() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -250,7 +250,7 @@ void testRewriteBasicPomWithTagBaseAndVaryingScmUrls() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -266,7 +266,7 @@ void testRewriteBasicPomWithGitFromTag() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -277,7 +277,7 @@ void testRewriteBasicPomWithEmptyScm() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -287,7 +287,7 @@ void testRewriteInterpolatedVersions() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -321,7 +321,7 @@ void testRewriteBasicPomWithInheritedScm() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -336,7 +336,7 @@ void testRewritePomWithParentAndProperties() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } // MRELEASE-1109 @@ -350,7 +350,7 @@ void testRewritePomWithCiFriendlyReactor() throws Exception { builder.addReleaseVersion("groupId:subproject1", NEXT_VERSION); phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -363,7 +363,7 @@ void testRewritePomWithCiFriendlyReactorWithOnlyRevision() throws Exception { builder.addReleaseVersion("groupId:subproject1", NEXT_VERSION); phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } // MRELEASE-311 @@ -379,7 +379,7 @@ void testRewritePomWithDependencyPropertyCoordinate() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } // MRELEASE-305 @@ -394,7 +394,7 @@ void testRewritePomWithScmOfParentEndingWithASlash() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -409,7 +409,7 @@ void testRewritePomWithDeepSubprojects() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -421,7 +421,7 @@ void testRewritePomForFlatMultiModule() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } // MRELEASE-383 @@ -438,7 +438,7 @@ void testRewritePomWithCDATASectionOnWindows() throws Exception { phase.execute(builderReleaseDescriptor, new DefaultReleaseEnvironment(), reactorProjects); // compare POMS without line ending normalization - assertTrue(comparePomFiles(reactorProjects, false)); + comparePomFiles(reactorProjects, false); } protected ReleaseDescriptorBuilder createDescriptorFromProjects( @@ -460,7 +460,7 @@ void testRewritePomWithExternallyReleasedParent() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } // MRELEASE-454 @@ -472,7 +472,7 @@ void testRewritePomWithImportedDependencyManagementInReactor() throws Exception phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -484,7 +484,7 @@ void testRewritePomWithDifferentVersionsAcrossModules() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } @Test @@ -499,6 +499,6 @@ void testRewritePomWithCheckModificationExcludes() throws Exception { phase.execute(ReleaseUtils.buildReleaseDescriptor(builder), new DefaultReleaseEnvironment(), reactorProjects); - assertTrue(comparePomFiles(reactorProjects)); + comparePomFiles(reactorProjects); } }