[Asterisk-code-review] fax/pjsip/gateway g711 t38: Add new test. (testsuite[15])
Jenkins2
asteriskteam at digium.com
Tue Jul 10 11:14:24 CDT 2018
Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/9343 )
Change subject: fax/pjsip/gateway_g711_t38: Add new test.
......................................................................
fax/pjsip/gateway_g711_t38: Add new test.
This test verifies proper gateway functionality when the calling leg is
g711 and the answering leg is t38. Two asterisk instances are started and
a call is made from ast1 to ast2. Ast2 will start a fax gateway and send
the call back to ast1. Ast1 runs SendFax() on the calling side and
ReceiveFax() on the answering side. If the fax is successful, the test
passes.
ASTERISK-27657
Change-Id: Ic88e132a3ce5af1bc7640eba8b9ed17cfdcd620f
---
A tests/fax/pjsip/gateway_g711_t38/configs/ast1/extensions.conf
A tests/fax/pjsip/gateway_g711_t38/configs/ast1/pjsip.conf
A tests/fax/pjsip/gateway_g711_t38/configs/ast2/extensions.conf
A tests/fax/pjsip/gateway_g711_t38/configs/ast2/pjsip.conf
A tests/fax/pjsip/gateway_g711_t38/run-test
A tests/fax/pjsip/gateway_g711_t38/send.tiff
A tests/fax/pjsip/gateway_g711_t38/test-config.yaml
M tests/fax/pjsip/tests.yaml
8 files changed, 228 insertions(+), 0 deletions(-)
Approvals:
Kevin Harwell: Looks good to me, but someone else must approve
Joshua Colp: Looks good to me, approved
Jenkins2: Approved for Submit
diff --git a/tests/fax/pjsip/gateway_g711_t38/configs/ast1/extensions.conf b/tests/fax/pjsip/gateway_g711_t38/configs/ast1/extensions.conf
new file mode 100644
index 0000000..d2c1909
--- /dev/null
+++ b/tests/fax/pjsip/gateway_g711_t38/configs/ast1/extensions.conf
@@ -0,0 +1,18 @@
+[receivefax]
+exten = 1234,1,NoOp()
+same = n,ReceiveFax(${ASTDATADIR}/receive.tiff)
+same = n,Hangup()
+
+exten = h,1,NoOp()
+same = n,UserEvent(FaxStatus,operation: receive,status: ${FAXOPT(status)},statusstr: ${FAXOPT(statusstr)},error: ${FAXOPT(error)})
+same = n,Hangup()
+
+[sendfax]
+exten = 1234,1,NoOp()
+same = n,SendFax(${ASTDATADIR}/send.tiff)
+same = n,Hangup()
+
+exten = h,1,NoOp()
+same = n,UserEvent(FaxStatus,operation: send,status: ${FAXOPT(status)},statusstr: ${FAXOPT(statusstr)},error: ${FAXOPT(error)})
+same = n,Hangup()
+
diff --git a/tests/fax/pjsip/gateway_g711_t38/configs/ast1/pjsip.conf b/tests/fax/pjsip/gateway_g711_t38/configs/ast1/pjsip.conf
new file mode 100644
index 0000000..c2ce7f7
--- /dev/null
+++ b/tests/fax/pjsip/gateway_g711_t38/configs/ast1/pjsip.conf
@@ -0,0 +1,22 @@
+[global]
+type=global
+debug=yes
+
+[local-transport]
+type=transport
+protocol=udp
+bind=127.0.0.1
+
+[endpoint-template](!)
+type=endpoint
+allow=!all,ulaw,alaw
+context=receivefax
+media_address=127.0.0.1
+
+[ast2-t38](endpoint-template)
+t38_udptl=yes
+from_user=ast1-t38
+
+[ast2-g711](endpoint-template)
+t38_udptl=no
+from_user=ast1-g711
diff --git a/tests/fax/pjsip/gateway_g711_t38/configs/ast2/extensions.conf b/tests/fax/pjsip/gateway_g711_t38/configs/ast2/extensions.conf
new file mode 100644
index 0000000..679b24e
--- /dev/null
+++ b/tests/fax/pjsip/gateway_g711_t38/configs/ast2/extensions.conf
@@ -0,0 +1,14 @@
+[gatewayfax]
+exten = 1234,1,NoOp()
+same = n,Dial(PJSIP/ast1-t38/sip:1234 at 127.0.0.1,,b(setup^1))
+same = n,Hangup()
+
+exten = setup,1,NoOp()
+same = n,Set(FAXOPT(gateway)=yes)
+same = n,Set(CHANNEL(hangup_handler_push)=hangup,1)
+same = n,Return()
+
+exten = hangup,1,NoOp()
+same = n,UserEvent(FaxStatus,operation: gateway,status: ${FAXOPT(status)},statusstr: ${FAXOPT(statusstr)},error: ${FAXOPT(error)})
+same = n,Return()
+
diff --git a/tests/fax/pjsip/gateway_g711_t38/configs/ast2/pjsip.conf b/tests/fax/pjsip/gateway_g711_t38/configs/ast2/pjsip.conf
new file mode 100644
index 0000000..2a00ebf
--- /dev/null
+++ b/tests/fax/pjsip/gateway_g711_t38/configs/ast2/pjsip.conf
@@ -0,0 +1,22 @@
+[global]
+type=global
+debug=yes
+
+[local-transport]
+type=transport
+protocol=udp
+bind=127.0.0.2
+
+[endpoint-template](!)
+type=endpoint
+allow=!all,ulaw,alaw
+context=gatewayfax
+media_address=127.0.0.2
+
+[ast1-t38](endpoint-template)
+t38_udptl=yes
+from_user=ast2-t38
+
+[ast1-g711](endpoint-template)
+t38_udptl=no
+from_user=ast2-g711
diff --git a/tests/fax/pjsip/gateway_g711_t38/run-test b/tests/fax/pjsip/gateway_g711_t38/run-test
new file mode 100755
index 0000000..7e43d4e
--- /dev/null
+++ b/tests/fax/pjsip/gateway_g711_t38/run-test
@@ -0,0 +1,116 @@
+#!/usr/bin/env python
+# vim: sw=3 et:
+'''
+Copyright (C) 2018, Digium, Inc.
+Matthew Nicholson <mnicholson at digium.com>
+Richard Mudgett <rmudgett at digium.com>
+
+This program is free software, distributed under the terms of
+the GNU General Public License Version 2.
+'''
+
+import sys
+import logging
+import os
+import re
+import shutil
+
+from twisted.internet import reactor
+
+sys.path.append("lib/python")
+from asterisk.asterisk import Asterisk
+from asterisk.test_case import TestCase
+
+logger = logging.getLogger(__name__)
+
+
+class GatewayTest(TestCase):
+ event_count = 0
+ success_count = 0
+
+ def __init__(self):
+ TestCase.__init__(self)
+ self.reactor_timeout = 120
+ self.create_asterisk(2)
+
+ # copy the tiff file we are going to send to a good known location
+ shutil.copy("%s/send.tiff" % (os.path.dirname(os.path.realpath(__file__)),),
+ "%s%s" % (self.ast[0].base, self.ast[0].directories['astdatadir']))
+
+ def ami_connect(self, ami):
+ if ami.id == 0:
+
+ logger.info("sending a call from ast1 to ast2")
+
+ ami.registerEvent('UserEvent', self.fax_result)
+ df = ami.originate("PJSIP/ast2-g711/sip:1234 at 127.0.0.2",
+ "sendfax", "1234", 1)
+
+ def handle_failure(reason):
+ logger.error("error sending originate:")
+ logger.error(reason.getTraceback())
+ self.stop_reactor()
+
+ return reason
+
+ df.addErrback(handle_failure)
+ else:
+ ami.registerEvent('UserEvent', self.fax_gateway_result)
+
+ def fax_gateway_result(self, ami, event):
+ if event['userevent'] != 'FaxStatus':
+ return
+
+ self.event_count += 1
+
+ if event['status'] == "SUCCESS":
+ logger.info("successful gateway operation")
+ self.success_count += 1
+ else:
+ logger.error("gateway error:")
+ logger.error("status: %s" % (event['status'],))
+ logger.error("error: %s" % (event['error'],))
+ logger.error("statusstr: %s" % (event['statusstr'],))
+
+ self.are_we_there_yet()
+
+ def fax_result(self, ami, event):
+ if event['userevent'] != 'FaxStatus':
+ return
+
+ self.event_count += 1
+
+ if event['status'] == "SUCCESS":
+ logger.info("successful %s through gateway" % (event['operation'],))
+ self.success_count += 1
+ else:
+ logger.error("error sending fax through gateway:")
+ logger.error("operation: %s" % (event['operation'],))
+ logger.error("status: %s" % (event['status'],))
+ logger.error("error: %s" % (event['error'],))
+ logger.error("statusstr: %s" % (event['statusstr'],))
+
+ self.are_we_there_yet()
+
+ def are_we_there_yet(self):
+ if self.event_count == 3:
+ if self.success_count == 3:
+ self.passed = True
+ self.stop_reactor()
+
+ def run(self):
+ TestCase.run(self)
+ self.create_ami_factory(2)
+
+
+def main():
+ test = GatewayTest()
+ reactor.run()
+
+ if not test.passed:
+ return 1
+
+ return 0
+
+if __name__ == "__main__":
+ sys.exit(main() or 0)
diff --git a/tests/fax/pjsip/gateway_g711_t38/send.tiff b/tests/fax/pjsip/gateway_g711_t38/send.tiff
new file mode 100644
index 0000000..a5fb4fe
--- /dev/null
+++ b/tests/fax/pjsip/gateway_g711_t38/send.tiff
Binary files differ
diff --git a/tests/fax/pjsip/gateway_g711_t38/test-config.yaml b/tests/fax/pjsip/gateway_g711_t38/test-config.yaml
new file mode 100644
index 0000000..15a597b
--- /dev/null
+++ b/tests/fax/pjsip/gateway_g711_t38/test-config.yaml
@@ -0,0 +1,35 @@
+testinfo:
+ summary: 'Test the fax gateway.'
+ description: |
+ This test verifies proper gateway functionality when the calling leg
+ is g711 and the answering leg is t38. Two asterisk instances are
+ started and a call is made from ast1 to ast2. Ast2 will start a fax
+ gateway and send the call back to ast1. Ast1 runs SendFax() on the
+ calling side and ReceiveFax() on the answering side. If the fax is
+ successful, the test passes.
+
+ The call flow looks like this:
+ |send |
+ |g711 g711|
+ |----------->|
+ | gateway|
+ |t38 t38|
+ |<-----------|
+ \|receive |/
+
+properties:
+ dependencies:
+ - python : 'twisted'
+ - python : 'starpy'
+ - custom : 'fax'
+ - asterisk : 'app_dial'
+ - asterisk : 'app_stack'
+ - asterisk : 'app_userevent'
+ - asterisk : 'chan_pjsip'
+ - asterisk : 'func_channel'
+ - asterisk : 'res_fax'
+ - asterisk : 'res_pjsip_t38'
+ tags:
+ - pjsip
+ - fax
+ - fax_gateway
diff --git a/tests/fax/pjsip/tests.yaml b/tests/fax/pjsip/tests.yaml
index e6022b1..f16dae3 100644
--- a/tests/fax/pjsip/tests.yaml
+++ b/tests/fax/pjsip/tests.yaml
@@ -2,6 +2,7 @@
tests:
- test: 'ast-2016-003'
- test: 'directmedia_reinvite_t38'
+ - test: 'gateway_g711_t38'
- test: 'gateway_native_t38'
- test: 'gateway_t38_g711'
- test: 't38'
--
To view, visit https://gerrit.asterisk.org/9343
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings
Gerrit-Project: testsuite
Gerrit-Branch: 15
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic88e132a3ce5af1bc7640eba8b9ed17cfdcd620f
Gerrit-Change-Number: 9343
Gerrit-PatchSet: 2
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180710/5b3458a2/attachment-0001.html>
More information about the asterisk-code-review
mailing list