camel-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From davscl...@apache.org
Subject git commit: Added to mock javadoc about the situation for expect no messages
Date Fri, 12 Sep 2014 09:01:38 GMT
Repository: camel
Updated Branches:
  refs/heads/camel-2.14.x 5222fed62 -> beed4cba6


Added to mock javadoc about the situation for expect no messages


Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/beed4cba
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/beed4cba
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/beed4cba

Branch: refs/heads/camel-2.14.x
Commit: beed4cba6a446e61d5033a1d9e77c04e4c9c4a8a
Parents: 5222fed
Author: Claus Ibsen <davsclaus@apache.org>
Authored: Fri Sep 12 11:00:31 2014 +0200
Committer: Claus Ibsen <davsclaus@apache.org>
Committed: Fri Sep 12 11:01:31 2014 +0200

----------------------------------------------------------------------
 .../apache/camel/component/mock/MockEndpoint.java  | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/beed4cba/camel-core/src/main/java/org/apache/camel/component/mock/MockEndpoint.java
----------------------------------------------------------------------
diff --git a/camel-core/src/main/java/org/apache/camel/component/mock/MockEndpoint.java b/camel-core/src/main/java/org/apache/camel/component/mock/MockEndpoint.java
index 804159b..03d20a2 100644
--- a/camel-core/src/main/java/org/apache/camel/component/mock/MockEndpoint.java
+++ b/camel-core/src/main/java/org/apache/camel/component/mock/MockEndpoint.java
@@ -74,6 +74,15 @@ import org.slf4j.LoggerFactory;
  * set expectations before the test is being started (eg before the mock receives messages).
  * The latter is used after the test has been executed, to verify the expectations; or
  * other assertions which you can perform after the test has been completed.
+ * <p/>
+ * <b>Beware:</b> If you want to expect a mock does not receive any messages,
by calling
+ * {@link #setExpectedMessageCount(int)} with <tt>0</tt>, then take extra care,
+ * as <tt>0</tt> matches when the tests starts, so you need to set a assert period
time
+ * to let the test run for a while to make sure there are still no messages arrived; for
+ * that use {@link #setAssertPeriod(long)}.
+ * An alternative is to use <a href="http://camel.apache.org/notifybuilder.html">NotifyBuilder</a>,
and use the notifier
+ * to know when Camel is done routing some messages, before you call the {@link #assertIsSatisfied()}
method on the mocks.
+ * This allows you to not use a fixed assert period, to speedup testing times.
  *
  * @version 
  */
@@ -1031,6 +1040,14 @@ public class MockEndpoint extends DefaultEndpoint implements BrowsableEndpoint
{
      * Specifies the expected number of message exchanges that should be
      * received by this endpoint.
      * <p/>
+     * <b>Beware:</b> If you want to expect that <tt>0</tt> messages,
then take extra care,
+     * as <tt>0</tt> matches when the tests starts, so you need to set a assert
period time
+     * to let the test run for a while to make sure there are still no messages arrived;
for
+     * that use {@link #setAssertPeriod(long)}.
+     * An alternative is to use <a href="http://camel.apache.org/notifybuilder.html">NotifyBuilder</a>,
and use the notifier
+     * to know when Camel is done routing some messages, before you call the {@link #assertIsSatisfied()}
method on the mocks.
+     * This allows you to not use a fixed assert period, to speedup testing times.
+     * <p/>
      * If you want to assert that <b>exactly</b> n'th message arrives to this
mock
      * endpoint, then see also the {@link #setAssertPeriod(long)} method for further details.
      *


Mime
View raw message