From d455b9894ae508d5aa859b7b8ae42debaadb8137 Mon Sep 17 00:00:00 2001 From: Marc Philipp Date: Mon, 1 Nov 2021 10:50:57 +0100 Subject: [PATCH] Always update snapshots and remove debug logging as it might leak secrets --- .../java/platform/tooling/support/tests/JavaVersionsTests.java | 2 +- .../java/platform/tooling/support/tests/MavenStarterTests.java | 2 +- .../platform/tooling/support/tests/MultiReleaseJarTests.java | 3 ++- .../tooling/support/tests/VintageMavenIntegrationTests.java | 2 +- 4 files changed, 5 insertions(+), 4 deletions(-) diff --git a/platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/JavaVersionsTests.java b/platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/JavaVersionsTests.java index 5b9a2d76aa80..ca545d46368f 100644 --- a/platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/JavaVersionsTests.java +++ b/platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/JavaVersionsTests.java @@ -54,7 +54,7 @@ List execute(String version, Path javaHome) { .setProject("java-versions") // .setWorkspace("java-versions-" + version) // .addArguments("-Dmaven.repo=" + MavenRepo.dir()) // - .addArguments("--debug", "--batch-mode", "verify") // + .addArguments("--update-snapshots", "--batch-mode", "verify") // .setTimeout(TOOL_TIMEOUT) // .setJavaHome(javaHome) // .build().run(); diff --git a/platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/MavenStarterTests.java b/platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/MavenStarterTests.java index 95e0c8493bac..55bfe5020d23 100644 --- a/platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/MavenStarterTests.java +++ b/platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/MavenStarterTests.java @@ -34,7 +34,7 @@ void verifyMavenStarterProject() { .setTool(Request.maven()) // .setProject("maven-starter") // .addArguments("-Dmaven.repo=" + MavenRepo.dir()) // - .addArguments("--debug", "--batch-mode", "verify") // + .addArguments("--update-snapshots", "--batch-mode", "verify") // .setTimeout(TOOL_TIMEOUT) // .setJavaHome(Helper.getJavaHome("8").orElseThrow(TestAbortedException::new)) // .build() // diff --git a/platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/MultiReleaseJarTests.java b/platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/MultiReleaseJarTests.java index 34d0e2a7f53e..a371d9f8a93a 100644 --- a/platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/MultiReleaseJarTests.java +++ b/platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/MultiReleaseJarTests.java @@ -84,7 +84,8 @@ private Result mvn(String variant) { .setTool(Request.maven()) // .setProject("multi-release-jar") // .addArguments("-Dmaven.repo=" + MavenRepo.dir()) // - .addArguments("--show-version", "--errors", "--batch-mode", "--file", variant, "test") // + .addArguments("--update-snapshots", "--show-version", "--errors", "--batch-mode", "--file", variant, + "test") // .setTimeout(TOOL_TIMEOUT) // .build() // .run(); diff --git a/platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/VintageMavenIntegrationTests.java b/platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/VintageMavenIntegrationTests.java index 7f2936bc90ad..267175b9e285 100644 --- a/platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/VintageMavenIntegrationTests.java +++ b/platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/VintageMavenIntegrationTests.java @@ -59,7 +59,7 @@ private Result run(String version) { .setJavaHome(Helper.getJavaHome("8").orElseThrow(TestAbortedException::new)) // .setProject("vintage") // .setWorkspace("vintage-maven-" + version) // - .addArguments("clean", "test", "--debug", "--batch-mode") // + .addArguments("clean", "test", "--update-snapshots", "--batch-mode") // .addArguments("-Dmaven.repo=" + MavenRepo.dir()) // .addArguments("-Djunit4Version=" + version) // .setTimeout(TOOL_TIMEOUT) //