[Asterisk-code-review] testsuite: Cleanup tests/channels/pjsip/refer send to vm (testsuite[master])

Anonymous Coward asteriskteam at digium.com
Thu Mar 3 07:18:36 CST 2016


Anonymous Coward #1000019 has submitted this change and it was merged.

Change subject: testsuite: Cleanup tests/channels/pjsip/refer_send_to_vm
......................................................................


testsuite: Cleanup tests/channels/pjsip/refer_send_to_vm

* Add debug dialplan lines.
* Add more test dependencies.

Change-Id: I4b113c33a1727851bfa6181b63018a4461904160
---
M tests/channels/pjsip/refer_send_to_vm/configs/ast1/extensions.conf
M tests/channels/pjsip/refer_send_to_vm/test-config.yaml
2 files changed, 12 insertions(+), 4 deletions(-)

Approvals:
  Kevin Harwell: Looks good to me, but someone else must approve
  Anonymous Coward #1000019: Verified
  Joshua Colp: Looks good to me, approved



diff --git a/tests/channels/pjsip/refer_send_to_vm/configs/ast1/extensions.conf b/tests/channels/pjsip/refer_send_to_vm/configs/ast1/extensions.conf
index 1b639dc..1127d18 100644
--- a/tests/channels/pjsip/refer_send_to_vm/configs/ast1/extensions.conf
+++ b/tests/channels/pjsip/refer_send_to_vm/configs/ast1/extensions.conf
@@ -6,6 +6,8 @@
       same => n,Hangup()
 
 exten => carol,1,NoOp()
+      same => n,NoOp(REDIRECTING(reason) is '${REDIRECTING(reason)}')
+      same => n,NoOp(PJSIP_HEADER(read,X-Digium-Call-Feature) is '${PJSIP_HEADER(read,X-Digium-Call-Feature)}')
       same => n,GotoIf($[${REDIRECTING(reason)} = "send_to_vm" | "${PJSIP_HEADER(read,X-Digium-Call-Feature)}" = "feature_send_to_vm"]?vm:notvm)
       same => n(vm),UserEvent(Result, Status: passed)
       same => n,Hangup()
diff --git a/tests/channels/pjsip/refer_send_to_vm/test-config.yaml b/tests/channels/pjsip/refer_send_to_vm/test-config.yaml
index acbfe54..02e70bc 100644
--- a/tests/channels/pjsip/refer_send_to_vm/test-config.yaml
+++ b/tests/channels/pjsip/refer_send_to_vm/test-config.yaml
@@ -10,10 +10,16 @@
 properties:
     minversion: '12.2.0'
     dependencies:
-         - app : 'sipp'
-         - asterisk : 'res_pjsip'
-         - asterisk : 'res_pjsip_refer'
-         - asterisk : 'res_pjsip_send_to_voicemail'
+         - python: 'twisted'
+         - python: 'starpy'
+         - app: 'sipp'
+         - asterisk: 'app_dial'
+         - asterisk: 'app_userevent'
+         - asterisk: 'func_callerid'
+         - asterisk: 'res_pjsip'
+         - asterisk: 'res_pjsip_header_funcs'
+         - asterisk: 'res_pjsip_refer'
+         - asterisk: 'res_pjsip_send_to_voicemail'
     tags:
         - pjsip
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4b113c33a1727851bfa6181b63018a4461904160
Gerrit-PatchSet: 2
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>



More information about the asterisk-code-review mailing list