[Asterisk-code-review] stasis bridge: Ensure the test gives time for all events to ... (testsuite[master])

[Joshua Colp]

Hello Anonymous Coward #1000019,

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

    https://gerrit.asterisk.org/2215

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

Change subject: stasis_bridge: Ensure the test gives time for all events to be received.
......................................................................

stasis_bridge: Ensure the test gives time for all events to be received.

This test terminates based on the hangup of the underlying channels
used during the test. This presents a race condition where the test
will terminate before the Stasis messages have been conveyed over the
ARI WebSocket.

This change makes it so an additional channel is originated and waits
for 10 seconds before continuing. This gives ample time for all events
and messages to be conveyed but does not alter the behavior of the
rest of the test.

Change-Id: Icf776149b9d0052b0db5be271608f508d2ee97ff
---
M tests/rest_api/external_interaction/bridge_app/x_option/stasis_bridge/configs/ast1/extensions.conf
M tests/rest_api/external_interaction/bridge_app/x_option/stasis_bridge/test-config.yaml
2 files changed, 9 insertions(+), 0 deletions(-)


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

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Icf776149b9d0052b0db5be271608f508d2ee97ff
Gerrit-PatchSet: 2
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>