Merge pull request #358 from ergebnis/fix/fixture

Fix: Run `friendsofphp/php-cs-fixer` on fixtures

.github/workflows/integrate.yaml

@@ -79,6 +79,9 @@ jobs:
       - name: "Run friendsofphp/php-cs-fixer"
         run: "vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.php --diff --diff-format=udiff --dry-run --verbose"
 
+      - name: "Run friendsofphp/php-cs-fixer on fixtures"
+        run: "vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.fixture.php --diff --diff-format=udiff --dry-run --verbose"
+
   dependency-analysis:
     name: "Dependency Analysis"
 

.php-cs-fixer.fixture.php

@@ -21,14 +21,15 @@
     'final_internal_class' => false,
     'final_public_method_for_abstract_class' => false,
     'header_comment' => false,
-    'lowercase_constants' => false,
     'lowercase_keywords' => false,
     'magic_method_casing' => false,
     'protected_to_private' => false,
     'static_lambda' => false,
 ]);
 
-$config->getFinder()->in(__DIR__ . '/test/Fixture');
+$config->getFinder()
+    ->in(__DIR__ . '/test/Fixture')
+    ->notPath('Classes/PHPUnit/Framework/TestCaseWithSuffixRule/Success/ImplicitlyAbstractTestCase.php');
 
 $config->setCacheFile(__DIR__ . '/.build/php-cs-fixer/.php_cs.fixture.cache');
 

Makefile

@@ -15,6 +15,7 @@ coding-standards: vendor ## Normalizes composer.json with ergebnis/composer-norm
 	yamllint -c .yamllint.yaml --strict .
 	mkdir -p .build/php-cs-fixer
 	vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.php --diff --diff-format=udiff --verbose
+	vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.fixture.php --diff --diff-format=udiff --verbose
 
 .PHONY: dependency-analysis
 dependency-analysis: vendor ## Runs a dependency analysis with maglnet/composer-require-checker