[Asterisk-code-review] bob does not exist: sometimes fails due to sipp scenario bei... (testsuite[master])

Kevin Harwell asteriskteam at digium.com
Thu Feb 18 13:40:19 CST 2016


Hello Anonymous Coward #1000019, Joshua Colp,

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

    https://gerrit.asterisk.org/2277

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

Change subject: bob_does_not_exist: sometimes fails due to sipp scenario being closed early
......................................................................

bob_does_not_exist: sometimes fails due to sipp scenario being closed early

The 'tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_exist'
would sporadically fail when the tcp sipp scenarios were killed too early. This
occurred because the test was being stopped upon receiving the first of the
expected hangup events. When this happened pjsip reported the socket as closed,
resulting in a "CONGESTED" hangup cause.

Added a 'trigger-on-count' parameter to the offending event action chain so that
the stop action won't trigger until all events have been received.

Note, while the 'tests/apps/queues/transfer/caller_attended/' test has not
failed (at least I don't think it has) it too could potentially suffer from
the same problem, so I added the parameter there as well.

SWP-8433

Change-Id: I0b8f3922bfca5446db298d50363a89a13c06e89d
---
M tests/apps/queues/transfer/caller_attended/test-config.yaml
M tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_exist/test-config.yaml
2 files changed, 2 insertions(+), 0 deletions(-)


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

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



More information about the asterisk-code-review mailing list