From 9eb2382d018a8da1aba91fa9c95fdd0b59320fe6 Mon Sep 17 00:00:00 2001 From: Albumen Kevin Date: Tue, 25 May 2021 10:13:22 +0800 Subject: [PATCH] Improve UT cases (#7856) * Opt ConsulRegistryTest Retry * Change zk remove listener check key --- .../support/zookeeper/ZookeeperDynamicConfigurationTest.java | 4 ++-- .../org/apache/dubbo/registry/consul/ConsulRegistryTest.java | 2 ++ 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/dubbo-configcenter/dubbo-configcenter-zookeeper/src/test/java/org/apache/dubbo/configcenter/support/zookeeper/ZookeeperDynamicConfigurationTest.java b/dubbo-configcenter/dubbo-configcenter-zookeeper/src/test/java/org/apache/dubbo/configcenter/support/zookeeper/ZookeeperDynamicConfigurationTest.java index a605055602..518ab9a8df 100644 --- a/dubbo-configcenter/dubbo-configcenter-zookeeper/src/test/java/org/apache/dubbo/configcenter/support/zookeeper/ZookeeperDynamicConfigurationTest.java +++ b/dubbo-configcenter/dubbo-configcenter-zookeeper/src/test/java/org/apache/dubbo/configcenter/support/zookeeper/ZookeeperDynamicConfigurationTest.java @@ -128,9 +128,9 @@ public class ZookeeperDynamicConfigurationTest { @Test public void tesRemoveListener() throws Exception { - String key1 = "service:version:group.configurators.remove"; + String key1 = "key1.remove"; String path1 = "/dubbo/config/dubbo/" + key1; - String key2 = "appname.tag-router.remove"; + String key2 = "key2.remove"; String path2 = "/dubbo/config/dubbo/" + key2; CountDownLatch latch = new CountDownLatch(2); diff --git a/dubbo-registry/dubbo-registry-consul/src/test/java/org/apache/dubbo/registry/consul/ConsulRegistryTest.java b/dubbo-registry/dubbo-registry-consul/src/test/java/org/apache/dubbo/registry/consul/ConsulRegistryTest.java index 792f258480..2acf67a141 100644 --- a/dubbo-registry/dubbo-registry-consul/src/test/java/org/apache/dubbo/registry/consul/ConsulRegistryTest.java +++ b/dubbo-registry/dubbo-registry-consul/src/test/java/org/apache/dubbo/registry/consul/ConsulRegistryTest.java @@ -57,8 +57,10 @@ public class ConsulRegistryTest { this.consul = ConsulStarterBuilder.consulStarter() .build() .start(); + exception = null; } catch (Exception e) { exception = e; + e.printStackTrace(); } if (exception == null) { break;