[asterisk-dev] [Code Review] 2371: Allow running test suite locally - initial work

[opticron]

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



/asterisk/trunk/lib/python/asterisk/TestCase.py
<https://reviewboard.asterisk.org/r/2371/#comment15703>

    This can go away.



/asterisk/trunk/lib/python/asterisk/asterisk.py
<https://reviewboard.asterisk.org/r/2371/#comment15704>

    This simple string replacement can be done with string.replace(). No need to dive into regex land.



/asterisk/trunk/lib/python/asterisk/asterisk.py
<https://reviewboard.asterisk.org/r/2371/#comment15705>

    Idem.


- opticron


On March 23, 2013, 4:23 a.m., Tzafrir Cohen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2371/
> -----------------------------------------------------------
> 
> (Updated March 23, 2013, 4:23 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Repository: testsuite
> 
> 
> Description
> -------
> 
> Place anything needed for the test suite (except the actual sources of Asterisk) under the test suite directory.
> 
> Fix parts in the python code. Add a wrapper script (run_local) for the required automation.
> 
> Work is still needed on asttest (add a destdir option? Consolidate all the run-test scripts to run it through a common function?) but I'm not exactly sure how the information is passed to Lua.
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/tests/channels/SIP/sip_channel_params/run-test 3678 
>   /asterisk/trunk/tests/channels/SIP/tcpauthlimit/run-test 3678 
>   /asterisk/trunk/tests/channels/SIP/tcpauthtimeout/run-test 3678 
>   /asterisk/trunk/tests/fax/local_channel_t38_queryoption/run-test 3678 
>   /asterisk/trunk/tests/cdr/blind-transfer-accountcode/run-test 3678 
>   /asterisk/trunk/tests/cdr/originate-cdr-disposition/run-test 3678 
>   /asterisk/trunk/tests/channels/SIP/handle_response_refer/run-test 3678 
>   /asterisk/trunk/tests/channels/SIP/rfc2833_dtmf_detect/run-test 3678 
>   /asterisk/trunk/run-local PRE-CREATION 
>   /asterisk/trunk/tests/apps/queues/macro_gosub_test/run-test 3678 
>   /asterisk/trunk/tests/apps/queues/position_priority_maxlen/run-test 3678 
>   /asterisk/trunk/tests/apps/queues/ringinuse_and_pause/run-test 3678 
>   /asterisk/trunk/tests/apps/queues/wrapup_time/run-test 3678 
>   /asterisk/trunk/tests/cdr/app_dial_G_flag/run-test 3678 
>   /asterisk/trunk/tests/cdr/app_queue/run-test 3678 
>   /asterisk/trunk/README.txt 3678 
>   /asterisk/trunk/lib/python/asterisk/TestCase.py 3678 
>   /asterisk/trunk/lib/python/asterisk/asterisk.py 3678 
>   /asterisk/trunk/tests/manager/action-events-response/run-test 3678 
>   /asterisk/trunk/tests/manager/authlimit/run-test 3678 
>   /asterisk/trunk/tests/manager/authtimeout/run-test 3678 
>   /asterisk/trunk/tests/manager/response-time/run-test 3678 
> 
> Diff: https://reviewboard.asterisk.org/r/2371/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Tzafrir Cohen
> 
>

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