[Asterisk-code-review] tests/test stasis endpoints: Remove expected duplicate events (asterisk[master])

Matt Jordan asteriskteam at digium.com
Mon Dec 28 15:11:01 CST 2015


Matt Jordan has submitted this change and it was merged.

Change subject: tests/test_stasis_endpoints: Remove expected duplicate events
......................................................................


tests/test_stasis_endpoints: Remove expected duplicate events

The cache_clear test was written to expect duplicate Stasis messages
sent from the technology endpoint to the all caching topic. This patch
fixes the test to no longer expect these duplicate messages.

ASTERISK-25137

Change-Id: I58075d70d6cdf42e792e0fb63ba624720bfce981
---
M tests/test_stasis_endpoints.c
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Richard Mudgett: Looks good to me, but someone else must approve
  Matt Jordan: Looks good to me, approved; Verified
  Joshua Colp: Looks good to me, but someone else must approve



diff --git a/tests/test_stasis_endpoints.c b/tests/test_stasis_endpoints.c
index 848d86b..6e97275 100644
--- a/tests/test_stasis_endpoints.c
+++ b/tests/test_stasis_endpoints.c
@@ -186,9 +186,10 @@
 
 	/* Note: there's a few messages between the creation and the clear.
 	 * Wait for all of them... */
-	message_index = stasis_message_sink_wait_for(sink, message_index + 4,
+	message_index = stasis_message_sink_wait_for(sink, message_index + 2,
 		cache_update, __func__, STASIS_SINK_DEFAULT_WAIT);
 	ast_test_validate(test, 0 <= message_index);
+
 	/* Now we should have a cache removal entry */
 	msg = sink->messages[message_index];
 	type = stasis_message_type(msg);

-- 
To view, visit https://gerrit.asterisk.org/1874
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I58075d70d6cdf42e792e0fb63ba624720bfce981
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>



More information about the asterisk-code-review mailing list