[asterisk-dev] [Code Review] Add bridge tests to check transfer capabilites

jrose reviewboard at asterisk.org
Tue Feb 19 15:37:20 CST 2013


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


Hmmmmm. 4 calls in the bridge module. 3 CDRs and 3 LINKEDID_END events... I think you can guess what I'm getting at.

I'm still getting a reactor timeout and we still aren't getting to the fourth call. That said, the test passes on my box now, but this is still incorrect behavior because the bridge module is failing. Clearly the bridge module needs failure tokens because it can fail on timeout like it has been and it will never report the error.  Then when the rest of the modules report success (or at least one of them reports success and none of the others report failure) the test appears to pass even though it critically failed like this.

- jrose


On Feb. 19, 2013, 2:40 p.m., opticron wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2268/
> -----------------------------------------------------------
> 
> (Updated Feb. 19, 2013, 2:40 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> Test blind and attended transfers from Alice and Bob to ensure that the "T" and "t" options allow the correct parties to perform transfers when used from the Dial application and from BRIDGE_FEATURES.
> 
> 
> This addresses bug SWP-5268.
>     https://issues.asterisk.org/jira/browse/SWP-5268
> 
> 
> Diffs
> -----
> 
>   asterisk/trunk/lib/python/asterisk/BridgeTestCase.py 3642 
>   asterisk/trunk/tests/bridge/tests.yaml 3642 
>   asterisk/trunk/tests/bridge/transfer_capabilities/configs/ast1/extensions.conf PRE-CREATION 
>   asterisk/trunk/tests/bridge/transfer_capabilities/test-config.yaml PRE-CREATION 
> 
> Diff: https://reviewboard.asterisk.org/r/2268/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> opticron
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130219/995a6e7c/attachment.htm>


More information about the asterisk-dev mailing list