[Asterisk-code-review] stasis bridge: Hangup from dialplan to give time for Stasis ... (testsuite[master])

[Joshua Colp]

Joshua Colp has uploaded a new change for review.

  https://gerrit.asterisk.org/2215

Change subject: stasis_bridge: Hangup from dialplan to give time for Stasis messages.
......................................................................

stasis_bridge: Hangup from dialplan to give time for Stasis messages.

This test terminates based on the hangup of the underlying channels
used during the test. This presents a race condition where the test
will terminate before the Stasis messages have been conveyed over the
ARI WebSocket.

This change makes it so the channels are hung up a period of time after
they are bridged together using Bridge(). This gives ample time for
the Stasis messages to be conveyed.

Change-Id: Icf776149b9d0052b0db5be271608f508d2ee97ff
---
M tests/rest_api/external_interaction/bridge_app/x_option/stasis_bridge/configs/ast1/extensions.conf
M tests/rest_api/external_interaction/bridge_app/x_option/stasis_bridge/test-config.yaml
2 files changed, 1 insertion(+), 8 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/15/2215/1

diff --git a/tests/rest_api/external_interaction/bridge_app/x_option/stasis_bridge/configs/ast1/extensions.conf b/tests/rest_api/external_interaction/bridge_app/x_option/stasis_bridge/configs/ast1/extensions.conf
index 2e9465e..c502ffb 100644
--- a/tests/rest_api/external_interaction/bridge_app/x_option/stasis_bridge/configs/ast1/extensions.conf
+++ b/tests/rest_api/external_interaction/bridge_app/x_option/stasis_bridge/configs/ast1/extensions.conf
@@ -11,7 +11,7 @@
 
 exten => echo,1,NoOp()
 	same => n,Answer()
-        same => n,Echo()
+        same => n,Wait(10)
 	same => n,Hangup()
 
 exten => dummy,1,Answer
diff --git a/tests/rest_api/external_interaction/bridge_app/x_option/stasis_bridge/test-config.yaml b/tests/rest_api/external_interaction/bridge_app/x_option/stasis_bridge/test-config.yaml
index 19eaf16..2af6556 100644
--- a/tests/rest_api/external_interaction/bridge_app/x_option/stasis_bridge/test-config.yaml
+++ b/tests/rest_api/external_interaction/bridge_app/x_option/stasis_bridge/test-config.yaml
@@ -132,13 +132,6 @@
                 nomatch:
                     Bridgecreator: 'Stasis'
             count: 1
-        ari-requests:
-            -
-                method: 'delete'
-                uri: 'channels/bob;2'
-            -
-                method: 'delete'
-                uri: 'channels/dummy;2'
     # Ensure alice leaves the non-stasis bridge
     -
         ami-events:

-- 
To view, visit https://gerrit.asterisk.org/2215
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Icf776149b9d0052b0db5be271608f508d2ee97ff
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Joshua Colp <jcolp at digium.com>