[asterisk-dev] [Code Review]: CDR Overhaul - Convert all python CDR tests to use CDRTestCase

[jrose]

> On Dec. 22, 2011, 10:28 a.m., Matt Jordan wrote:
> > /asterisk/trunk/tests/cdr/console_dial_sip_busy/configs/ast1/extensions.conf, lines 9-13
> > <https://reviewboard.asterisk.org/r/1637/diff/1/?file=22334#file22334line9>
> >
> >     Do you need to test overriding the accountcode in this test?  If not, you may want to just remove this - it makes it easier to maintain if a test only tests what it needs to (particularly when you test this in other tests as well)

Unfortunately, the use of accountcode is necessary for most of these tests in order to split up the CDR records from tests that were previously using multiple servers. In addition to that, in this particular case the Master would have a local channel using "Echo" that will end at approximately the same time as the the channel we are wanting to check the CDR on, so without splitting that CDR log off somehow, that channel would create a CDR log at roughly the same time as the one that is being checked by the original tests do.


> On Dec. 22, 2011, 10:28 a.m., Matt Jordan wrote:
> > /asterisk/trunk/tests/cdr/cdr_userfield/run-test, line 22
> > <https://reviewboard.asterisk.org/r/1637/diff/1/?file=22324#file22324line22>
> >
> >     Class name doesn't reflect test name (should be CDRUserField, if you use the same nomenclature)

Just an oversight.  Fixed.


- jrose


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


On Dec. 21, 2011, 4:14 p.m., jrose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1637/
> -----------------------------------------------------------
> 
> (Updated Dec. 21, 2011, 4:14 p.m.)
> 
> 
> Review request for Asterisk Developers, Paul Belanger and Matt Jordan.
> 
> 
> Summary
> -------
> 
> This rather obnoxiously large patch converts every python (read: not lua) CDR test in the testsuite to use the new CDRTestCase class made by Paul and myself. It significantly reduces the amount of duplicate code within the CDR tests by offloading result checking and common pass/failure conditions to the CDRTestCase class.
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/tests/cdr/console_dial_sip_busy/test-config.yaml 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_congestion/configs/ast1/branch-1.4/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_busy/configs/ast2/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_busy/run-test 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_busy/configs/ast2/branch-1.4/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_busy/configs/ast2/extensions.conf 2951 
>   /asterisk/trunk/tests/cdr/cdr_userfield/configs/ast1/extensions.conf 2951 
>   /asterisk/trunk/tests/cdr/cdr_userfield/configs/ast1/manager.conf 2951 
>   /asterisk/trunk/tests/cdr/cdr_userfield/run-test 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_answer/configs/ast1/branch-1.4/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_answer/configs/ast1/extensions.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_answer/configs/ast1/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_answer/configs/ast2/branch-1.4/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_answer/configs/ast2/extensions.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_answer/configs/ast2/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_answer/run-test 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_answer/test-config.yaml 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_busy/configs/ast1/branch-1.4/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_busy/configs/ast1/extensions.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_busy/configs/ast1/manager.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_busy/configs/ast1/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/cdr_unanswered_yes/configs/ast1/extensions.conf 2951 
>   /asterisk/trunk/tests/cdr/cdr_unanswered_yes/configs/ast1/manager.conf 2951 
>   /asterisk/trunk/tests/cdr/cdr_unanswered_yes/run-test 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_congestion/configs/ast1/extensions.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_congestion/configs/ast1/manager.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_congestion/configs/ast1/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_congestion/configs/ast2/branch-1.4/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_congestion/configs/ast2/extensions.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_congestion/configs/ast2/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_congestion/run-test 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_congestion/test-config.yaml 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_transfer/configs/ast1/branch-1.4/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_transfer/configs/ast1/extensions.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_transfer/configs/ast1/manager.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_transfer/configs/ast1/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_transfer/configs/ast2/branch-1.4/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_transfer/configs/ast2/extensions.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_transfer/configs/ast2/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_dial_sip_transfer/run-test 2951 
>   /asterisk/trunk/tests/cdr/console_fork_after_busy_forward/configs/ast1/extensions.conf 2951 
>   /asterisk/trunk/tests/cdr/console_fork_after_busy_forward/configs/ast1/manager.conf 2951 
>   /asterisk/trunk/tests/cdr/console_fork_after_busy_forward/configs/ast1/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_fork_after_busy_forward/configs/ast2/branch-1.4/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_fork_after_busy_forward/configs/ast2/extensions.conf 2951 
>   /asterisk/trunk/tests/cdr/console_fork_after_busy_forward/configs/ast2/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_fork_after_busy_forward/run-test 2951 
>   /asterisk/trunk/tests/cdr/console_fork_after_busy_forward/test-config.yaml 2951 
>   /asterisk/trunk/tests/cdr/console_fork_before_dial/configs/ast1/branch-1.4/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_fork_before_dial/configs/ast1/extensions.conf 2951 
>   /asterisk/trunk/tests/cdr/console_fork_before_dial/configs/ast1/manager.conf 2951 
>   /asterisk/trunk/tests/cdr/console_fork_before_dial/configs/ast1/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_fork_before_dial/configs/ast2/branch-1.4/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_fork_before_dial/configs/ast2/extensions.conf 2951 
>   /asterisk/trunk/tests/cdr/console_fork_before_dial/configs/ast2/sip.conf 2951 
>   /asterisk/trunk/tests/cdr/console_fork_before_dial/run-test 2951 
>   /asterisk/trunk/tests/cdr/console_fork_before_dial/test-config.yaml 2951 
>   /asterisk/trunk/tests/cdr/nocdr/configs/ast1/manager.conf 2951 
>   /asterisk/trunk/tests/cdr/nocdr/run-test 2951 
>   /asterisk/trunk/tests/cdr/tests.yaml 2951 
> 
> Diff: https://reviewboard.asterisk.org/r/1637/diff
> 
> 
> Testing
> -------
> 
> All of the CDR tests have been ran to make sure they behave as expected and they have been checked to ensure that they produce the same expected CDR files (for the most part.  Some elements have changed due to necessity so that soundcard dependency could be removed) and that these expectations are validated accordingly.
> 
> 
> Thanks,
> 
> jrose
> 
>

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