[Asterisk-code-review] Add local attended transfer tests with direct media, hold, a... (testsuite[master])

John Bigelow asteriskteam at digium.com
Fri May 22 10:45:21 CDT 2015


John Bigelow has posted comments on this change.

Change subject: Add local attended transfer tests with direct media, hold, and REFER/Replaces.
......................................................................


Patch Set 2:

(2 comments)

https://gerrit.asterisk.org/#/c/483/2/tests/channels/pjsip/transfers/attended_transfer/nominal/packet_sniffer.py
File tests/channels/pjsip/transfers/attended_transfer/nominal/packet_sniffer.py:

Line 283:             LOGGER.debug("Endpoints in remote RTP bridge: %s" %
> Use "".format instead of string interpolation.
Done


Line 293:                     k = "Endpoint%d" % (num + 1)
> Use "".format instead of string interpolation.
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibd1f9a3ea8bf6b80fe95efe3fcd880a55d5a7cae
Gerrit-PatchSet: 2
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: John Bigelow <jbigelow at digium.com>
Gerrit-Reviewer: Ashley Sanders <asanders at digium.com>
Gerrit-Reviewer: John Bigelow <jbigelow at digium.com>
Gerrit-Reviewer: Samuel Galarneau <sgalarneau at digium.com>
Gerrit-HasComments: Yes



More information about the asterisk-code-review mailing list