Mark Michelson has uploaded a new patch set (#2). Change subject: sip_attended_transfer now supports pre-12 Asterisk versions. ......................................................................
sip_attended_transfer now supports pre-12 Asterisk versions. The sip_attended transfer test was recently rewritten to prevent it from bouncing during automatic test runs. The rewrite attempted to break into two tests in an attempt to separate the logic of different versions from one another. Ashley pointed out on my original Asterisk 11 version of the patch that there are only small difference between the Asterisk 11 and 12 versions of the test, resulting in a lot of repeated boilerplate code that could otherwise be avoided. This change alters the 12+ specific test by separating the bridge logic for different versions into their own classes. I have verified that the test passes using both Asterisk 11 and Asterisk 13. Change-Id: I958c52cebb94f9cfc8dc8ed81311ae62efb2679d Fix a typo where a function call was evaluated as a boolean. Change-Id: I958c52cebb94f9cfc8dc8ed81311ae62efb2679d --- M tests/channels/SIP/sip_attended_transfer/attended_transfer.py M tests/channels/SIP/sip_attended_transfer/test-config.yaml 2 files changed, 125 insertions(+), 45 deletions(-) git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/29/29/2 -- To view, visit https://gerrit.asterisk.org/29 To unsubscribe, visit https://gerrit.asterisk.org/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I958c52cebb94f9cfc8dc8ed81311ae62efb2679d Gerrit-PatchSet: 2 Gerrit-Project: testsuite Gerrit-Branch: master Gerrit-Owner: Mark Michelson <mmichel...@digium.com> Gerrit-Reviewer: Mark Michelson <mmichel...@digium.com> -- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev