[Asterisk-code-review] Stasis attended transfers: No StasisEnd event for transfer t... (testsuite[master])
Kevin Harwell
asteriskteam at digium.com
Tue Jun 23 09:53:47 CDT 2015
Kevin Harwell has uploaded a new patch set (#2).
Change subject: Stasis attended transfers: No StasisEnd event for transfer target channel
......................................................................
Stasis attended transfers: No StasisEnd event for transfer target channel
This required a change in Asterisk since the transfer target was not being
hung up, thus no event. However, these tests were altered to now not expect
a test failure. This patch also fixes a few typos/copy paste errors found
in the test that were keeping them from passing. As well, in a couple of
tests the local attended transfer channels needed to be hung up. In the
'non_stasis_app_to_stasis_bridge' the hangup of 'bob' was moved to be done
once the local channel entered the bridge. This made the test a little
more deterministic.
With the changes it was verified that the tests pass and that Asterisk gets
shutdown gracefully (i.e. is not killed).
ASTERISK-24782 #close
Reported by: John Bigelow
ASTERISK-25167
Reported by: Corey Farrell
Change-Id: Ife584d91394ff69a3662004471421761789c4901
---
M tests/rest_api/external_interaction/attended_transfer/non_stasis_app_to_stasis_bridge/test-config.yaml
M tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/different_stasis_app/test-config.yaml
M tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/same_stasis_app/test-config.yaml
3 files changed, 27 insertions(+), 31 deletions(-)
git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/93/693/2
--
To view, visit https://gerrit.asterisk.org/693
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ife584d91394ff69a3662004471421761789c4901
Gerrit-PatchSet: 2
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
More information about the asterisk-code-review
mailing list