[asterisk-dev] [Code Review] MixMonitor external test

David Vossel dvossel at digium.com
Thu Apr 29 15:16:09 CDT 2010


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

(Updated 2010-04-29 15:16:09.384984)


Review request for Asterisk Developers.


Changes
-------

Update includes verification that the second audiofile was released during hangup by the MixMonitor thread.  This is done through the addition of a second test extension.  The new extension is called after a period of time to playback the second audiofile... If that playback is successful then we know the previous MixMonitor thread exited properly.


Summary
-------

MixMonitor external test.

Tests MixMonitor with StopMixMonitor, verifying that StopMixMonitor releases the audio file so it can be processed by the dialplan.
Tests MixMonitor without StopMixMonitor, verifying that MixMonitor stops correctly on hangup.
Tests the size of the two audio files to see if they are what we expect given the length of time processed.

Future improvements to this test will include using AUDIOHOOK_INHERIT.


Diffs (updated)
-----

  /asterisk/trunk/tests/mixmonitor/configs/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/mixmonitor/run-test PRE-CREATION 
  /asterisk/trunk/tests/mixmonitor/test-config.yaml PRE-CREATION 

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


Testing
-------

Test passes with trunk and 1.4


Thanks,

David




More information about the asterisk-dev mailing list