[Asterisk-code-review] pjsip 'bob does not exists' times out to early (testsuite[master])

Kevin Harwell asteriskteam at digium.com
Fri Oct 2 18:15:46 CDT 2015


Kevin Harwell has uploaded a new change for review.

  https://gerrit.asterisk.org/1369

Change subject: pjsip 'bob does not exists' times out to early
......................................................................

pjsip 'bob does not exists' times out to early

Increased the test's timeout value to be more than the sipp scenario's
timeout value. This way the test can properly shutdown instead of being
killed by a reactor timeout.

Change-Id: I4d700516cdc1b0cd082dc5fb81fae1171f946430
---
M tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_exist/test-config.yaml
1 file changed, 48 insertions(+), 52 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/69/1369/1

diff --git a/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_exist/test-config.yaml b/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_exist/test-config.yaml
index 6367ab6..c472367 100644
--- a/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_exist/test-config.yaml
+++ b/tests/channels/pjsip/basic_calls/outgoing/off-nominal/bob_does_not_exist/test-config.yaml
@@ -25,10 +25,10 @@
             typename: 'pluggable_modules.Originator'
         -
             config-section: 'ami-config'
-            typename: 'ami.AMIEventModule'
+            typename: 'pluggable_modules.EventActionModule'
 
 test-object-config:
-   reactor-timeout: 60
+   reactor-timeout: 70
    fail-on-any: False
    test-iterations:
        -
@@ -85,60 +85,56 @@
 ami-config:
     # UUT events
     -
-        type: 'headermatch'
-        id: '0'
-        conditions:
-            match:
-                Event: 'UserEvent'
-                UserEvent: 'DIALSTATUS'
-        requirements:
-            match:
-                Result: 'CANCEL'
-        count: '4'
+        ami-events:
+            conditions:
+                match:
+                    Event: 'UserEvent'
+                    UserEvent: 'DIALSTATUS'
+            requirements:
+                match:
+                    Result: 'CANCEL'
+            count: '4'
     -
-        type: 'headermatch'
-        id: '0'
-        conditions:
-            match:
-                Event: 'SoftHangupRequest'
-                Channel: 'Local/bob-*'
-        requirements:
-            match:
-                Cause: '16'
-        count: '4'
+        ami-events:
+            conditions:
+                match:
+                    Event: 'SoftHangupRequest'
+                    Channel: 'Local/bob-*'
+            requirements:
+                match:
+                    Cause: '16'
+            count: '4'
     -
-        type: 'headermatch'
-        id: '0'
-        conditions:
-            match:
-                Event: 'HangupRequest'
-                Channel: 'Local/bob-*'
-        requirements:
-            match:
-                Cause: '0'
-        count: '4'
+        ami-events:
+            conditions:
+                match:
+                    Event: 'HangupRequest'
+                    Channel: 'Local/bob-*'
+            requirements:
+                match:
+                    Cause: '0'
+            count: '4'
     -
-        type: 'headermatch'
-        id: '0'
-        conditions:
-            match:
-                Event: 'Hangup'
-                Channel: 'PJSIP/bob-*'
-        requirements:
-            match:
-                Cause: '0'
-        count: '4'
+        ami-events:
+            conditions:
+                match:
+                    Event: 'Hangup'
+                    Channel: 'PJSIP/bob-*'
+            requirements:
+                match:
+                    Cause: '0'
+            count: '4'
     -
-        type: 'headermatch'
-        id: '0'
-        conditions:
-            match:
-                Event: 'Hangup'
-                Channel: 'Local/bob-*'
-        requirements:
-            match:
-                Cause: '0'
-        count: '8'
+        ami-events:
+            conditions:
+                match:
+                    Event: 'Hangup'
+                    Channel: 'Local/bob-*'
+            requirements:
+                match:
+                    Cause: '0'
+            count: '8'
+        stop_test:
 
 properties:
     minversion: '12.0.0'

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4d700516cdc1b0cd082dc5fb81fae1171f946430
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>



More information about the asterisk-code-review mailing list