<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/2792/">https://reviewboard.asterisk.org/r/2792/</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.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers.</div>
<div>By Mark Michelson.</div>
<p style="color: grey;"><i>Updated Sept. 3, 2013, 2:20 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;">Addressed review feedback.</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-22315">ASTERISK-22315</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;">The queue_baseline, set_penalty, and position_priority_maxlen tests were failing with Asterisk 12. In each case, the reason was that the tests had not been modified to take into account the new names of manager events or the new names of manager event headers. The queue_baseline and set_penalty tests have small modifications to make them pass.
The position_priority_maxlen test, on the other hand, has been converted from lua to python. This wasn't necessary to get the test to pass, but the test now runs in ~15 seconds instead of ~80 seconds. The lua script had difficulty terminating Asterisk once the test actually completed, so the test would run until the test timeout was reached. The python test library shuts Asterisk down more quickly once the test cases are complete, and so the test ends much more quickly.</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;">I ran the three tests against both Asterisk 11 and Asterisk 12 after making the changes. Tests pass on both versions of Asterisk.</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/queues/position_priority_maxlen/configs/ast1/extensions.conf <span style="color: grey">(4085)</span></li>
<li>/asterisk/trunk/tests/apps/queues/position_priority_maxlen/run-test <span style="color: grey">(4085)</span></li>
<li>/asterisk/trunk/tests/apps/queues/position_priority_maxlen/test.lua <span style="color: grey">(4085)</span></li>
<li>/asterisk/trunk/tests/apps/queues/queue_baseline/run-test <span style="color: grey">(4085)</span></li>
<li>/asterisk/trunk/tests/apps/queues/set_penalty/run-test <span style="color: grey">(4085)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2792/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>