[asterisk-dev] [Code Review] External tests for func_srv and explicit SIP outgoing destination

[Mark Michelson]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/609/
-----------------------------------------------------------

(Updated 2010-04-07 15:25:52.959913)


Review request for Asterisk Developers.


Changes
-------

I added a new test extension to the func_srv test to test the case where SRVQUERY is not used prior to using SRVRESULT to reflect the latest changes on review 608. Tests still pass.


Summary
-------

Two new external tests for Asterisk, meant to exercise the new code added in review 608.

The first has a simple dialplan that queries for known SRV records and then compares what is returned to what we expect to have been returned. This simultaneously exercises the ordering of SRV records by ast_srv_lookup while also testing the validity of the data retrieved by SRVQUERY and SRVRESULT.

The second runs several test calls through a dialplan that formats some SIP dialstrings quite differently. For each iteration of the test, we run two SIPp UAS scenarios. One is the UAS that is reached if using the parameters from sip.conf, and the other is the one that is reached if we override the destination in the dialstring. Each UAS will answer the call and send a specific DTMF digit. We read this digit and based on which digit we receive, we can determine if the proper UAS answered.


Diffs (updated)
-----

  /asterisk/trunk/tests/func_srv/configs/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/func_srv/run-test PRE-CREATION 
  /asterisk/trunk/tests/func_srv/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/func_srv/test.lua PRE-CREATION 
  /asterisk/trunk/tests/sip_outbound_address/configs/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/sip_outbound_address/configs/sip.conf PRE-CREATION 
  /asterisk/trunk/tests/sip_outbound_address/run-test PRE-CREATION 
  /asterisk/trunk/tests/sip_outbound_address/sipp/dtmf_2833_1.pcap UNKNOWN 
  /asterisk/trunk/tests/sip_outbound_address/sipp/dtmf_2833_2.pcap UNKNOWN 
  /asterisk/trunk/tests/sip_outbound_address/sipp/dtmf_2833_pound.pcap UNKNOWN 
  /asterisk/trunk/tests/sip_outbound_address/sipp/uac.xml PRE-CREATION 
  /asterisk/trunk/tests/sip_outbound_address/sipp/uac1.xml PRE-CREATION 
  /asterisk/trunk/tests/sip_outbound_address/sipp/uac2.xml PRE-CREATION 
  /asterisk/trunk/tests/sip_outbound_address/sipp/uac3.xml PRE-CREATION 
  /asterisk/trunk/tests/sip_outbound_address/sipp/uac4.xml PRE-CREATION 
  /asterisk/trunk/tests/sip_outbound_address/sipp/uac5.xml PRE-CREATION 
  /asterisk/trunk/tests/sip_outbound_address/sipp/uac6.xml PRE-CREATION 
  /asterisk/trunk/tests/sip_outbound_address/sipp/uas1.xml PRE-CREATION 
  /asterisk/trunk/tests/sip_outbound_address/sipp/uas2.xml PRE-CREATION 
  /asterisk/trunk/tests/sip_outbound_address/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/sip_outbound_address/test.lua PRE-CREATION 

Diff: https://reviewboard.asterisk.org/r/609/diff


Testing
-------

The tests both pass.


Thanks,

Mark