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





 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Also, I&#39;d like to get away of the habit of &#39;copying&#39; tests to create a different test case. So in this example of CDRs, we could create another class CDRTestCase() that inherits TestCase().  Removing 95% of the duplicate code.

Moving forward I see maintaining of the testsuite to be an issue if we keep coping tests.</pre>
 <br />







<p>- Paul</p>


<br />
<p>On August 1st, 2011, 11:17 a.m., jrose wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/media/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 and Paul Belanger.</div>
<div>By jrose.</div>


<p style="color: grey;"><i>Updated Aug. 1, 2011, 11:17 a.m.</i></p>




<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;">https://reviewboard.asterisk.org/r/1331/

This is a test for a feature that logs congested calls as CONGESTION rather than as FAILURE.  This should have been a five minute rewrite, but I found a number or problems along the way involving things like long directory names.

This test is essentially a copy of the other congestion test, only with a sip setting change and a difference in expectations as well as a difference in the minversion requirement.  This test should (important qualifier) only run on trunk builds, but that is still untested and I&#39;m not entirely sure how to indicate versions above 1.8 right now in testsuite.

Perhaps it would be more proper to work this into the other test and change the expectation depending on the minversion instead (and just always have the cdr.conf file set to have congestion=yes)

congestion = yes in cdr.conf is still an unmerged feature in trunk, so in order to test this without getting failure every time, it is necessary to apply the patch in r/1331 above to trunk.

Also, I&#39;m not 100% sure if the folder name for this test is appropriate.  I had to change the folder name from something else in order to make it not randomly fail (presumably from too long of a path name, but I don&#39;t really see why that&#39;s a problem)</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 the test until I got the CDR log reporting as expected and for the test to be able to read that like it should be.  It isn&#39;t very complicated since it works basically the same as the other congestion test.</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/cdr/console_dial_sip_congestion_log/configs/ast1/cdr.conf <span style="color: grey">(PRE-CREATION)</span></li>

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

 <li>/asterisk/trunk/tests/cdr/console_dial_sip_congestion_log/configs/ast1/manager.conf <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/asterisk/trunk/tests/cdr/console_dial_sip_congestion_log/configs/ast1/sip.conf <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/asterisk/trunk/tests/cdr/console_dial_sip_congestion_log/configs/ast2/cdr.conf <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/asterisk/trunk/tests/cdr/console_dial_sip_congestion_log/configs/ast2/extensions.conf <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/asterisk/trunk/tests/cdr/console_dial_sip_congestion_log/configs/ast2/sip.conf <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/asterisk/trunk/tests/cdr/console_dial_sip_congestion_log/run-test <span style="color: grey">(PRE-CREATION)</span></li>

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

 <li>/asterisk/trunk/tests/cdr/console_dial_sip_congestion_log/test-output.txt <span style="color: grey">(PRE-CREATION)</span></li>

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

</ul>

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




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








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