Implement UnwrapLtrim mutator (#1395)

resources/schema.json

@@ -349,6 +349,7 @@
                 "UnwrapArrayUnique": { "$ref": "#/definitions/default-mutator-config" },
                 "UnwrapArrayValues": { "$ref": "#/definitions/default-mutator-config" },
                 "UnwrapLcFirst": { "$ref": "#/definitions/default-mutator-config" },
+                "UnwrapLtrim": { "$ref": "#/definitions/default-mutator-config" },
                 "UnwrapStrRepeat": { "$ref": "#/definitions/default-mutator-config" },
                 "UnwrapStrReplace": { "$ref": "#/definitions/default-mutator-config" },
                 "UnwrapStrToLower": { "$ref": "#/definitions/default-mutator-config" },

src/Mutator/ProfileList.php

@@ -239,6 +239,7 @@ final class ProfileList
         Mutator\Unwrap\UnwrapArrayUnique::class,
         Mutator\Unwrap\UnwrapArrayValues::class,
         Mutator\Unwrap\UnwrapLcFirst::class,
+        Mutator\Unwrap\UnwrapLtrim::class,
         Mutator\Unwrap\UnwrapStrRepeat::class,
         Mutator\Unwrap\UnwrapStrReplace::class,
         Mutator\Unwrap\UnwrapStrToLower::class,
@@ -421,6 +422,7 @@ final class ProfileList
         'UnwrapArrayUnique' => Mutator\Unwrap\UnwrapArrayUnique::class,
         'UnwrapArrayValues' => Mutator\Unwrap\UnwrapArrayValues::class,
         'UnwrapLcFirst' => Mutator\Unwrap\UnwrapLcFirst::class,
+        'UnwrapLtrim' => Mutator\Unwrap\UnwrapLtrim::class,
         'UnwrapStrRepeat' => Mutator\Unwrap\UnwrapStrRepeat::class,
         'UnwrapStrReplace' => Mutator\Unwrap\UnwrapStrReplace::class,
         'UnwrapStrToLower' => Mutator\Unwrap\UnwrapStrToLower::class,

src/Mutator/Unwrap/UnwrapLtrim.php

@@ -0,0 +1,72 @@
+<?php
+/**
+ * This code is licensed under the BSD 3-Clause License.
+ *
+ * Copyright (c) 2017, Maks Rafalko
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are met:
+ *
+ * * Redistributions of source code must retain the above copyright notice, this
+ *   list of conditions and the following disclaimer.
+ *
+ * * Redistributions in binary form must reproduce the above copyright notice,
+ *   this list of conditions and the following disclaimer in the documentation
+ *   and/or other materials provided with the distribution.
+ *
+ * * Neither the name of the copyright holder nor the names of its
+ *   contributors may be used to endorse or promote products derived from
+ *   this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+ * DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+ * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+ * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
+ * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+declare(strict_types=1);
+
+namespace Infection\Mutator\Unwrap;
+
+use Infection\Mutator\Definition;
+use Infection\Mutator\MutatorCategory;
+
+/**
+ * @internal
+ */
+final class UnwrapLtrim extends AbstractUnwrapMutator
+{
+    public static function getDefinition(): ?Definition
+    {
+        return new Definition(
+            <<<'TXT'
+Replaces a `ltrim` function call with its first operand. For example:
+
+```php
+$x = ltrim(' Hello!');
+```
+
+Will be mutated to:
+
+```php
+$x = ' Hello!';
+```
+TXT
+            ,
+            MutatorCategory::SEMANTIC_REDUCTION,
+            null
+        );
+    }
+
+    protected function getFunctionName(): string
+    {
+        return 'ltrim';
+    }
+}

tests/phpunit/Mutator/Unwrap/UnwrapLtrimTest.php

@@ -0,0 +1,191 @@
+<?php
+/**
+ * This code is licensed under the BSD 3-Clause License.
+ *
+ * Copyright (c) 2017, Maks Rafalko
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are met:
+ *
+ * * Redistributions of source code must retain the above copyright notice, this
+ *   list of conditions and the following disclaimer.
+ *
+ * * Redistributions in binary form must reproduce the above copyright notice,
+ *   this list of conditions and the following disclaimer in the documentation
+ *   and/or other materials provided with the distribution.
+ *
+ * * Neither the name of the copyright holder nor the names of its
+ *   contributors may be used to endorse or promote products derived from
+ *   this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+ * DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+ * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+ * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
+ * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+declare(strict_types=1);
+
+namespace Infection\Tests\Mutator\Unwrap;
+
+use Infection\Tests\Mutator\BaseMutatorTestCase;
+
+final class UnwrapLtrimTest extends BaseMutatorTestCase
+{
+    /**
+     * @dataProvider mutationsProvider
+     *
+     * @param string|string[] $expected
+     */
+    public function test_it_can_mutate(string $input, $expected = []): void
+    {
+        $this->doTest($input, $expected);
+    }
+
+    public function mutationsProvider(): iterable
+    {
+        yield 'It mutates correctly when provided with a string' => [
+            <<<'PHP'
+<?php
+
+$a = ltrim(' Good Afternoon! ');
+PHP
+            ,
+            <<<'PHP'
+<?php
+
+$a = ' Good Afternoon! ';
+PHP
+        ];
+
+        yield 'It mutates correctly when provided with a constant' => [
+            <<<'PHP'
+<?php
+
+$a = ltrim(\Class_With_Const::Const);
+PHP
+            ,
+            <<<'PHP'
+<?php
+
+$a = \Class_With_Const::Const;
+PHP
+        ];
+
+        yield 'It mutates correctly when a backslash is in front of trim' => [
+            <<<'PHP'
+<?php
+
+$a = \ltrim(' Good Afternoon! ');
+PHP
+            ,
+            <<<'PHP'
+<?php
+
+$a = ' Good Afternoon! ';
+PHP
+        ];
+
+        yield 'It mutates correctly within if statements' => [
+            <<<'PHP'
+<?php
+
+$a = ' Good Afternoon! ';
+if (ltrim($a) === $a) {
+    return true;
+}
+PHP
+            ,
+            <<<'PHP'
+<?php
+
+$a = ' Good Afternoon! ';
+if ($a === $a) {
+    return true;
+}
+PHP
+        ];
+
+        yield 'It mutates correctly when trim is wrongly capitalized' => [
+            <<<'PHP'
+<?php
+
+$a = LtRiM(' Good Afternoon! ');
+PHP
+            ,
+            <<<'PHP'
+<?php
+
+$a = ' Good Afternoon! ';
+PHP
+        ];
+
+        yield 'It mutates correctly when trim uses another function as input' => [
+            <<<'PHP'
+<?php
+
+$a = ltrim($foo->bar());
+PHP
+            ,
+            <<<'PHP'
+<?php
+
+$a = $foo->bar();
+PHP
+        ];
+
+        yield 'It mutates correctly when provided with a more complex situation' => [
+            <<<'PHP'
+<?php
+
+$a = ltrim(array_reduce($words, function (string $carry, string $item) {
+    return $carry . substr($item, 0, 1);
+}));
+PHP
+            ,
+            <<<'PHP'
+<?php
+
+$a = array_reduce($words, function (string $carry, string $item) {
+    return $carry . substr($item, 0, 1);
+});
+PHP
+        ];
+
+        yield 'It does not mutate other *trim calls' => [
+            <<<'PHP'
+<?php
+
+$a = rtrim(' Good Afternoon! ');
+PHP
+        ];
+
+        yield 'It does not mutate functions named ltrim' => [
+            <<<'PHP'
+<?php
+
+function ltrim($string)
+{
+}
+PHP
+        ];
+
+        yield 'It does not break when provided with a variable function name' => [
+            <<<'PHP'
+<?php
+
+$a = 'ltrim';
+
+$b = $a(' FooBar ');
+PHP
+            ,
+        ];
+    }
+}