[asterisk-dev] [Code Review] 2813: Test Suite: off-nominal outgoing pjsip tests

Mark Michelson reviewboard at asterisk.org
Thu Sep 5 14:33:00 CDT 2013


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

Ship it!


The only thing I could find to point out was a minor grammatical oddity.

For what Asterisk publishes, this test does its job. It would be good if Asterisk could publish events for when channels are destroyed and for when SIP sessions are destroyed so that tests like this could also be a canary for potential refleaks/memory leaks in the code.


/asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_answer/test-config.yaml
<https://reviewboard.asterisk.org/r/2813/#comment18782>

    s/does responds/responds/


- Mark Michelson


On Aug. 30, 2013, 8:55 p.m., jbigelow wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2813/
> -----------------------------------------------------------
> 
> (Updated Aug. 30, 2013, 8:55 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-22284
>     https://issues.asterisk.org/jira/browse/ASTERISK-22284
> 
> 
> Repository: testsuite
> 
> 
> Description
> -------
> 
> These tests cover off-nominal outgoing PJSIP tests. Each test has iterations for IPv4/IPv6 and UDP/TCP. These cover all of the off-nominal outgoing tests (1-5) as listed on the test plan page of the wiki. Tests 2 and 4 are essentially the same.
> 
> Note: The tests.yaml contain directories / tests for other outgoing tests which are on review 2808, so ignore them here.
> 
> Tests:
> * bob_does_not_exist - Originate an outgoing call to SIPp ("Bob"). Bob does not respond to the INVITE. This ensures that Asterisk tears down the calls after the default value of timer B expires.
> 
> * bob_does_not_answer - Originate an outgoing call to SIPp ("Bob"). Bob does responds with a 180 ringing but does not answer. This ensures that Asterisk sends a CANCEL and sets the DIALSTATUS to NOANSWER.
> 
> * bob_is_busy - Originate an outgoing call to SIPp ("Bob"). Bob responds with a 486 Busy Here. This ensures that Asterisk responds with an ACK to the 486 and the hang up cause is 17 / User busy.
> 
> * bob_incompatible_codecs - Originate an outgoing call to SIPp ("Bob"). Bob answers with a codec that Asterisk is incompatible with. This ensures that Asterisk sends a BYE after sending an ACK in response to the 200 OK and that the hang up cause is 58 / "Bearer capability not available".
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/tests.yaml 4098 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/tests.yaml PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/tests.yaml PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_is_busy/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_is_busy/sipp/receive.xml PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_is_busy/configs/ast1/pjsip.conf PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_is_busy/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_incompatible_codecs/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_incompatible_codecs/sipp/receive.xml PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_incompatible_codecs/configs/ast1/pjsip.conf PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_incompatible_codecs/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_exist/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_exist/sipp/receive.xml PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_exist/configs/ast1/pjsip.conf PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_exist/configs/ast1/extensions.conf PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_answer/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_answer/sipp/receive.xml PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_answer/configs/ast1/pjsip.conf PRE-CREATION 
>   /asterisk/trunk/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_answer/configs/ast1/extensions.conf PRE-CREATION 
> 
> Diff: https://reviewboard.asterisk.org/r/2813/diff/
> 
> 
> Testing
> -------
> 
> * Ensure all pass on multiple executions
> * Ensure the logs displayed calls using IPv4, IPV6, UDP, & TCP
> 
> 
> Thanks,
> 
> jbigelow
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130905/dc301b1c/attachment-0001.htm>


More information about the asterisk-dev mailing list