<p style="white-space: pre-wrap; word-wrap: break-word;">Up to you whether to add the extra check (unless someone else wants to block on it). Otherwise when you feel this is ready please cherry pick to the 13, 17, and master branches.</p><p><a href="https://gerrit.asterisk.org/c/testsuite/+/13164">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.asterisk.org/c/testsuite/+/13164/1/tests/manager/playdtmf/test-config.yaml">File tests/manager/playdtmf/test-config.yaml:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/testsuite/+/13164/1/tests/manager/playdtmf/test-config.yaml@52">Patch Set #1, Line 52:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">        ami-events:<br>            count: 1<br>            conditions:<br>                match:<br>                    Event: 'UserEvent'<br>                    UserEvent: 'valid'<br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I don't believe this is necessary? In the dialplan the "valid" and "invalid" events are exclusive, i […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Not necessary I'd say as it currently does test the feature. However, adding an event for 'invalid' would make the test more robust, and help to guard against regressions. For example, let's say a future patch accidentally makes it so the two cases are not mutually exclusive, and the 'invalid' case/userevent gets triggered. The test would still pass since there is no event check for the 'invalid' userevent.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/testsuite/+/13164">change 13164</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/+/13164"/><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: I50f29a7c86ad10ef0f94d1aebb6eca4d905b4ec4 </div>
<div style="display:none"> Gerrit-Change-Number: 13164 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: lvl <digium@lvlconsultancy.nl> </div>
<div style="display:none"> Gerrit-Reviewer: Benjamin Keith Ford <bford@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 19 Nov 2019 16:11:58 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: Benjamin Keith Ford <bford@digium.com> </div>
<div style="display:none"> Comment-In-Reply-To: lvl <digium@lvlconsultancy.nl> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>