<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/3795/">https://reviewboard.asterisk.org/r/3795/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers.</div>
<div>By Tyler Austin Cambron.</div>
<p style="color: grey;"><i>Updated July 22, 2014, 3:01 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Corrected test to use the MixMonitor AMI command rather than call MixMonitor through the dialplan by adding a small bit of python code. The test-config.yaml has been updated to work with updates to the SoundChecker pluggable module. Also updated the size to check for in the test-config to be slightly more accurate.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://issues.asterisk.org/jira/browse/Asterisk-24028">Asterisk-24028</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
testsuite
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This is a test that verifies that the recording feature of MixMonitor is functioning properly. The test uses the SoundChecker pluggable module, which is currently still up for review, so the test could change depending on changes with the pluggable module. Assuming the pluggable module is working correctly, this test uses the pluggable module to verify that audio has been recorded from a local channel call and that the recording has correctly created a sound file that contains the full recording. The pluggable module will appear in the diff, specifically to show how the test uses the pluggable module. Please do not review the module on this thread, as it is being reviewed in another thread posted by cwolfe.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The test begins by executing dialplan, where MixMonitor is started and a playback of tt-monkeys begins. MixMonitor records this audio and stores it into a file called theRecording.wav. When the recording is finished, the dialplan executes a hangup, which triggers (through the yaml file) a size check and energy check on the newly created file, which is done using the SoundChecker pluggable module. The test logs confirm that a call is made, the audio starts and is recorded, the file is saved, and then the pluggable module verifies that the file has the correct size and energy, showing that there is actual audio in the file. I also went and found the file manually, and the recording fully played back tt-monkeys.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/asterisk/trunk/tests/apps/tests.yaml <span style="color: grey">(5297)</span></li>
<li>/asterisk/trunk/tests/apps/mixmonitor_record/test-config.yaml <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/apps/mixmonitor_record/start_mix.py <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/apps/mixmonitor_record/configs/ast1/extensions.conf <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/lib/python/asterisk/pluggable_modules.py <span style="color: grey">(5297)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/3795/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>