[Asterisk-code-review] Test receiving an early BYE during an attended transfer (testsuite[master])

[Kevin Harwell]

Hello Richard Mudgett, Jenkins2, Joshua Colp, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/5823

to look at the new patch set (#2).

Change subject: Test receiving an early BYE during an attended transfer
......................................................................

Test receiving an early BYE during an attended transfer

Asterisk used to fail the transfer and drop the call if an early BYE (sometime
just after the 202 accepted, but before the Notify 200 OK frag) was received.
Added a test that makes sure the transfer does not fail when an early BYE is
received.

This patch modified the testsuite's pjsua phones wrapper to allow configuring
of when to send the transferer's BYE during the transfer process. A new option,
'hangup-reason', has been added that specifies when to send the BYE. Allowed
values are:

  accepted - Sent right after the refer is accepted
  trying - Sent right after receiving a sip frag 100 Trying
  ok - Sent right after receiving a sip frag 200 OK
  final - Sent upon receiving the final message

A test was also added that sends a BYE right after the REFER is accepted. As
noted in test-config.yaml this test is a direct copy of the 'same_stasis_app'
test except for when the BYE is sent.

ASTERISK-27053

Change-Id: Ifbf1d6e8f91dfc00128a2ccd7eaff6402d170650
---
M lib/python/asterisk/phones.py
M tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/same_stasis_app/test-config.yaml
A tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/same_stasis_app_accepted/configs/ast1/extensions.conf
A tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/same_stasis_app_accepted/configs/ast1/pjsip.conf
A tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/same_stasis_app_accepted/test-config.yaml
M tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/tests.yaml
6 files changed, 618 insertions(+), 17 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/23/5823/2
-- 
To view, visit https://gerrit.asterisk.org/5823
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ifbf1d6e8f91dfc00128a2ccd7eaff6402d170650
Gerrit-Change-Number: 5823
Gerrit-PatchSet: 2
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20170616/f4464776/attachment-0001.html>