<p>Michael Bradeen <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/c/testsuite/+/18070">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">unsolicited/notify_after_register: properly handle test scenario<br><br>There were two issues with the existing tests, affecting both the<br>standard and realtime versions of the test.<br><br>First, the xml contained a syntax error which caused an error on<br>current versions of sipp<br><br>Second, the scenario itself can not be handled by one sipp instance<br>as the NOTIFY will not match the call-ID of the REGISTER.<br><br>This is handled by splitting the test into two scenarios, one that<br>Registers with a contact port of 5062 then quits and a second that<br>waits for a NOTIFY on port 5062 then sends a 200 OK.<br><br>The REGISTER scenario waits 500ms for the NOTIFY scenario to start.<br><br>If both scenarios succeed, the test passes.<br><br>Change-Id: I23c7fbea995d99211552321f2d4c7e02806792e4<br>---<br>D tests/channels/pjsip/subscriptions/mwi/unsolicited/notify_after_register/sipp/notify_after_register.xml<br>A tests/channels/pjsip/subscriptions/mwi/unsolicited/notify_after_register/sipp/receive_notify.xml<br>A tests/channels/pjsip/subscriptions/mwi/unsolicited/notify_after_register/sipp/register_and_quit_two.xml<br>M tests/channels/pjsip/subscriptions/mwi/unsolicited/notify_after_register/test-config.yaml<br>D tests/channels/pjsip/subscriptions/mwi/unsolicited/notify_after_register_realtime/sipp/notify_after_register.xml<br>A tests/channels/pjsip/subscriptions/mwi/unsolicited/notify_after_register_realtime/sipp/receive_notify.xml<br>A tests/channels/pjsip/subscriptions/mwi/unsolicited/notify_after_register_realtime/sipp/register_and_quit_two.xml<br>M tests/channels/pjsip/subscriptions/mwi/unsolicited/notify_after_register_realtime/test-config.yaml<br>8 files changed, 106 insertions(+), 84 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/70/18070/2</pre><p>To view, visit <a href="https://gerrit.asterisk.org/c/testsuite/+/18070">change 18070</a>. To unsubscribe, or for help writing mail filters, 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/c/testsuite/+/18070"/><meta itemprop="name" content="View Change"/></div></div>
<div style="display:none"> Gerrit-Project: testsuite </div>
<div style="display:none"> Gerrit-Branch: 16 </div>
<div style="display:none"> Gerrit-Change-Id: I23c7fbea995d99211552321f2d4c7e02806792e4 </div>
<div style="display:none"> Gerrit-Change-Number: 18070 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Michael Bradeen <mbradeen@sangoma.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>