[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