[Asterisk-code-review] chanspy barge: rewrite to remove pjsua dependency (testsuite[13])

Kevin Harwell asteriskteam at digium.com
Mon Oct 29 13:01:55 CDT 2018


Hello Jenkins2, Corey Farrell, 

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

    https://gerrit.asterisk.org/10551

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

Change subject: chanspy_barge: rewrite to remove pjsua dependency
......................................................................

chanspy_barge: rewrite to remove pjsua dependency

When Asterisk was upgraded to use pjproject 2.8 a patch that was made to pjsua
caused pjsua to default to a TCP transport. This caused any testsuite test that
spawned pjsua as an endpoint to start to fail as they were using UDP (e.g.
chanspy_barge). An easy solution here is to set those particular tests to pass
the '--no-tcp' option to pjsua.

However, at some point we want to remove the dependency to pjsua altogther from
the testsuite. That being the case the chanspy_barge test has been rewritten to
not longer use pjsua. Instead this test just now uses local channels to test the
"barge" functionality.

ASTERISK~28096

Change-Id: Ia4ab4e37b867cc6b4cd7351e22adc3862d4b34eb
---
M tests/apps/chanspy/chanspy_barge/configs/ast1/extensions.conf
D tests/apps/chanspy/chanspy_barge/configs/ast1/rtp.conf
D tests/apps/chanspy/chanspy_barge/configs/ast1/sip.conf
D tests/apps/chanspy/chanspy_barge/run-test
D tests/apps/chanspy/chanspy_barge/sounds/talking.wav
M tests/apps/chanspy/chanspy_barge/test-config.yaml
6 files changed, 71 insertions(+), 214 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/51/10551/3
-- 
To view, visit https://gerrit.asterisk.org/10551
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: testsuite
Gerrit-Branch: 13
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia4ab4e37b867cc6b4cd7351e22adc3862d4b34eb
Gerrit-Change-Number: 10551
Gerrit-PatchSet: 3
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2 (1000185)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20181029/04d04e64/attachment.html>


More information about the asterisk-code-review mailing list