Skip to content

[MNG-5760] Add -r/--resume to automatically resume from the last fa… #93

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
michael-o wants to merge 1 commit into from
Closed

[MNG-5760] Add -r/--resume to automatically resume from the last fa… #93

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 8 additions & 7 deletions core-it-suite/src/test/java/org/apache/maven/it/MavenITmng5760ResumeFeatureTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ public MavenITmng5760ResumeFeatureTest() throws IOException {
*/
public void testShouldSuggestToResumeWithoutArgs() throws Exception
{
final Verifier verifier = newVerifier( parentDependentTestDir.getAbsolutePath() );
Verifier verifier = newVerifier( parentDependentTestDir.getAbsolutePath() );
verifier.addCliOption( "-Dmodule-b.fail=true" );

try
Expand All @@ -78,17 +78,18 @@ public void testShouldSuggestToResumeWithoutArgs() throws Exception
}

// New build with -r should resume the build from module-b, skipping module-a since it has succeeded already.
verifier.getCliOptions().clear();
verifier = newVerifier( parentDependentTestDir.getAbsolutePath() );
verifier.addCliOption( "-r" );
verifier.executeGoal( "test" );
verifyTextNotInLog( verifier, "Building module-a 1.0" );
verifier.verifyTextInLog( "Building module-b 1.0" );
verifier.verifyTextInLog( "Building module-c 1.0" );
verifier.resetStreams();
}

public void testShouldSkipSuccessfulProjects() throws Exception
{
final Verifier verifier = newVerifier( parentDependentTestDir.getAbsolutePath() );
Verifier verifier = newVerifier( parentDependentTestDir.getAbsolutePath() );
verifier.addCliOption( "-Dmodule-a.fail=true" );
verifier.addCliOption( "--fail-at-end");

Expand All @@ -106,9 +107,8 @@ public void testShouldSkipSuccessfulProjects() throws Exception
verifier.resetStreams();
}

verifier.getCliOptions().clear();

// Let module-b and module-c fail, if they would have been built...
verifier = newVerifier( parentDependentTestDir.getAbsolutePath() );
verifier.addCliOption( "-Dmodule-b.fail=true" );
verifier.addCliOption( "-Dmodule-c.fail=true" );
// ... but adding -r should exclude those two from the build because the previous Maven invocation
Expand All @@ -129,7 +129,7 @@ public void testShouldSkipSuccessfulModulesWhenTheFirstModuleFailed() throws Exc
// In this multi-module project, the submodules are not dependent on the parent.
// This results in the parent to be built last, and module-a to be built first.
// This enables us to let the first module in the reactor (module-a) fail.
final Verifier verifier = newVerifier( parentIndependentTestDir.getAbsolutePath() );
Verifier verifier = newVerifier( parentIndependentTestDir.getAbsolutePath() );
verifier.addCliOption( "-Dmodule-a.fail=true" );
verifier.addCliOption( "--fail-at-end");

Expand All @@ -147,11 +147,12 @@ public void testShouldSkipSuccessfulModulesWhenTheFirstModuleFailed() throws Exc
verifier.resetStreams();
}

verifier.getCliOptions().clear();
verifier = newVerifier( parentIndependentTestDir.getAbsolutePath() );
verifier.addCliOption( "-r" );
verifier.executeGoal( "test" );
verifier.verifyTextInLog( "Building module-a 1.0" );
verifyTextNotInLog( verifier, "Building module-b 1.0" );
verifier.resetStreams();
}

public void testShouldNotCrashWithoutProject() throws Exception
Expand Down