[TASK] Switch skip condition with annotations in RedisBackendTest 95/57795/3
authorMichael Kersten <michael.kersten@3m5.de>
Sat, 4 Aug 2018 07:19:43 +0000 (09:19 +0200)
committerTymoteusz Motylewski <t.motylewski@gmail.com>
Sat, 4 Aug 2018 07:58:48 +0000 (09:58 +0200)
Use requires annotation for skipping test if php extension redis
is not loaded.

Resolves: #85748
Releases: master
Change-Id: I587d2e6f9928defef9417c44358cc6816345caf3
Reviewed-on: https://review.typo3.org/57795
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Tested-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Build/Scripts/annotationChecker.php
typo3/sysext/core/Tests/Functional/Cache/Backend/RedisBackendTest.php

index c70c714..4e89a94 100755 (executable)
@@ -57,7 +57,7 @@ class NodeVisitor extends NodeVisitorAbstract
                     // PHPDocumentor 2 tags
                     'api', 'author', 'category', 'copyright', 'deprecated', 'example', 'filesource', 'global', 'ignore', 'internal', 'license', 'link', 'method', 'package', 'param', 'property', 'property-read', 'property-write', 'return', 'see', 'since', 'source', 'subpackage', 'throws', 'todo', 'TODO', 'usedby', 'uses', 'var', 'version',
                     // PHPUnit tags
-                    'codeCoverageIgnore', 'codeCoverageIgnoreStart', 'codeCoverageIgnoreEnd', 'test', 'covers', 'dataProvider', 'group', 'skip', 'depends', 'expectedException', 'before',
+                    'codeCoverageIgnore', 'codeCoverageIgnoreStart', 'codeCoverageIgnoreEnd', 'test', 'covers', 'dataProvider', 'group', 'skip', 'depends', 'expectedException', 'before', 'requires',
                     // codeception tags
                     'env',
                     // PHPCheckStyle
index 45f41a2..3378503 100644 (file)
@@ -24,6 +24,8 @@ use TYPO3\TestingFramework\Core\Functional\FunctionalTestCase;
  * Warning:
  * These functional tests use and flush redis database numbers 0 and 1 on the
  * redis host specified by environment variable typo3RedisHost
+ *
+ * @requires extension redis
  */
 class RedisBackendTest extends FunctionalTestCase
 {
@@ -34,9 +36,6 @@ class RedisBackendTest extends FunctionalTestCase
     {
         // Note this functional does NOT call parent::setUp() since it does
         // not need a full blown instance and database
-        if (!extension_loaded('redis')) {
-            $this->markTestSkipped('redis extension was not available');
-        }
         if (!getenv('typo3TestingRedisHost')) {
             $this->markTestSkipped('environment variable "typo3TestingRedisHost" must be set to run this test');
         }