You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by or...@apache.org on 2023/06/23 14:53:45 UTC

[camel] 12/17: (chores) camel-mock: do not catch Throwable

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

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit f1521f968dc284624ef58ddd004e966577762a62
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Fri Jun 23 14:51:12 2023 +0200

    (chores) camel-mock: do not catch Throwable
---
 .../src/main/java/org/apache/camel/component/mock/MockEndpoint.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/components/camel-mock/src/main/java/org/apache/camel/component/mock/MockEndpoint.java b/components/camel-mock/src/main/java/org/apache/camel/component/mock/MockEndpoint.java
index 0bb05d5ea2a..d3738d21c90 100644
--- a/components/camel-mock/src/main/java/org/apache/camel/component/mock/MockEndpoint.java
+++ b/components/camel-mock/src/main/java/org/apache/camel/component/mock/MockEndpoint.java
@@ -1334,7 +1334,7 @@ public class MockEndpoint extends DefaultEndpoint implements BrowsableEndpoint,
             if (!skip) {
                 try {
                     test.run();
-                } catch (Throwable e) {
+                } catch (Exception e) {
                     // the test failed so we do not match
                     return false;
                 }
@@ -1634,8 +1634,8 @@ public class MockEndpoint extends DefaultEndpoint implements BrowsableEndpoint,
                     }
                 }
             }
-        } catch (Throwable e) {
-            // must catch java.lang.Throwable as AssertionError extends java.lang.Error
+        } catch (AssertionError | Exception e) {
+            // AssertionError extends java.lang.Error
             failures.add(e);
         } finally {
             // make sure latch is counted down to avoid test hanging forever