[asterisk-commits] bridge wait: Check for RINGING INBAND test event. (testsuite[master])

[SVN commits to the Asterisk project]

Joshua Colp has submitted this change and it was merged.

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(-)

Approvals:
  Kevin Harwell: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved; Verified



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: merged
Gerrit-Change-Id: If33efe8161b139e8c0c79690bb6eb6cf189cca9b
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>