[Asterisk-code-review] rest api: Fix ami bridge tests. (testsuite[master])

Mark Michelson asteriskteam at digium.com
Thu May 4 16:52:51 CDT 2017


Hello Jenkins2,

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

    https://gerrit.asterisk.org/5582

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

Change subject: rest_api: Fix ami_bridge tests.
......................................................................

rest_api: Fix ami_bridge tests.

The ARI ami_bridge tests relied on broken behavior of
the AMI Bridge action itself. The behavior would result in
dialplan priorities being skipped over when they shouldn't
be, causing the 'alice' channel in each test to get hung
up quickly. Since the behavior has now been fixed the
channel properly executes dialplan and results in the
Echo() application being executed until the test fails.

The tests have now been changed to restore the expected
behavior by hanging up the 'alice' channel if it is sent
into the dialplan.

ASTERISK-24529

Change-Id: Ieb289a4623592ab6986411bf83325876baa6a71c
---
M tests/rest_api/external_interaction/ami_bridge/stasis_app/two_channel_different_stasis_app/configs/ast1/extensions.conf
M tests/rest_api/external_interaction/ami_bridge/stasis_app/two_channel_same_stasis_app/configs/ast1/extensions.conf
M tests/rest_api/external_interaction/ami_bridge/stasis_bridge/different_stasis_app/configs/ast1/extensions.conf
M tests/rest_api/external_interaction/ami_bridge/stasis_bridge/non_stasis_bridge/configs/ast1/extensions.conf
M tests/rest_api/external_interaction/ami_bridge/stasis_bridge/same_stasis_app/configs/ast1/extensions.conf
M tests/rest_api/external_interaction/ami_bridge/stasis_bridge/two_channel_different_stasis_app/configs/ast1/extensions.conf
M tests/rest_api/external_interaction/ami_bridge/stasis_bridge/two_channel_same_stasis_app/configs/ast1/extensions.conf
7 files changed, 21 insertions(+), 7 deletions(-)


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

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ieb289a4623592ab6986411bf83325876baa6a71c
Gerrit-PatchSet: 2
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Jenkins2



More information about the asterisk-code-review mailing list