[asterisk-commits] testsuite: Attempt to fix max initial qualify time test fai... (testsuite[master])

[SVN commits to the Asterisk project]

Mark Michelson has submitted this change and it was merged.

Change subject: testsuite:  Attempt to fix max_initial_qualify_time test failure
......................................................................


testsuite:  Attempt to fix max_initial_qualify_time test failure

Converted the test to use EventActionModule and increased
the qualify_timeout.

Change-Id: I0e62d3b3f59f53a570abffb2401380fe56ebcb24
---
M tests/channels/pjsip/qualify/max_initial_qualify_time/configs/ast1/pjsip.conf
M tests/channels/pjsip/qualify/max_initial_qualify_time/test-config.yaml
2 files changed, 28 insertions(+), 26 deletions(-)

Approvals:
  Mark Michelson: Looks good to me, approved; Verified
  Matt Jordan: Looks good to me, but someone else must approve



diff --git a/tests/channels/pjsip/qualify/max_initial_qualify_time/configs/ast1/pjsip.conf b/tests/channels/pjsip/qualify/max_initial_qualify_time/configs/ast1/pjsip.conf
index 14ad57e..979a212 100644
--- a/tests/channels/pjsip/qualify/max_initial_qualify_time/configs/ast1/pjsip.conf
+++ b/tests/channels/pjsip/qualify/max_initial_qualify_time/configs/ast1/pjsip.conf
@@ -1,7 +1,7 @@
 
 [global]
 type=global
-max_initial_qualify_time = 1
+max_initial_qualify_time = 5
 
 [mytransport]
 type=transport
@@ -16,4 +16,4 @@
 type=aor
 contact=sip:127.0.0.1:5061
 qualify_frequency=6000
-qualify_timeout = 2
+qualify_timeout = 5
diff --git a/tests/channels/pjsip/qualify/max_initial_qualify_time/test-config.yaml b/tests/channels/pjsip/qualify/max_initial_qualify_time/test-config.yaml
index de78457..9969ee5 100644
--- a/tests/channels/pjsip/qualify/max_initial_qualify_time/test-config.yaml
+++ b/tests/channels/pjsip/qualify/max_initial_qualify_time/test-config.yaml
@@ -12,11 +12,12 @@
     modules:
         -
             config-section: 'ami-config'
-            typename: 'ami.AMIEventModule'
+            typename: 'pluggable_modules.EventActionModule'
 
 test-object-config:
+    connect-ami: True
     fail-on-any: False
-    reactor-timeout: 10
+    reactor-timeout: 15
     test-iterations:
         -
             scenarios:
@@ -24,29 +25,30 @@
 
 ami-config:
     -
-        type: 'headermatch'
-        id: '0'
-        conditions:
-            match:
-                Event: 'TestEvent'
-                State: 'AOR_CONTACT_UPDATE'
-        requirements:
-            match:
-                Contact: 'sipp@@sip:127.0.0.1:5061'
-                Status: 'Reachable'
-        count: '1'
+        ami-start:
     -
-        type: 'headermatch'
-        id: '0'
-        conditions:
-            match:
-                Event: 'TestEvent'
-                State: 'AOR_CONTACT_QUALIFY_RESULT'
-        requirements:
-            match:
-                Contact: 'sipp@@sip:127.0.0.1:5061'
-                Status: 'Reachable'
-        count: '1'
+        ami-events:
+            conditions:
+                match:
+                    Event: 'TestEvent'
+                    State: 'AOR_CONTACT_UPDATE'
+            requirements:
+                match:
+                    Contact: 'sipp@@sip:127.0.0.1:5061'
+                    Status: 'Reachable'
+            count: '1'
+    -
+        ami-events:
+            conditions:
+                match:
+                    Event: 'TestEvent'
+                    State: 'AOR_CONTACT_QUALIFY_RESULT'
+            requirements:
+                match:
+                    Contact: 'sipp@@sip:127.0.0.1:5061'
+                    Status: 'Reachable'
+            count: '1'
+        stop_test:
 
 properties:
     minversion: '13.4.0'

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I0e62d3b3f59f53a570abffb2401380fe56ebcb24
Gerrit-PatchSet: 3
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: George Joseph <george.joseph at fairview5.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>