Merge pull request #242 from PHPCSStandards/testutils/skipjscsstest-a…

…void-risky-warning Tests/UtilityMethodTestCase::skipJSCSSTestsOnPHPCS4: get rid of "risky test" warning
PHPCSStandards · Dec 22, 2020 · edf6a35 · edf6a35
2 parents 58bb724 + 9cf84a1
commit edf6a35
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/Tests/TestUtils/UtilityMethodTestCase/SkipCSJSTest.php b/Tests/TestUtils/UtilityMethodTestCase/SkipCSJSTest.php
@@ -46,8 +46,6 @@ public function skipJSCSSTestsOnPHPCS4()
     /**
      * Test that the skipJSCSSTestsOnPHPCS4() skips JS/CSS file tests on PHPCS 4.x.
      *
-     * @doesNotPerformAssertions
-     *
      * @return void
      */
     public function testSkipJsCss()
@@ -68,6 +66,9 @@ public function testSkipJsCss()
                 // PHPUnit 4.
                 $this->setExpectedException($exception, $msg);
             }
+        } else {
+            // Get rid of the "does not perform assertions" warning when run with PHPCS 3.x.
+            $this->assertTrue(true);
         }
 
         parent::skipJSCSSTestsOnPHPCS4();