[asterisk-dev] [Code Review] 3420: Testsuite: Call Files' Max Retries

[opticron]

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



./asterisk/trunk/tests/pbx/call_file_retries_alwaysdelete/configs/ast1/extensions.conf
<https://reviewboard.asterisk.org/r/3420/#comment21317>

    Where does this check that the call file was not deleted?



./asterisk/trunk/tests/pbx/call_file_retries_alwaysdelete/test-config.yaml
<https://reviewboard.asterisk.org/r/3420/#comment21316>

    What does this test rely on that doesn't exist in the 1.8 branch? This applies to the other tests as well.



./asterisk/trunk/tests/pbx/call_file_retries_archive/configs/ast1/extensions.conf
<https://reviewboard.asterisk.org/r/3420/#comment21318>

    Where does this check that the call file was archived correctly?



./asterisk/trunk/tests/pbx/call_file_retries_success/configs/ast1/extensions.conf
<https://reviewboard.asterisk.org/r/3420/#comment21320>

    The Answer() here is never called and an unconditional goto would work just as well since the condition is checked again under failure at test_set.


- opticron


On April 4, 2014, 6:15 p.m., Scott Emidy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3420/
> -----------------------------------------------------------
> 
> (Updated April 4, 2014, 6:15 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-23218
>     https://issues.asterisk.org/jira/browse/ASTERISK-23218
> 
> 
> Repository: testsuite
> 
> 
> Description
> -------
> 
> These tests involved checking that call files max retries are functioning as planned through four tests:
> 
> 1) The first test (call_file_retries_fail) required that the call file originates a local channel to a dialplan extension that will always fail, and checks to make sure that it ran through all of its max retries.
> 
> 2) The second test (call_file_retries_success) involves a call file that originates a local channel that will fail once, but then is answered before it hits its max retries.
> 
> 3) The third test (call_file_retries_alwaysdelete) consists of checking whether or not the call file was deleted from the [astspooldir]'s outgoing folder when the alwaysdelete option is set to 'no'.
> 
> 4) The fourth and final test (call_file_retries_archive) consists of checking whether or not the call file was placed in [astspooldir]'s outgoing_done folder when archive is set to 'yes'.
> 
> 
> Diffs
> -----
> 
>   ./asterisk/trunk/tests/pbx/tests.yaml 4935 
>   ./asterisk/trunk/tests/pbx/call_file_retries_success/test-config.yaml PRE-CREATION 
>   ./asterisk/trunk/tests/pbx/call_file_retries_success/configs/ast1/extensions.conf PRE-CREATION 
>   ./asterisk/trunk/tests/pbx/call_file_retries_fail/test-config.yaml PRE-CREATION 
>   ./asterisk/trunk/tests/pbx/call_file_retries_fail/configs/ast1/extensions.conf PRE-CREATION 
>   ./asterisk/trunk/tests/pbx/call_file_retries_archive/test-config.yaml PRE-CREATION 
>   ./asterisk/trunk/tests/pbx/call_file_retries_archive/configs/ast1/extensions.conf PRE-CREATION 
>   ./asterisk/trunk/tests/pbx/call_file_retries_alwaysdelete/test-config.yaml PRE-CREATION 
>   ./asterisk/trunk/tests/pbx/call_file_retries_alwaysdelete/configs/ast1/extensions.conf PRE-CREATION 
>   ./asterisk/trunk/lib/python/asterisk/pluggable_modules.py 4935 
> 
> Diff: https://reviewboard.asterisk.org/r/3420/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Scott Emidy
> 
>

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