[Asterisk-code-review] Revert "res pjsip empty: Add test for reply to empty SIP INF... (testsuite[master])

Joshua Colp asteriskteam at digium.com
Thu May 19 05:35:16 CDT 2016


Joshua Colp has uploaded a new change for review.

  https://gerrit.asterisk.org/2874

Change subject: Revert "res_pjsip_empty: Add test for reply to empty SIP INFO packets"
......................................................................

Revert "res_pjsip_empty: Add test for reply to empty SIP INFO packets"

The functionality to implement this has not yet gone in.

This reverts commit 7e603c0f85b90ecf7466d9c30e7958beccd8d1e2.

Change-Id: Ib6e87d76748430e265116ded0c46bd1f83604fb7
---
D tests/channels/pjsip/info_empty/configs/ast1/extensions.conf
D tests/channels/pjsip/info_empty/configs/ast1/pjsip.conf
D tests/channels/pjsip/info_empty/sipp/empty.xml
D tests/channels/pjsip/info_empty/test-config.yaml
M tests/channels/pjsip/tests.yaml
5 files changed, 0 insertions(+), 189 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/74/2874/1

diff --git a/tests/channels/pjsip/info_empty/configs/ast1/extensions.conf b/tests/channels/pjsip/info_empty/configs/ast1/extensions.conf
deleted file mode 100644
index 7476ff3..0000000
--- a/tests/channels/pjsip/info_empty/configs/ast1/extensions.conf
+++ /dev/null
@@ -1,5 +0,0 @@
-[default]
-
-exten => empty,1,NoOp()
-    same => n,Answer()
-    same => n,MusicOnHold()
diff --git a/tests/channels/pjsip/info_empty/configs/ast1/pjsip.conf b/tests/channels/pjsip/info_empty/configs/ast1/pjsip.conf
deleted file mode 100644
index 33e61b9..0000000
--- a/tests/channels/pjsip/info_empty/configs/ast1/pjsip.conf
+++ /dev/null
@@ -1,25 +0,0 @@
-[local]
-type=transport
-protocol=udp
-bind=0.0.0.0
-
-[endpoint_t](!)
-type=endpoint
-context=default
-transport=local
-direct_media=no
-disallow=all
-allow=ulaw
-dtmf_mode=info
-
-[aor_t](!)
-type=aor
-max_contacts=1
-
-;; test
-
-[test](endpoint_t)
-aors=test
-
-[test](aor_t)
-contact=sip:test at 127.0.0.1:5061
diff --git a/tests/channels/pjsip/info_empty/sipp/empty.xml b/tests/channels/pjsip/info_empty/sipp/empty.xml
deleted file mode 100644
index 5fb32ec..0000000
--- a/tests/channels/pjsip/info_empty/sipp/empty.xml
+++ /dev/null
@@ -1,133 +0,0 @@
-<?xml version="1.0" encoding="ISO-8859-1" ?>
-<!DOCTYPE scenario SYSTEM "sipp.dtd">
-
-<scenario name="INFO Test">
-  <send retrans="500">
-    <![CDATA[
-
-      INVITE sip:empty@[remote_ip]:[remote_port] SIP/2.0
-      Via: SIP/2.0/[transport] [local_ip]:[local_port];branch=[branch]
-      From: test <sip:test@[local_ip]:[local_port]>;tag=[call_number]
-      To: empty <sip:empty@[remote_ip]:[remote_port]>
-      Call-ID: [call_id]
-      CSeq: 1 INVITE
-      Contact: sip:test@[local_ip]:[local_port]
-      Max-Forwards: 70
-      Subject: INFO Test
-      Content-Length: [len]
-
-      v=0
-      o=phoneA 53655765 2353687637 IN IP[local_ip_type] [local_ip]
-      s=-
-      c=IN IP[media_ip_type] [media_ip]
-      t=0 0
-      m=audio 6000 RTP/AVP 0
-      a=rtpmap:0 PCMU/8000
-
-    ]]>
-  </send>
-
-  <recv response="100" optional="true">
-  </recv>
-
-  <recv response="200" rtd="true">
-  </recv>
-
-  <send>
-    <![CDATA[
-
-      ACK sip:empty@[remote_ip]:[remote_port] SIP/2.0
-      Via: SIP/2.0/[transport] [local_ip]:[local_port];branch=[branch]
-      From: test <sip:test@[local_ip]:[local_port]>;tag=[call_number]
-      To: empty <sip:empty@[remote_ip]:[remote_port]>[peer_tag_param]
-      Call-ID: [call_id]
-      CSeq: 1 ACK
-      Contact: sip:test@[local_ip]:[local_port]
-      Max-Forwards: 70
-      Subject: INFO Test
-      Content-Type: application/sdp
-      Content-Length: 0
-
-    ]]>
-  </send>
-
-  <send retrans="500">
-    <![CDATA[
-
-      INFO sip:test@[remote_ip]:[remote_port] SIP/2.0
-      Via: SIP/2.0/[transport] [local_ip]:[local_port];branch=[branch]
-      From: empty <sip:empty@[local_ip]:[local_port]>;tag=[call_number]
-      To: test <sip:test@[remote_ip]:[remote_port]>[peer_tag_param]
-      Call-ID: [call_id]
-      CSeq: 2 INFO
-      Contact: sip:test@[local_ip]:[local_port]
-      Max-Forwards: 70
-      Content-Length: [len]
-
-    ]]>
-  </send>
-
-  <recv response="200" crlf="true">
-  </recv>
-
-  <send retrans="500">
-    <![CDATA[
-
-      INFO sip:test@[remote_ip]:[remote_port] SIP/2.0
-      Via: SIP/2.0/[transport] [local_ip]:[local_port];branch=[branch]
-      From: empty <sip:empty@[local_ip]:[local_port]>;tag=[call_number]
-      To: test <sip:test@[remote_ip]:[remote_port]>[peer_tag_param]
-      Call-ID: [call_id]
-      CSeq: 3 INFO
-      Contact: sip:test@[local_ip]:[local_port]
-      Max-Forwards: 70
-      Content-Length: [len]
-
-    ]]>
-  </send>
-
-  <recv response="200" crlf="true">
-  </recv>
-
-  <send retrans="500">
-    <![CDATA[
-
-      INFO sip:test@[remote_ip]:[remote_port] SIP/2.0
-      Via: SIP/2.0/[transport] [local_ip]:[local_port];branch=[branch]
-      From: empty <sip:empty@[local_ip]:[local_port]>;tag=[call_number]
-      To: test <sip:test@[remote_ip]:[remote_port]>[peer_tag_param]
-      Call-ID: [call_id]
-      CSeq: 17 INFO
-      Contact: sip:test@[local_ip]:[local_port]
-      Max-Forwards: 70
-      Content-Length: [len]
-
-    ]]>
-  </send>
-
-  <recv response="200" crlf="true">
-  </recv>
-
-  <pause/>
-
-  <send retrans="500">
-    <![CDATA[
-
-      BYE sip:empty@[remote_ip]:[remote_port] SIP/2.0
-      Via: SIP/2.0/[transport] [local_ip]:[local_port];branch=[branch]
-      From: test <sip:test@[local_ip]:[local_port]>;tag=[call_number]
-      To: empty <sip:empty@[remote_ip]:[remote_port]>[peer_tag_param]
-      Call-ID: [call_id]
-      CSeq: 18 BYE
-      Contact: sip:test@[local_ip][local_port]
-      Max-Forwards: 70
-      Subject: INFO Test
-      Content-Length: 0
-
-    ]]>
-  </send>
-
-  <recv response="200" crlf="true">
-  </recv>
-
-</scenario>
diff --git a/tests/channels/pjsip/info_empty/test-config.yaml b/tests/channels/pjsip/info_empty/test-config.yaml
deleted file mode 100644
index 8e9f286..0000000
--- a/tests/channels/pjsip/info_empty/test-config.yaml
+++ /dev/null
@@ -1,25 +0,0 @@
-testinfo:
-    summary: 'Test that asterisk correctly handles EMTPY INFO in SIP packets when using PJSIP.'
-    description:
-            This test checks that after establishing a call via an INVITE, when
-            an INFO packet is recieved with no 'Content-Type' header, the
-            emtpy INFO module should respond with a 200 - OK packet.
-
-test-modules:
-    test-object:
-        config-section: test-object-config
-        typename: 'sipp.SIPpTestCase'
-
-test-object-config:
-    test-iterations:
-        -
-            scenarios:
-                - { 'key-args': {'scenario': 'empty.xml', '-i': '127.0.0.1', '-p': '5061', '-d': '3000'} }
-
-properties:
-    minversion: '13.10.0'
-    dependencies:
-        - app : 'sipp'
-        - asterisk : 'res_pjsip'
-    tags:
-        - pjsip
diff --git a/tests/channels/pjsip/tests.yaml b/tests/channels/pjsip/tests.yaml
index 88a9a06..b44fafb 100644
--- a/tests/channels/pjsip/tests.yaml
+++ b/tests/channels/pjsip/tests.yaml
@@ -37,7 +37,6 @@
     - test: 'incoming_call_on_second_transport'
     - test: 'incoming_calls_without_auth'
     - test: 'info_dtmf'
-    - test: 'info_empty'
     - test: 'keep_alive'
     - test: 'reinvite_early'
     - test: 'reinvite_pending'

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib6e87d76748430e265116ded0c46bd1f83604fb7
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Joshua Colp <jcolp at digium.com>



More information about the asterisk-code-review mailing list