Skip to content

Commit

Permalink
Adjust ITs for new multi-mod behavior
Browse files Browse the repository at this point in the history
Signed-off-by: Scott Kurz <[email protected]>
  • Loading branch information
scottkurz committed Aug 6, 2022
1 parent 727966a commit b77d937
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ public static void setUpBeforeClass() throws Exception {
}

@Test
public void purgeLocallyInstalledModule_DevMode_Test() throws Exception {
public void purgeUpstreamSourcePart_DevMode_Test() throws Exception {

// install everything to m2
runCommand("mvn install");
Expand All @@ -58,7 +58,7 @@ public void purgeLocallyInstalledModule_DevMode_Test() throws Exception {
startProcess(null, true, "mvn io.openliberty.tools:liberty-maven-plugin:"+System.getProperty("mavenPluginVersion")+":", false);

// TODO when https://github.com/OpenLiberty/ci.maven/issues/1203 is fixed, check for compilation error instead
assertTrue(getLogTail(logFile), verifyLogMessageExists("Could not resolve dependencies for project", 5000));
assertTrue(getLogTail(logFile), verifyLogMessageExists("error: package io.openliberty.guides.multimodules.lib does not exist", 5000));

assertFalse(getLogTail(logFile), targetClass.exists());
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ public static void setUpBeforeClass() throws Exception {
}

@Test
public void purgeLocallyInstalledModule_LibertyRun_Test() throws Exception {
public void purgeUpstreamSourcePart_LibertyRun_Test() throws Exception {

// install everything to m2
runCommand("mvn install");
Expand All @@ -58,7 +58,7 @@ public void purgeLocallyInstalledModule_LibertyRun_Test() throws Exception {
startProcess(null, false, "mvn io.openliberty.tools:liberty-maven-plugin:"+System.getProperty("mavenPluginVersion")+":", false);

// TODO when https://github.com/OpenLiberty/ci.maven/issues/1203 is fixed, check for compilation error instead
assertTrue(getLogTail(logFile), verifyLogMessageExists("Could not resolve dependencies for project", 5000));
assertTrue(getLogTail(logFile), verifyLogMessageExists("error: package io.openliberty.guides.multimodules.lib does not exist", 5000));

assertFalse(getLogTail(logFile), targetClass.exists());
}
Expand Down

0 comments on commit b77d937

Please sign in to comment.