[Asterisk-code-review] bridge: Add test for confirming hangup after feature timeout (testsuite[master])

[Joshua Colp]

Joshua Colp has uploaded a new change for review.

  https://gerrit.asterisk.org/1134

Change subject: bridge: Add test for confirming hangup after feature timeout
......................................................................

bridge: Add test for confirming hangup after feature timeout

This test creates a chain of Local channels and sets up a
scenario where DTMF is sent that causes a feature timeout to
occur. When the DTMF is received at the far end the channel is
hung up. All channels are confirmed to have been hung up when
this happens.

ASTERISK-25341 #close

Change-Id: Icfa456ea9de7b452c57fa91919ca03c56120d523
---
A tests/bridge/feature_timeout_hangup/configs/ast1/extensions.conf
A tests/bridge/feature_timeout_hangup/configs/ast1/features.conf
A tests/bridge/feature_timeout_hangup/test-config.yaml
M tests/bridge/tests.yaml
4 files changed, 92 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/34/1134/1

diff --git a/tests/bridge/feature_timeout_hangup/configs/ast1/extensions.conf b/tests/bridge/feature_timeout_hangup/configs/ast1/extensions.conf
new file mode 100644
index 0000000..06f67a5
--- /dev/null
+++ b/tests/bridge/feature_timeout_hangup/configs/ast1/extensions.conf
@@ -0,0 +1,15 @@
+[collect_context]
+exten => collect,1,Answer()
+exten => collect,2,Read(TACOS,demo-congrats,1)
+exten => h,1,UserEvent(CollectResult,result:pass)
+
+[outgoing_context]
+exten => playback,1,Dial(Local/collect at collect_context,,tT)
+exten => h,1,UserEvent(OutgoingResult,result:pass)
+
+[dtmf_context]
+exten => dtmf,1,Answer
+exten => dtmf,2,Wait(2)
+exten => dtmf,3,SendDTMF(3)
+exten => dtmf,4,Wait(36)
+exten => h,1,UserEvent(DTMFResult,result:pass)
diff --git a/tests/bridge/feature_timeout_hangup/configs/ast1/features.conf b/tests/bridge/feature_timeout_hangup/configs/ast1/features.conf
new file mode 100644
index 0000000..6b59cbd
--- /dev/null
+++ b/tests/bridge/feature_timeout_hangup/configs/ast1/features.conf
@@ -0,0 +1,4 @@
+[featuremap]
+blindxfer => #2
+atxfer => #1
+
diff --git a/tests/bridge/feature_timeout_hangup/test-config.yaml b/tests/bridge/feature_timeout_hangup/test-config.yaml
new file mode 100644
index 0000000..16d504b
--- /dev/null
+++ b/tests/bridge/feature_timeout_hangup/test-config.yaml
@@ -0,0 +1,72 @@
+testinfo:
+    summary:     'Test that feature timeout does not cause a hangup to get dropped'
+    description: |
+        'This test creates a small chain of Local channels with the DTMF based transfer
+         features enabled. The features are configured to require a # DTMF and a digit in
+         order to trigger the transfers. Once the chain is answered SendDTMF is used to
+         send a single # which causes a feature timeout to occur. The # DTMF is streamed
+         out to the called dialplan logic. This dialplan logic immediately hangs up when
+         a single DTMF digit is received. This test confirms that all channels involved
+         are hung up when this leg of the chain is hung up.'
+
+test-modules:
+    test-object:
+        config-section: test-object-config
+        typename: 'test_case.SimpleTestCase'
+    modules:
+        -
+            typename: 'pluggable_modules.HangupMonitor'
+            config-section: 'hangup-monitor'
+        -
+            typename: 'ami.AMIEventModule'
+            config-section: 'ami-config'
+
+test-object-config:
+    spawn-after-hangup: True
+    test-iterations:
+        -
+            channel: 'Local/playback at outgoing_context'
+            exten: 'dtmf'
+            context: 'dtmf_context'
+            priority: '1'
+
+hangup-monitor:
+    ids: '0'
+
+ami-config:
+    -
+        type: 'headermatch'
+        conditions:
+            match:
+                Event: 'UserEvent'
+                UserEvent: 'OutgoingResult'
+        requirements:
+            match:
+                Result: 'pass'
+        count: 1
+    -
+        type: 'headermatch'
+        conditions:
+            match:
+                Event: 'UserEvent'
+                UserEvent: 'DTMFResult'
+        requirements:
+            match:
+                Result: 'pass'
+        count: 1
+    -
+        type: 'headermatch'
+        conditions:
+            match:
+                Event: 'UserEvent'
+                UserEvent: 'CollectResult'
+        requirements:
+            match:
+                Result: 'pass'
+        count: 1
+
+properties:
+    minversion: '13.6.0'
+    tags:
+        - bridge
+
diff --git a/tests/bridge/tests.yaml b/tests/bridge/tests.yaml
index dfcfdf1..1557301 100644
--- a/tests/bridge/tests.yaml
+++ b/tests/bridge/tests.yaml
@@ -23,3 +23,4 @@
     - test: 'bridge_action'
     - test: 'atxfer_threeway_nominal'
     - test: 'atxfer_retries'
+    - test: 'feature_timeout_hangup'

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

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