[Asterisk-code-review] pjsip/ami/pjsip_notify/cli: Added a test for SIP Notify via the CLI (...testsuite[13])

George Joseph asteriskteam at digium.com
Fri Jul 19 12:22:52 CDT 2019


George Joseph has posted comments on this change. ( https://gerrit.asterisk.org/c/testsuite/+/11598 )

Change subject: pjsip/ami/pjsip_notify/cli: Added a test for SIP Notify via the CLI
......................................................................


Patch Set 1: Code-Review-1

(1 comment)

https://gerrit.asterisk.org/#/c/11598/1/tests/channels/pjsip/ami/pjsip_notify/cli/sipp/options.xml 
File tests/channels/pjsip/ami/pjsip_notify/cli/sipp/options.xml:

https://gerrit.asterisk.org/#/c/11598/1/tests/channels/pjsip/ami/pjsip_notify/cli/sipp/options.xml@1 
PS1, Line 1: <?xml version="1.0" encoding="ISO-8859-1" ?>
shouldn't this be named "notify.xml" instead of "options.xml"?



-- 
To view, visit https://gerrit.asterisk.org/c/testsuite/+/11598
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: testsuite
Gerrit-Branch: 13
Gerrit-Change-Id: I94312ba52b24020a29f609bbff75198b74475a6f
Gerrit-Change-Number: 11598
Gerrit-PatchSet: 1
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Comment-Date: Fri, 19 Jul 2019 17:22:52 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190719/0bebb49a/attachment.html>


More information about the asterisk-code-review mailing list