[Asterisk-code-review] Skip rest api tests that are failing. (testsuite[master])

Corey Farrell asteriskteam at digium.com
Tue Jun 16 18:15:08 CDT 2015


Corey Farrell has uploaded a new change for review.

  https://gerrit.asterisk.org/667

Change subject: Skip rest_api tests that are failing.
......................................................................

Skip rest_api tests that are failing.

These 3 tests were marked as expected failures.  Set them to skip
instead since they also produce reference leaks and timeout on
shutdown.

ASTERISK-25167

Change-Id: Ie1ca7da1fd01031846994f3b18ac551f2b51320d
---
M tests/rest_api/external_interaction/attended_transfer/non_stasis_app_to_stasis_bridge/test-config.yaml
M tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/different_stasis_app/test-config.yaml
M tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/same_stasis_app/test-config.yaml
3 files changed, 3 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/67/667/1

diff --git a/tests/rest_api/external_interaction/attended_transfer/non_stasis_app_to_stasis_bridge/test-config.yaml b/tests/rest_api/external_interaction/attended_transfer/non_stasis_app_to_stasis_bridge/test-config.yaml
index 502fd7e..c1890da 100644
--- a/tests/rest_api/external_interaction/attended_transfer/non_stasis_app_to_stasis_bridge/test-config.yaml
+++ b/tests/rest_api/external_interaction/attended_transfer/non_stasis_app_to_stasis_bridge/test-config.yaml
@@ -22,6 +22,7 @@
          This test verifies that a non-Stasis app can be transferred (attended)
          to a Stasis bridge. It also verifies that the StasisStart & StasisEnd
          events occur for the alice, bob, and local replacement channels."
+    skip: 'Need to troubleshoot failure'
     issues:
         - jira: 'ASTERISK-24782'
 
@@ -304,6 +305,4 @@
     tags:
         - ARI
         - pjsip
-    # Expect test to fail until ASTERISK-24782 is resolved.
-    expected-result: False
 
diff --git a/tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/different_stasis_app/test-config.yaml b/tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/different_stasis_app/test-config.yaml
index 36e7987..cf520ad 100644
--- a/tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/different_stasis_app/test-config.yaml
+++ b/tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/different_stasis_app/test-config.yaml
@@ -31,6 +31,7 @@
          to a Stasis bridge via different Stasis apps. It also verifies that
          the StasisStart & StasisEnd events occur for the alice(both), bob,
          carol, and local replacement channels."
+    skip: 'Need to troubleshoot failure'
     issues:
         - jira: 'ASTERISK-24782'
 
@@ -498,6 +499,4 @@
     tags:
         - ARI
         - pjsip
-    # Expect test to fail until ASTERISK-24782 is resolved.
-    expected-result: False
 
diff --git a/tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/same_stasis_app/test-config.yaml b/tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/same_stasis_app/test-config.yaml
index fd68ee7..7c81648 100644
--- a/tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/same_stasis_app/test-config.yaml
+++ b/tests/rest_api/external_interaction/attended_transfer/stasis_bridge_to_stasis_bridge/same_stasis_app/test-config.yaml
@@ -30,6 +30,7 @@
          to a Stasis bridge in the same Stasis app. It also verifies that the
          StasisStart & StasisEnd events occur for the alice(both), bob, carol,
          and local replacement channels."
+    skip: 'Need to troubleshoot failure'
     issues:
         - jira: 'ASTERISK-24782'
 
@@ -503,6 +504,4 @@
     tags:
         - ARI
         - pjsip
-    # Expect test to fail until ASTERISK-24782 is resolved.
-    expected-result: False
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie1ca7da1fd01031846994f3b18ac551f2b51320d
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Corey Farrell <git at cfware.com>



More information about the asterisk-code-review mailing list