[Asterisk-code-review] sendtext: Update enhanced test to use an application conten... (testsuite[13])
Jenkins2
asteriskteam at digium.com
Wed Jun 13 14:41:20 CDT 2018
Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/9082 )
Change subject: sendtext: Update enhanced test to use an application content type
......................................................................
sendtext: Update enhanced test to use an application content type
app_sendtext now allows content types other than text/plain so the
test for enagnced messaging has been updated to use
application/x-asterisk+json.
Change-Id: I749c7c7b72b41d46220c9dcd0e3368bd57805ed3
---
M tests/apps/sendtext/enhanced/configs/ast1/extensions.conf
M tests/apps/sendtext/enhanced/sipp/message_recv.xml
2 files changed, 2 insertions(+), 2 deletions(-)
Approvals:
Joshua Colp: Looks good to me, but someone else must approve
George Joseph: Looks good to me, approved
Jenkins2: Approved for Submit
diff --git a/tests/apps/sendtext/enhanced/configs/ast1/extensions.conf b/tests/apps/sendtext/enhanced/configs/ast1/extensions.conf
index e600589..22ba40e 100644
--- a/tests/apps/sendtext/enhanced/configs/ast1/extensions.conf
+++ b/tests/apps/sendtext/enhanced/configs/ast1/extensions.conf
@@ -4,7 +4,7 @@
exten = sendtext,1,Answer()
same = n,Set(SENDTEXT_FROM_DISPLAYNAME=From Someone Else)
same = n,Set(SENDTEXT_TO_DISPLAYNAME=To Someone Else)
-same = n,Set(SENDTEXT_CONTENT_TYPE=text/json)
+same = n,Set(SENDTEXT_CONTENT_TYPE=application/x-asterisk+json)
same = n,Set(SENDTEXT_BODY={"foo":666, "bar":"666"})
; The body supplied in the parameter should be ignored.
same = n,SendText({"abc":1, "def":"ghi"})
diff --git a/tests/apps/sendtext/enhanced/sipp/message_recv.xml b/tests/apps/sendtext/enhanced/sipp/message_recv.xml
index a1e10af..cccc7c0 100644
--- a/tests/apps/sendtext/enhanced/sipp/message_recv.xml
+++ b/tests/apps/sendtext/enhanced/sipp/message_recv.xml
@@ -49,7 +49,7 @@
<action>
<ereg regexp="From Someone Else" search_in="hdr" header="From:" check_it="true" assign_to="dummy"/>
<ereg regexp="To Someone Else" search_in="hdr" header="To:" check_it="true" assign_to="dummy"/>
- <ereg regexp="text/json" search_in="hdr" header="Content-Type:" check_it="true" assign_to="dummy"/>
+ <ereg regexp="application/x-asterisk\+json" search_in="hdr" header="Content-Type:" check_it="true" assign_to="dummy"/>
<ereg regexp="\{\"foo\":666, \"bar\":\"666\"\}" search_in="body" check_it="true" assign_to="dummy"/>
</action>
</recv>
--
To view, visit https://gerrit.asterisk.org/9082
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-Project: testsuite
Gerrit-Branch: 13
Gerrit-MessageType: merged
Gerrit-Change-Id: I749c7c7b72b41d46220c9dcd0e3368bd57805ed3
Gerrit-Change-Number: 9082
Gerrit-PatchSet: 1
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180613/3e21ba0d/attachment.html>
More information about the asterisk-code-review
mailing list