Workaround for ecs issue caused by PHP_CodeSniffer & php-cs-fixer

See deprecated-packages/symplify#3130

CHANGELOG.md

@@ -5,6 +5,7 @@
 <!-- There is always Unreleased section on the top. Subsections (Added, Changed, Fixed, Removed) should be added as needed. -->
 
 ## Unreleased
+- `PhpUnitExpectationFixer` is used only on PHP 8.0+ See [symplify#3130](https://github.com/symplify/symplify/issues/3130).
 
 ## 3.0.0 - 2021-03-02
 - **[BC break]** Change YAML config to PHP. See [UPGRADE-3.0.md](UPGRADE-3.0.md) for step-by-step upgrade howto.

ecs.php

@@ -444,8 +444,13 @@
         $services->set(PhpUnitMockShortWillReturnFixer::class);
         // Use expectedException*() methods instead of @expectedException* annotation (both following fixers must be applied to do so)
         $services->set(PhpUnitNoExpectationAnnotationFixer::class);
-        // Usages of ->setExpectedException* methods MUST be replaced by ->expectException* methods
-        $services->set(PhpUnitExpectationFixer::class);
+
+        // Following check fails on PHP <8.0. See https://github.com/symplify/symplify/issues/3130
+        if (PHP_VERSION_ID >= 80000) {
+            // Usages of ->setExpectedException* methods MUST be replaced by ->expectException* methods
+            $services->set(PhpUnitExpectationFixer::class);
+        }
+
         // Visibility of setUp() and tearDown() method should be kept protected
         $services->set(PhpUnitSetUpTearDownVisibilityFixer::class);
         // Calls to `PHPUnit\Framework\TestCase` static methods must all be of the same type (`$this->...`)