[asterisk-dev] [Code Review] 4014: Changes to CDR and CEL unit tests to prevent FRACKs.

[Mark Michelson]

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

(Updated Sept. 23, 2014, 9:29 a.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers and Matt Jordan.


Changes
-------

Committed in revision 423783


Repository: Asterisk


Description
-------

In Asterisk 13, the format rewrite caused CDR and CEL unit tests to have FRACKs because they have no formats on the channels they create. In revision 423414 of Asterisk 13, I introduced an independent change that is intended to prevent unnecessary FRACKs by returning early when attempting to set up translation paths if any channel has no formats set. Unfortunately, this resulted in huge breakage of the CDR and CEL unit tests.

The change introduced in this review is to set the ulaw format on channels created by the CEL and CDR unit tests. This way, the tests are passing again. Event better, they now pass FRACK-free.


Diffs
-----

  /branches/13/tests/test_cel.c 423656 
  /branches/13/tests/test_cdr.c 423656 

Diff: https://reviewboard.asterisk.org/r/4014/diff/


Testing
-------

Ran CDR and CEL unit tests with and without the change. Without this patch, things are pretty disastrous in the CEL tests (i.e. Asterisk crashes). And the CDR tests have FRACKs in them. With the patch, all CDR and CEL unit tests pass with no FRACKs.


Thanks,

Mark Michelson

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