From f723f67c3f3611681d2280bca4e4f722d17d35a2 Mon Sep 17 00:00:00 2001
From: Daniele Alessandri <suppakilla@gmail.com>
Date: Thu, 3 Sep 2020 12:21:35 +0200
Subject: [PATCH] [tests] Use new utility method to skip PUB/SUB tests on CI.

---
 tests/Predis/PubSub/ConsumerTest.php | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

diff --git a/tests/Predis/PubSub/ConsumerTest.php b/tests/Predis/PubSub/ConsumerTest.php
index 32c7c2d2..55354a2c 100644
--- a/tests/Predis/PubSub/ConsumerTest.php
+++ b/tests/Predis/PubSub/ConsumerTest.php
@@ -364,11 +364,9 @@ class ConsumerTest extends PredisTestCase
      */
     public function testPubSubAgainstRedisServer(): void
     {
-        if (getenv('GITHUB_ACTIONS') || getenv('TRAVIS')) {
-            $this->markTestSkipped(
-                'Test temporarily skipped on CI environments, see note in the body of the test'
-            );
-        }
+        $this->markTestSkippedOnCIEnvironment(
+            'Test temporarily skipped on CI environments, see note in the body of the test' // TODO
+        );
 
         $parameters = array(
             'host' => constant('REDIS_SERVER_HOST'),
@@ -415,11 +413,9 @@ class ConsumerTest extends PredisTestCase
      */
     public function testPubSubAgainstRedisServerBlocking(): void
     {
-        if (getenv('GITHUB_ACTIONS') || getenv('TRAVIS')) {
-            $this->markTestSkipped(
-                'Test temporarily skipped on CI environments, see note in the body of the test'
-            );
-        }
+        $this->markTestSkippedOnCIEnvironment(
+            'Test temporarily skipped on CI environments, see note in the body of the test' // TODO
+        );
 
         $parameters = array(
             'host' => constant('REDIS_SERVER_HOST'),