[Asterisk-code-review] testsuite: Fix tests/fax/pjsip/directmedia reinvite t38 (testsuite[master])

Richard Mudgett asteriskteam at digium.com
Wed Feb 17 17:54:23 CST 2016


Richard Mudgett has posted comments on this change.

Change subject: testsuite: Fix tests/fax/pjsip/directmedia_reinvite_t38
......................................................................


Patch Set 1:

> I don't think this will fix the test failure, but merely make it
 > slightly more rare.
 > 
 > How many times did you run the test with this change?

Dang it.  After running it several times before posting this patch it never failed.  Because of your query I set it up to loop 100 times and it failed the first time the loop ran it.

I've come up with another way to ensure that the reinvite completes before the call is disconnected.  I'll make the dialplan run Echo instead of Wait and make the endpoint_A scenario initiate the BYE.

-- 
To view, visit https://gerrit.asterisk.org/2271
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib0b35ca580aaca95ce6732dbf060fd46c634e352
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-HasComments: No



More information about the asterisk-code-review mailing list