[Asterisk-code-review] tests/rest api/events/variables: Stop test on StasisEnd. (testsuite[master])

Anonymous Coward asteriskteam at digium.com
Mon Nov 28 09:03:07 CST 2016


Anonymous Coward #1000019 has submitted this change and it was merged. ( https://gerrit.asterisk.org/4504 )

Change subject: tests/rest_api/events/variables: Stop test on StasisEnd.
......................................................................


tests/rest_api/events/variables: Stop test on StasisEnd.

There was a race condition in this test where a StasisEnd
would not be received as Asterisk, and the test, were being
terminated during that time as a result of the channel hanging
up.

This change makes the test (and Asterisk) stop as a result of
the test receiving StasisEnd instead.

ASTERISK-26492

Change-Id: Ie5faa3a2aea1280b6c89d570a592f9dfdedc40b5
---
M tests/rest_api/events/variables/test-config.yaml
1 file changed, 12 insertions(+), 0 deletions(-)

Approvals:
  George Joseph: Looks good to me, approved
  Anonymous Coward #1000019: Verified



diff --git a/tests/rest_api/events/variables/test-config.yaml b/tests/rest_api/events/variables/test-config.yaml
index 7b08777..0ba628a 100644
--- a/tests/rest_api/events/variables/test-config.yaml
+++ b/tests/rest_api/events/variables/test-config.yaml
@@ -17,8 +17,12 @@
         -
             config-section: ari-config
             typename: ari.WebSocketEventModule
+        -
+            config-section: ari-test-stopper
+            typename: pluggable_modules.EventActionModule
 
 test-object-config:
+    stop-on-end: False
     test-iterations:
         -
             endpoint: Local/s at default
@@ -30,6 +34,14 @@
                     CAN_YOU_SEE_ME: 'YES'
                     CHANNEL(language): gobbledegook
 
+ari-test-stopper:
+    -
+        ari-events:
+            match:
+                type: StasisEnd
+                application: testsuite
+        stop_test:
+
 ari-config:
     events:
         -   conditions:

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie5faa3a2aea1280b6c89d570a592f9dfdedc40b5
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>



More information about the asterisk-code-review mailing list