Author: mjordan
Date: Fri Aug 22 12:08:05 2014
New Revision: 5514

URL: http://svnview.digium.com/svn/testsuite?view=rev&rev=5514
Log:
tests/apps/control_playback: Fix tests

This patch fixes two problems with the tests:
(1) With the patch that removes the implicit answer in ControlPlayback, the
    originates will now fail if the channel is not answered. This didn't
    cause the tests to fail, but does create a lot of origination spam.
(2) The queueing up of the AMI action that controls the playback was done
    rather optimistically. This is now done once we know for sure that the
    sound file is being played back via a TestEvent.

Modified:
    
asterisk/trunk/tests/apps/control_playback/control_forward/configs/ast1/extensions.conf
    asterisk/trunk/tests/apps/control_playback/control_forward/test-config.yaml
    
asterisk/trunk/tests/apps/control_playback/control_pause/configs/ast1/extensions.conf
    asterisk/trunk/tests/apps/control_playback/control_pause/test-config.yaml
    
asterisk/trunk/tests/apps/control_playback/control_restart/configs/ast1/extensions.conf
    asterisk/trunk/tests/apps/control_playback/control_restart/test-config.yaml
    
asterisk/trunk/tests/apps/control_playback/control_reverse/configs/ast1/extensions.conf
    asterisk/trunk/tests/apps/control_playback/control_reverse/test-config.yaml
    
asterisk/trunk/tests/apps/control_playback/control_stop/configs/ast1/extensions.conf
    asterisk/trunk/tests/apps/control_playback/control_stop/test-config.yaml
    
asterisk/trunk/tests/apps/control_playback/nominal/configs/ast1/extensions.conf

Modified: 
asterisk/trunk/tests/apps/control_playback/control_forward/configs/ast1/extensions.conf
URL: 
http://svnview.digium.com/svn/testsuite/asterisk/trunk/tests/apps/control_playback/control_forward/configs/ast1/extensions.conf?view=diff&rev=5514&r1=5513&r2=5514
==============================================================================
--- 
asterisk/trunk/tests/apps/control_playback/control_forward/configs/ast1/extensions.conf
 (original)
+++ 
asterisk/trunk/tests/apps/control_playback/control_forward/configs/ast1/extensions.conf
 Fri Aug 22 12:08:05 2014
@@ -2,6 +2,7 @@
 
 exten => test_user,1,NoOp()
        ; Use DTMF 1 to FF 5 seconds
+       same => n,Answer()
        same => n,ControlPlayback(tt-monkeys,5000,1)
        same => 
n,ExecIf($[${CPLAYBACKOFFSET}=-1]?UserEvent(TestResult,result:pass_eof)
        same => 
n,ExecIf($[${CPLAYBACKSTATUS}=SUCCESS]?UserEvent(TestResult,result:pass_user)
@@ -9,6 +10,7 @@
 
 exten => test_default_user,1,NoOp()
        ; Test default skipms of 3000; default FF of #
+       same => n,Answer()
        same => n,ControlPlayback(tt-monkeys)
        same => 
n,ExecIf($[${CPLAYBACKOFFSET}=-1]?UserEvent(TestResult,result:pass_default_eof)
        same => 
n,ExecIf($[${CPLAYBACKSTATUS}=SUCCESS]?UserEvent(TestResult,result:pass_default_user)
@@ -16,6 +18,7 @@
 
 exten => test_remote,1,NoOp()
        ; Remote AMI connection controls test
+       same => n,Answer()
        same => n,ControlPlayback(tt-monkeys)
        same => 
n,ExecIf($[${CPLAYBACKOFFSET}=-1]?UserEvent(TestResult,result:pass_remote_eof)
        same => 
n,ExecIf($[${CPLAYBACKSTATUS}=SUCCESS]?UserEvent(TestResult,result:pass_remote)

Modified: 
asterisk/trunk/tests/apps/control_playback/control_forward/test-config.yaml
URL: 
http://svnview.digium.com/svn/testsuite/asterisk/trunk/tests/apps/control_playback/control_forward/test-config.yaml?view=diff&rev=5514&r1=5513&r2=5514
==============================================================================
--- asterisk/trunk/tests/apps/control_playback/control_forward/test-config.yaml 
(original)
+++ asterisk/trunk/tests/apps/control_playback/control_forward/test-config.yaml 
Fri Aug 22 12:08:05 2014
@@ -167,9 +167,9 @@
               type: 'headermatch'
               conditions:
                 match:
-                  Event: 'Newexten'
-                  Context: 'default'
-                  Extension: 'wait'
+                  Event: 'TestEvent'
+                  State: 'PLAYBACK'
+                  Message: 'tt-monkeys'
               actions:
                 -
                   action-type: 'send-ami-message'

Modified: 
asterisk/trunk/tests/apps/control_playback/control_pause/configs/ast1/extensions.conf
URL: 
http://svnview.digium.com/svn/testsuite/asterisk/trunk/tests/apps/control_playback/control_pause/configs/ast1/extensions.conf?view=diff&rev=5514&r1=5513&r2=5514
==============================================================================
--- 
asterisk/trunk/tests/apps/control_playback/control_pause/configs/ast1/extensions.conf
 (original)
+++ 
asterisk/trunk/tests/apps/control_playback/control_pause/configs/ast1/extensions.conf
 Fri Aug 22 12:08:05 2014
@@ -2,6 +2,7 @@
 
 exten => test_user,1,NoOp()
        ; Use DTMF 3 to pause playback
+       same => n,Answer()
        same => n,ControlPlayback(tt-monkeys,,,,,3)
        same => 
n,ExecIf($[${CPLAYBACKOFFSET}=-1]?UserEvent(TestResult,result:pass_eof)
        same => 
n,ExecIf($[${CPLAYBACKSTATUS}=SUCCESS]?UserEvent(TestResult,result:pass_user)
@@ -9,6 +10,7 @@
 
 exten => test_remote,1,NoOp()
        ; Remote AMI connection controls test
+       same => n,Answer()
        same => n,ControlPlayback(tt-monkeys)
        same => 
n,ExecIf($[${CPLAYBACKOFFSET}=-1]?UserEvent(TestResult,result:pass_remote_eof)
        same => 
n,ExecIf($[${CPLAYBACKSTATUS}=SUCCESS]?UserEvent(TestResult,result:pass_remote)

Modified: 
asterisk/trunk/tests/apps/control_playback/control_pause/test-config.yaml
URL: 
http://svnview.digium.com/svn/testsuite/asterisk/trunk/tests/apps/control_playback/control_pause/test-config.yaml?view=diff&rev=5514&r1=5513&r2=5514
==============================================================================
--- asterisk/trunk/tests/apps/control_playback/control_pause/test-config.yaml 
(original)
+++ asterisk/trunk/tests/apps/control_playback/control_pause/test-config.yaml 
Fri Aug 22 12:08:05 2014
@@ -113,9 +113,9 @@
               type: 'headermatch'
               conditions:
                 match:
-                  Event: 'Newexten'
-                  Context: 'default'
-                  Extension: 'wait'
+                  Event: 'TestEvent'
+                  State: 'PLAYBACK'
+                  Message: 'tt-monkeys'
               actions:
                 -
                   action-type: 'send-ami-message'

Modified: 
asterisk/trunk/tests/apps/control_playback/control_restart/configs/ast1/extensions.conf
URL: 
http://svnview.digium.com/svn/testsuite/asterisk/trunk/tests/apps/control_playback/control_restart/configs/ast1/extensions.conf?view=diff&rev=5514&r1=5513&r2=5514
==============================================================================
--- 
asterisk/trunk/tests/apps/control_playback/control_restart/configs/ast1/extensions.conf
 (original)
+++ 
asterisk/trunk/tests/apps/control_playback/control_restart/configs/ast1/extensions.conf
 Fri Aug 22 12:08:05 2014
@@ -2,6 +2,7 @@
 
 exten => test_user,1,NoOp()
        ; Use DTMF 8 to restart playback
+       same => n,Answer()
        same => n,ControlPlayback(tt-monkeys,,,,,,8)
        same => 
n,ExecIf($[${CPLAYBACKOFFSET}=-1]?UserEvent(TestResult,result:pass_eof)
        same => 
n,ExecIf($[${CPLAYBACKSTATUS}=SUCCESS]?UserEvent(TestResult,result:pass_user)
@@ -9,6 +10,7 @@
 
 exten => test_remote,1,NoOp()
        ; Remote AMI connection controls test
+       same => n,Answer()
        same => n,ControlPlayback(tt-monkeys)
        same => 
n,ExecIf($[${CPLAYBACKOFFSET}=-1]?UserEvent(TestResult,result:pass_remote_eof)
        same => 
n,ExecIf($[${CPLAYBACKSTATUS}=SUCCESS]?UserEvent(TestResult,result:pass_remote)

Modified: 
asterisk/trunk/tests/apps/control_playback/control_restart/test-config.yaml
URL: 
http://svnview.digium.com/svn/testsuite/asterisk/trunk/tests/apps/control_playback/control_restart/test-config.yaml?view=diff&rev=5514&r1=5513&r2=5514
==============================================================================
--- asterisk/trunk/tests/apps/control_playback/control_restart/test-config.yaml 
(original)
+++ asterisk/trunk/tests/apps/control_playback/control_restart/test-config.yaml 
Fri Aug 22 12:08:05 2014
@@ -97,9 +97,9 @@
               type: 'headermatch'
               conditions:
                 match:
-                  Event: 'Newexten'
-                  Context: 'default'
-                  Extension: 'wait'
+                  Event: 'TestEvent'
+                  State: 'PLAYBACK'
+                  Message: 'tt-monkeys'
               actions:
                 -
                   action-type: 'send-ami-message'

Modified: 
asterisk/trunk/tests/apps/control_playback/control_reverse/configs/ast1/extensions.conf
URL: 
http://svnview.digium.com/svn/testsuite/asterisk/trunk/tests/apps/control_playback/control_reverse/configs/ast1/extensions.conf?view=diff&rev=5514&r1=5513&r2=5514
==============================================================================
--- 
asterisk/trunk/tests/apps/control_playback/control_reverse/configs/ast1/extensions.conf
 (original)
+++ 
asterisk/trunk/tests/apps/control_playback/control_reverse/configs/ast1/extensions.conf
 Fri Aug 22 12:08:05 2014
@@ -2,6 +2,7 @@
 
 exten => test_user,1,NoOp()
        ; Use DTMF 1 to Rewind 5 seconds
+       same => n,Answer()
        same => n,ControlPlayback(tt-monkeys,5000,,1)
        same => 
n,ExecIf($[${CPLAYBACKOFFSET}=-1]?UserEvent(TestResult,result:pass_eof)
        same => 
n,ExecIf($[${CPLAYBACKSTATUS}=SUCCESS]?UserEvent(TestResult,result:pass_user)
@@ -9,6 +10,7 @@
 
 exten => test_default_user,1,NoOp()
        ; Test default skipms of 3000; default Rewind of *
+       same => n,Answer()
        same => n,ControlPlayback(tt-monkeys)
        same => 
n,ExecIf($[${CPLAYBACKOFFSET}=-1]?UserEvent(TestResult,result:pass_default_eof)
        same => 
n,ExecIf($[${CPLAYBACKSTATUS}=SUCCESS]?UserEvent(TestResult,result:pass_default_user)
@@ -16,6 +18,7 @@
 
 exten => test_remote,1,NoOp()
        ; Remote AMI connection controls test
+       same => n,Answer()
        same => n,ControlPlayback(tt-monkeys)
        same => 
n,ExecIf($[${CPLAYBACKOFFSET}=-1]?UserEvent(TestResult,result:pass_remote_eof)
        same => 
n,ExecIf($[${CPLAYBACKSTATUS}=SUCCESS]?UserEvent(TestResult,result:pass_remote)

Modified: 
asterisk/trunk/tests/apps/control_playback/control_reverse/test-config.yaml
URL: 
http://svnview.digium.com/svn/testsuite/asterisk/trunk/tests/apps/control_playback/control_reverse/test-config.yaml?view=diff&rev=5514&r1=5513&r2=5514
==============================================================================
--- asterisk/trunk/tests/apps/control_playback/control_reverse/test-config.yaml 
(original)
+++ asterisk/trunk/tests/apps/control_playback/control_reverse/test-config.yaml 
Fri Aug 22 12:08:05 2014
@@ -167,9 +167,9 @@
               type: 'headermatch'
               conditions:
                 match:
-                  Event: 'Newexten'
-                  Context: 'default'
-                  Extension: 'wait'
+                  Event: 'TestEvent'
+                  State: 'PLAYBACK'
+                  Message: 'tt-monkeys'
               actions:
                 -
                   action-type: 'send-ami-message'

Modified: 
asterisk/trunk/tests/apps/control_playback/control_stop/configs/ast1/extensions.conf
URL: 
http://svnview.digium.com/svn/testsuite/asterisk/trunk/tests/apps/control_playback/control_stop/configs/ast1/extensions.conf?view=diff&rev=5514&r1=5513&r2=5514
==============================================================================
--- 
asterisk/trunk/tests/apps/control_playback/control_stop/configs/ast1/extensions.conf
 (original)
+++ 
asterisk/trunk/tests/apps/control_playback/control_stop/configs/ast1/extensions.conf
 Fri Aug 22 12:08:05 2014
@@ -2,6 +2,7 @@
 
 exten => test_user,1,NoOp()
        ; Use DTMF 0 to stop
+       same => n,Answer()
        same => n,ControlPlayback(tt-monkeys,,,,0)
        same => 
n,ExecIf($[${CPLAYBACKOFFSET}!=-1]?UserEvent(TestResult,result:pass_eof)
        same => 
n,ExecIf($[${CPLAYBACKSTATUS}=USERSTOPPED]?UserEvent(TestResult,result:pass_user)
@@ -10,6 +11,7 @@
 
 exten => test_remote,1,NoOp()
        ; Remote AMI connection stops test
+       same => n,Answer()
        same => n,ControlPlayback(tt-monkeys)
        same => 
n,ExecIf($[${CPLAYBACKOFFSET}!=-1]?UserEvent(TestResult,result:pass_remote_eof)
        same => 
n,ExecIf($[${CPLAYBACKSTATUS}=REMOTESTOPPED]?UserEvent(TestResult,result:pass_remote)

Modified: 
asterisk/trunk/tests/apps/control_playback/control_stop/test-config.yaml
URL: 
http://svnview.digium.com/svn/testsuite/asterisk/trunk/tests/apps/control_playback/control_stop/test-config.yaml?view=diff&rev=5514&r1=5513&r2=5514
==============================================================================
--- asterisk/trunk/tests/apps/control_playback/control_stop/test-config.yaml 
(original)
+++ asterisk/trunk/tests/apps/control_playback/control_stop/test-config.yaml 
Fri Aug 22 12:08:05 2014
@@ -108,9 +108,9 @@
               type: 'headermatch'
               conditions:
                 match:
-                  Event: 'Newexten'
-                  Context: 'default'
-                  Extension: 'wait'
+                  Event: 'TestEvent'
+                  State: 'PLAYBACK'
+                  Message: 'tt-monkeys'
               actions:
                 -
                   action-type: 'send-ami-message'

Modified: 
asterisk/trunk/tests/apps/control_playback/nominal/configs/ast1/extensions.conf
URL: 
http://svnview.digium.com/svn/testsuite/asterisk/trunk/tests/apps/control_playback/nominal/configs/ast1/extensions.conf?view=diff&rev=5514&r1=5513&r2=5514
==============================================================================
--- 
asterisk/trunk/tests/apps/control_playback/nominal/configs/ast1/extensions.conf 
(original)
+++ 
asterisk/trunk/tests/apps/control_playback/nominal/configs/ast1/extensions.conf 
Fri Aug 22 12:08:05 2014
@@ -1,6 +1,7 @@
 [default]
 
 exten => test,1,NoOp()
+       same => n,Answer()
        same => n,ControlPlayback(tt-monkeys)
        same => 
n,ExecIf($[${CPLAYBACKOFFSET}=-1]?UserEvent(Offset,result:offset_pass)
        same => 
n,ExecIf($[${CPLAYBACKSTATUS}=SUCCESS]?UserEvent(Status,result:status_pass)


-- 
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

svn-commits mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/svn-commits

Reply via email to