Skip to content

Commit dfbd8c4

Browse files
committed
fix
1 parent 24d7b1c commit dfbd8c4

File tree

2 files changed

+17
-1
lines changed

2 files changed

+17
-1
lines changed

spring-cloud-starter-tencent-polaris-config/src/main/java/com/tencent/cloud/polaris/config/adapter/PolarisConfigFileLocator.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ public class PolarisConfigFileLocator implements PropertySourceLocator {
6969
// this class provides customized logic for some customers to configure special business group files
7070
private final PolarisConfigCustomExtensionLayer polarisConfigCustomExtensionLayer = PolarisServiceLoaderUtil.getPolarisConfigCustomExtensionLayer();
7171

72-
private volatile CompositePropertySource compositePropertySourceCache = null;
72+
private volatile static CompositePropertySource compositePropertySourceCache = null;
7373

7474
public PolarisConfigFileLocator(PolarisConfigProperties polarisConfigProperties,
7575
PolarisContextProperties polarisContextProperties, ConfigFileService configFileService, Environment environment) {

spring-cloud-starter-tencent-polaris-config/src/test/java/com/tencent/cloud/polaris/config/adapter/PolarisConfigFileLocatorTest.java

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818

1919
package com.tencent.cloud.polaris.config.adapter;
2020

21+
import java.lang.reflect.Field;
2122
import java.util.Arrays;
2223
import java.util.HashMap;
2324
import java.util.LinkedList;
@@ -68,6 +69,7 @@ public void setUp() {
6869

6970
@Test
7071
public void testLoadApplicationPropertiesFile() {
72+
clearCompositePropertySourceCache();
7173
PolarisConfigFileLocator locator = new PolarisConfigFileLocator(polarisConfigProperties, polarisContextProperties,
7274
configFileService, environment);
7375

@@ -105,6 +107,7 @@ public void testLoadApplicationPropertiesFile() {
105107

106108
@Test
107109
public void testActiveProfileFilesPriorityBiggerThanDefault() {
110+
clearCompositePropertySourceCache();
108111
PolarisConfigFileLocator locator = new PolarisConfigFileLocator(polarisConfigProperties, polarisContextProperties,
109112
configFileService, environment);
110113

@@ -154,6 +157,7 @@ public void testActiveProfileFilesPriorityBiggerThanDefault() {
154157

155158
@Test
156159
public void testGetCustomFiles() {
160+
clearCompositePropertySourceCache();
157161
PolarisConfigFileLocator locator = new PolarisConfigFileLocator(polarisConfigProperties, polarisContextProperties,
158162
configFileService, environment);
159163

@@ -208,6 +212,7 @@ public void testGetCustomFiles() {
208212

209213
@Test
210214
public void testGetCustomGroupFiles() {
215+
clearCompositePropertySourceCache();
211216
PolarisConfigFileLocator locator = new PolarisConfigFileLocator(polarisConfigProperties, polarisContextProperties,
212217
configFileService, environment);
213218

@@ -260,4 +265,15 @@ public void testGetCustomGroupFiles() {
260265
assertThat(propertySource.getProperty("k2")).isEqualTo("v2");
261266
assertThat(propertySource.getProperty("k3")).isEqualTo("v3");
262267
}
268+
269+
private void clearCompositePropertySourceCache() {
270+
try {
271+
Class<?> clazz = PolarisConfigFileLocator.class;
272+
Field field = clazz.getDeclaredField("compositePropertySourceCache");
273+
field.setAccessible(true);
274+
field.set(null, null);
275+
} catch (Exception e) {
276+
// ignore
277+
}
278+
}
263279
}

0 commit comments

Comments
 (0)