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

[Anonymous Coward]

Anonymous Coward #1000019 has submitted this change and it was merged.

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


pjsip 'bob does not exists' times out too 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(-)

Approvals:
  Anonymous Coward #1000019: Verified
  Matt Jordan: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved



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: merged
Gerrit-Change-Id: I4d700516cdc1b0cd082dc5fb81fae1171f946430
Gerrit-PatchSet: 2
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>