Add mutator LogicalNot (#680)

kubawerlos · Dec 4, 2021 · da63a01 · da63a01
1 parent 8d4aaea
commit da63a01
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 2 deletions.
diff --git a/.dev-tools/infection.json b/.dev-tools/infection.json
@@ -78,6 +78,7 @@
         },
         "LogicalLowerAnd": true,
         "LogicalLowerOr": true,
+        "LogicalNot": true,
         "LogicalOr": true,
         "MBString": true,
         "MethodCallRemoval": true,

diff --git a/.dev-tools/src/InfectionConfigBuilder.php b/.dev-tools/src/InfectionConfigBuilder.php
@@ -26,7 +26,6 @@ final class InfectionConfigBuilder
         'IntegerNegation',
         'LessThan',
         'LessThanOrEqualTo',
-        'LogicalNot',
         'Minus',
         'NotIdentical',
         'NotIdenticalNotEqual',

diff --git a/README.md b/README.md
@@ -3,7 +3,7 @@
 [![Latest stable version](https://img.shields.io/packagist/v/kubawerlos/php-cs-fixer-custom-fixers.svg?label=current%20version)](https://packagist.org/packages/kubawerlos/php-cs-fixer-custom-fixers)
 [![PHP version](https://img.shields.io/packagist/php-v/kubawerlos/php-cs-fixer-custom-fixers.svg)](https://php.net)
 [![License](https://img.shields.io/github/license/kubawerlos/php-cs-fixer-custom-fixers.svg)](LICENSE)
-![Tests](https://img.shields.io/badge/tests-3108-brightgreen.svg)
+![Tests](https://img.shields.io/badge/tests-3110-brightgreen.svg)
 [![Downloads](https://img.shields.io/packagist/dt/kubawerlos/php-cs-fixer-custom-fixers.svg)](https://packagist.org/packages/kubawerlos/php-cs-fixer-custom-fixers)
 
 [![CI Status](https://github.com/kubawerlos/php-cs-fixer-custom-fixers/workflows/CI/badge.svg?branch=main&event=push)](https://github.com/kubawerlos/php-cs-fixer-custom-fixers/actions)

diff --git a/tests/Analyzer/DataProviderAnalyzerTest.php b/tests/Analyzer/DataProviderAnalyzerTest.php
@@ -127,6 +127,17 @@ public function testFoo() {}
             }',
         ];
 
+        yield 'data provider being constant' => [
+            [],
+            '<?php class FooTest extends TestCase {
+                private const provider = [];
+                /**
+                 * @dataProvider provider
+                 */
+                public function testFoo() {}
+            }',
+        ];
+
         yield 'ignore anonymous function' => [
             [
                 new DataProviderAnalysis('provider2', 93, [65]),

diff --git a/tests/Fixer/StringableInterfaceFixerTest.php b/tests/Fixer/StringableInterfaceFixerTest.php
@@ -51,6 +51,13 @@ function () { return 0; };
                     }
                 }'];
 
+        yield ['<?php class Foo
+            {
+                public function bar() {
+                    $ohject->__toString();
+                }
+            }'];
+
         yield [
             '<?php class Foo implements \Stringable
             {