<p>Kevin Harwell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7705">View Change</a></p><p>Patch set 5:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p style="white-space: pre-wrap; word-wrap: break-word;">Am I wrong in thinking this test could be written in yaml using the SIPpTestCase class and EventActionModule passing in the sipp call_id as one of the keyword parameters? For instance see the following test as examples of using those modules:</p><p style="white-space: pre-wrap; word-wrap: break-word;">tests/channels/pjsip/transfers/attended_transfer/nominal/callee_local/test-config.yaml</p><p style="white-space: pre-wrap; word-wrap: break-word;">If not at the very least the testsuite already has an ami client handler. Instead of using the SIPpTest class use SIPpTestCase class. You can then register to listen for ami message as well as send ami actions. See the following test as an example on how to use the SIPpTestCase with a custom class that listens for ami events and sends ami actions:</p><p style="white-space: pre-wrap; word-wrap: break-word;">tests/bridge/hold_redirect/devstate_check.py<br>tests/bridge/hold_redirect/test-config.yaml</p><p style="white-space: pre-wrap; word-wrap: break-word;">This would alleviate the need for a custom ami client handler.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7705">change 7705</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/7705"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: testsuite </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Idccbf32ed6670a5205ee99bd7413c7fe0804efb1 </div>
<div style="display:none"> Gerrit-Change-Number: 7705 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: Yasuhiko Kamata <yasuhiko.kamata@nxtg.co.jp> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Yasuhiko Kamata <yasuhiko.kamata@nxtg.co.jp> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 18 Jan 2018 23:58:17 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>