This is an automated email from the ASF dual-hosted git repository.

xiaoheng pushed a commit to branch 3.0
in repository https://gitbox.apache.org/repos/asf/dubbo.git


The following commit(s) were added to refs/heads/3.0 by this push:
     new 4f77724  perf: Ignore unnecessary exception stack trace in testcase 
(#8420)
4f77724 is described below

commit 4f77724de850101f5196fd2576a86f07eee64f58
Author: Xiong, Pin <[email protected]>
AuthorDate: Tue Aug 10 21:22:58 2021 -0500

    perf: Ignore unnecessary exception stack trace in testcase (#8420)
    
    1. Ignore of printing exception stack trace in 
ConfigCenterConfigTest#testOverrideConfig
    2. Ignore of printing exception stack trace in 
MethodConfigTest#testVerifyMethodConfigOfService
    3. Ignore of printing exception stack trace in 
ReferenceConfigTest#test1ReferenceRetry
---
 .../src/test/java/org/apache/dubbo/config/ConfigCenterConfigTest.java | 2 +-
 .../src/test/java/org/apache/dubbo/config/MethodConfigTest.java       | 2 +-
 .../src/test/java/org/apache/dubbo/config/ReferenceConfigTest.java    | 4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/dubbo-config/dubbo-config-api/src/test/java/org/apache/dubbo/config/ConfigCenterConfigTest.java
 
b/dubbo-config/dubbo-config-api/src/test/java/org/apache/dubbo/config/ConfigCenterConfigTest.java
index 96f3590..014d764 100644
--- 
a/dubbo-config/dubbo-config-api/src/test/java/org/apache/dubbo/config/ConfigCenterConfigTest.java
+++ 
b/dubbo-config/dubbo-config-api/src/test/java/org/apache/dubbo/config/ConfigCenterConfigTest.java
@@ -92,7 +92,7 @@ public class ConfigCenterConfigTest {
                         .configCenter(configCenter)
                         .initialize();
             } catch (Exception e) {
-                e.printStackTrace();
+                // ignore
             }
 
             Collection<ConfigCenterConfig> configCenters = 
ApplicationModel.getConfigManager().getConfigCenters();
diff --git 
a/dubbo-config/dubbo-config-api/src/test/java/org/apache/dubbo/config/MethodConfigTest.java
 
b/dubbo-config/dubbo-config-api/src/test/java/org/apache/dubbo/config/MethodConfigTest.java
index 0209408..e27a359 100644
--- 
a/dubbo-config/dubbo-config-api/src/test/java/org/apache/dubbo/config/MethodConfigTest.java
+++ 
b/dubbo-config/dubbo-config-api/src/test/java/org/apache/dubbo/config/MethodConfigTest.java
@@ -429,7 +429,7 @@ public class MethodConfigTest {
                 .initialize();
             Assertions.fail("Method config verification should failed");
         } catch (Exception e) {
-            e.printStackTrace();
+            // ignore
             Throwable cause = e.getCause();
             Assertions.assertEquals(IllegalStateException.class, 
cause.getClass());
             Assertions.assertTrue(cause.getMessage().contains("not found 
method"), cause.toString());
diff --git 
a/dubbo-config/dubbo-config-api/src/test/java/org/apache/dubbo/config/ReferenceConfigTest.java
 
b/dubbo-config/dubbo-config-api/src/test/java/org/apache/dubbo/config/ReferenceConfigTest.java
index e7c0618..b6c45dc 100644
--- 
a/dubbo-config/dubbo-config-api/src/test/java/org/apache/dubbo/config/ReferenceConfigTest.java
+++ 
b/dubbo-config/dubbo-config-api/src/test/java/org/apache/dubbo/config/ReferenceConfigTest.java
@@ -135,7 +135,7 @@ public class ReferenceConfigTest {
             demoService = rc.get();
             success = true;
         } catch (Exception e) {
-            e.printStackTrace();
+            // ignore
         }
         Assertions.assertFalse(success);
         Assertions.assertNull(demoService);
@@ -152,7 +152,7 @@ public class ReferenceConfigTest {
             demoService = rc.get();
             success = true;
         } catch (Exception e) {
-            e.printStackTrace();
+            // ignore
         } finally {
             rc.destroy();
             sc.unexport();

Reply via email to