[asterisk-dev] [Code Review] 2821: testsuite: Update tests/bridge/automon and tests/bridge/automixmon to accomodate Asterisk 12 CEL changes

svnbot reviewboard at asterisk.org
Thu Sep 12 13:13:07 CDT 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2821/
-----------------------------------------------------------

(Updated Sept. 12, 2013, 1:13 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers, kmoore and Matt Jordan.


Changes
-------

Committed in revision 4170


Bugs: ASTERISK-22325
    https://issues.asterisk.org/jira/browse/ASTERISK-22325


Repository: testsuite


Description
-------

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.


Diffs
-----

  /asterisk/trunk/tests/bridge/automixmon/configs/ast1/extensions.conf 4102 
  /asterisk/trunk/tests/bridge/automixmon/test-config.yaml 4102 
  /asterisk/trunk/tests/bridge/automon/configs/ast1/extensions.conf 4102 
  /asterisk/trunk/tests/bridge/automixmon_bridgefeatures/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/bridge/automixmon_bridgefeatures/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/bridge/tests.yaml 4102 
  /asterisk/trunk/tests/bridge/automon_bridgefeatures/test-config.yaml PRE-CREATION 
  /asterisk/trunk/tests/bridge/automon_bridgefeatures/configs/ast1/extensions.conf PRE-CREATION 
  /asterisk/trunk/tests/bridge/automon/test-config.yaml 4102 

Diff: https://reviewboard.asterisk.org/r/2821/diff/


Testing
-------

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't fail under any of the potential orderings of CEL logs that I have encountered so far.


Thanks,

jrose

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130912/72cc5db3/attachment.htm>


More information about the asterisk-dev mailing list