[asterisk-dev] [Code Review] Add tests for Path header support

[Matt Jordan]

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

Review request for Asterisk Developers, Olle E Johansson and Klaus Darilion.


Summary
-------

This patch adds a test to the Asterisk Test Suite to support the Path header extension in https://reviewboard.asterisk.org/r/2235/. The tests covers two scenarios:

1. A UA sends a REGISTER request to Asterisk. The test verifies that the Path header in the request is copied into the 200 OK response and that the response indicates support for 'path'. It then verifies that a subsequent request sent to the UA (An INVITE for a new call) contains the path information in the Route header.

2. If path support is enabled globally, it verifies that outbound REGISTER requests from Asterisk indicate support for 'path'.


This addresses bug ASTERISK-16884.
    https://issues.asterisk.org/jira/browse/ASTERISK-16884


Diffs
-----

  /asterisk/trunk/tests/channels/SIP/path/sipp/registrar.xml PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/path/sipp/ua1_invite_recv.xml PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/path/configs/ast1/sip.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/path/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/path/sipp/ua1_register.xml PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/path/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/channels/SIP/tests.yaml 3635 

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


Testing
-------

Tests were set up in Bamboo to run on Olle's branch. Note that prior test runs were expected to fail, as the branch didn't yet send the Path header in the REGISTER response, which the test checks for.

A successful test run *should* be http://bamboo.asterisk.org/browse/ASTTEAM-OOLONGPATHSUPPORTTRUNK-12/ (when its done running, anyway)


Thanks,

Matt

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130213/8cc170e5/attachment.htm>