From 80f39bea84e5b1a32f937fa5294b944dacad91b6 Mon Sep 17 00:00:00 2001 From: Glenn Renfro Date: Wed, 8 Jul 2020 12:41:56 -0400 Subject: [PATCH] Removed cfenv from dependencies Updated tests that used it. resolves #681 --- spring-cloud-task-core/pom.xml | 14 ++++++--- .../micrometer/AbstractMicrometerTest.java | 31 +++++++++++++++++-- spring-cloud-task-dependencies/pom.xml | 9 ------ 3 files changed, 38 insertions(+), 16 deletions(-) diff --git a/spring-cloud-task-core/pom.xml b/spring-cloud-task-core/pom.xml index 4b3892d9..6356abda 100755 --- a/spring-cloud-task-core/pom.xml +++ b/spring-cloud-task-core/pom.xml @@ -13,6 +13,9 @@ jar Spring Cloud Task Core Spring Cloud Task + + 1.49 + @@ -109,11 +112,6 @@ io.micrometer micrometer-core - - io.pivotal.cfenv - java-cfenv-test-support - test - org.junit.jupiter junit-jupiter-params @@ -124,6 +122,12 @@ junit-jupiter-engine test + + org.jmockit + jmockit + ${jmock-version} + test + diff --git a/spring-cloud-task-core/src/test/java/org/springframework/cloud/task/micrometer/AbstractMicrometerTest.java b/spring-cloud-task-core/src/test/java/org/springframework/cloud/task/micrometer/AbstractMicrometerTest.java index 9e02efaa..9af009ef 100644 --- a/spring-cloud-task-core/src/test/java/org/springframework/cloud/task/micrometer/AbstractMicrometerTest.java +++ b/spring-cloud-task-core/src/test/java/org/springframework/cloud/task/micrometer/AbstractMicrometerTest.java @@ -18,13 +18,16 @@ package org.springframework.cloud.task.micrometer; import java.io.IOException; import java.nio.charset.Charset; +import java.util.HashMap; +import java.util.Map; import io.micrometer.core.instrument.Clock; import io.micrometer.core.instrument.Meter; import io.micrometer.core.instrument.Metrics; import io.micrometer.core.instrument.simple.SimpleConfig; import io.micrometer.core.instrument.simple.SimpleMeterRegistry; -import io.pivotal.cfenv.test.CfEnvTestUtils; +import mockit.Mock; +import mockit.MockUp; import org.junit.jupiter.api.AfterEach; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.BeforeEach; @@ -80,7 +83,31 @@ public class AbstractMicrometerTest { String serviceJson = StreamUtils.copyToString(new DefaultResourceLoader() .getResource("classpath:/micrometer/pcf-scs-info.json").getInputStream(), Charset.forName("UTF-8")); - CfEnvTestUtils.mockVcapServicesFromString(serviceJson); + mockVcapServicesFromString(serviceJson); + } + + public static MockUp mockVcapServicesFromString(String serviceJson) { + final Map env = System.getenv(); + return new MockUp() { + @Mock + public String getenv(String name) { + if (name.equalsIgnoreCase("VCAP_SERVICES")) { + return serviceJson; + } + else { + return name.equalsIgnoreCase("VCAP_APPLICATION") + ? "{\"instance_id\":\"123\"}" : (String) env.get(name); + } + } + + @Mock + public Map getenv() { + Map finalMap = new HashMap(); + finalMap.putAll(env); + finalMap.put("VCAP_SERVICES", serviceJson); + return finalMap; + } + }; } @SpringBootApplication diff --git a/spring-cloud-task-dependencies/pom.xml b/spring-cloud-task-dependencies/pom.xml index 3ed9a4a7..d6b9e731 100644 --- a/spring-cloud-task-dependencies/pom.xml +++ b/spring-cloud-task-dependencies/pom.xml @@ -14,10 +14,6 @@ - - 2.1.0.RELEASE - - @@ -40,11 +36,6 @@ spring-cloud-task-stream 2.3.0-SNAPSHOT - - io.pivotal.cfenv - java-cfenv-test-support - ${java-cfenv-boot.version} -