Skip to content

Commit

Permalink
Enhancement: Implement UnwrapArrayDiffAssoc mutator (#562)
Browse files Browse the repository at this point in the history
  • Loading branch information
localheinz authored and maks-rafalko committed Dec 8, 2018
1 parent 9fc6fed commit bb6e754
Show file tree
Hide file tree
Showing 3 changed files with 257 additions and 0 deletions.
54 changes: 54 additions & 0 deletions src/Mutator/Unwrap/UnwrapArrayDiffAssoc.php
@@ -0,0 +1,54 @@
<?php
/**
* This code is licensed under the BSD 3-Clause License.
*
* Copyright (c) 2017-2018, 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 PhpParser\Node;

/**
* @internal
*/
final class UnwrapArrayDiffAssoc extends AbstractUnwrapMutator
{
protected function getFunctionName(): string
{
return 'array_diff_assoc';
}

protected function getParameterIndexes(Node $node): \Generator
{
yield 0;
}
}
2 changes: 2 additions & 0 deletions src/Mutator/Util/MutatorProfile.php
Expand Up @@ -192,6 +192,7 @@ final class MutatorProfile
Mutator\Unwrap\UnwrapArrayChunk::class,
Mutator\Unwrap\UnwrapArrayCombine::class,
Mutator\Unwrap\UnwrapArrayDiff::class,
Mutator\Unwrap\UnwrapArrayDiffAssoc::class,
Mutator\Unwrap\UnwrapArrayDiffKey::class,
Mutator\Unwrap\UnwrapArrayFilter::class,
Mutator\Unwrap\UnwrapArrayFlip::class,
Expand Down Expand Up @@ -335,6 +336,7 @@ final class MutatorProfile
'UnwrapArrayChunk' => Mutator\Unwrap\UnwrapArrayChunk::class,
'UnwrapArrayCombine' => Mutator\Unwrap\UnwrapArrayCombine::class,
'UnwrapArrayDiff' => Mutator\Unwrap\UnwrapArrayDiff::class,
'UnwrapArrayDiffAssoc' => Mutator\Unwrap\UnwrapArrayDiffAssoc::class,
'UnwrapArrayDiffKey' => Mutator\Unwrap\UnwrapArrayDiffKey::class,
'UnwrapArrayFilter' => Mutator\Unwrap\UnwrapArrayFilter::class,
'UnwrapArrayFlip' => Mutator\Unwrap\UnwrapArrayFlip::class,
Expand Down
201 changes: 201 additions & 0 deletions tests/Mutator/Unwrap/UnwrapArrayDiffAssocTest.php
@@ -0,0 +1,201 @@
<?php
/**
* This code is licensed under the BSD 3-Clause License.
*
* Copyright (c) 2017-2018, 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\AbstractMutatorTestCase;

/**
* @internal
*/
final class UnwrapArrayDiffAssocTest extends AbstractMutatorTestCase
{
/**
* @dataProvider provideMutationCases
*/
public function test_mutator($input, $expected = null): void
{
$this->doTest($input, $expected);
}

public function provideMutationCases(): \Generator
{
yield 'It mutates correctly when provided with an array' => [
<<<'PHP'
<?php
$a = array_diff_assoc(['foo' => 'bar'], ['baz' => 'bar']);
PHP
,
<<<'PHP'
<?php
$a = ['foo' => 'bar'];
PHP
];

yield 'It mutates correctly when provided with a constant' => [
<<<'PHP'
<?php
$a = array_diff_assoc(\Class_With_Const::Const, ['baz' => 'bar']);
PHP
,
<<<'PHP'
<?php
$a = \Class_With_Const::Const;
PHP
];

yield 'It mutates correctly when a backslash is in front of array_diff_assoc' => [
<<<'PHP'
<?php
$a = \array_diff_assoc(['foo' => 'bar'], ['baz' => 'bar']);
PHP
,
<<<'PHP'
<?php
$a = ['foo' => 'bar'];
PHP
];

yield 'It mutates correctly within if statements' => [
<<<'PHP'
<?php
$a = ['foo' => 'bar'];
if (array_diff_assoc($a, ['baz' => 'bar']) === $a) {
return true;
}
PHP
,
<<<'PHP'
<?php
$a = ['foo' => 'bar'];
if ($a === $a) {
return true;
}
PHP
];

yield 'It mutates correctly when array_diff_assoc is wrongly capitalized' => [
<<<'PHP'
<?php
$a = aRraY_dIfF_aSsOc(['foo' => 'bar'], ['baz' => 'bar']);
PHP
,
<<<'PHP'
<?php
$a = ['foo' => 'bar'];
PHP
];

yield 'It mutates correctly when array_diff_assoc uses functions as input' => [
<<<'PHP'
<?php
$a = array_diff_assoc($foo->bar(), $foo->baz());
PHP
,
<<<'PHP'
<?php
$a = $foo->bar();
PHP
];

yield 'It mutates correctly when provided with a more complex situation' => [
<<<'PHP'
<?php
$a = array_map('strtolower', array_diff_assoc(['foo' => 'bar'], ['baz' => 'bar']));
PHP
,
<<<'PHP'
<?php
$a = array_map('strtolower', ['foo' => 'bar']);
PHP
];

yield 'It mutates correctly when more than two parameters are present' => [
<<<'PHP'
<?php
$a = array_diff_assoc(['foo' => 'bar'], ['baz' => 'bar'], ['qux' => 'bar']);
PHP
,
<<<'PHP'
<?php
$a = ['foo' => 'bar'];
PHP
];

yield 'It does not mutate other array_ calls' => [
<<<'PHP'
<?php
$a = array_map('strtolower', ['foo' => 'bar']);
PHP
];

yield 'It does not mutate functions named array_diff_assoc' => [
<<<'PHP'
<?php
function array_diff_assoc($array, $array1, $array2)
{
}
PHP
];

yield 'It does not mutate when a variable function name is used' => [
<<<'PHP'
<?php
$a = 'array_diff_assoc';
$b = $a(['foo' => 'bar'], ['baz' => 'bar']);
PHP
];
}
}

0 comments on commit bb6e754

Please sign in to comment.