From a58cc723e2796352eb0aceac625bbf31c1982aee Mon Sep 17 00:00:00 2001 From: wind57 Date: Wed, 23 Aug 2023 19:14:01 +0300 Subject: [PATCH 01/10] dirty --- .../cloud/kubernetes/commons/DummyTest.java | 28 ++++++ spring-cloud-kubernetes-test-support/pom.xml | 8 ++ .../DisabledTestsCondition.java | 1 + .../main/resources/junit-platform.properties | 1 - .../test/support/TestsDiscovery.java | 97 +++++++++++++++++++ 5 files changed, 134 insertions(+), 1 deletion(-) create mode 100644 spring-cloud-kubernetes-commons/src/test/java/org/springframework/cloud/kubernetes/commons/DummyTest.java delete mode 100644 spring-cloud-kubernetes-test-support/src/main/resources/junit-platform.properties create mode 100644 spring-cloud-kubernetes-test-support/src/test/java/org/springframework/cloud/kubernetes/test/support/TestsDiscovery.java diff --git a/spring-cloud-kubernetes-commons/src/test/java/org/springframework/cloud/kubernetes/commons/DummyTest.java b/spring-cloud-kubernetes-commons/src/test/java/org/springframework/cloud/kubernetes/commons/DummyTest.java new file mode 100644 index 0000000000..5e9b83a09c --- /dev/null +++ b/spring-cloud-kubernetes-commons/src/test/java/org/springframework/cloud/kubernetes/commons/DummyTest.java @@ -0,0 +1,28 @@ +/* + * Copyright 2013-2022 the original author or authors. + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * https://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.springframework.cloud.kubernetes.commons; + +import org.junit.Test; + +public class DummyTest { + + @Test + public void test() { + + } + +} diff --git a/spring-cloud-kubernetes-test-support/pom.xml b/spring-cloud-kubernetes-test-support/pom.xml index 4de47972f4..e761d293e0 100644 --- a/spring-cloud-kubernetes-test-support/pom.xml +++ b/spring-cloud-kubernetes-test-support/pom.xml @@ -16,6 +16,14 @@ 1.18.0 + + + org.junit.platform + junit-platform-launcher + test + + + org.springframework.boot spring-boot-starter-test diff --git a/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/tests/commons/junit_extension/DisabledTestsCondition.java b/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/tests/commons/junit_extension/DisabledTestsCondition.java index f141f07c93..b4f92e152a 100644 --- a/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/tests/commons/junit_extension/DisabledTestsCondition.java +++ b/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/tests/commons/junit_extension/DisabledTestsCondition.java @@ -35,6 +35,7 @@ * * @author wind57 */ +@Deprecated public class DisabledTestsCondition implements ExecutionCondition { private static final boolean SKIP_RUNNING_TESTS = "true".equals(System.getProperty("spring.cloud.k8s.skip.tests")); diff --git a/spring-cloud-kubernetes-test-support/src/main/resources/junit-platform.properties b/spring-cloud-kubernetes-test-support/src/main/resources/junit-platform.properties deleted file mode 100644 index 6efc0d5e85..0000000000 --- a/spring-cloud-kubernetes-test-support/src/main/resources/junit-platform.properties +++ /dev/null @@ -1 +0,0 @@ -junit.jupiter.extensions.autodetection.enabled=true diff --git a/spring-cloud-kubernetes-test-support/src/test/java/org/springframework/cloud/kubernetes/test/support/TestsDiscovery.java b/spring-cloud-kubernetes-test-support/src/test/java/org/springframework/cloud/kubernetes/test/support/TestsDiscovery.java new file mode 100644 index 0000000000..02d21522b6 --- /dev/null +++ b/spring-cloud-kubernetes-test-support/src/test/java/org/springframework/cloud/kubernetes/test/support/TestsDiscovery.java @@ -0,0 +1,97 @@ +/* + * Copyright 2013-2023 the original author or authors. + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * https://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.springframework.cloud.kubernetes.test.support; + +import java.io.File; +import java.net.URI; +import java.net.URL; +import java.net.URLClassLoader; +import java.nio.file.Files; +import java.nio.file.Path; +import java.nio.file.Paths; +import java.util.HashSet; +import java.util.List; +import java.util.Set; +import java.util.stream.Collectors; +import java.util.stream.Stream; + +import org.junit.jupiter.api.Test; +import org.junit.platform.engine.discovery.DiscoverySelectors; +import org.junit.platform.launcher.Launcher; +import org.junit.platform.launcher.LauncherDiscoveryRequest; +import org.junit.platform.launcher.TestIdentifier; +import org.junit.platform.launcher.TestPlan; +import org.junit.platform.launcher.core.LauncherDiscoveryRequestBuilder; +import org.junit.platform.launcher.core.LauncherFactory; + +/** + * This is not really a test, but a way for us to discover all the tests using ... + * + * @author wind57 + */ +class TestsDiscovery { + + @Test + void discoverTests() throws Exception { + + List classpath = entireClasspath(); + + List targetClasses = classpath.stream().filter(x -> x.contains("target/classes")).toList(); + List targetTestClasses = targetClasses.stream().map(x -> x.replace("classes", "test-classes")).toList(); + List jars = targetTestClasses.stream().filter(x -> x.contains(".jar")).toList(); + + List urls = Stream.of(targetClasses, targetTestClasses, jars).flatMap(List::stream) + .map(x -> toURL(new File(x).toPath().toUri())).toList(); + + Set paths = Stream.of(targetClasses, targetTestClasses, jars).flatMap(List::stream) + .map(Paths::get).collect(Collectors.toSet()); + + replaceClassloader(urls); + + LauncherDiscoveryRequest request = LauncherDiscoveryRequestBuilder.request() + .selectors(DiscoverySelectors.selectClasspathRoots(paths)) + .build(); + + Launcher launcher = LauncherFactory.openSession().getLauncher(); + TestPlan testPlan = launcher.discover(request); + testPlan.getRoots().stream().flatMap(x -> testPlan.getChildren(x).stream()) + .map(TestIdentifier::getLegacyReportingName).sorted().forEach(System.out::println); + + } + + private static void replaceClassloader(List classpathURLs) { + ClassLoader parentClassLoader = Thread.currentThread().getContextClassLoader(); + URLClassLoader classLoader = URLClassLoader.newInstance(classpathURLs.toArray(new URL[0]), parentClassLoader); + Thread.currentThread().setContextClassLoader(classLoader); + } + + // /tmp/deps.txt are created by the pipeline + private static List entireClasspath() throws Exception { + return Files.lines(Paths.get("/tmp/deps.txt")).distinct().collect(Collectors.toList()); + } + + private static URL toURL(URI uri) { + try { + return uri.toURL(); + } + catch (Exception e) { + throw new RuntimeException(e); + } + } + +} From 2b2fd43658656661ca1b866ccf7c850b53b00cfa Mon Sep 17 00:00:00 2001 From: wind57 Date: Fri, 25 Aug 2023 08:55:27 +0300 Subject: [PATCH 02/10] first --- .../action.yaml | 24 ++++++++++---- .../pom.xml | 5 --- .../pom.xml | 5 --- .../pom.xml | 5 --- .../pom.xml | 5 --- .../pom.xml | 6 ---- spring-cloud-kubernetes-test-support/pom.xml | 24 ++++++++++++-- .../integration/tests/commons/K8SUtils.java | 2 +- .../tests/commons/fabric8_client/Util.java | 2 +- .../tests/commons/native_client/Util.java | 2 +- .../tests/discovery}/TestsDiscovery.java | 33 +++++++------------ src/checkstyle/checkstyle-suppressions.xml | 1 + 12 files changed, 55 insertions(+), 59 deletions(-) rename spring-cloud-kubernetes-test-support/src/{test/java/org/springframework/cloud/kubernetes/test/support => main/java/org/springframework/cloud/kubernetes/tests/discovery}/TestsDiscovery.java (76%) diff --git a/.github/workflows/composites/maven-build-with-dry-run-for-tests/action.yaml b/.github/workflows/composites/maven-build-with-dry-run-for-tests/action.yaml index dd709b8062..369427a2f1 100644 --- a/.github/workflows/composites/maven-build-with-dry-run-for-tests/action.yaml +++ b/.github/workflows/composites/maven-build-with-dry-run-for-tests/action.yaml @@ -3,14 +3,26 @@ description: maven-build-with-dry-run-for-tests runs: using: "composite" steps: - - name: maven-build-with-dry-run-for-tests + + - name: run 'package' on the project + shell: bash + run: | + ./mvnw package -B \ + -Dskip.build.image=true \ + -DskipTests -DskipITs \ + -T 1C -q + + - name: find all classpath entries + shell: bash + run: | + ./mvnw -q exec:exec -Dexec.classpathScope="test" -Dexec.executable="echo" -Dexec.args="%classpath" | tr : "\n" > /tmp/deps.txt + + - name: find all tests shell: bash run: | - # find all the tests that are supposed to be run, but don't actually run them. - # this is achieved via: 'spring.cloud.k8s.skip.tests=true' in DisabledTestsCondition - ./mvnw test -B -Dskip.build.image=true -Dspring.cloud.k8s.skip.tests=true \ - -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=error \ - -T 1C -q > /tmp/tests.txt + cd spring-cloud-kubernetes-test-support + ./mvnw -q exec:java -Dexec.mainClass="org.springframework.cloud.kubernetes.tests.discovery.TestsDiscovery" > /tmp/tests.txt + cd .. - name: show result if: always() diff --git a/spring-cloud-kubernetes-client-discovery/pom.xml b/spring-cloud-kubernetes-client-discovery/pom.xml index 365907aede..72974abed3 100644 --- a/spring-cloud-kubernetes-client-discovery/pom.xml +++ b/spring-cloud-kubernetes-client-discovery/pom.xml @@ -50,11 +50,6 @@ spring-boot-starter-test test - - org.junit.vintage - junit-vintage-engine - test - org.springframework.boot spring-boot-starter-web diff --git a/spring-cloud-kubernetes-controllers/spring-cloud-kubernetes-configuration-watcher/pom.xml b/spring-cloud-kubernetes-controllers/spring-cloud-kubernetes-configuration-watcher/pom.xml index b2149b83a1..cc9c63cc6b 100644 --- a/spring-cloud-kubernetes-controllers/spring-cloud-kubernetes-configuration-watcher/pom.xml +++ b/spring-cloud-kubernetes-controllers/spring-cloud-kubernetes-configuration-watcher/pom.xml @@ -66,11 +66,6 @@ spring-rabbit-test test - - org.junit.vintage - junit-vintage-engine - test - org.mockito mockito-inline diff --git a/spring-cloud-kubernetes-examples/kubernetes-leader-election-example/pom.xml b/spring-cloud-kubernetes-examples/kubernetes-leader-election-example/pom.xml index 30c5e5a978..d6bc61dbc0 100644 --- a/spring-cloud-kubernetes-examples/kubernetes-leader-election-example/pom.xml +++ b/spring-cloud-kubernetes-examples/kubernetes-leader-election-example/pom.xml @@ -49,11 +49,6 @@ spring-boot-starter-test test - - org.junit.vintage - junit-vintage-engine - test - diff --git a/spring-cloud-kubernetes-fabric8-config/pom.xml b/spring-cloud-kubernetes-fabric8-config/pom.xml index 1e327e30d8..40d1e2b30c 100644 --- a/spring-cloud-kubernetes-fabric8-config/pom.xml +++ b/spring-cloud-kubernetes-fabric8-config/pom.xml @@ -97,11 +97,6 @@ spring-boot-starter-test test - - org.junit.vintage - junit-vintage-engine - test - org.springframework.cloud spring-cloud-test-support diff --git a/spring-cloud-kubernetes-fabric8-discovery/pom.xml b/spring-cloud-kubernetes-fabric8-discovery/pom.xml index ef4323e057..9d9a513ef6 100644 --- a/spring-cloud-kubernetes-fabric8-discovery/pom.xml +++ b/spring-cloud-kubernetes-fabric8-discovery/pom.xml @@ -68,12 +68,6 @@ test - - org.junit.vintage - junit-vintage-engine - test - - org.springframework.boot spring-boot-starter-web diff --git a/spring-cloud-kubernetes-test-support/pom.xml b/spring-cloud-kubernetes-test-support/pom.xml index e761d293e0..61e165db2d 100644 --- a/spring-cloud-kubernetes-test-support/pom.xml +++ b/spring-cloud-kubernetes-test-support/pom.xml @@ -13,17 +13,15 @@ 4.0.3 - 1.18.0 + 1.19.0 org.junit.platform junit-platform-launcher - test - org.springframework.boot spring-boot-starter-test @@ -51,4 +49,24 @@ + + + + org.codehaus.mojo + exec-maven-plugin + 3.1.0 + + + + java + + + + + org.springframework.cloud.kubernetes.tests.discovery.TestsDiscovery + + + + + diff --git a/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/integration/tests/commons/K8SUtils.java b/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/integration/tests/commons/K8SUtils.java index ad86be2176..dee9bf7fc4 100644 --- a/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/integration/tests/commons/K8SUtils.java +++ b/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/integration/tests/commons/K8SUtils.java @@ -69,7 +69,7 @@ import org.springframework.util.StringUtils; import static org.awaitility.Awaitility.await; -import static org.junit.Assert.fail; +import static org.junit.jupiter.api.Assertions.fail; /** * @author Ryan Baxter diff --git a/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/integration/tests/commons/fabric8_client/Util.java b/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/integration/tests/commons/fabric8_client/Util.java index 85a04deba9..74060aa53d 100644 --- a/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/integration/tests/commons/fabric8_client/Util.java +++ b/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/integration/tests/commons/fabric8_client/Util.java @@ -50,7 +50,7 @@ import org.springframework.cloud.kubernetes.integration.tests.commons.Phase; import static org.awaitility.Awaitility.await; -import static org.junit.Assert.fail; +import static org.junit.jupiter.api.Assertions.fail; import static org.springframework.cloud.kubernetes.integration.tests.commons.Commons.loadImage; import static org.springframework.cloud.kubernetes.integration.tests.commons.Commons.pomVersion; import static org.springframework.cloud.kubernetes.integration.tests.commons.Commons.pullImage; diff --git a/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/integration/tests/commons/native_client/Util.java b/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/integration/tests/commons/native_client/Util.java index 0dc5efee3d..0cddfbbb87 100644 --- a/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/integration/tests/commons/native_client/Util.java +++ b/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/integration/tests/commons/native_client/Util.java @@ -60,7 +60,7 @@ import org.springframework.cloud.kubernetes.integration.tests.commons.Phase; import static org.awaitility.Awaitility.await; -import static org.junit.Assert.fail; +import static org.junit.jupiter.api.Assertions.fail; import static org.springframework.cloud.kubernetes.integration.tests.commons.Commons.loadImage; import static org.springframework.cloud.kubernetes.integration.tests.commons.Commons.pomVersion; import static org.springframework.cloud.kubernetes.integration.tests.commons.Commons.pullImage; diff --git a/spring-cloud-kubernetes-test-support/src/test/java/org/springframework/cloud/kubernetes/test/support/TestsDiscovery.java b/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/tests/discovery/TestsDiscovery.java similarity index 76% rename from spring-cloud-kubernetes-test-support/src/test/java/org/springframework/cloud/kubernetes/test/support/TestsDiscovery.java rename to spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/tests/discovery/TestsDiscovery.java index 02d21522b6..528613c37b 100644 --- a/spring-cloud-kubernetes-test-support/src/test/java/org/springframework/cloud/kubernetes/test/support/TestsDiscovery.java +++ b/spring-cloud-kubernetes-test-support/src/main/java/org/springframework/cloud/kubernetes/tests/discovery/TestsDiscovery.java @@ -14,7 +14,7 @@ * limitations under the License. */ -package org.springframework.cloud.kubernetes.test.support; +package org.springframework.cloud.kubernetes.tests.discovery; import java.io.File; import java.net.URI; @@ -23,13 +23,11 @@ import java.nio.file.Files; import java.nio.file.Path; import java.nio.file.Paths; -import java.util.HashSet; import java.util.List; import java.util.Set; import java.util.stream.Collectors; import java.util.stream.Stream; -import org.junit.jupiter.api.Test; import org.junit.platform.engine.discovery.DiscoverySelectors; import org.junit.platform.launcher.Launcher; import org.junit.platform.launcher.LauncherDiscoveryRequest; @@ -39,39 +37,32 @@ import org.junit.platform.launcher.core.LauncherFactory; /** - * This is not really a test, but a way for us to discover all the tests using ... - * * @author wind57 */ -class TestsDiscovery { - - @Test - void discoverTests() throws Exception { +public class TestsDiscovery { - List classpath = entireClasspath(); - - List targetClasses = classpath.stream().filter(x -> x.contains("target/classes")).toList(); + public static void main(String[] args) throws Exception { + List targetClasses = entireClasspath().stream().filter(x -> x.contains("target/classes")).toList(); List targetTestClasses = targetClasses.stream().map(x -> x.replace("classes", "test-classes")).toList(); - List jars = targetTestClasses.stream().filter(x -> x.contains(".jar")).toList(); + List jars = entireClasspath().stream().filter(x -> x.contains(".jar")).toList(); List urls = Stream.of(targetClasses, targetTestClasses, jars).flatMap(List::stream) - .map(x -> toURL(new File(x).toPath().toUri())).toList(); + .map(x -> toURL(new File(x).toPath().toUri())).toList(); - Set paths = Stream.of(targetClasses, targetTestClasses, jars).flatMap(List::stream) - .map(Paths::get).collect(Collectors.toSet()); + Set paths = Stream.of(targetClasses, targetTestClasses, jars).flatMap(List::stream).map(Paths::get) + .collect(Collectors.toSet()); replaceClassloader(urls); LauncherDiscoveryRequest request = LauncherDiscoveryRequestBuilder.request() - .selectors(DiscoverySelectors.selectClasspathRoots(paths)) - .build(); + .selectors(DiscoverySelectors.selectClasspathRoots(paths)).build(); Launcher launcher = LauncherFactory.openSession().getLauncher(); TestPlan testPlan = launcher.discover(request); testPlan.getRoots().stream().flatMap(x -> testPlan.getChildren(x).stream()) - .map(TestIdentifier::getLegacyReportingName).sorted().forEach(System.out::println); - + .map(TestIdentifier::getLegacyReportingName).sorted().forEach(test -> { + System.out.println("spring.cloud.k8s.test.to.run -> " + test); + }); } private static void replaceClassloader(List classpathURLs) { diff --git a/src/checkstyle/checkstyle-suppressions.xml b/src/checkstyle/checkstyle-suppressions.xml index deeaaf2fd4..97dc2deb9d 100644 --- a/src/checkstyle/checkstyle-suppressions.xml +++ b/src/checkstyle/checkstyle-suppressions.xml @@ -16,4 +16,5 @@ + From 54edbb40991d632ac16b0f5b333e24d7c396110c Mon Sep 17 00:00:00 2001 From: wind57 Date: Fri, 25 Aug 2023 08:56:40 +0300 Subject: [PATCH 03/10] clean --- .../cloud/kubernetes/commons/DummyTest.java | 28 ------------------- 1 file changed, 28 deletions(-) delete mode 100644 spring-cloud-kubernetes-commons/src/test/java/org/springframework/cloud/kubernetes/commons/DummyTest.java diff --git a/spring-cloud-kubernetes-commons/src/test/java/org/springframework/cloud/kubernetes/commons/DummyTest.java b/spring-cloud-kubernetes-commons/src/test/java/org/springframework/cloud/kubernetes/commons/DummyTest.java deleted file mode 100644 index 5e9b83a09c..0000000000 --- a/spring-cloud-kubernetes-commons/src/test/java/org/springframework/cloud/kubernetes/commons/DummyTest.java +++ /dev/null @@ -1,28 +0,0 @@ -/* - * Copyright 2013-2022 the original author or authors. - * - * Licensed under the Apache License, Version 2.0 (the "License"); - * you may not use this file except in compliance with the License. - * You may obtain a copy of the License at - * - * https://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ - -package org.springframework.cloud.kubernetes.commons; - -import org.junit.Test; - -public class DummyTest { - - @Test - public void test() { - - } - -} From 1457d68f35786f5037d8e65617056b95480814f6 Mon Sep 17 00:00:00 2001 From: wind57 Date: Fri, 25 Aug 2023 12:17:46 +0300 Subject: [PATCH 04/10] fix --- .../composites/maven-build-with-dry-run-for-tests/action.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/composites/maven-build-with-dry-run-for-tests/action.yaml b/.github/workflows/composites/maven-build-with-dry-run-for-tests/action.yaml index 369427a2f1..e007645ba8 100644 --- a/.github/workflows/composites/maven-build-with-dry-run-for-tests/action.yaml +++ b/.github/workflows/composites/maven-build-with-dry-run-for-tests/action.yaml @@ -21,7 +21,7 @@ runs: shell: bash run: | cd spring-cloud-kubernetes-test-support - ./mvnw -q exec:java -Dexec.mainClass="org.springframework.cloud.kubernetes.tests.discovery.TestsDiscovery" > /tmp/tests.txt + .././mvnw -q exec:java -Dexec.mainClass="org.springframework.cloud.kubernetes.tests.discovery.TestsDiscovery" > /tmp/tests.txt cd .. - name: show result From cc3c7003a0fa20007cb2ceeaa703eb42fb73cb87 Mon Sep 17 00:00:00 2001 From: wind57 Date: Fri, 25 Aug 2023 14:06:09 +0300 Subject: [PATCH 05/10] placeholder commit From 73b3e277c5a919a34697b5013cedb9292e17012b Mon Sep 17 00:00:00 2001 From: wind57 Date: Fri, 25 Aug 2023 14:12:12 +0300 Subject: [PATCH 06/10] fix --- .../composites/maven-build-with-dry-run-for-tests/action.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/composites/maven-build-with-dry-run-for-tests/action.yaml b/.github/workflows/composites/maven-build-with-dry-run-for-tests/action.yaml index e007645ba8..f2b6696192 100644 --- a/.github/workflows/composites/maven-build-with-dry-run-for-tests/action.yaml +++ b/.github/workflows/composites/maven-build-with-dry-run-for-tests/action.yaml @@ -7,7 +7,7 @@ runs: - name: run 'package' on the project shell: bash run: | - ./mvnw package -B \ + ./mvnw install -B \ -Dskip.build.image=true \ -DskipTests -DskipITs \ -T 1C -q From 7d6fded02eb867b7fd17870d6e4c4645aff96123 Mon Sep 17 00:00:00 2001 From: wind57 Date: Fri, 25 Aug 2023 19:11:05 +0300 Subject: [PATCH 07/10] fix failure --- .../run-and-save-test-times-when-cache-present/action.yaml | 6 +++--- .github/workflows/composites/test-times/action.yaml | 4 ++-- .github/workflows/maven.yaml | 4 ++-- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/.github/workflows/composites/run-and-save-test-times-when-cache-present/action.yaml b/.github/workflows/composites/run-and-save-test-times-when-cache-present/action.yaml index 4093ab1aee..147475f458 100644 --- a/.github/workflows/composites/run-and-save-test-times-when-cache-present/action.yaml +++ b/.github/workflows/composites/run-and-save-test-times-when-cache-present/action.yaml @@ -8,8 +8,8 @@ runs: uses: actions/cache/restore@v3 with: path: /tmp/sorted.txt - key: ${{ runner.os }}-spring-cloud-kubernetes-existing-test-times-cache-${{ github.run_id }} - restore-keys: ${{ runner.os }}-spring-cloud-kubernetes-existing-test-times-cache- + key: ${{ runner.os }}-spring-cloud-k8s-existing-test-times-cache-${{ github.run_id }} + restore-keys: ${{ runner.os }}-spring-cloud-k8s-existing-test-times-cache- - name: show cached test times shell: bash @@ -138,7 +138,7 @@ runs: # the previous cache will contain more tests then we currently have, so some # matrix instances will have no tests to run if [[ "$sum" -eq "0" ]]; then - echo "no tests to run in current index, most probably this PR removed some tests" + echo "no tests (with known times) to run in current index" tests_to_run_in_current_index='none' fi diff --git a/.github/workflows/composites/test-times/action.yaml b/.github/workflows/composites/test-times/action.yaml index 97f9a32074..33d2c13bfe 100644 --- a/.github/workflows/composites/test-times/action.yaml +++ b/.github/workflows/composites/test-times/action.yaml @@ -47,7 +47,7 @@ runs: # save with the current run_id, but restore it without it. This means two things: # 1) if we re-run, cache will be available - # 2) if there is a new run, we restore as '${{ runner.os }}-spring-cloud-kubernetes-existing-test-times-cache-' + # 2) if there is a new run, we restore as '${{ runner.os }}-spring-cloud-k8s-existing-test-times-cache-' # meaning there could be many of them already present and this is not an exact match # github in this case will pick up the latest one, exactly what we want. - name: save test times in cache @@ -55,4 +55,4 @@ runs: if: env.BASE_BRANCH != '2.1.x' with: path: /tmp/sorted.txt - key: ${{ runner.os }}-spring-cloud-kubernetes-existing-test-times-cache-${{ github.run_id }} + key: ${{ runner.os }}-spring-cloud-k8s-existing-test-times-cache-${{ github.run_id }} diff --git a/.github/workflows/maven.yaml b/.github/workflows/maven.yaml index 65802dcf54..11dd61fedf 100644 --- a/.github/workflows/maven.yaml +++ b/.github/workflows/maven.yaml @@ -67,8 +67,8 @@ jobs: uses: actions/cache/restore@v3 with: path: /tmp/sorted.txt - key: ${{ runner.os }}-spring-cloud-kubernetes-existing-test-times-cache-${{ github.run_id }} - restore-keys: ${{ runner.os }}-spring-cloud-kubernetes-existing-test-times-cache- + key: ${{ runner.os }}-spring-cloud-k8s-existing-test-times-cache-${{ github.run_id }} + restore-keys: ${{ runner.os }}-spring-cloud-k8s-existing-test-times-cache- - name: check test times cache exists id: check_files From 5d848aab7279fe269f0fd08560d8a5daea571c86 Mon Sep 17 00:00:00 2001 From: wind57 Date: Sat, 26 Aug 2023 09:34:22 +0300 Subject: [PATCH 08/10] fix failure --- ....java => KubernetesProfileEnvironmentPostProcessorTest.java} | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) rename spring-cloud-kubernetes-commons/src/test/java/org/springframework/cloud/kubernetes/commons/profile/{AbstractKubernetesProfileEnvironmentPostProcessorTest.java => KubernetesProfileEnvironmentPostProcessorTest.java} (99%) diff --git a/spring-cloud-kubernetes-commons/src/test/java/org/springframework/cloud/kubernetes/commons/profile/AbstractKubernetesProfileEnvironmentPostProcessorTest.java b/spring-cloud-kubernetes-commons/src/test/java/org/springframework/cloud/kubernetes/commons/profile/KubernetesProfileEnvironmentPostProcessorTest.java similarity index 99% rename from spring-cloud-kubernetes-commons/src/test/java/org/springframework/cloud/kubernetes/commons/profile/AbstractKubernetesProfileEnvironmentPostProcessorTest.java rename to spring-cloud-kubernetes-commons/src/test/java/org/springframework/cloud/kubernetes/commons/profile/KubernetesProfileEnvironmentPostProcessorTest.java index c66fcf72f1..aaad3614e8 100644 --- a/spring-cloud-kubernetes-commons/src/test/java/org/springframework/cloud/kubernetes/commons/profile/AbstractKubernetesProfileEnvironmentPostProcessorTest.java +++ b/spring-cloud-kubernetes-commons/src/test/java/org/springframework/cloud/kubernetes/commons/profile/KubernetesProfileEnvironmentPostProcessorTest.java @@ -39,7 +39,7 @@ /** * @author wind57 */ -public class AbstractKubernetesProfileEnvironmentPostProcessorTest { +public class KubernetesProfileEnvironmentPostProcessorTest { private static final String FOUNT_IT = "foundIt"; From 0b24a1f4bbf85039476d797966c9e20dae7cc0c9 Mon Sep 17 00:00:00 2001 From: wind57 Date: Sat, 26 Aug 2023 10:25:40 +0300 Subject: [PATCH 09/10] placeholder commit From a32bf85ca17d83cb9c99dab6c200296ddf37f13d Mon Sep 17 00:00:00 2001 From: wind57 Date: Sat, 26 Aug 2023 10:52:48 +0300 Subject: [PATCH 10/10] placeholder commit