From 2bcbf0e344aab31ccc96b6e3c323331d0e8afcee Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Sun, 10 Mar 2019 13:16:27 +0100
Subject: [PATCH 01/18] #654 mbstring mutator

---
 src/Mutator/Extensions/MBString.php       | 217 ++++++++++
 src/Mutator/Util/MutatorProfile.php       |   9 +
 tests/Mutator/Extensions/MBStringTest.php | 462 ++++++++++++++++++++++
 3 files changed, 688 insertions(+)
 create mode 100644 src/Mutator/Extensions/MBString.php
 create mode 100644 tests/Mutator/Extensions/MBStringTest.php

diff --git a/src/Mutator/Extensions/MBString.php b/src/Mutator/Extensions/MBString.php
new file mode 100644
index 000000000..b4b0742ac
--- /dev/null
+++ b/src/Mutator/Extensions/MBString.php
@@ -0,0 +1,217 @@
+<?php
+/**
+ * This code is licensed under the BSD 3-Clause License.
+ *
+ * Copyright (c) 2017-2019, 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\Extensions;
+
+use Generator;
+use Infection\Mutator\Util\Mutator;
+use Infection\Mutator\Util\MutatorConfig;
+use PhpParser\Node;
+
+/**
+ * @internal
+ */
+final class MBString extends Mutator
+{
+    private const MB_CASES = [
+        'MB_CASE_UPPER' => 0,
+        'MB_CASE_LOWER' => 1,
+        'MB_CASE_TITLE' => 2,
+        'MB_CASE_FOLD' => 3,
+        'MB_CASE_UPPER_SIMPLE' => 4,
+        'MB_CASE_LOWER_SIMPLE' => 5,
+        'MB_CASE_TITLE_SIMPLE' => 6,
+        'MB_CASE_FOLD_SIMPLE' => 7,
+    ];
+
+    private $converters;
+
+    public function __construct(MutatorConfig $config)
+    {
+        parent::__construct($config);
+        $this->setupConverters();
+    }
+
+    /**
+     * @return Node|Node[]|Generator
+     */
+    public function mutate(Node $node)
+    {
+        yield from $this->converters[$this->getFunctionName($node)]($node);
+    }
+
+    protected function mutatesNode(Node $node): bool
+    {
+        return isset($this->converters[$this->getFunctionName($node)]);
+    }
+
+    private function setupConverters(): void
+    {
+        $this->converters = [
+            'mb_chr' => $this->mapNameSkipArg('chr', 1),
+            'mb_ereg_match' => $this->mapNameSkipArg('preg_match', 2),
+            'mb_ereg_replace_callback' => $this->mapNameSkipArg('preg_replace_callback', 3),
+            'mb_ereg_replace' => $this->mapNameSkipArg('ereg_replace', 3),
+            'mb_ereg' => $this->mapNameSkipArg('ereg', 3),
+            'mb_eregi_replace' => $this->mapNameSkipArg('eregi_replace', 3),
+            'mb_eregi' => $this->mapNameSkipArg('eregi', 3),
+            'mb_ord' => $this->mapNameSkipArg('ord', 1),
+            'mb_parse_str' => $this->mapName('parse_str'),
+            'mb_send_mail' => $this->mapName('mail'),
+            'mb_split' => $this->mapName('split'),
+            'mb_strcut' => $this->mapNameSkipArg('substr', 3),
+            'mb_stripos' => $this->mapNameSkipArg('stripos', 3),
+            'mb_stristr' => $this->mapNameSkipArg('stristr', 3),
+            'mb_strlen' => $this->mapNameSkipArg('strlen', 1),
+            'mb_strpos' => $this->mapNameSkipArg('strpos', 3),
+            'mb_strrchr' => $this->mapNameSkipArg('strrchr', 2),
+            'mb_strrichr' => $this->mapNameSkipArg('strrchr', 2),
+            'mb_strripos' => $this->mapNameSkipArg('strripos', 3),
+            'mb_strrpos' => $this->mapNameSkipArg('strrpos', 3),
+            'mb_strstr' => $this->mapNameSkipArg('strstr', 3),
+            'mb_strtolower' => $this->mapNameSkipArg('strtolower', 1),
+            'mb_strtoupper' => $this->mapNameSkipArg('strtoupper', 1),
+            'mb_substr_count' => $this->mapNameSkipArg('substr_count', 2),
+            'mb_substr' => $this->mapNameSkipArg('substr', 3),
+            'mb_convert_case' => $this->mapConvertCase(),
+        ];
+    }
+
+    private function mapName(string $functionName): callable
+    {
+        return function (Node\Expr\FuncCall $node) use ($functionName): Generator {
+            yield $this->createNode($node, $functionName, $node->args);
+        };
+    }
+
+    private function mapNameSkipArg(string $functionName, int $skipArgs): callable
+    {
+        return function (Node\Expr\FuncCall $node) use ($functionName, $skipArgs): Generator {
+            $args = $node->args;
+
+            if ($skipArgs !== null) {
+                $args = \array_slice($args, 0, $skipArgs);
+            }
+
+            yield $this->createNode($node, $functionName, $args);
+        };
+    }
+
+    private function mapConvertCase(): callable
+    {
+        return function (Node\Expr\FuncCall $node): Generator {
+            $modeValue = $this->getConvertCaseModeValue($node);
+
+            if ($modeValue === null) {
+                return;
+            }
+
+            $functionName = $this->getConvertCaseFunctionName($modeValue);
+
+            if ($functionName === null) {
+                return;
+            }
+
+            yield $this->createNode($node, $functionName, [$node->args[0]]);
+        };
+    }
+
+    private function getConvertCaseModeValue(Node\Expr\FuncCall $node): ?int
+    {
+        if (\count($node->args) < 2) {
+            return null;
+        }
+
+        $mode = $node->args[1]->value;
+
+        $modeValue = null;
+
+        if ($mode instanceof Node\Expr\ConstFetch) {
+            $modeName = $mode->name->toString();
+
+            if (\defined($modeName)) {
+                return \constant($modeName);
+            }
+
+            if (isset(self::MB_CASES[$modeName])) {
+                return self::MB_CASES[$modeName];
+            }
+        }
+
+        return null;
+    }
+
+    private function getConvertCaseFunctionName(int $mode): ?string
+    {
+        if ($this->isInMbCaseMode($mode, 'MB_CASE_UPPER', 'MB_CASE_UPPER_SIMPLE')) {
+            return 'strtoupper';
+        }
+
+        if ($this->isInMbCaseMode($mode, 'MB_CASE_LOWER', 'MB_CASE_LOWER_SIMPLE', 'MB_CASE_FOLD', 'MB_CASE_FOLD_SIMPLE')) {
+            return 'strtolower';
+        }
+
+        if ($this->isInMbCaseMode($mode, 'MB_CASE_TITLE', 'MB_CASE_TITLE_SIMPLE')) {
+            return 'ucwords';
+        }
+
+        return null;
+    }
+
+    private function isInMbCaseMode(int $mode, string ...$cases): bool
+    {
+        $modes = \array_flip(self::MB_CASES);
+
+        return isset($modes[$mode]) && \in_array($modes[$mode], $cases);
+    }
+
+    private function getFunctionName(Node $node): ?string
+    {
+        if (!$node instanceof Node\Expr\FuncCall || !$node->name instanceof Node\Name) {
+            return null;
+        }
+
+        return \strtolower($node->name->toString());
+    }
+
+    private function createNode(Node\Expr\FuncCall $node, string $functionName, array $args): Node\Expr\FuncCall
+    {
+        return new Node\Expr\FuncCall(
+            new Node\Name($functionName, $node->name->getAttributes()),
+            $args,
+            $node->getAttributes()
+        );
+    }
+}
diff --git a/src/Mutator/Util/MutatorProfile.php b/src/Mutator/Util/MutatorProfile.php
index ba113192e..a8f46427c 100644
--- a/src/Mutator/Util/MutatorProfile.php
+++ b/src/Mutator/Util/MutatorProfile.php
@@ -60,6 +60,7 @@ final class MutatorProfile
         '@zero_iteration' => self::ZERO_ITERATION,
         '@cast' => self::CAST,
         '@unwrap' => self::UNWRAP,
+        '@extensions' => self::EXTENSIONS,
 
         //Special Profiles
         '@default' => self::DEFAULT,
@@ -232,6 +233,10 @@ final class MutatorProfile
         Mutator\Unwrap\UnwrapUcWords::class,
     ];
 
+    public const EXTENSIONS = [
+        Mutator\Extensions\MBString::class,
+    ];
+
     public const DEFAULT = [
         '@arithmetic',
         '@boolean',
@@ -246,6 +251,7 @@ final class MutatorProfile
         '@return_value',
         '@sort',
         '@zero_iteration',
+        '@extensions',
     ];
 
     public const FULL_MUTATOR_LIST = [
@@ -395,5 +401,8 @@ final class MutatorProfile
         'UnwrapTrim' => Mutator\Unwrap\UnwrapTrim::class,
         'UnwrapUcFirst' => Mutator\Unwrap\UnwrapUcFirst::class,
         'UnwrapUcWords' => Mutator\Unwrap\UnwrapUcWords::class,
+
+        // Extensions
+        'MBString' => Mutator\Extensions\MBString::class,
     ];
 }
diff --git a/tests/Mutator/Extensions/MBStringTest.php b/tests/Mutator/Extensions/MBStringTest.php
new file mode 100644
index 000000000..2ba851173
--- /dev/null
+++ b/tests/Mutator/Extensions/MBStringTest.php
@@ -0,0 +1,462 @@
+<?php
+/**
+ * This code is licensed under the BSD 3-Clause License.
+ *
+ * Copyright (c) 2017-2019, 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\Extensions;
+
+use Infection\Tests\Mutator\AbstractMutatorTestCase;
+
+/**
+ * @internal
+ */
+final class MBStringTest extends AbstractMutatorTestCase
+{
+    /**
+     * @dataProvider provideMutationCases
+     */
+    public function test_mutator(string $input, string $expected = null): void
+    {
+        $this->doTest($input, $expected);
+    }
+
+    public function provideMutationCases(): \Generator
+    {
+        yield 'It converts mb_strlen with leading slash' => [
+            "<?php \mb_strlen('test');",
+            "<?php\n\nstrlen('test');",
+        ];
+
+        yield 'It converts mb_strlen with leading slash in namespace' => [
+            "<?php namespace Test; \mb_strlen('test');",
+            "<?php\n\nnamespace Test;\n\nstrlen('test');",
+        ];
+
+        yield 'It does not convert standard functions like strpos' => [
+            "<?php strpos('test');",
+        ];
+
+        yield 'It converts mb_strlen with encoding to strlen' => [
+            "<?php mb_strlen('test', 'utf-8');",
+            "<?php\n\nstrlen('test');",
+        ];
+
+        // mb_chr-> chr
+
+        yield 'It converts mb_chr to chr' => [
+            '<?php mb_chr(74);',
+            "<?php\n\nchr(74);",
+        ];
+
+        yield 'It converts mb_chr with encoding to chr' => [
+            "<?php mb_chr(74, 'utf-8');",
+            "<?php\n\nchr(74);",
+        ];
+
+        // mb_ereg_match -> preg_match
+
+        yield 'It converts mb_ereg_match to preg_match' => [
+            "<?php mb_ereg_match('#t#', 'test');",
+            "<?php\n\npreg_match('#t#', 'test');",
+        ];
+
+        yield 'It converts mb_ereg_match with encoding argument to preg_match' => [
+            "<?php mb_ereg_match('#t#', 'trim', 'mb');",
+            "<?php\n\npreg_match('#t#', 'trim');",
+        ];
+
+        // mb_ereg_replace_callback -> preg_replace_callback
+
+        yield 'It converts mb_ereg_replace_callback to preg_replace_callback' => [
+            "<?php mb_ereg_replace_callback('#t#', 'trim', 'test');",
+            "<?php\n\npreg_replace_callback('#t#', 'trim', 'test');",
+        ];
+
+        yield 'It converts mb_ereg_replace_callback with options to preg_replace_callback' => [
+            "<?php mb_ereg_replace_callback('#t#', 'trim', 'test', 'msr');",
+            "<?php\n\npreg_replace_callback('#t#', 'trim', 'test');",
+        ];
+
+        // mb_ereg_replace -> ereg_replace
+
+        yield 'It converts mb_ereg_replace to ereg_replace' => [
+            "<?php mb_ereg_replace('#t#', 'T', 'test');",
+            "<?php\n\nereg_replace('#t#', 'T', 'test');",
+        ];
+
+        yield 'It converts mb_ereg_replace with options to ereg_replace' => [
+            "<?php mb_ereg_replace('#t#', 'T', 'test', 'mb');",
+            "<?php\n\nereg_replace('#t#', 'T', 'test');",
+        ];
+
+        // mb_ereg -> ereg
+
+        yield 'It converts mb_ereg to ereg' => [
+            "<?php mb_ereg('#t#', 'test');",
+            "<?php\n\nereg('#t#', 'test');",
+        ];
+
+        yield 'It converts mb_ereg with regs argument to ereg' => [
+            "<?php mb_ereg('#t#', 'test', \$regs);",
+            "<?php\n\nereg('#t#', 'test', \$regs);",
+        ];
+
+        // mb_eregi_replace -> eregi_replace
+
+        yield 'It converts mb_eregi_replace to eregi_replace' => [
+            "<?php mb_eregi_replace('#t#', 'T', 'test');",
+            "<?php\n\neregi_replace('#t#', 'T', 'test');",
+        ];
+
+        yield 'It converts mb_eregi_replace with options to eregi_replace' => [
+            "<?php mb_eregi_replace('#t#', 'T', 'test', 'msr');",
+            "<?php\n\neregi_replace('#t#', 'T', 'test');",
+        ];
+
+        // mb_eregi -> eregi
+
+        yield 'It converts mb_eregi to eregi' => [
+            "<?php mb_eregi('#t#', 'test');",
+            "<?php\n\neregi('#t#', 'test');",
+        ];
+
+        yield 'It converts mb_eregi with regs parameter to eregi' => [
+            "<?php mb_eregi('#t#', 'test', \$regs);",
+            "<?php\n\neregi('#t#', 'test', \$regs);",
+        ];
+
+        // mb_ord -> ord
+
+        yield 'It converts mb_ord to ord' => [
+            "<?php mb_ord('T');",
+            "<?php\n\nord('T');",
+        ];
+
+        yield 'It converts mb_ord with encoding to ord' => [
+            "<?php mb_ord('T', 'utf-8');",
+            "<?php\n\nord('T');",
+        ];
+
+        // mb_parse_str -> parse_str
+
+        yield 'It converts mb_parse_str to parse_str' => [
+            "<?php mb_parse_str('T');",
+            "<?php\n\nparse_str('T');",
+        ];
+
+        yield 'It converts mb_parse_str with params argument to parse_str' => [
+            "<?php mb_parse_str('T', \$params);",
+            "<?php\n\nparse_str('T', \$params);",
+        ];
+
+        // mb_send_mail -> mail
+
+        yield 'It converts mb_send_mail to mail' => [
+            "<?php mb_send_mail('to', 'subject', 'msg');",
+            "<?php\n\nmail('to', 'subject', 'msg');",
+        ];
+
+        yield 'It converts mb_send_mail with additional parameters to mail' => [
+            "<?php mb_send_mail('to', 'subject', 'msg', [], []);",
+            "<?php\n\nmail('to', 'subject', 'msg', [], []);",
+        ];
+
+        // mb_split -> split
+
+        yield 'It converts mb_split to split' => [
+            "<?php mb_split('to', 'subject');",
+            "<?php\n\nsplit('to', 'subject');",
+        ];
+
+        yield 'It converts mb_split with limit parameter to split' => [
+            "<?php mb_split('to', 'subject', 2);",
+            "<?php\n\nsplit('to', 'subject', 2);",
+        ];
+
+        // mb_strcut -> substr
+
+        yield 'It converts mb_strcut to substr' => [
+            "<?php mb_strcut('subject', 1);",
+            "<?php\n\nsubstr('subject', 1);",
+        ];
+
+        yield 'It converts mb_strcut with limit to substr' => [
+            "<?php mb_strcut('subject', 1, 20);",
+            "<?php\n\nsubstr('subject', 1, 20);",
+        ];
+
+        yield 'It converts mb_strcut with encoding to substr' => [
+            "<?php mb_strcut('subject', 1, 20, 'utf-8');",
+            "<?php\n\nsubstr('subject', 1, 20);",
+        ];
+
+        // mb_strpos -> strpos
+
+        yield 'It converts mb_strpos to strpos' => [
+            "<?php mb_strpos('subject', 'b');",
+            "<?php\n\nstrpos('subject', 'b');",
+        ];
+
+        yield 'It converts mb_strpos with offset to strpos' => [
+            "<?php mb_strpos('subject', 'b', 3);",
+            "<?php\n\nstrpos('subject', 'b', 3);",
+        ];
+
+        yield 'It converts mb_strpos with encoding to strpos' => [
+            "<?php mb_strpos('subject', 'b', 3, 'utf-8');",
+            "<?php\n\nstrpos('subject', 'b', 3);",
+        ];
+
+        // mb_stripos -> stripos
+
+        yield 'It converts mb_stripos to stripos' => [
+            "<?php mb_stripos('subject', 'b');",
+            "<?php\n\nstripos('subject', 'b');",
+        ];
+
+        yield 'It converts mb_stripos with offset to stripos' => [
+            "<?php mb_stripos('subject', 'b', 3);",
+            "<?php\n\nstripos('subject', 'b', 3);",
+        ];
+
+        yield 'It converts mb_stripos with encoding to stripos' => [
+            "<?php mb_stripos('subject', 'b', 3, 'utf-8');",
+            "<?php\n\nstripos('subject', 'b', 3);",
+        ];
+
+        // mb_stristr -> stristr
+
+        yield 'It converts mb_stristr to stristr' => [
+            "<?php mb_stristr('subject', 'b');",
+            "<?php\n\nstristr('subject', 'b');",
+        ];
+
+        yield 'It converts mb_stristr with part argument to stristr' => [
+            "<?php mb_stristr('subject', 'b', false);",
+            "<?php\n\nstristr('subject', 'b', false);",
+        ];
+
+        yield 'It converts mb_stristr with encoding to stristr' => [
+            "<?php mb_stristr('subject', 'b', false, 'utf-8');",
+            "<?php\n\nstristr('subject', 'b', false);",
+        ];
+
+        // mb_strripos -> strripos
+
+        yield 'It converts mb_strripos to strripos' => [
+            "<?php mb_strripos('subject', 'b');",
+            "<?php\n\nstrripos('subject', 'b');",
+        ];
+
+        yield 'It converts mb_strripos with offset argument to strripos' => [
+            "<?php mb_strripos('subject', 'b', 2);",
+            "<?php\n\nstrripos('subject', 'b', 2);",
+        ];
+
+        yield 'It converts mb_strripos with encoding to strripos' => [
+            "<?php mb_strripos('subject', 'b', 2, 'utf-8');",
+            "<?php\n\nstrripos('subject', 'b', 2);",
+        ];
+
+        // mb_strrpos -> strrpos
+
+        yield 'It converts mb_strrpos to strrpos' => [
+            "<?php mb_strrpos('subject', 'b');",
+            "<?php\n\nstrrpos('subject', 'b');",
+        ];
+
+        yield 'It converts mb_strrpos with offset argument to strrpos' => [
+            "<?php mb_strrpos('subject', 'b', 2);",
+            "<?php\n\nstrrpos('subject', 'b', 2);",
+        ];
+
+        yield 'It converts mb_strrpos with encoding to strrpos' => [
+            "<?php mb_strrpos('subject', 'b', 2, 'utf-8');",
+            "<?php\n\nstrrpos('subject', 'b', 2);",
+        ];
+
+        // mb_strstr -> strstr
+
+        yield 'It converts mb_strstr to strstr' => [
+            "<?php mb_strstr('subject', 'b');",
+            "<?php\n\nstrstr('subject', 'b');",
+        ];
+
+        yield 'It converts mb_strstr with part argument to strstr' => [
+            "<?php mb_strstr('subject', 'b', false);",
+            "<?php\n\nstrstr('subject', 'b', false);",
+        ];
+
+        yield 'It converts mb_strstr with encoding to strstr' => [
+            "<?php mb_strstr('subject', 'b', false, 'utf-8');",
+            "<?php\n\nstrstr('subject', 'b', false);",
+        ];
+
+        // mb_strtolower -> strtolower
+
+        yield 'It converts mb_strtolower to strtolower' => [
+            "<?php mb_strtolower('test');",
+            "<?php\n\nstrtolower('test');",
+        ];
+
+        yield 'It converts mb_strtolower with encoding to strtolower' => [
+            "<?php mb_strtolower('test', 'utf-8');",
+            "<?php\n\nstrtolower('test');",
+        ];
+
+        // mb_strtoupper -> strtoupper
+
+        yield 'It converts mb_strtoupper to strtoupper' => [
+            "<?php mb_strtoupper('test');",
+            "<?php\n\nstrtoupper('test');",
+        ];
+
+        yield 'It converts mb_strtoupper with encoding to strtoupper' => [
+            "<?php mb_strtoupper('test', 'utf-8');",
+            "<?php\n\nstrtoupper('test');",
+        ];
+
+        // mb_substr_count -> substr_count
+
+        yield 'It converts mb_substr_count to substr_count' => [
+            "<?php mb_substr_count('test', 't');",
+            "<?php\n\nsubstr_count('test', 't');",
+        ];
+
+        yield 'It converts mb_substr_count with encoding to substr_count' => [
+            "<?php mb_substr_count('test', 't', 'utf-8');",
+            "<?php\n\nsubstr_count('test', 't');",
+        ];
+
+        // mb_substr -> substr
+
+        yield 'It converts mb_substr to substr' => [
+            "<?php mb_substr('test', 2);",
+            "<?php\n\nsubstr('test', 2);",
+        ];
+
+        yield 'It converts mb_substr with length argument to substr' => [
+            "<?php mb_substr('test', 2, 10);",
+            "<?php\n\nsubstr('test', 2, 10);",
+        ];
+
+        yield 'It converts mb_substr with encoding argument to substr' => [
+            "<?php mb_substr('test', 2, 10, 'utf-8');",
+            "<?php\n\nsubstr('test', 2, 10);",
+        ];
+
+        // mb_strrchr -> strrchr
+
+        yield 'It converts mb_strrchr to strrchr' => [
+            "<?php mb_strrchr('subject', 'b');",
+            "<?php\n\nstrrchr('subject', 'b');",
+        ];
+
+        yield 'It converts mb_strrchr with part argument to strrchr' => [
+            "<?php mb_strrchr('subject', 'b', false);",
+            "<?php\n\nstrrchr('subject', 'b');",
+        ];
+
+        yield 'It converts mb_strrchr with encoding to strrchr' => [
+            "<?php mb_strrchr('subject', 'b', false, 'utf-8');",
+            "<?php\n\nstrrchr('subject', 'b');",
+        ];
+
+        // mb_strrichr -> strrchr
+
+        yield 'It converts mb_strrichr to strrchr' => [
+            "<?php mb_strrichr('subject', 'b');",
+            "<?php\n\nstrrchr('subject', 'b');",
+        ];
+
+        yield 'It converts mb_strrichr with part argument to strrchr' => [
+            "<?php mb_strrichr('subject', 'b', false);",
+            "<?php\n\nstrrchr('subject', 'b');",
+        ];
+
+        yield 'It converts mb_strrichr with encoding to strrchr' => [
+            "<?php mb_strrichr('subject', 'b', false, 'utf-8');",
+            "<?php\n\nstrrchr('subject', 'b');",
+        ];
+
+        // mb_convert_case
+
+        yield 'It converts mb_convert_case with MB_CASE_UPPER mode to strtoupper' => [
+            "<?php mb_convert_case('test', MB_CASE_UPPER);",
+            "<?php\n\nstrtoupper('test');",
+        ];
+
+        yield 'It converts mb_convert_case with MB_CASE_UPPER_SIMPLE mode to strtoupper' => [
+            "<?php mb_convert_case('test', MB_CASE_UPPER_SIMPLE);",
+            "<?php\n\nstrtoupper('test');",
+        ];
+
+        yield 'It converts mb_convert_case with MB_CASE_LOWER mode to strtolower' => [
+            "<?php mb_convert_case('test', E_ERROR);",
+            "<?php\n\nstrtolower('test');",
+        ];
+
+        yield 'It converts mb_convert_case with MB_CASE_LOWER_SIMPLE mode to strtolower' => [
+            "<?php mb_convert_case('test', MB_CASE_LOWER);",
+            "<?php\n\nstrtolower('test');",
+        ];
+
+        yield 'It converts mb_convert_case with MB_CASE_TITLE mode to ucwords' => [
+            "<?php mb_convert_case('test', MB_CASE_TITLE);",
+            "<?php\n\nucwords('test');",
+        ];
+
+        yield 'It converts mb_convert_case with MB_CASE_TITLE_SIMPLE mode to ucwords' => [
+            "<?php mb_convert_case('test', \MB_CASE_TITLE_SIMPLE);",
+            "<?php\n\nucwords('test');",
+        ];
+
+        yield 'It converts mb_convert_case with MB_CASE_FOLD mode to strtolower' => [
+            "<?php mb_convert_case('test', MB_CASE_FOLD);",
+            "<?php\n\nstrtolower('test');",
+        ];
+
+        yield 'It converts mb_convert_case with MB_CASE_FOLD_SIMPLE mode to strtolower' => [
+            "<?php mb_convert_case('test', MB_CASE_FOLD_SIMPLE);",
+            "<?php\n\nstrtolower('test');",
+        ];
+
+        yield 'It does not convert mb_convert_case with mode as variable' => [
+            "<?php mb_convert_case('test', \$mode);",
+        ];
+
+        yield 'It does not convert mb_convert_case with missing mode argument' => [
+            "<?php mb_convert_case('test');",
+        ];
+    }
+}

From 66883f7c505bea731550bcd69bf4c95c8649414b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Sun, 10 Mar 2019 13:44:08 +0100
Subject: [PATCH 02/18] #654 mbstring mutator configuration

---
 src/Mutator/Extensions/MBString.php       | 15 ++++++++++++---
 tests/Mutator/Extensions/MBStringTest.php | 11 +++++++++--
 2 files changed, 21 insertions(+), 5 deletions(-)

diff --git a/src/Mutator/Extensions/MBString.php b/src/Mutator/Extensions/MBString.php
index b4b0742ac..3dd3e686a 100644
--- a/src/Mutator/Extensions/MBString.php
+++ b/src/Mutator/Extensions/MBString.php
@@ -61,7 +61,10 @@ final class MBString extends Mutator
     public function __construct(MutatorConfig $config)
     {
         parent::__construct($config);
-        $this->setupConverters();
+
+        $settings = $this->getSettings();
+
+        $this->setupConverters($settings);
     }
 
     /**
@@ -77,9 +80,9 @@ protected function mutatesNode(Node $node): bool
         return isset($this->converters[$this->getFunctionName($node)]);
     }
 
-    private function setupConverters(): void
+    private function setupConverters(array $functionsMap): void
     {
-        $this->converters = [
+        $converters = [
             'mb_chr' => $this->mapNameSkipArg('chr', 1),
             'mb_ereg_match' => $this->mapNameSkipArg('preg_match', 2),
             'mb_ereg_replace_callback' => $this->mapNameSkipArg('preg_replace_callback', 3),
@@ -107,6 +110,12 @@ private function setupConverters(): void
             'mb_substr' => $this->mapNameSkipArg('substr', 3),
             'mb_convert_case' => $this->mapConvertCase(),
         ];
+
+        $functionsToRemove = \array_filter($functionsMap, function ($isOn) {
+            return !$isOn;
+        });
+
+        $this->converters = \array_diff_key($converters, $functionsToRemove);
     }
 
     private function mapName(string $functionName): callable
diff --git a/tests/Mutator/Extensions/MBStringTest.php b/tests/Mutator/Extensions/MBStringTest.php
index 2ba851173..f8180700f 100644
--- a/tests/Mutator/Extensions/MBStringTest.php
+++ b/tests/Mutator/Extensions/MBStringTest.php
@@ -45,9 +45,9 @@ final class MBStringTest extends AbstractMutatorTestCase
     /**
      * @dataProvider provideMutationCases
      */
-    public function test_mutator(string $input, string $expected = null): void
+    public function test_mutator(string $input, string $expected = null, array $settings = []): void
     {
-        $this->doTest($input, $expected);
+        $this->doTest($input, $expected, $settings);
     }
 
     public function provideMutationCases(): \Generator
@@ -69,6 +69,13 @@ public function provideMutationCases(): \Generator
         yield 'It converts mb_strlen with encoding to strlen' => [
             "<?php mb_strlen('test', 'utf-8');",
             "<?php\n\nstrlen('test');",
+            ['settings' => ['mb_strlen' => true]],
+        ];
+
+        yield 'It does not convert mb_strlen when disabled' => [
+            "<?php mb_strlen('test');",
+            null,
+            ['settings' => ['mb_strlen' => false]],
         ];
 
         // mb_chr-> chr

From 150ee972ed69fed6bdafe3ff23854ca97234a2c8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Sun, 10 Mar 2019 14:04:16 +0100
Subject: [PATCH 03/18] #654 fix code analysis issues

---
 src/Mutator/Extensions/MBString.php | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/src/Mutator/Extensions/MBString.php b/src/Mutator/Extensions/MBString.php
index 3dd3e686a..adf45ea9c 100644
--- a/src/Mutator/Extensions/MBString.php
+++ b/src/Mutator/Extensions/MBString.php
@@ -128,13 +128,7 @@ private function mapName(string $functionName): callable
     private function mapNameSkipArg(string $functionName, int $skipArgs): callable
     {
         return function (Node\Expr\FuncCall $node) use ($functionName, $skipArgs): Generator {
-            $args = $node->args;
-
-            if ($skipArgs !== null) {
-                $args = \array_slice($args, 0, $skipArgs);
-            }
-
-            yield $this->createNode($node, $functionName, $args);
+            yield $this->createNode($node, $functionName, \array_slice($node->args, 0, $skipArgs));
         };
     }
 
@@ -165,8 +159,6 @@ private function getConvertCaseModeValue(Node\Expr\FuncCall $node): ?int
 
         $mode = $node->args[1]->value;
 
-        $modeValue = null;
-
         if ($mode instanceof Node\Expr\ConstFetch) {
             $modeName = $mode->name->toString();
 

From 97d64bc69a5fade49403fede6f5cfe11bcbc4a35 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Sun, 10 Mar 2019 14:09:08 +0100
Subject: [PATCH 04/18] #654 update json scheme for MBString mutator

---
 resources/schema.json | 99 +++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 99 insertions(+)

diff --git a/resources/schema.json b/resources/schema.json
index 5a37a82f4..5cfec4967 100644
--- a/resources/schema.json
+++ b/resources/schema.json
@@ -144,6 +144,105 @@
                             }
                         ]
                     }
+                },
+                "MBString": {
+                    "type": {
+                        "anyOf": [
+                            {
+                                "type": "boolean"
+                            },
+                            {
+                                "type": "object",
+                                "additionalProperties": false,
+                                "properties": {
+                                    "settings": {
+                                        "type": "object",
+                                        "additionalProperties": false,
+                                        "properties": {
+                                            "mb_chr": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_ereg_match": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_ereg_replace_callback": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_ereg_replace": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_ereg": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_eregi_replace": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_eregi": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_ord": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_parse_str": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_send_mail": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_split": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_strcut": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_stripos": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_stristr": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_strlen": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_strpos": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_strrchr": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_strrichr": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_strripos": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_strrpos": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_strstr": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_strtolower": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_strtoupper": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_substr_count": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_substr": {
+                                                "type": "boolean"
+                                            },
+                                            "mb_convert_case": {
+                                                "type": "boolean"
+                                            }
+                                        }
+                                    }
+                                }
+                            }
+                        ]
+                    }
                 }
             }
         },

From 45269900f3a63f4938db6102f7f7a4ff51693dab Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Sun, 10 Mar 2019 14:16:19 +0100
Subject: [PATCH 05/18] #654 fix spaceing

---
 src/Mutator/Extensions/MBString.php | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/Mutator/Extensions/MBString.php b/src/Mutator/Extensions/MBString.php
index adf45ea9c..287e851f5 100644
--- a/src/Mutator/Extensions/MBString.php
+++ b/src/Mutator/Extensions/MBString.php
@@ -111,7 +111,7 @@ private function setupConverters(array $functionsMap): void
             'mb_convert_case' => $this->mapConvertCase(),
         ];
 
-        $functionsToRemove = \array_filter($functionsMap, function ($isOn) {
+        $functionsToRemove = \array_filter($functionsMap, function($isOn) {
             return !$isOn;
         });
 
@@ -120,21 +120,21 @@ private function setupConverters(array $functionsMap): void
 
     private function mapName(string $functionName): callable
     {
-        return function (Node\Expr\FuncCall $node) use ($functionName): Generator {
+        return function(Node\Expr\FuncCall $node) use ($functionName): Generator {
             yield $this->createNode($node, $functionName, $node->args);
         };
     }
 
     private function mapNameSkipArg(string $functionName, int $skipArgs): callable
     {
-        return function (Node\Expr\FuncCall $node) use ($functionName, $skipArgs): Generator {
+        return function(Node\Expr\FuncCall $node) use ($functionName, $skipArgs): Generator {
             yield $this->createNode($node, $functionName, \array_slice($node->args, 0, $skipArgs));
         };
     }
 
     private function mapConvertCase(): callable
     {
-        return function (Node\Expr\FuncCall $node): Generator {
+        return function(Node\Expr\FuncCall $node): Generator {
             $modeValue = $this->getConvertCaseModeValue($node);
 
             if ($modeValue === null) {

From 733b40e66b4e20d2c933f449f23465a62bbc28c6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Sun, 10 Mar 2019 18:17:16 +0100
Subject: [PATCH 06/18] #654 remove mutants

---
 src/Mutator/Extensions/MBString.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/Mutator/Extensions/MBString.php b/src/Mutator/Extensions/MBString.php
index 287e851f5..0fe91e649 100644
--- a/src/Mutator/Extensions/MBString.php
+++ b/src/Mutator/Extensions/MBString.php
@@ -87,9 +87,9 @@ private function setupConverters(array $functionsMap): void
             'mb_ereg_match' => $this->mapNameSkipArg('preg_match', 2),
             'mb_ereg_replace_callback' => $this->mapNameSkipArg('preg_replace_callback', 3),
             'mb_ereg_replace' => $this->mapNameSkipArg('ereg_replace', 3),
-            'mb_ereg' => $this->mapNameSkipArg('ereg', 3),
+            'mb_ereg' => $this->mapName('ereg'),
             'mb_eregi_replace' => $this->mapNameSkipArg('eregi_replace', 3),
-            'mb_eregi' => $this->mapNameSkipArg('eregi', 3),
+            'mb_eregi' => $this->mapName('eregi'),
             'mb_ord' => $this->mapNameSkipArg('ord', 1),
             'mb_parse_str' => $this->mapName('parse_str'),
             'mb_send_mail' => $this->mapName('mail'),

From f6f7ff4a6e4659cef5130afc95a5ee95039ecdeb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Wed, 13 Mar 2019 00:58:49 +0100
Subject: [PATCH 07/18] #654 drop ereg functions in favoure of preg functions

---
 src/Mutator/Extensions/MBString.php       |  56 +++++-
 tests/Mutator/Extensions/MBStringTest.php | 224 +++++++++++++++-------
 2 files changed, 200 insertions(+), 80 deletions(-)

diff --git a/src/Mutator/Extensions/MBString.php b/src/Mutator/Extensions/MBString.php
index 0fe91e649..9018e1598 100644
--- a/src/Mutator/Extensions/MBString.php
+++ b/src/Mutator/Extensions/MBString.php
@@ -84,12 +84,12 @@ private function setupConverters(array $functionsMap): void
     {
         $converters = [
             'mb_chr' => $this->mapNameSkipArg('chr', 1),
-            'mb_ereg_match' => $this->mapNameSkipArg('preg_match', 2),
-            'mb_ereg_replace_callback' => $this->mapNameSkipArg('preg_replace_callback', 3),
-            'mb_ereg_replace' => $this->mapNameSkipArg('ereg_replace', 3),
-            'mb_ereg' => $this->mapName('ereg'),
-            'mb_eregi_replace' => $this->mapNameSkipArg('eregi_replace', 3),
-            'mb_eregi' => $this->mapName('eregi'),
+            'mb_ereg_match' => $this->mapEreg($this->mapNameSkipArg('preg_match', 2), '^', '', [$this, 'warpEqOne']),
+            'mb_ereg_replace_callback' => $this->mapEreg($this->mapNameSkipArg('preg_replace_callback', 3)),
+            'mb_ereg_replace' => $this->mapEreg($this->mapNameSkipArg('preg_replace', 3)),
+            'mb_ereg' => $this->mapEreg($this->mapName('preg_match'), '', '', [$this, 'warpTernary']),
+            'mb_eregi_replace' => $this->mapEreg($this->mapNameSkipArg('preg_replace', 3), '', 'i'),
+            'mb_eregi' => $this->mapEreg($this->mapName('preg_match'), '', 'i', [$this, 'warpTernary']),
             'mb_ord' => $this->mapNameSkipArg('ord', 1),
             'mb_parse_str' => $this->mapName('parse_str'),
             'mb_send_mail' => $this->mapName('mail'),
@@ -132,6 +132,50 @@ private function mapNameSkipArg(string $functionName, int $skipArgs): callable
         };
     }
 
+    private function mapEreg(callable $baseConverter, string $prefix = '', string $suffix = '', callable $warp = null): callable
+    {
+        return function (Node\Expr\FuncCall $node) use ($baseConverter, $prefix, $suffix, $warp): Generator {
+            foreach ($baseConverter($node) as $newNode) {
+                /* @var Node\Expr\FuncCall $newNode */
+                $newNode->args[0] = new Node\Arg(
+                    new Node\Expr\BinaryOp\Concat(
+                        new Node\Expr\BinaryOp\Concat(
+                            new Node\Scalar\String_("/$prefix"),
+                            new Node\Expr\FuncCall(
+                                new Node\Name('\str_replace'),
+                                [
+                                    new Node\Arg(new Node\Scalar\String_('/')),
+                                    new Node\Arg(new Node\Scalar\String_('\/')),
+                                    new Node\Arg($newNode->args[0]->value),
+                                ]
+                            )
+                        ),
+                        new Node\Scalar\String_("/$suffix")
+                    )
+                );
+
+                yield $warp ? $warp($newNode) : $newNode;
+            }
+        };
+    }
+
+    private function warpEqOne(Node\Expr\FuncCall $node): Node
+    {
+        return new Node\Expr\BinaryOp\Identical(
+            $node,
+            new Node\Scalar\LNumber(1)
+        );
+    }
+
+    private function warpTernary(Node\Expr\FuncCall $node): Node
+    {
+        return new Node\Expr\Ternary(
+            $node,
+            new Node\Scalar\LNumber(1),
+            new Node\Expr\ConstFetch(new Node\Name('false'))
+        );
+    }
+
     private function mapConvertCase(): callable
     {
         return function(Node\Expr\FuncCall $node): Generator {
diff --git a/tests/Mutator/Extensions/MBStringTest.php b/tests/Mutator/Extensions/MBStringTest.php
index f8180700f..451190711 100644
--- a/tests/Mutator/Extensions/MBStringTest.php
+++ b/tests/Mutator/Extensions/MBStringTest.php
@@ -35,6 +35,7 @@
 
 namespace Infection\Tests\Mutator\Extensions;
 
+use Generator;
 use Infection\Tests\Mutator\AbstractMutatorTestCase;
 
 /**
@@ -50,7 +51,7 @@ public function test_mutator(string $input, string $expected = null, array $sett
         $this->doTest($input, $expected, $settings);
     }
 
-    public function provideMutationCases(): \Generator
+    public function provideMutationCases(): Generator
     {
         yield 'It converts mb_strlen with leading slash' => [
             "<?php \mb_strlen('test');",
@@ -78,8 +79,59 @@ public function provideMutationCases(): \Generator
             ['settings' => ['mb_strlen' => false]],
         ];
 
-        // mb_chr-> chr
+        yield from $this->provideMutationCasesForChr();
 
+        yield from $this->provideMutationCasesForEregMatch();
+
+        yield from $this->provideMutationCasesForEregReplaceCallback();
+
+        yield from $this->provideMutationCasesForEregReplace();
+
+        yield from $this->provideMutationCasesForEreg();
+
+        yield from $this->provideMutationCasesForEregiReplace();
+
+        yield from $this->provideMutationCasesForEregi();
+
+        yield from $this->provideMutationCasesForOrd();
+
+        yield from $this->provideMutationCasesForParseStr();
+
+        yield from $this->provideMutationCasesForSendMail();
+
+        yield from $this->provideMutationCasesForSplit();
+
+        yield from $this->provideMutationCasesForStrCut();
+
+        yield from $this->provideMutationCasesForStrPos();
+
+        yield from $this->provideMutationCasesForStrIPos();
+
+        yield from $this->provideMutationCasesForStrIStr();
+
+        yield from $this->provideMutationCasesForStrRiPos();
+
+        yield from $this->provideMutationCasesForStrRPos();
+
+        yield from $this->provideMutationCasesForStrStr();
+
+        yield from $this->provideMutationCasesForStrToLower();
+
+        yield from $this->provideMutationCasesForStrToUpper();
+
+        yield from $this->provideMutationCasesForSubStrCount();
+
+        yield from $this->provideMutationCasesForSubStr();
+
+        yield from $this->provideMutationCasesForStrRChr();
+
+        yield from $this->provideMutationCasesForStrRiChr();
+
+        yield from $this->provideMutationCasesForConvertCase();
+    }
+
+    private function provideMutationCasesForChr(): Generator
+    {
         yield 'It converts mb_chr to chr' => [
             '<?php mb_chr(74);',
             "<?php\n\nchr(74);",
@@ -89,81 +141,88 @@ public function provideMutationCases(): \Generator
             "<?php mb_chr(74, 'utf-8');",
             "<?php\n\nchr(74);",
         ];
+    }
 
-        // mb_ereg_match -> preg_match
-
+    private function provideMutationCasesForEregMatch(): Generator
+    {
         yield 'It converts mb_ereg_match to preg_match' => [
-            "<?php mb_ereg_match('#t#', 'test');",
-            "<?php\n\npreg_match('#t#', 'test');",
+            "<?php mb_ereg_match(M_2_PI, 'test');",
+            "<?php\n\npreg_match('/^' . \str_replace('/', '\\\\/', M_2_PI) . '/', 'test') === 1;",
         ];
 
         yield 'It converts mb_ereg_match with encoding argument to preg_match' => [
-            "<?php mb_ereg_match('#t#', 'trim', 'mb');",
-            "<?php\n\npreg_match('#t#', 'trim');",
+            "<?php mb_ereg_match(M_2_PI, 'trim', 'mb');",
+            "<?php\n\npreg_match('/^' . \str_replace('/', '\\\\/', M_2_PI) . '/', 'trim') === 1;",
         ];
+    }
 
-        // mb_ereg_replace_callback -> preg_replace_callback
-
+    private function provideMutationCasesForEregReplaceCallback(): Generator
+    {
         yield 'It converts mb_ereg_replace_callback to preg_replace_callback' => [
-            "<?php mb_ereg_replace_callback('#t#', 'trim', 'test');",
-            "<?php\n\npreg_replace_callback('#t#', 'trim', 'test');",
+            "<?php mb_ereg_replace_callback('[A-Z/#]', 'trim', 'test');",
+            "<?php\n\npreg_replace_callback('/' . \str_replace('/', '\\\\/', '[A-Z/#]') . '/', 'trim', 'test');",
         ];
 
         yield 'It converts mb_ereg_replace_callback with options to preg_replace_callback' => [
-            "<?php mb_ereg_replace_callback('#t#', 'trim', 'test', 'msr');",
-            "<?php\n\npreg_replace_callback('#t#', 'trim', 'test');",
+            "<?php mb_ereg_replace_callback('[A-Z/#]', 'trim', 'test', 'msr');",
+            "<?php\n\npreg_replace_callback('/' . \str_replace('/', '\\\\/', '[A-Z/#]') . '/', 'trim', 'test');",
         ];
+    }
 
-        // mb_ereg_replace -> ereg_replace
-
+    private function provideMutationCasesForEregReplace(): Generator
+    {
         yield 'It converts mb_ereg_replace to ereg_replace' => [
-            "<?php mb_ereg_replace('#t#', 'T', 'test');",
-            "<?php\n\nereg_replace('#t#', 'T', 'test');",
+            "<?php mb_ereg_replace(\$pattern, 'T', 'test');",
+            "<?php\n\npreg_replace('/' . \str_replace('/', '\\\\/', \$pattern) . '/', 'T', 'test');",
         ];
 
         yield 'It converts mb_ereg_replace with options to ereg_replace' => [
-            "<?php mb_ereg_replace('#t#', 'T', 'test', 'mb');",
-            "<?php\n\nereg_replace('#t#', 'T', 'test');",
+            "<?php mb_ereg_replace(\$pattern, 'T', 'test', 'mb');",
+            "<?php\n\npreg_replace('/' . \str_replace('/', '\\\\/', \$pattern) . '/', 'T', 'test');",
         ];
+    }
 
-        // mb_ereg -> ereg
-
+    private function provideMutationCasesForEreg(): Generator
+    {
         yield 'It converts mb_ereg to ereg' => [
-            "<?php mb_ereg('#t#', 'test');",
-            "<?php\n\nereg('#t#', 'test');",
+            "<?php mb_ereg(getPattern(\$x, 1), 'test');",
+            "<?php\n\npreg_match('/' . \str_replace('/', '\\\\/', getPattern(\$x, 1)) . '/', 'test') ? 1 : false;",
         ];
 
         yield 'It converts mb_ereg with regs argument to ereg' => [
-            "<?php mb_ereg('#t#', 'test', \$regs);",
-            "<?php\n\nereg('#t#', 'test', \$regs);",
+            "<?php mb_ereg(getPattern(\$x, 1), 'test', \$regs);",
+            "<?php\n\npreg_match('/' . \str_replace('/', '\\\\/', getPattern(\$x, 1)) . '/', 'test', \$regs) ? 1 : false;",
         ];
+    }
 
-        // mb_eregi_replace -> eregi_replace
-
+    private function provideMutationCasesForEregiReplace(): Generator
+    {
         yield 'It converts mb_eregi_replace to eregi_replace' => [
-            "<?php mb_eregi_replace('#t#', 'T', 'test');",
-            "<?php\n\neregi_replace('#t#', 'T', 'test');",
+            "<?php mb_eregi_replace(\$a . \$b, 'T', 'test');",
+            "<?php\n\npreg_replace('/' . \str_replace('/', '\\\\/', \$a . \$b) . '/i', 'T', 'test');",
         ];
 
         yield 'It converts mb_eregi_replace with options to eregi_replace' => [
-            "<?php mb_eregi_replace('#t#', 'T', 'test', 'msr');",
-            "<?php\n\neregi_replace('#t#', 'T', 'test');",
+            "<?php mb_eregi_replace(\$a . \$b, 'T', 'test', 'msr');",
+            "<?php\n\npreg_replace('/' . \str_replace('/', '\\\\/', \$a . \$b) . '/i', 'T', 'test');",
         ];
+    }
 
-        // mb_eregi -> eregi
-
+    private function provideMutationCasesForEregi(): Generator
+    {
         yield 'It converts mb_eregi to eregi' => [
-            "<?php mb_eregi('#t#', 'test');",
-            "<?php\n\neregi('#t#', 'test');",
+            "<?php mb_eregi(\DateTime::getLastErrors(), 'test');",
+            "<?php\n\npreg_match('/' . \str_replace('/', '\\\\/', \DateTime::getLastErrors()) . '/i', 'test') ? 1 : false;",
         ];
 
         yield 'It converts mb_eregi with regs parameter to eregi' => [
-            "<?php mb_eregi('#t#', 'test', \$regs);",
-            "<?php\n\neregi('#t#', 'test', \$regs);",
+            "<?php mb_eregi(\DateTime::getLastErrors(), 'test', \$regs);",
+            "<?php\n\npreg_match('/' . \str_replace('/', '\\\\/', \DateTime::getLastErrors()) . '/i', 'test', \$regs) ? 1 : false;",
         ];
+    }
 
-        // mb_ord -> ord
-
+    private function provideMutationCasesForOrd(): Generator
+    {
         yield 'It converts mb_ord to ord' => [
             "<?php mb_ord('T');",
             "<?php\n\nord('T');",
@@ -173,9 +232,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_ord('T', 'utf-8');",
             "<?php\n\nord('T');",
         ];
+    }
 
-        // mb_parse_str -> parse_str
-
+    private function provideMutationCasesForParseStr(): Generator
+    {
         yield 'It converts mb_parse_str to parse_str' => [
             "<?php mb_parse_str('T');",
             "<?php\n\nparse_str('T');",
@@ -185,9 +245,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_parse_str('T', \$params);",
             "<?php\n\nparse_str('T', \$params);",
         ];
+    }
 
-        // mb_send_mail -> mail
-
+    private function provideMutationCasesForSendMail(): Generator
+    {
         yield 'It converts mb_send_mail to mail' => [
             "<?php mb_send_mail('to', 'subject', 'msg');",
             "<?php\n\nmail('to', 'subject', 'msg');",
@@ -197,9 +258,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_send_mail('to', 'subject', 'msg', [], []);",
             "<?php\n\nmail('to', 'subject', 'msg', [], []);",
         ];
+    }
 
-        // mb_split -> split
-
+    private function provideMutationCasesForSplit(): Generator
+    {
         yield 'It converts mb_split to split' => [
             "<?php mb_split('to', 'subject');",
             "<?php\n\nsplit('to', 'subject');",
@@ -209,9 +271,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_split('to', 'subject', 2);",
             "<?php\n\nsplit('to', 'subject', 2);",
         ];
+    }
 
-        // mb_strcut -> substr
-
+    private function provideMutationCasesForStrCut(): Generator
+    {
         yield 'It converts mb_strcut to substr' => [
             "<?php mb_strcut('subject', 1);",
             "<?php\n\nsubstr('subject', 1);",
@@ -226,9 +289,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_strcut('subject', 1, 20, 'utf-8');",
             "<?php\n\nsubstr('subject', 1, 20);",
         ];
+    }
 
-        // mb_strpos -> strpos
-
+    private function provideMutationCasesForStrPos(): Generator
+    {
         yield 'It converts mb_strpos to strpos' => [
             "<?php mb_strpos('subject', 'b');",
             "<?php\n\nstrpos('subject', 'b');",
@@ -243,9 +307,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_strpos('subject', 'b', 3, 'utf-8');",
             "<?php\n\nstrpos('subject', 'b', 3);",
         ];
+    }
 
-        // mb_stripos -> stripos
-
+    private function provideMutationCasesForStrIPos(): Generator
+    {
         yield 'It converts mb_stripos to stripos' => [
             "<?php mb_stripos('subject', 'b');",
             "<?php\n\nstripos('subject', 'b');",
@@ -260,9 +325,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_stripos('subject', 'b', 3, 'utf-8');",
             "<?php\n\nstripos('subject', 'b', 3);",
         ];
+    }
 
-        // mb_stristr -> stristr
-
+    private function provideMutationCasesForStrIStr(): Generator
+    {
         yield 'It converts mb_stristr to stristr' => [
             "<?php mb_stristr('subject', 'b');",
             "<?php\n\nstristr('subject', 'b');",
@@ -277,9 +343,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_stristr('subject', 'b', false, 'utf-8');",
             "<?php\n\nstristr('subject', 'b', false);",
         ];
+    }
 
-        // mb_strripos -> strripos
-
+    private function provideMutationCasesForStrRiPos(): Generator
+    {
         yield 'It converts mb_strripos to strripos' => [
             "<?php mb_strripos('subject', 'b');",
             "<?php\n\nstrripos('subject', 'b');",
@@ -294,9 +361,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_strripos('subject', 'b', 2, 'utf-8');",
             "<?php\n\nstrripos('subject', 'b', 2);",
         ];
+    }
 
-        // mb_strrpos -> strrpos
-
+    private function provideMutationCasesForStrRPos(): Generator
+    {
         yield 'It converts mb_strrpos to strrpos' => [
             "<?php mb_strrpos('subject', 'b');",
             "<?php\n\nstrrpos('subject', 'b');",
@@ -311,9 +379,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_strrpos('subject', 'b', 2, 'utf-8');",
             "<?php\n\nstrrpos('subject', 'b', 2);",
         ];
+    }
 
-        // mb_strstr -> strstr
-
+    private function provideMutationCasesForStrStr(): Generator
+    {
         yield 'It converts mb_strstr to strstr' => [
             "<?php mb_strstr('subject', 'b');",
             "<?php\n\nstrstr('subject', 'b');",
@@ -328,9 +397,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_strstr('subject', 'b', false, 'utf-8');",
             "<?php\n\nstrstr('subject', 'b', false);",
         ];
+    }
 
-        // mb_strtolower -> strtolower
-
+    private function provideMutationCasesForStrToLower(): Generator
+    {
         yield 'It converts mb_strtolower to strtolower' => [
             "<?php mb_strtolower('test');",
             "<?php\n\nstrtolower('test');",
@@ -340,9 +410,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_strtolower('test', 'utf-8');",
             "<?php\n\nstrtolower('test');",
         ];
+    }
 
-        // mb_strtoupper -> strtoupper
-
+    private function provideMutationCasesForStrToUpper(): Generator
+    {
         yield 'It converts mb_strtoupper to strtoupper' => [
             "<?php mb_strtoupper('test');",
             "<?php\n\nstrtoupper('test');",
@@ -352,9 +423,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_strtoupper('test', 'utf-8');",
             "<?php\n\nstrtoupper('test');",
         ];
+    }
 
-        // mb_substr_count -> substr_count
-
+    private function provideMutationCasesForSubStrCount(): Generator
+    {
         yield 'It converts mb_substr_count to substr_count' => [
             "<?php mb_substr_count('test', 't');",
             "<?php\n\nsubstr_count('test', 't');",
@@ -364,9 +436,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_substr_count('test', 't', 'utf-8');",
             "<?php\n\nsubstr_count('test', 't');",
         ];
+    }
 
-        // mb_substr -> substr
-
+    private function provideMutationCasesForSubStr(): Generator
+    {
         yield 'It converts mb_substr to substr' => [
             "<?php mb_substr('test', 2);",
             "<?php\n\nsubstr('test', 2);",
@@ -381,9 +454,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_substr('test', 2, 10, 'utf-8');",
             "<?php\n\nsubstr('test', 2, 10);",
         ];
+    }
 
-        // mb_strrchr -> strrchr
-
+    private function provideMutationCasesForStrRChr(): Generator
+    {
         yield 'It converts mb_strrchr to strrchr' => [
             "<?php mb_strrchr('subject', 'b');",
             "<?php\n\nstrrchr('subject', 'b');",
@@ -398,9 +472,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_strrchr('subject', 'b', false, 'utf-8');",
             "<?php\n\nstrrchr('subject', 'b');",
         ];
+    }
 
-        // mb_strrichr -> strrchr
-
+    private function provideMutationCasesForStrRiChr(): Generator
+    {
         yield 'It converts mb_strrichr to strrchr' => [
             "<?php mb_strrichr('subject', 'b');",
             "<?php\n\nstrrchr('subject', 'b');",
@@ -415,9 +490,10 @@ public function provideMutationCases(): \Generator
             "<?php mb_strrichr('subject', 'b', false, 'utf-8');",
             "<?php\n\nstrrchr('subject', 'b');",
         ];
+    }
 
-        // mb_convert_case
-
+    private function provideMutationCasesForConvertCase(): Generator
+    {
         yield 'It converts mb_convert_case with MB_CASE_UPPER mode to strtoupper' => [
             "<?php mb_convert_case('test', MB_CASE_UPPER);",
             "<?php\n\nstrtoupper('test');",

From 66433097d5531d3a1a5ee70a37aaf4dcce2bb5cb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Wed, 13 Mar 2019 01:08:10 +0100
Subject: [PATCH 08/18] #654 fix styles

---
 src/Mutator/Extensions/MBString.php | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/Mutator/Extensions/MBString.php b/src/Mutator/Extensions/MBString.php
index 9018e1598..633630c85 100644
--- a/src/Mutator/Extensions/MBString.php
+++ b/src/Mutator/Extensions/MBString.php
@@ -111,7 +111,7 @@ private function setupConverters(array $functionsMap): void
             'mb_convert_case' => $this->mapConvertCase(),
         ];
 
-        $functionsToRemove = \array_filter($functionsMap, function($isOn) {
+        $functionsToRemove = \array_filter($functionsMap, function ($isOn) {
             return !$isOn;
         });
 
@@ -120,21 +120,21 @@ private function setupConverters(array $functionsMap): void
 
     private function mapName(string $functionName): callable
     {
-        return function(Node\Expr\FuncCall $node) use ($functionName): Generator {
+        return function (Node\Expr\FuncCall $node) use ($functionName): Generator {
             yield $this->createNode($node, $functionName, $node->args);
         };
     }
 
     private function mapNameSkipArg(string $functionName, int $skipArgs): callable
     {
-        return function(Node\Expr\FuncCall $node) use ($functionName, $skipArgs): Generator {
+        return function (Node\Expr\FuncCall $node) use ($functionName, $skipArgs): Generator {
             yield $this->createNode($node, $functionName, \array_slice($node->args, 0, $skipArgs));
         };
     }
 
-    private function mapEreg(callable $baseConverter, string $prefix = '', string $suffix = '', callable $warp = null): callable
+    private function mapEreg(callable $baseConverter, string $prefix = '', string $modifiers = '', callable $warp = null): callable
     {
-        return function (Node\Expr\FuncCall $node) use ($baseConverter, $prefix, $suffix, $warp): Generator {
+        return function (Node\Expr\FuncCall $node) use ($baseConverter, $prefix, $modifiers, $warp): Generator {
             foreach ($baseConverter($node) as $newNode) {
                 /* @var Node\Expr\FuncCall $newNode */
                 $newNode->args[0] = new Node\Arg(
@@ -150,7 +150,7 @@ private function mapEreg(callable $baseConverter, string $prefix = '', string $s
                                 ]
                             )
                         ),
-                        new Node\Scalar\String_("/$suffix")
+                        new Node\Scalar\String_("/$modifiers")
                     )
                 );
 
@@ -178,7 +178,7 @@ private function warpTernary(Node\Expr\FuncCall $node): Node
 
     private function mapConvertCase(): callable
     {
-        return function(Node\Expr\FuncCall $node): Generator {
+        return function (Node\Expr\FuncCall $node): Generator {
             $modeValue = $this->getConvertCaseModeValue($node);
 
             if ($modeValue === null) {

From 21f83cb746bdc7b9b5b126d7261eacaf26322574 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Wed, 13 Mar 2019 23:07:06 +0100
Subject: [PATCH 09/18] #654 drop support fro mb_ereg* functions

---
 resources/schema.json                     | 18 -----
 src/Mutator/Extensions/MBString.php       | 50 -------------
 tests/Mutator/Extensions/MBStringTest.php | 90 -----------------------
 3 files changed, 158 deletions(-)

diff --git a/resources/schema.json b/resources/schema.json
index 5cfec4967..c28edbc7c 100644
--- a/resources/schema.json
+++ b/resources/schema.json
@@ -162,24 +162,6 @@
                                             "mb_chr": {
                                                 "type": "boolean"
                                             },
-                                            "mb_ereg_match": {
-                                                "type": "boolean"
-                                            },
-                                            "mb_ereg_replace_callback": {
-                                                "type": "boolean"
-                                            },
-                                            "mb_ereg_replace": {
-                                                "type": "boolean"
-                                            },
-                                            "mb_ereg": {
-                                                "type": "boolean"
-                                            },
-                                            "mb_eregi_replace": {
-                                                "type": "boolean"
-                                            },
-                                            "mb_eregi": {
-                                                "type": "boolean"
-                                            },
                                             "mb_ord": {
                                                 "type": "boolean"
                                             },
diff --git a/src/Mutator/Extensions/MBString.php b/src/Mutator/Extensions/MBString.php
index 633630c85..9b789792c 100644
--- a/src/Mutator/Extensions/MBString.php
+++ b/src/Mutator/Extensions/MBString.php
@@ -84,12 +84,6 @@ private function setupConverters(array $functionsMap): void
     {
         $converters = [
             'mb_chr' => $this->mapNameSkipArg('chr', 1),
-            'mb_ereg_match' => $this->mapEreg($this->mapNameSkipArg('preg_match', 2), '^', '', [$this, 'warpEqOne']),
-            'mb_ereg_replace_callback' => $this->mapEreg($this->mapNameSkipArg('preg_replace_callback', 3)),
-            'mb_ereg_replace' => $this->mapEreg($this->mapNameSkipArg('preg_replace', 3)),
-            'mb_ereg' => $this->mapEreg($this->mapName('preg_match'), '', '', [$this, 'warpTernary']),
-            'mb_eregi_replace' => $this->mapEreg($this->mapNameSkipArg('preg_replace', 3), '', 'i'),
-            'mb_eregi' => $this->mapEreg($this->mapName('preg_match'), '', 'i', [$this, 'warpTernary']),
             'mb_ord' => $this->mapNameSkipArg('ord', 1),
             'mb_parse_str' => $this->mapName('parse_str'),
             'mb_send_mail' => $this->mapName('mail'),
@@ -132,50 +126,6 @@ private function mapNameSkipArg(string $functionName, int $skipArgs): callable
         };
     }
 
-    private function mapEreg(callable $baseConverter, string $prefix = '', string $modifiers = '', callable $warp = null): callable
-    {
-        return function (Node\Expr\FuncCall $node) use ($baseConverter, $prefix, $modifiers, $warp): Generator {
-            foreach ($baseConverter($node) as $newNode) {
-                /* @var Node\Expr\FuncCall $newNode */
-                $newNode->args[0] = new Node\Arg(
-                    new Node\Expr\BinaryOp\Concat(
-                        new Node\Expr\BinaryOp\Concat(
-                            new Node\Scalar\String_("/$prefix"),
-                            new Node\Expr\FuncCall(
-                                new Node\Name('\str_replace'),
-                                [
-                                    new Node\Arg(new Node\Scalar\String_('/')),
-                                    new Node\Arg(new Node\Scalar\String_('\/')),
-                                    new Node\Arg($newNode->args[0]->value),
-                                ]
-                            )
-                        ),
-                        new Node\Scalar\String_("/$modifiers")
-                    )
-                );
-
-                yield $warp ? $warp($newNode) : $newNode;
-            }
-        };
-    }
-
-    private function warpEqOne(Node\Expr\FuncCall $node): Node
-    {
-        return new Node\Expr\BinaryOp\Identical(
-            $node,
-            new Node\Scalar\LNumber(1)
-        );
-    }
-
-    private function warpTernary(Node\Expr\FuncCall $node): Node
-    {
-        return new Node\Expr\Ternary(
-            $node,
-            new Node\Scalar\LNumber(1),
-            new Node\Expr\ConstFetch(new Node\Name('false'))
-        );
-    }
-
     private function mapConvertCase(): callable
     {
         return function (Node\Expr\FuncCall $node): Generator {
diff --git a/tests/Mutator/Extensions/MBStringTest.php b/tests/Mutator/Extensions/MBStringTest.php
index 451190711..5bd128fce 100644
--- a/tests/Mutator/Extensions/MBStringTest.php
+++ b/tests/Mutator/Extensions/MBStringTest.php
@@ -81,18 +81,6 @@ public function provideMutationCases(): Generator
 
         yield from $this->provideMutationCasesForChr();
 
-        yield from $this->provideMutationCasesForEregMatch();
-
-        yield from $this->provideMutationCasesForEregReplaceCallback();
-
-        yield from $this->provideMutationCasesForEregReplace();
-
-        yield from $this->provideMutationCasesForEreg();
-
-        yield from $this->provideMutationCasesForEregiReplace();
-
-        yield from $this->provideMutationCasesForEregi();
-
         yield from $this->provideMutationCasesForOrd();
 
         yield from $this->provideMutationCasesForParseStr();
@@ -143,84 +131,6 @@ private function provideMutationCasesForChr(): Generator
         ];
     }
 
-    private function provideMutationCasesForEregMatch(): Generator
-    {
-        yield 'It converts mb_ereg_match to preg_match' => [
-            "<?php mb_ereg_match(M_2_PI, 'test');",
-            "<?php\n\npreg_match('/^' . \str_replace('/', '\\\\/', M_2_PI) . '/', 'test') === 1;",
-        ];
-
-        yield 'It converts mb_ereg_match with encoding argument to preg_match' => [
-            "<?php mb_ereg_match(M_2_PI, 'trim', 'mb');",
-            "<?php\n\npreg_match('/^' . \str_replace('/', '\\\\/', M_2_PI) . '/', 'trim') === 1;",
-        ];
-    }
-
-    private function provideMutationCasesForEregReplaceCallback(): Generator
-    {
-        yield 'It converts mb_ereg_replace_callback to preg_replace_callback' => [
-            "<?php mb_ereg_replace_callback('[A-Z/#]', 'trim', 'test');",
-            "<?php\n\npreg_replace_callback('/' . \str_replace('/', '\\\\/', '[A-Z/#]') . '/', 'trim', 'test');",
-        ];
-
-        yield 'It converts mb_ereg_replace_callback with options to preg_replace_callback' => [
-            "<?php mb_ereg_replace_callback('[A-Z/#]', 'trim', 'test', 'msr');",
-            "<?php\n\npreg_replace_callback('/' . \str_replace('/', '\\\\/', '[A-Z/#]') . '/', 'trim', 'test');",
-        ];
-    }
-
-    private function provideMutationCasesForEregReplace(): Generator
-    {
-        yield 'It converts mb_ereg_replace to ereg_replace' => [
-            "<?php mb_ereg_replace(\$pattern, 'T', 'test');",
-            "<?php\n\npreg_replace('/' . \str_replace('/', '\\\\/', \$pattern) . '/', 'T', 'test');",
-        ];
-
-        yield 'It converts mb_ereg_replace with options to ereg_replace' => [
-            "<?php mb_ereg_replace(\$pattern, 'T', 'test', 'mb');",
-            "<?php\n\npreg_replace('/' . \str_replace('/', '\\\\/', \$pattern) . '/', 'T', 'test');",
-        ];
-    }
-
-    private function provideMutationCasesForEreg(): Generator
-    {
-        yield 'It converts mb_ereg to ereg' => [
-            "<?php mb_ereg(getPattern(\$x, 1), 'test');",
-            "<?php\n\npreg_match('/' . \str_replace('/', '\\\\/', getPattern(\$x, 1)) . '/', 'test') ? 1 : false;",
-        ];
-
-        yield 'It converts mb_ereg with regs argument to ereg' => [
-            "<?php mb_ereg(getPattern(\$x, 1), 'test', \$regs);",
-            "<?php\n\npreg_match('/' . \str_replace('/', '\\\\/', getPattern(\$x, 1)) . '/', 'test', \$regs) ? 1 : false;",
-        ];
-    }
-
-    private function provideMutationCasesForEregiReplace(): Generator
-    {
-        yield 'It converts mb_eregi_replace to eregi_replace' => [
-            "<?php mb_eregi_replace(\$a . \$b, 'T', 'test');",
-            "<?php\n\npreg_replace('/' . \str_replace('/', '\\\\/', \$a . \$b) . '/i', 'T', 'test');",
-        ];
-
-        yield 'It converts mb_eregi_replace with options to eregi_replace' => [
-            "<?php mb_eregi_replace(\$a . \$b, 'T', 'test', 'msr');",
-            "<?php\n\npreg_replace('/' . \str_replace('/', '\\\\/', \$a . \$b) . '/i', 'T', 'test');",
-        ];
-    }
-
-    private function provideMutationCasesForEregi(): Generator
-    {
-        yield 'It converts mb_eregi to eregi' => [
-            "<?php mb_eregi(\DateTime::getLastErrors(), 'test');",
-            "<?php\n\npreg_match('/' . \str_replace('/', '\\\\/', \DateTime::getLastErrors()) . '/i', 'test') ? 1 : false;",
-        ];
-
-        yield 'It converts mb_eregi with regs parameter to eregi' => [
-            "<?php mb_eregi(\DateTime::getLastErrors(), 'test', \$regs);",
-            "<?php\n\npreg_match('/' . \str_replace('/', '\\\\/', \DateTime::getLastErrors()) . '/i', 'test', \$regs) ? 1 : false;",
-        ];
-    }
-
     private function provideMutationCasesForOrd(): Generator
     {
         yield 'It converts mb_ord to ord' => [

From 13af845844833c21fa8e75fb63245e2205562c3a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Thu, 14 Mar 2019 00:05:12 +0100
Subject: [PATCH 10/18] #654 mutate mb_convert_case with integer mode

---
 src/Mutator/Extensions/MBString.php       | 2 ++
 tests/Mutator/Extensions/MBStringTest.php | 7 ++++++-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/Mutator/Extensions/MBString.php b/src/Mutator/Extensions/MBString.php
index 9b789792c..a7b51e4f5 100644
--- a/src/Mutator/Extensions/MBString.php
+++ b/src/Mutator/Extensions/MBString.php
@@ -163,6 +163,8 @@ private function getConvertCaseModeValue(Node\Expr\FuncCall $node): ?int
             if (isset(self::MB_CASES[$modeName])) {
                 return self::MB_CASES[$modeName];
             }
+        } elseif ($mode instanceof Node\Scalar\LNumber) {
+            return $mode->value;
         }
 
         return null;
diff --git a/tests/Mutator/Extensions/MBStringTest.php b/tests/Mutator/Extensions/MBStringTest.php
index 5bd128fce..14ed4e064 100644
--- a/tests/Mutator/Extensions/MBStringTest.php
+++ b/tests/Mutator/Extensions/MBStringTest.php
@@ -414,11 +414,16 @@ private function provideMutationCasesForConvertCase(): Generator
             "<?php\n\nstrtoupper('test');",
         ];
 
-        yield 'It converts mb_convert_case with MB_CASE_LOWER mode to strtolower' => [
+        yield 'It converts mb_convert_case with constant similar MB_CASE_LOWER mode to strtolower' => [
             "<?php mb_convert_case('test', E_ERROR);",
             "<?php\n\nstrtolower('test');",
         ];
 
+        yield 'It converts mb_convert_case with numeric MB_CASE_LOWER (1) mode to strtolower' => [
+            "<?php mb_convert_case('test', 1);",
+            "<?php\n\nstrtolower('test');",
+        ];
+
         yield 'It converts mb_convert_case with MB_CASE_LOWER_SIMPLE mode to strtolower' => [
             "<?php mb_convert_case('test', MB_CASE_LOWER);",
             "<?php\n\nstrtolower('test');",

From c4fdde9e6cb2cb83a3d618ab768efccb93241d4e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Tue, 19 Mar 2019 00:30:48 +0100
Subject: [PATCH 11/18] #654 remove functions that cannot be easily mapped

---
 src/Mutator/Extensions/MBString.php       |  2 --
 tests/Mutator/Extensions/MBStringTest.php | 35 -----------------------
 2 files changed, 37 deletions(-)

diff --git a/src/Mutator/Extensions/MBString.php b/src/Mutator/Extensions/MBString.php
index a7b51e4f5..d332870b1 100644
--- a/src/Mutator/Extensions/MBString.php
+++ b/src/Mutator/Extensions/MBString.php
@@ -87,14 +87,12 @@ private function setupConverters(array $functionsMap): void
             'mb_ord' => $this->mapNameSkipArg('ord', 1),
             'mb_parse_str' => $this->mapName('parse_str'),
             'mb_send_mail' => $this->mapName('mail'),
-            'mb_split' => $this->mapName('split'),
             'mb_strcut' => $this->mapNameSkipArg('substr', 3),
             'mb_stripos' => $this->mapNameSkipArg('stripos', 3),
             'mb_stristr' => $this->mapNameSkipArg('stristr', 3),
             'mb_strlen' => $this->mapNameSkipArg('strlen', 1),
             'mb_strpos' => $this->mapNameSkipArg('strpos', 3),
             'mb_strrchr' => $this->mapNameSkipArg('strrchr', 2),
-            'mb_strrichr' => $this->mapNameSkipArg('strrchr', 2),
             'mb_strripos' => $this->mapNameSkipArg('strripos', 3),
             'mb_strrpos' => $this->mapNameSkipArg('strrpos', 3),
             'mb_strstr' => $this->mapNameSkipArg('strstr', 3),
diff --git a/tests/Mutator/Extensions/MBStringTest.php b/tests/Mutator/Extensions/MBStringTest.php
index 14ed4e064..5f2deb203 100644
--- a/tests/Mutator/Extensions/MBStringTest.php
+++ b/tests/Mutator/Extensions/MBStringTest.php
@@ -87,8 +87,6 @@ public function provideMutationCases(): Generator
 
         yield from $this->provideMutationCasesForSendMail();
 
-        yield from $this->provideMutationCasesForSplit();
-
         yield from $this->provideMutationCasesForStrCut();
 
         yield from $this->provideMutationCasesForStrPos();
@@ -113,8 +111,6 @@ public function provideMutationCases(): Generator
 
         yield from $this->provideMutationCasesForStrRChr();
 
-        yield from $this->provideMutationCasesForStrRiChr();
-
         yield from $this->provideMutationCasesForConvertCase();
     }
 
@@ -170,19 +166,6 @@ private function provideMutationCasesForSendMail(): Generator
         ];
     }
 
-    private function provideMutationCasesForSplit(): Generator
-    {
-        yield 'It converts mb_split to split' => [
-            "<?php mb_split('to', 'subject');",
-            "<?php\n\nsplit('to', 'subject');",
-        ];
-
-        yield 'It converts mb_split with limit parameter to split' => [
-            "<?php mb_split('to', 'subject', 2);",
-            "<?php\n\nsplit('to', 'subject', 2);",
-        ];
-    }
-
     private function provideMutationCasesForStrCut(): Generator
     {
         yield 'It converts mb_strcut to substr' => [
@@ -384,24 +367,6 @@ private function provideMutationCasesForStrRChr(): Generator
         ];
     }
 
-    private function provideMutationCasesForStrRiChr(): Generator
-    {
-        yield 'It converts mb_strrichr to strrchr' => [
-            "<?php mb_strrichr('subject', 'b');",
-            "<?php\n\nstrrchr('subject', 'b');",
-        ];
-
-        yield 'It converts mb_strrichr with part argument to strrchr' => [
-            "<?php mb_strrichr('subject', 'b', false);",
-            "<?php\n\nstrrchr('subject', 'b');",
-        ];
-
-        yield 'It converts mb_strrichr with encoding to strrchr' => [
-            "<?php mb_strrichr('subject', 'b', false, 'utf-8');",
-            "<?php\n\nstrrchr('subject', 'b');",
-        ];
-    }
-
     private function provideMutationCasesForConvertCase(): Generator
     {
         yield 'It converts mb_convert_case with MB_CASE_UPPER mode to strtoupper' => [

From 1d6bb5ed541bb670f865ac10b8499df879bc812a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Thu, 21 Mar 2019 23:36:04 +0100
Subject: [PATCH 12/18] #654 rm mb_split from json schema

---
 resources/schema.json | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/resources/schema.json b/resources/schema.json
index c28edbc7c..5e046901f 100644
--- a/resources/schema.json
+++ b/resources/schema.json
@@ -171,9 +171,6 @@
                                             "mb_send_mail": {
                                                 "type": "boolean"
                                             },
-                                            "mb_split": {
-                                                "type": "boolean"
-                                            },
                                             "mb_strcut": {
                                                 "type": "boolean"
                                             },

From 7420b4c2f334706a2390ef0b34c8f0805930d2c6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Thu, 21 Mar 2019 23:43:06 +0100
Subject: [PATCH 13/18] #654 rm mb_strrichr from json schema

---
 resources/schema.json | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/resources/schema.json b/resources/schema.json
index 5e046901f..14b93a998 100644
--- a/resources/schema.json
+++ b/resources/schema.json
@@ -189,9 +189,6 @@
                                             "mb_strrchr": {
                                                 "type": "boolean"
                                             },
-                                            "mb_strrichr": {
-                                                "type": "boolean"
-                                            },
                                             "mb_strripos": {
                                                 "type": "boolean"
                                             },

From c683158e2bdebb981b839029bbf8edbd6fb410b2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Tue, 26 Mar 2019 23:12:29 +0100
Subject: [PATCH 14/18] #654 code style fixes

---
 src/Mutator/Extensions/MBString.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Mutator/Extensions/MBString.php b/src/Mutator/Extensions/MBString.php
index d332870b1..d8035ec70 100644
--- a/src/Mutator/Extensions/MBString.php
+++ b/src/Mutator/Extensions/MBString.php
@@ -103,7 +103,7 @@ private function setupConverters(array $functionsMap): void
             'mb_convert_case' => $this->mapConvertCase(),
         ];
 
-        $functionsToRemove = \array_filter($functionsMap, function ($isOn) {
+        $functionsToRemove = \array_filter($functionsMap, static function ($isOn) {
             return !$isOn;
         });
 

From cacb538cf535dcaac117097929a4f3762bcba8d1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Wed, 27 Mar 2019 22:14:32 +0100
Subject: [PATCH 15/18] #654 simplify MBString mutator

---
 src/Mutator/Extensions/MBString.php       | 89 ++++++++++-------------
 tests/Mutator/Extensions/MBStringTest.php | 20 +++++
 2 files changed, 58 insertions(+), 51 deletions(-)

diff --git a/src/Mutator/Extensions/MBString.php b/src/Mutator/Extensions/MBString.php
index d8035ec70..afa803372 100644
--- a/src/Mutator/Extensions/MBString.php
+++ b/src/Mutator/Extensions/MBString.php
@@ -45,17 +45,6 @@
  */
 final class MBString extends Mutator
 {
-    private const MB_CASES = [
-        'MB_CASE_UPPER' => 0,
-        'MB_CASE_LOWER' => 1,
-        'MB_CASE_TITLE' => 2,
-        'MB_CASE_FOLD' => 3,
-        'MB_CASE_UPPER_SIMPLE' => 4,
-        'MB_CASE_LOWER_SIMPLE' => 5,
-        'MB_CASE_TITLE_SIMPLE' => 6,
-        'MB_CASE_FOLD_SIMPLE' => 7,
-    ];
-
     private $converters;
 
     public function __construct(MutatorConfig $config)
@@ -83,23 +72,23 @@ protected function mutatesNode(Node $node): bool
     private function setupConverters(array $functionsMap): void
     {
         $converters = [
-            'mb_chr' => $this->mapNameSkipArg('chr', 1),
-            'mb_ord' => $this->mapNameSkipArg('ord', 1),
-            'mb_parse_str' => $this->mapName('parse_str'),
-            'mb_send_mail' => $this->mapName('mail'),
-            'mb_strcut' => $this->mapNameSkipArg('substr', 3),
-            'mb_stripos' => $this->mapNameSkipArg('stripos', 3),
-            'mb_stristr' => $this->mapNameSkipArg('stristr', 3),
-            'mb_strlen' => $this->mapNameSkipArg('strlen', 1),
-            'mb_strpos' => $this->mapNameSkipArg('strpos', 3),
-            'mb_strrchr' => $this->mapNameSkipArg('strrchr', 2),
-            'mb_strripos' => $this->mapNameSkipArg('strripos', 3),
-            'mb_strrpos' => $this->mapNameSkipArg('strrpos', 3),
-            'mb_strstr' => $this->mapNameSkipArg('strstr', 3),
-            'mb_strtolower' => $this->mapNameSkipArg('strtolower', 1),
-            'mb_strtoupper' => $this->mapNameSkipArg('strtoupper', 1),
-            'mb_substr_count' => $this->mapNameSkipArg('substr_count', 2),
-            'mb_substr' => $this->mapNameSkipArg('substr', 3),
+            'mb_chr' => $this->mapFunctionAndRemoveExtraArgs('chr', 1),
+            'mb_ord' => $this->mapFunctionAndRemoveExtraArgs('ord', 1),
+            'mb_parse_str' => $this->mapFunction('parse_str'),
+            'mb_send_mail' => $this->mapFunction('mail'),
+            'mb_strcut' => $this->mapFunctionAndRemoveExtraArgs('substr', 3),
+            'mb_stripos' => $this->mapFunctionAndRemoveExtraArgs('stripos', 3),
+            'mb_stristr' => $this->mapFunctionAndRemoveExtraArgs('stristr', 3),
+            'mb_strlen' => $this->mapFunctionAndRemoveExtraArgs('strlen', 1),
+            'mb_strpos' => $this->mapFunctionAndRemoveExtraArgs('strpos', 3),
+            'mb_strrchr' => $this->mapFunctionAndRemoveExtraArgs('strrchr', 2),
+            'mb_strripos' => $this->mapFunctionAndRemoveExtraArgs('strripos', 3),
+            'mb_strrpos' => $this->mapFunctionAndRemoveExtraArgs('strrpos', 3),
+            'mb_strstr' => $this->mapFunctionAndRemoveExtraArgs('strstr', 3),
+            'mb_strtolower' => $this->mapFunctionAndRemoveExtraArgs('strtolower', 1),
+            'mb_strtoupper' => $this->mapFunctionAndRemoveExtraArgs('strtoupper', 1),
+            'mb_substr_count' => $this->mapFunctionAndRemoveExtraArgs('substr_count', 2),
+            'mb_substr' => $this->mapFunctionAndRemoveExtraArgs('substr', 3),
             'mb_convert_case' => $this->mapConvertCase(),
         ];
 
@@ -110,17 +99,17 @@ private function setupConverters(array $functionsMap): void
         $this->converters = \array_diff_key($converters, $functionsToRemove);
     }
 
-    private function mapName(string $functionName): callable
+    private function mapFunction(string $newFunctionName): callable
     {
-        return function (Node\Expr\FuncCall $node) use ($functionName): Generator {
-            yield $this->createNode($node, $functionName, $node->args);
+        return function (Node\Expr\FuncCall $node) use ($newFunctionName): Generator {
+            yield $this->mapFunctionCall($node, $newFunctionName, $node->args);
         };
     }
 
-    private function mapNameSkipArg(string $functionName, int $skipArgs): callable
+    private function mapFunctionAndRemoveExtraArgs(string $newFunctionName, int $argsAtMost): callable
     {
-        return function (Node\Expr\FuncCall $node) use ($functionName, $skipArgs): Generator {
-            yield $this->createNode($node, $functionName, \array_slice($node->args, 0, $skipArgs));
+        return function (Node\Expr\FuncCall $node) use ($newFunctionName, $argsAtMost): Generator {
+            yield $this->mapFunctionCall($node, $newFunctionName, \array_slice($node->args, 0, $argsAtMost));
         };
     }
 
@@ -139,7 +128,7 @@ private function mapConvertCase(): callable
                 return;
             }
 
-            yield $this->createNode($node, $functionName, [$node->args[0]]);
+            yield $this->mapFunctionCall($node, $functionName, [$node->args[0]]);
         };
     }
 
@@ -151,20 +140,14 @@ private function getConvertCaseModeValue(Node\Expr\FuncCall $node): ?int
 
         $mode = $node->args[1]->value;
 
-        if ($mode instanceof Node\Expr\ConstFetch) {
-            $modeName = $mode->name->toString();
-
-            if (\defined($modeName)) {
-                return \constant($modeName);
-            }
-
-            if (isset(self::MB_CASES[$modeName])) {
-                return self::MB_CASES[$modeName];
-            }
-        } elseif ($mode instanceof Node\Scalar\LNumber) {
+        if ($mode instanceof Node\Scalar\LNumber) {
             return $mode->value;
         }
 
+        if ($mode instanceof Node\Expr\ConstFetch) {
+            return \constant($mode->name->toString());
+        }
+
         return null;
     }
 
@@ -187,9 +170,13 @@ private function getConvertCaseFunctionName(int $mode): ?string
 
     private function isInMbCaseMode(int $mode, string ...$cases): bool
     {
-        $modes = \array_flip(self::MB_CASES);
+        foreach ($cases as $constant) {
+            if (\defined($constant) && \constant($constant) === $mode) {
+                return true;
+            }
+        }
 
-        return isset($modes[$mode]) && \in_array($modes[$mode], $cases);
+        return false;
     }
 
     private function getFunctionName(Node $node): ?string
@@ -198,13 +185,13 @@ private function getFunctionName(Node $node): ?string
             return null;
         }
 
-        return \strtolower($node->name->toString());
+        return $node->name->toLowerString();
     }
 
-    private function createNode(Node\Expr\FuncCall $node, string $functionName, array $args): Node\Expr\FuncCall
+    private function mapFunctionCall(Node\Expr\FuncCall $node, string $newFuncName, array $args): Node\Expr\FuncCall
     {
         return new Node\Expr\FuncCall(
-            new Node\Name($functionName, $node->name->getAttributes()),
+            new Node\Name($newFuncName, $node->name->getAttributes()),
             $args,
             $node->getAttributes()
         );
diff --git a/tests/Mutator/Extensions/MBStringTest.php b/tests/Mutator/Extensions/MBStringTest.php
index 5f2deb203..f65fac191 100644
--- a/tests/Mutator/Extensions/MBStringTest.php
+++ b/tests/Mutator/Extensions/MBStringTest.php
@@ -43,6 +43,11 @@
  */
 final class MBStringTest extends AbstractMutatorTestCase
 {
+    public static function setUpBeforeClass(): void
+    {
+        self::defineMissingMbCaseConstants();
+    }
+
     /**
      * @dataProvider provideMutationCases
      */
@@ -422,4 +427,19 @@ private function provideMutationCasesForConvertCase(): Generator
             "<?php mb_convert_case('test');",
         ];
     }
+
+    private static function defineMissingMbCaseConstants(): void
+    {
+        foreach ([
+            'MB_CASE_FOLD' => 3,
+            'MB_CASE_UPPER_SIMPLE' => 4,
+            'MB_CASE_LOWER_SIMPLE' => 5,
+            'MB_CASE_TITLE_SIMPLE' => 6,
+            'MB_CASE_FOLD_SIMPLE' => 7,
+        ] as $constantName => $constantValue) {
+            if (!\defined($constantName)) {
+                \define($constantName, $constantValue);
+            }
+        }
+    }
 }

From 3b486ca253fe11f1f23c795e3b7900a0e2c96568 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Thu, 28 Mar 2019 07:17:39 +0100
Subject: [PATCH 16/18] #654 remove getFunctionName fx

---
 src/Mutator/Extensions/MBString.php | 19 ++++++++-----------
 1 file changed, 8 insertions(+), 11 deletions(-)

diff --git a/src/Mutator/Extensions/MBString.php b/src/Mutator/Extensions/MBString.php
index afa803372..c10fc4417 100644
--- a/src/Mutator/Extensions/MBString.php
+++ b/src/Mutator/Extensions/MBString.php
@@ -57,16 +57,22 @@ public function __construct(MutatorConfig $config)
     }
 
     /**
+     * @param Node\Expr\FuncCall $node
+     *
      * @return Node|Node[]|Generator
      */
     public function mutate(Node $node)
     {
-        yield from $this->converters[$this->getFunctionName($node)]($node);
+        yield from $this->converters[$node->name->toLowerString()]($node);
     }
 
     protected function mutatesNode(Node $node): bool
     {
-        return isset($this->converters[$this->getFunctionName($node)]);
+        if (!$node instanceof Node\Expr\FuncCall || !$node->name instanceof Node\Name) {
+            return false;
+        }
+
+        return isset($this->converters[$node->name->toLowerString()]);
     }
 
     private function setupConverters(array $functionsMap): void
@@ -179,15 +185,6 @@ private function isInMbCaseMode(int $mode, string ...$cases): bool
         return false;
     }
 
-    private function getFunctionName(Node $node): ?string
-    {
-        if (!$node instanceof Node\Expr\FuncCall || !$node->name instanceof Node\Name) {
-            return null;
-        }
-
-        return $node->name->toLowerString();
-    }
-
     private function mapFunctionCall(Node\Expr\FuncCall $node, string $newFuncName, array $args): Node\Expr\FuncCall
     {
         return new Node\Expr\FuncCall(

From a6a4e4b5794c66ce8bddb3006bac5c89645cff29 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Sat, 30 Mar 2019 11:22:45 +0100
Subject: [PATCH 17/18] #654 add tests cases for capitalization

---
 tests/Mutator/Extensions/MBStringTest.php | 85 +++++++++++++++++++++++
 1 file changed, 85 insertions(+)

diff --git a/tests/Mutator/Extensions/MBStringTest.php b/tests/Mutator/Extensions/MBStringTest.php
index f65fac191..298c7bdb5 100644
--- a/tests/Mutator/Extensions/MBStringTest.php
+++ b/tests/Mutator/Extensions/MBStringTest.php
@@ -126,6 +126,11 @@ private function provideMutationCasesForChr(): Generator
             "<?php\n\nchr(74);",
         ];
 
+        yield 'It converts correctly when mb_chr is wrongly capitalized' => [
+            '<?php mB_cHr(74);',
+            "<?php\n\nchr(74);",
+        ];
+
         yield 'It converts mb_chr with encoding to chr' => [
             "<?php mb_chr(74, 'utf-8');",
             "<?php\n\nchr(74);",
@@ -139,6 +144,11 @@ private function provideMutationCasesForOrd(): Generator
             "<?php\n\nord('T');",
         ];
 
+        yield 'It converts correctly when mb_ord is wrongly capitalized' => [
+            "<?php MB_ord('T');",
+            "<?php\n\nord('T');",
+        ];
+
         yield 'It converts mb_ord with encoding to ord' => [
             "<?php mb_ord('T', 'utf-8');",
             "<?php\n\nord('T');",
@@ -152,6 +162,11 @@ private function provideMutationCasesForParseStr(): Generator
             "<?php\n\nparse_str('T');",
         ];
 
+        yield 'It converts correctly when mb_parse_str is wrongly capitalize' => [
+            "<?php mb_pARse_Str('T');",
+            "<?php\n\nparse_str('T');",
+        ];
+
         yield 'It converts mb_parse_str with params argument to parse_str' => [
             "<?php mb_parse_str('T', \$params);",
             "<?php\n\nparse_str('T', \$params);",
@@ -165,6 +180,11 @@ private function provideMutationCasesForSendMail(): Generator
             "<?php\n\nmail('to', 'subject', 'msg');",
         ];
 
+        yield 'It converts correctly when mb_send_mail is wrongly capitalize' => [
+            "<?php mb_SEND_mail('to', 'subject', 'msg');",
+            "<?php\n\nmail('to', 'subject', 'msg');",
+        ];
+
         yield 'It converts mb_send_mail with additional parameters to mail' => [
             "<?php mb_send_mail('to', 'subject', 'msg', [], []);",
             "<?php\n\nmail('to', 'subject', 'msg', [], []);",
@@ -178,6 +198,11 @@ private function provideMutationCasesForStrCut(): Generator
             "<?php\n\nsubstr('subject', 1);",
         ];
 
+        yield 'It converts correctly when mb_strcut is wrongly capitalize' => [
+            "<?php MB_strcut('subject', 1);",
+            "<?php\n\nsubstr('subject', 1);",
+        ];
+
         yield 'It converts mb_strcut with limit to substr' => [
             "<?php mb_strcut('subject', 1, 20);",
             "<?php\n\nsubstr('subject', 1, 20);",
@@ -196,6 +221,11 @@ private function provideMutationCasesForStrPos(): Generator
             "<?php\n\nstrpos('subject', 'b');",
         ];
 
+        yield 'It converts correctly when mb_strpos is wrongly capitalize' => [
+            "<?php mb_StRpOs('subject', 'b');",
+            "<?php\n\nstrpos('subject', 'b');",
+        ];
+
         yield 'It converts mb_strpos with offset to strpos' => [
             "<?php mb_strpos('subject', 'b', 3);",
             "<?php\n\nstrpos('subject', 'b', 3);",
@@ -214,6 +244,11 @@ private function provideMutationCasesForStrIPos(): Generator
             "<?php\n\nstripos('subject', 'b');",
         ];
 
+        yield 'It converts correctly when mb_stripos is wrongly capitalize' => [
+            "<?php mB_sTRIpos('subject', 'b');",
+            "<?php\n\nstripos('subject', 'b');",
+        ];
+
         yield 'It converts mb_stripos with offset to stripos' => [
             "<?php mb_stripos('subject', 'b', 3);",
             "<?php\n\nstripos('subject', 'b', 3);",
@@ -232,6 +267,11 @@ private function provideMutationCasesForStrIStr(): Generator
             "<?php\n\nstristr('subject', 'b');",
         ];
 
+        yield 'It converts correctly when mb_stristr is wrongly capitalize' => [
+            "<?php mb_strISTR('subject', 'b');",
+            "<?php\n\nstristr('subject', 'b');",
+        ];
+
         yield 'It converts mb_stristr with part argument to stristr' => [
             "<?php mb_stristr('subject', 'b', false);",
             "<?php\n\nstristr('subject', 'b', false);",
@@ -250,6 +290,11 @@ private function provideMutationCasesForStrRiPos(): Generator
             "<?php\n\nstrripos('subject', 'b');",
         ];
 
+        yield 'It converts correctly when mb_strripos is wrongly capitalize' => [
+            "<?php MB_sTrRipos('subject', 'b');",
+            "<?php\n\nstrripos('subject', 'b');",
+        ];
+
         yield 'It converts mb_strripos with offset argument to strripos' => [
             "<?php mb_strripos('subject', 'b', 2);",
             "<?php\n\nstrripos('subject', 'b', 2);",
@@ -268,6 +313,11 @@ private function provideMutationCasesForStrRPos(): Generator
             "<?php\n\nstrrpos('subject', 'b');",
         ];
 
+        yield 'It converts correctly when mb_strrpos is wrongly capitalize' => [
+            "<?php mb_StRrPos('subject', 'b');",
+            "<?php\n\nstrrpos('subject', 'b');",
+        ];
+
         yield 'It converts mb_strrpos with offset argument to strrpos' => [
             "<?php mb_strrpos('subject', 'b', 2);",
             "<?php\n\nstrrpos('subject', 'b', 2);",
@@ -286,6 +336,11 @@ private function provideMutationCasesForStrStr(): Generator
             "<?php\n\nstrstr('subject', 'b');",
         ];
 
+        yield 'It converts correctly when mb_strstr is wrongly capitalize' => [
+            "<?php Mb_STRstr('subject', 'b');",
+            "<?php\n\nstrstr('subject', 'b');",
+        ];
+
         yield 'It converts mb_strstr with part argument to strstr' => [
             "<?php mb_strstr('subject', 'b', false);",
             "<?php\n\nstrstr('subject', 'b', false);",
@@ -304,6 +359,11 @@ private function provideMutationCasesForStrToLower(): Generator
             "<?php\n\nstrtolower('test');",
         ];
 
+        yield 'It converts correctly when mb_strtolower is wrongly capitalize' => [
+            "<?php mB_StrTOloWer('test');",
+            "<?php\n\nstrtolower('test');",
+        ];
+
         yield 'It converts mb_strtolower with encoding to strtolower' => [
             "<?php mb_strtolower('test', 'utf-8');",
             "<?php\n\nstrtolower('test');",
@@ -317,6 +377,11 @@ private function provideMutationCasesForStrToUpper(): Generator
             "<?php\n\nstrtoupper('test');",
         ];
 
+        yield 'It converts correctly when mb_strtoupper is wrongly capitalize' => [
+            "<?php Mb_StrToupPer('test');",
+            "<?php\n\nstrtoupper('test');",
+        ];
+
         yield 'It converts mb_strtoupper with encoding to strtoupper' => [
             "<?php mb_strtoupper('test', 'utf-8');",
             "<?php\n\nstrtoupper('test');",
@@ -330,6 +395,11 @@ private function provideMutationCasesForSubStrCount(): Generator
             "<?php\n\nsubstr_count('test', 't');",
         ];
 
+        yield 'It converts correctly when mb_substr_count is wrongly capitalize' => [
+            "<?php MB_substr_COunt('test', 't');",
+            "<?php\n\nsubstr_count('test', 't');",
+        ];
+
         yield 'It converts mb_substr_count with encoding to substr_count' => [
             "<?php mb_substr_count('test', 't', 'utf-8');",
             "<?php\n\nsubstr_count('test', 't');",
@@ -343,6 +413,11 @@ private function provideMutationCasesForSubStr(): Generator
             "<?php\n\nsubstr('test', 2);",
         ];
 
+        yield 'It converts correctly when mb_substr is wrongly capitalize' => [
+            "<?php mB_SuBsTr('test', 2);",
+            "<?php\n\nsubstr('test', 2);",
+        ];
+
         yield 'It converts mb_substr with length argument to substr' => [
             "<?php mb_substr('test', 2, 10);",
             "<?php\n\nsubstr('test', 2, 10);",
@@ -361,6 +436,11 @@ private function provideMutationCasesForStrRChr(): Generator
             "<?php\n\nstrrchr('subject', 'b');",
         ];
 
+        yield 'It converts correctly when mb_strrchr is wrongly capitalize' => [
+            "<?php MB_StRrcHr('subject', 'b');",
+            "<?php\n\nstrrchr('subject', 'b');",
+        ];
+
         yield 'It converts mb_strrchr with part argument to strrchr' => [
             "<?php mb_strrchr('subject', 'b', false);",
             "<?php\n\nstrrchr('subject', 'b');",
@@ -379,6 +459,11 @@ private function provideMutationCasesForConvertCase(): Generator
             "<?php\n\nstrtoupper('test');",
         ];
 
+        yield 'It converts correctly when mb_parse_str is wrongly capitalize' => [
+            "<?php Mb_CoNvErT_Case('test', MB_CASE_UPPER);",
+            "<?php\n\nstrtoupper('test');",
+        ];
+
         yield 'It converts mb_convert_case with MB_CASE_UPPER_SIMPLE mode to strtoupper' => [
             "<?php mb_convert_case('test', MB_CASE_UPPER_SIMPLE);",
             "<?php\n\nstrtoupper('test');",

From 22053168f3391e0b0b654130c53ca6a290eec157 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C5=82=20Kowalik?= <maf.michal@gmail.com>
Date: Sat, 30 Mar 2019 12:05:28 +0100
Subject: [PATCH 18/18] #654 add test for calling functions via variable

---
 tests/Mutator/Extensions/MBStringTest.php | 68 +++++++++++++++++++++++
 1 file changed, 68 insertions(+)

diff --git a/tests/Mutator/Extensions/MBStringTest.php b/tests/Mutator/Extensions/MBStringTest.php
index 298c7bdb5..6a05a4f62 100644
--- a/tests/Mutator/Extensions/MBStringTest.php
+++ b/tests/Mutator/Extensions/MBStringTest.php
@@ -135,6 +135,10 @@ private function provideMutationCasesForChr(): Generator
             "<?php mb_chr(74, 'utf-8');",
             "<?php\n\nchr(74);",
         ];
+
+        yield 'It does not mutate mb_chr called via variable' => [
+            '<?php $a = "mb_chr"; $a(74);',
+        ];
     }
 
     private function provideMutationCasesForOrd(): Generator
@@ -153,6 +157,10 @@ private function provideMutationCasesForOrd(): Generator
             "<?php mb_ord('T', 'utf-8');",
             "<?php\n\nord('T');",
         ];
+
+        yield 'It does not mutate mb_ord called via variable' => [
+            '<?php $a = "mb_ord"; $a("T");',
+        ];
     }
 
     private function provideMutationCasesForParseStr(): Generator
@@ -171,6 +179,10 @@ private function provideMutationCasesForParseStr(): Generator
             "<?php mb_parse_str('T', \$params);",
             "<?php\n\nparse_str('T', \$params);",
         ];
+
+        yield 'It does not mutate mb_parse_str called via variable' => [
+            '<?php $a = "mb_parse_str"; $a("T");',
+        ];
     }
 
     private function provideMutationCasesForSendMail(): Generator
@@ -189,6 +201,10 @@ private function provideMutationCasesForSendMail(): Generator
             "<?php mb_send_mail('to', 'subject', 'msg', [], []);",
             "<?php\n\nmail('to', 'subject', 'msg', [], []);",
         ];
+
+        yield 'It does not mutate mb_send_mail called via variable' => [
+            '<?php $a = "mb_send_mail"; $a("to", "subject", "msg");',
+        ];
     }
 
     private function provideMutationCasesForStrCut(): Generator
@@ -212,6 +228,10 @@ private function provideMutationCasesForStrCut(): Generator
             "<?php mb_strcut('subject', 1, 20, 'utf-8');",
             "<?php\n\nsubstr('subject', 1, 20);",
         ];
+
+        yield 'It does not mutate mb_strcut called via variable' => [
+            '<?php $a = "mb_strcut"; $a("subject", 1);',
+        ];
     }
 
     private function provideMutationCasesForStrPos(): Generator
@@ -235,6 +255,10 @@ private function provideMutationCasesForStrPos(): Generator
             "<?php mb_strpos('subject', 'b', 3, 'utf-8');",
             "<?php\n\nstrpos('subject', 'b', 3);",
         ];
+
+        yield 'It does not mutate mb_strpos called via variable' => [
+            '<?php $a = "mb_strpos"; $a("subject", "b");',
+        ];
     }
 
     private function provideMutationCasesForStrIPos(): Generator
@@ -258,6 +282,10 @@ private function provideMutationCasesForStrIPos(): Generator
             "<?php mb_stripos('subject', 'b', 3, 'utf-8');",
             "<?php\n\nstripos('subject', 'b', 3);",
         ];
+
+        yield 'It does not mutate mb_stripos called via variable' => [
+            '<?php $a = "mb_stripos"; $a("subject", "b");',
+        ];
     }
 
     private function provideMutationCasesForStrIStr(): Generator
@@ -281,6 +309,10 @@ private function provideMutationCasesForStrIStr(): Generator
             "<?php mb_stristr('subject', 'b', false, 'utf-8');",
             "<?php\n\nstristr('subject', 'b', false);",
         ];
+
+        yield 'It does not mutate mb_stristr called via variable' => [
+            '<?php $a = "mb_stristr"; $a("subject", "b");',
+        ];
     }
 
     private function provideMutationCasesForStrRiPos(): Generator
@@ -304,6 +336,10 @@ private function provideMutationCasesForStrRiPos(): Generator
             "<?php mb_strripos('subject', 'b', 2, 'utf-8');",
             "<?php\n\nstrripos('subject', 'b', 2);",
         ];
+
+        yield 'It does not mutate mb_strripos called via variable' => [
+            '<?php $a = "mb_strripos"; $a("subject", "b");',
+        ];
     }
 
     private function provideMutationCasesForStrRPos(): Generator
@@ -327,6 +363,10 @@ private function provideMutationCasesForStrRPos(): Generator
             "<?php mb_strrpos('subject', 'b', 2, 'utf-8');",
             "<?php\n\nstrrpos('subject', 'b', 2);",
         ];
+
+        yield 'It does not mutate mb_strrpos called via variable' => [
+            '<?php $a = "mb_strrpos"; $a("subject", "b");',
+        ];
     }
 
     private function provideMutationCasesForStrStr(): Generator
@@ -350,6 +390,10 @@ private function provideMutationCasesForStrStr(): Generator
             "<?php mb_strstr('subject', 'b', false, 'utf-8');",
             "<?php\n\nstrstr('subject', 'b', false);",
         ];
+
+        yield 'It does not mutate mb_strstr called via variable' => [
+            '<?php $a = "mb_strstr"; $a("subject", "b");',
+        ];
     }
 
     private function provideMutationCasesForStrToLower(): Generator
@@ -368,6 +412,10 @@ private function provideMutationCasesForStrToLower(): Generator
             "<?php mb_strtolower('test', 'utf-8');",
             "<?php\n\nstrtolower('test');",
         ];
+
+        yield 'It does not mutate mb_strtolower called via variable' => [
+            '<?php $a = "mb_strtolower"; $a("test");',
+        ];
     }
 
     private function provideMutationCasesForStrToUpper(): Generator
@@ -386,6 +434,10 @@ private function provideMutationCasesForStrToUpper(): Generator
             "<?php mb_strtoupper('test', 'utf-8');",
             "<?php\n\nstrtoupper('test');",
         ];
+
+        yield 'It does not mutate mb_strtoupper called via variable' => [
+            '<?php $a = "mb_strtoupper"; $a("test");',
+        ];
     }
 
     private function provideMutationCasesForSubStrCount(): Generator
@@ -404,6 +456,10 @@ private function provideMutationCasesForSubStrCount(): Generator
             "<?php mb_substr_count('test', 't', 'utf-8');",
             "<?php\n\nsubstr_count('test', 't');",
         ];
+
+        yield 'It does not mutate mb_substr_count called via variable' => [
+            '<?php $a = "mb_substr_count"; $a("test", "t");',
+        ];
     }
 
     private function provideMutationCasesForSubStr(): Generator
@@ -427,6 +483,10 @@ private function provideMutationCasesForSubStr(): Generator
             "<?php mb_substr('test', 2, 10, 'utf-8');",
             "<?php\n\nsubstr('test', 2, 10);",
         ];
+
+        yield 'It does not mutate mb_substr called via variable' => [
+            '<?php $a = "mb_substr"; $a("test", 2, 10);',
+        ];
     }
 
     private function provideMutationCasesForStrRChr(): Generator
@@ -450,6 +510,10 @@ private function provideMutationCasesForStrRChr(): Generator
             "<?php mb_strrchr('subject', 'b', false, 'utf-8');",
             "<?php\n\nstrrchr('subject', 'b');",
         ];
+
+        yield 'It does not mutate mb_strrchr called via variable' => [
+            '<?php $a = "mb_strrchr"; $a("subject", "b");',
+        ];
     }
 
     private function provideMutationCasesForConvertCase(): Generator
@@ -511,6 +575,10 @@ private function provideMutationCasesForConvertCase(): Generator
         yield 'It does not convert mb_convert_case with missing mode argument' => [
             "<?php mb_convert_case('test');",
         ];
+
+        yield 'It does not mutate mb_convert_case called via variable' => [
+            '<?php $a = "mb_convert_case"; $a("test");',
+        ];
     }
 
     private static function defineMissingMbCaseConstants(): void