Skip to content

Commit

Permalink
Merge pull request #37679 from dmlloyd/cds
Browse files Browse the repository at this point in the history
Skip CDS test if the JDK cannot create AppCDS
  • Loading branch information
geoand authored Dec 12, 2023
2 parents 49ec297 + 02b1eae commit 419ed1a
Showing 1 changed file with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
import static io.quarkus.maven.it.ApplicationNameAndVersionTestUtil.assertApplicationPropertiesSetCorrectly;
import static org.assertj.core.api.Assertions.assertThat;
import static org.assertj.core.api.Assertions.fail;
import static org.assertj.core.api.Assumptions.assumeThat;
import static org.awaitility.Awaitility.await;

import java.io.BufferedReader;
Expand Down Expand Up @@ -617,6 +618,8 @@ public void testThatAppCDSAreUsable() throws Exception {

await().atMost(TestUtils.getDefaultTimeout(), TimeUnit.MINUTES)
.until(() -> result.getProcess() != null && !result.getProcess().isAlive());
// just skip if the JDK can't create appcds
assumeThat(running.log()).doesNotContainIgnoringCase("Unable to create AppCDS");
assertThat(running.log()).containsIgnoringCase("BUILD SUCCESS");
running.stop();

Expand Down

0 comments on commit 419ed1a

Please sign in to comment.