<p>Jenkins2 <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/6066">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  George Joseph: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved
  Jenkins2: Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">tests/apps/voicemail/play_message: Fix dialplan syntax error.<br><br>Change-Id: I48c20cea8c1148750a44e75d19e4f51984195087<br>---<br>M tests/apps/voicemail/play_message/configs/ast1/extensions.conf<br>M tests/apps/voicemail/play_message/run-test<br>2 files changed, 7 insertions(+), 3 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/tests/apps/voicemail/play_message/configs/ast1/extensions.conf b/tests/apps/voicemail/play_message/configs/ast1/extensions.conf<br>index 60ed5f0..7add36f 100644<br>--- a/tests/apps/voicemail/play_message/configs/ast1/extensions.conf<br>+++ b/tests/apps/voicemail/play_message/configs/ast1/extensions.conf<br>@@ -1,14 +1,14 @@<br> <br> [voicemail]<br>-exten => playmsg,1,NoOp()<br>+exten => playmsg,1,NoOp(MAILBOX="${MAILBOX}"  MAILCONTEXT="${MAILCONTEXT}"  MSGID="${MSGID}")<br>     same => n,GotoIf($[${LEN(${MAILCONTEXT})}=0]?nocontext:context)<br>     same => n(context),VoiceMailPlayMsg(${MAILBOX}@${MAILCONTEXT},${MSGID})<br>     same => n,Hangup()<br>     same => n(nocontext),VoiceMailPlayMsg(${MAILBOX},${MSGID})<br>     same => n,Hangup()<br> <br>-exten => h,1,NoOp()<br>-    same => n,GotoIf($[${VOICEMAIL_PLAYBACKSTATUS} = SUCCESS]?pass:fail)<br>+exten => h,1,NoOp(VOICEMAIL_PLAYBACKSTATUS="${VOICEMAIL_PLAYBACKSTATUS}")<br>+    same => n,GotoIf($["${VOICEMAIL_PLAYBACKSTATUS}" = "SUCCESS"]?pass:fail)<br>     same => n(fail),UserEvent(TestResult,result: fail, status: VoiceMailPlayMsg failed to playback message ${MSGID})<br>     same => n,Goto(out)<br>     same => n(pass),NoOp()<br>diff --git a/tests/apps/voicemail/play_message/run-test b/tests/apps/voicemail/play_message/run-test<br>index 7a0b657..32482e7 100755<br>--- a/tests/apps/voicemail/play_message/run-test<br>+++ b/tests/apps/voicemail/play_message/run-test<br>@@ -100,6 +100,10 @@<br>         if 'state' in event and 'message' in event:<br>             if event['state'] == 'PLAYBACK':<br>                 logger.debug("Playing back %s" % event.get('message'))<br>+            elif event['state'] == 'PLAYVOICE':<br>+                # Playing a recorded voicemail message<br>+                # or at least trying to play it.<br>+                logger.debug("Recorded msg %s" % event.get('message'))<br> <br>     def user_event_handler(self, ami, event):<br>         if 'userevent' not in event or 'result' not in event or 'status' not in event:<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/6066">change 6066</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/6066"/><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: merged </div>
<div style="display:none"> Gerrit-Change-Id: I48c20cea8c1148750a44e75d19e4f51984195087 </div>
<div style="display:none"> Gerrit-Change-Number: 6066 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>