Skip to content

Commit 2f1f27c

Browse files
committed
fix ut
1 parent c29db28 commit 2f1f27c

File tree

2 files changed

+21
-6
lines changed

2 files changed

+21
-6
lines changed

polaris-configuration/polaris-configuration-client/src/test/java/com/tencent/polaris/configuration/client/ConfigFileTest.java

Lines changed: 17 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,9 @@ public void onChange(ConfigFileChangeEvent event) {
150150
}
151151
});
152152

153-
configFile.onChange(ConfigFileTestUtils.assembleDefaultConfigFileMeta(), "hello2");
153+
com.tencent.polaris.api.plugin.configuration.ConfigFile newConfigFile = new com.tencent.polaris.api.plugin.configuration.ConfigFile("ns", "group", "file");
154+
newConfigFile.setContent("hello2");
155+
configFile.onChange(ConfigFileTestUtils.assembleDefaultConfigFileMeta(), newConfigFile);
154156

155157
TimeUnit.MILLISECONDS.sleep(100);
156158

@@ -207,7 +209,10 @@ public void onChange(ConfigFileChangeEvent event) {
207209
}
208210
});
209211

210-
configFile.onChange(ConfigFileTestUtils.assembleDefaultConfigFileMeta(), content);
212+
com.tencent.polaris.api.plugin.configuration.ConfigFile newConfigFile = new com.tencent.polaris.api.plugin.configuration.ConfigFile("ns", "group", "file");
213+
newConfigFile.setContent(content);
214+
215+
configFile.onChange(ConfigFileTestUtils.assembleDefaultConfigFileMeta(), newConfigFile);
211216

212217
TimeUnit.MILLISECONDS.sleep(100);
213218

@@ -260,7 +265,10 @@ public void onChange(ConfigFileChangeEvent event) {
260265
}
261266
});
262267

263-
configFile.onChange(ConfigFileTestUtils.assembleDefaultConfigFileMeta(), content);
268+
com.tencent.polaris.api.plugin.configuration.ConfigFile newConfigFile = new com.tencent.polaris.api.plugin.configuration.ConfigFile("ns", "group", "file");
269+
newConfigFile.setContent(content);
270+
271+
configFile.onChange(ConfigFileTestUtils.assembleDefaultConfigFileMeta(), newConfigFile);
264272

265273
TimeUnit.MILLISECONDS.sleep(100);
266274

@@ -290,7 +298,9 @@ public void onChange(ConfigFileChangeEvent event) {
290298
};
291299
configFile.addChangeListener(listener);
292300

293-
configFile.onChange(ConfigFileTestUtils.assembleDefaultConfigFileMeta(), content2);
301+
com.tencent.polaris.api.plugin.configuration.ConfigFile newConfigFile = new com.tencent.polaris.api.plugin.configuration.ConfigFile("ns", "group", "file");
302+
newConfigFile.setContent(content2);
303+
configFile.onChange(ConfigFileTestUtils.assembleDefaultConfigFileMeta(), newConfigFile);
294304

295305
TimeUnit.MILLISECONDS.sleep(100);
296306

@@ -302,7 +312,9 @@ public void onChange(ConfigFileChangeEvent event) {
302312
configFile.removeChangeListener(listener);
303313

304314
String content3 = "hello3";
305-
configFile.onChange(ConfigFileTestUtils.assembleDefaultConfigFileMeta(), content3);
315+
newConfigFile = new com.tencent.polaris.api.plugin.configuration.ConfigFile("ns", "group", "file");
316+
newConfigFile.setContent(content3);
317+
configFile.onChange(ConfigFileTestUtils.assembleDefaultConfigFileMeta(), newConfigFile);
306318

307319
TimeUnit.MILLISECONDS.sleep(100);
308320

polaris-configuration/polaris-configuration-client/src/test/java/com/tencent/polaris/configuration/client/ConfigPropertiesFileTest.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,10 @@ public void onChange(ConfigFileChangeEvent event) {
167167
}
168168
});
169169

170-
configFile.onChange(ConfigFileTestUtils.assembleDefaultConfigFileMeta(), newContent);
170+
com.tencent.polaris.api.plugin.configuration.ConfigFile newConfigFile = new com.tencent.polaris.api.plugin.configuration.ConfigFile("ns", "group", "file");
171+
newConfigFile.setContent(newContent);
172+
173+
configFile.onChange(ConfigFileTestUtils.assembleDefaultConfigFileMeta(), newConfigFile);
171174

172175
TimeUnit.MILLISECONDS.sleep(500);
173176

0 commit comments

Comments
 (0)