<p>George Joseph <strong>uploaded patch set #4</strong> to the change originally created by Kevin Harwell.</p><p><a href="https://gerrit.asterisk.org/9048">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">pjsip/statsd/contacts: Sporadically failing due to unexpected messages<br><br>After the pjsip qualify rewrite the contacts statsd test failed fairly<br>regularly because it was now receiving two messages that it did not receive<br>before the rewrite. Namely, the messages that occur on shutdown. However,<br>due to some current shutdown handling in Asterisk these final messages still<br>may or may not be received by the test.<br><br>Unfortunately, the way the statsd tests were written did not allow for optional<br>messages. Really it only allowed for a strict result set that was not too<br>configurable.<br><br>This patch alleviates the brittleness, and fixes the test, by creating<br>suitable condition based message handling and matching routines that the test<br>can now use for optional messages received.<br><br>An attempt was made to generically write the message matching routines in hopes<br>that other future event listeners and handlers could take advantage of it.<br><br>Change-Id: Iaae769c7a4fe2dcac4865eb7dc4e5b6a1b25900b<br>---<br>A lib/python/asterisk/matcher.py<br>A lib/python/asterisk/matcher_listener.py<br>M lib/python/asterisk/pluggable_modules.py<br>A lib/python/asterisk/self_test/test2_matcher.py<br>D lib/python/mockd.py<br>M self_test<br>M tests/channels/pjsip/statsd/contacts/test-config.yaml<br>M tests/channels/pjsip/statsd/registrations/test-config.yaml<br>8 files changed, 775 insertions(+), 188 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/48/9048/4</pre><p>To view, visit <a href="https://gerrit.asterisk.org/9048">change 9048</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/9048"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: testsuite </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: Iaae769c7a4fe2dcac4865eb7dc4e5b6a1b25900b </div>
<div style="display:none"> Gerrit-Change-Number: 9048 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>