<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/2821/">https://reviewboard.asterisk.org/r/2821/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On September 5th, 2013, 7:02 p.m. UTC, <b>Mark Michelson</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">With this and the other CEL-related tests I&#39;m putting &quot;Ship it!s&quot; on, I&#39;d recommend getting a second opinion from Matt or Kinsey since they are likely more familiar with the CEL workings in Asterisk and the testsuite. That being said, Ship it!</pre>
 </blockquote>







</blockquote>

<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I have to wait for r2799&#39;s library changes to go in before I commit these anyway :P</pre>
<br />










<p>- jrose</p>


<br />
<p>On September 3rd, 2013, 10:51 p.m. UTC, jrose wrote:</p>








<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, kmoore and Matt Jordan.</div>
<div>By jrose.</div>


<p style="color: grey;"><i>Updated Sept. 3, 2013, 10:51 p.m.</i></p>







<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-22325">ASTERISK-22325</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;">Uses python library changes from https://reviewboard.asterisk.org/r/2799/ to add AMI functionality for IDs/before and after ID event matching

The reasons for this change are basically the same as with disconnect and the added test module for each test is very similar.
In addition, the method of applying the features with the BRIDGE_FEATURES channel variable was split into a separate test for each test to be able to run the base tests against Asterisk 11. As a side benefit, there are four faster, smaller tests which test less now instead of two somewhat larger tests.</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;">Ran each test against 11 and 12 (the BRIDGE_FEATURES variants to make sure they would be skipped against 11). Ran the tests against 12 numerous times to make sure they wouldn&#39;t fail under any of the potential orderings of CEL logs that I have encountered so far.</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>/asterisk/trunk/tests/bridge/automixmon/configs/ast1/extensions.conf <span style="color: grey">(4102)</span></li>

 <li>/asterisk/trunk/tests/bridge/automixmon/test-config.yaml <span style="color: grey">(4102)</span></li>

 <li>/asterisk/trunk/tests/bridge/automon/configs/ast1/extensions.conf <span style="color: grey">(4102)</span></li>

 <li>/asterisk/trunk/tests/bridge/automixmon_bridgefeatures/test-config.yaml <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/asterisk/trunk/tests/bridge/automixmon_bridgefeatures/configs/ast1/extensions.conf <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/asterisk/trunk/tests/bridge/tests.yaml <span style="color: grey">(4102)</span></li>

 <li>/asterisk/trunk/tests/bridge/automon_bridgefeatures/test-config.yaml <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/asterisk/trunk/tests/bridge/automon_bridgefeatures/configs/ast1/extensions.conf <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/asterisk/trunk/tests/bridge/automon/test-config.yaml <span style="color: grey">(4102)</span></li>

</ul>

<p><a href="https://reviewboard.asterisk.org/r/2821/diff/" style="margin-left: 3em;">View Diff</a></p>







  </td>
 </tr>
</table>








  </div>
 </body>
</html>