[asterisk-dev] [Code Review] 3182: testsuite: LinkedID Propagation test

[Scott Griepentrog]

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

(Updated Feb. 10, 2014, 2:37 p.m.)


Review request for Asterisk Developers.


Changes
-------

Corrected reviewer issues, and updated to include fix for sporadic failures by querying the linkedid after channels are bridged.  Also added slight delays to channel creation and after each bridge action to allow for channel events to arrive and be processed.


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


Repository: testsuite


Description
-------

This monitors bridge enter/exit and CEL events, and builds
a picture of which channels are bridged together, and when
their LinkedID changes, and then verifies that each change
is correct.

Note: The BridgeEnter and CEL 'BRIDGE_ENTER' events can be
in either order - so wait for both to arrive before making
a judgement on the LinkedID.

This test can be added to any other bridging test to add a
layer of LinkedId Propagation checking to it.

This test has now been added to the existing bridge_action
test rather than have a separate test for it.


Diffs (updated)
-----

  /asterisk/trunk/tests/bridge/bridge_action/test-config.yaml 4675 
  /asterisk/trunk/tests/bridge/bridge_action/bridge_action.py 4675 
  /asterisk/trunk/lib/python/asterisk/linkedid_check.py PRE-CREATION 

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


Testing
-------

This test fails spuriously due to what appears to be CEL events being sent on BRIDGE_ENTER prior to the LinkedId being updated.  This bug will be fixed along with the other changes for 23120.


Thanks,

Scott Griepentrog

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140210/60dc98db/attachment.html>