[asterisk-dev] [Code Review] 3206: HEP: Add test for PJSIP HEP packet capture

Matt Jordan reviewboard at asterisk.org
Sun Feb 23 00:04:27 CST 2014


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

(Updated Feb. 23, 2014, 12:04 a.m.)


Review request for Asterisk Developers.


Changes
-------

Addressed Mark's findings.

Also added tests that check the authentication key field in a HEP packet, as well as IPv6 source/destination addresses. With these tests added, all possible fields now have a corresponding test.


Repository: testsuite


Description
-------

This patch adds the following:

* A pluggable module that emulates a HEP capture server for verifying packets.
* A function that can, in a rudimentary fashion, verify SIP packets. It's simplistic, but good enough to verify that Asterisk sent the server the SIP message traffic we sent to it from SIPp.
* A test that verifies that the res_hep/res_hep_pjsip modules forward SIP message traffic to our little capture server.


Diffs (updated)
-----

  /asterisk/trunk/tests/tests.yaml 4685 
  /asterisk/trunk/tests/hep/tests.yaml PRE-CREATION 
  /asterisk/trunk/tests/hep/pjsip/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/hep/pjsip/sipp/echo_with_deferred_sdp.xml PRE-CREATION 
  /asterisk/trunk/tests/hep/pjsip/configs/ast1/pjsip.conf PRE-CREATION 
  /asterisk/trunk/tests/hep/pjsip/configs/ast1/hep.conf PRE-CREATION 
  /asterisk/trunk/tests/hep/pjsip/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/hep/pjsip-ipv6/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/hep/pjsip-ipv6/sipp/echo_with_deferred_sdp.xml PRE-CREATION 
  /asterisk/trunk/tests/hep/pjsip-ipv6/configs/ast1/pjsip.conf PRE-CREATION 
  /asterisk/trunk/tests/hep/pjsip-ipv6/configs/ast1/hep.conf PRE-CREATION 
  /asterisk/trunk/tests/hep/pjsip-ipv6/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/hep/pjsip-auth/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/hep/pjsip-auth/sipp/echo_with_deferred_sdp.xml PRE-CREATION 
  /asterisk/trunk/tests/hep/pjsip-auth/configs/ast1/pjsip.conf PRE-CREATION 
  /asterisk/trunk/tests/hep/pjsip-auth/configs/ast1/hep.conf PRE-CREATION 
  /asterisk/trunk/tests/hep/pjsip-auth/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/hep/hep_capture_node.py PRE-CREATION 

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


Testing
-------


Thanks,

Matt Jordan

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140223/a41fc7d1/attachment-0001.html>


More information about the asterisk-dev mailing list