<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/2063/">https://reviewboard.asterisk.org/r/2063/</a>
</td>
</tr>
</table>
<br />
<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.</div>
<div>By Matt Jordan.</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 patch adds three tests for Queue that verify CDR/CEL records when a user is in a queue and no one answers:
1. It verifies that if you have multiple agents and they ignore the caller, the CDR disposition is NO ANSWER
2. It verifies that if you have multiple agents, one being paused, and the other ignores the caller, the CDR disposition is NO ANSWER (this currently fails in 1.8+ - the disposition is BUSY - see https://reviewboard.asterisk.org/r/2064)
3. It verifies that if you have multiple agents and they are all paused, that the CDR disposition is NO ANSWER
In all three scenarios, we verify the creation of the appropriate CHAN_START, APP_START, APP_END, and CHAN_END events.
Note that as an improvement to the cdr/cel pluggable modules, we now allow a record to be 'skipped' in the CDR/CEL files under test by specifying a blank line in the expected CDR/CEL results. While these tests don't currently use that feature, it was useful in getting certain tests set up, and could be useful in the future if you only want to know the expected result of the last entry in the CDR/CEL record (or if you have several CDR/CEL records that you don't care about and that can arrive in any order).</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/AST-906">AST-906</a>
</div>
<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/configs/cel.conf <span style="color: grey">(3356)</span></li>
<li>/asterisk/trunk/lib/python/asterisk/cdr.py <span style="color: grey">(3356)</span></li>
<li>/asterisk/trunk/lib/python/asterisk/cel.py <span style="color: grey">(3356)</span></li>
<li>/asterisk/trunk/tests/queues/queue_all_paused_no_answer/configs/ast1/cdr.conf <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/queues/queue_all_paused_no_answer/configs/ast1/cel.conf <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/queues/queue_all_paused_no_answer/configs/ast1/extensions.conf <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/queues/queue_all_paused_no_answer/configs/ast1/queues.conf <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/queues/queue_all_paused_no_answer/test-config.yaml <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/queues/queue_no_answer/configs/ast1/cdr.conf <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/queues/queue_no_answer/configs/ast1/cel.conf <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/queues/queue_no_answer/configs/ast1/extensions.conf <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/queues/queue_no_answer/configs/ast1/queues.conf <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/queues/queue_no_answer/test-config.yaml <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/queues/queue_one_paused_no_answer/configs/ast1/cdr.conf <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/queues/queue_one_paused_no_answer/configs/ast1/cel.conf <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/queues/queue_one_paused_no_answer/configs/ast1/extensions.conf <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/queues/queue_one_paused_no_answer/configs/ast1/queues.conf <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/queues/queue_one_paused_no_answer/test-config.yaml <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/queues/tests.yaml <span style="color: grey">(3357)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2063/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>