[Asterisk-code-review] bridge wait: Check for RINGING INBAND test event. (testsuite[master])

Mark Michelson asteriskteam at digium.com
Wed Jul 29 12:27:11 CDT 2015


Mark Michelson has uploaded a new change for review.

  https://gerrit.asterisk.org/982

Change subject: bridge_wait: Check for RINGING_INBAND test event.
......................................................................

bridge_wait: Check for RINGING_INBAND test event.

A recent change to Asterisk 13 and master made local channels incapable
of changing from the "UP" state to the "RINGING" state. The
bridge_wait_e_options test expected the local channel that entered the
BridgeWait() application to change state to "RINGING" as a way of
ensuring that the e(r) option worked as expected. However, since that
state change is now invalid, the event that is checked needs to be
altered, too. This changeset has the test expect the RINGING_INBAND test
event instead.

ASTERISK-25292
Reported by Kevin Harwell

Change-Id: If33efe8161b139e8c0c79690bb6eb6cf189cca9b
---
M tests/apps/bridge/bridge_wait/bridge_wait_e_options/test-config.yaml
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/82/982/1

diff --git a/tests/apps/bridge/bridge_wait/bridge_wait_e_options/test-config.yaml b/tests/apps/bridge/bridge_wait/bridge_wait_e_options/test-config.yaml
index c2637ec..b1293b3 100644
--- a/tests/apps/bridge/bridge_wait/bridge_wait_e_options/test-config.yaml
+++ b/tests/apps/bridge/bridge_wait/bridge_wait_e_options/test-config.yaml
@@ -217,9 +217,9 @@
               type: 'headermatch'
               conditions:
                 match:
-                  Event: 'Newstate'
+                  Event: 'TestEvent'
+                  State: 'RINGING_INBAND'
                   Channel: 'Local/er_option at holding_bridge-.*'
-                  ChannelStateDesc: 'Ringing'
               actions:
                 -
                   action-type: 'set-expected-result'

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If33efe8161b139e8c0c79690bb6eb6cf189cca9b
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Mark Michelson <mmichelson at digium.com>



More information about the asterisk-code-review mailing list