CoveragePass::filterWhitelist() - Code Metrics - Inspection of "refactor session" - doyolabs/code-coverage - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 146be5...1e12ef )

by ANTHONIUS

created 2019-06-14 06:45 UTC

CoveragePass::filterWhitelist() C

↳ Parent: CoveragePass

Complexity

Conditions	13
Paths	96

Size

Total Lines	37
Code Lines	22

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	13
CRAP Score	24.5711

Importance

Changes

Metric	Value
cc	13
eloc	22
nc	96
nop	3
dl	0
loc	37
ccs	13
cts	22
cp	0.5909
crap	24.5711
rs	6.6166
c	0
b	0
f	0

How to fix Complexity

<?php

/*
 * This file is part of the doyo/code-coverage project.
 *
 * (c) Anthonius Munthi <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace Doyo\Bridge\CodeCoverage\Compiler;

use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
use Symfony\Component\DependencyInjection\ContainerBuilder;
use Symfony\Component\DependencyInjection\Definition;
use Symfony\Component\DependencyInjection\Reference;

class CoveragePass implements CompilerPassInterface
{
    public function process(ContainerBuilder $container)
    {
        $this->processDriver($container);
        $this->processFilter($container);
    }

    private function processDriver(ContainerBuilder $container)
    {
        $driverClass = $container->get('runtime')->getDriverClass();
        $container->getParameterBag()->set('coverage.driver.class', $driverClass);

        $r = new \ReflectionClass($driverClass);
        if ($r->hasMethod('setFilter')) {
            $definition = $container->getDefinition('coverage.filter');
            $definition->addMethodCall('setFilter', [new Reference('coverage.filter')]);
        }
    }

    private function processFilter(ContainerBuilder $container)
    {
        $config     = $container->getParameter('config.filter');
        $definition = $container->getDefinition('coverage.filter');

        foreach ($config as $options) {
            $options['basePath'] = '';
            $this->filterWhitelist($definition, $options, 'add');
            $exclude = $options['exclude'];
            foreach ($exclude as $item) {
                $item['basePath'] = '';
                $this->filterWhitelist($definition, $item, 'remove');
            }
        }
    }

    private function filterWhitelist(Definition $definition, $options, $method)
    {
        $suffix    = $options['suffix'] ?: '.php';
        $prefix    = $options['prefix'] ?: '';
        $directory = $options['directory'];
        $file      = $options['file'];
        $type      = !is_null($options['directory']) ? 'directory' : 'file';

        if(!is_null($directory) && "" === $directory){
            $directory = getcwd();
        }
        
        if (
            !is_null($directory)
            && preg_match('/\/\*(\..+)/', $directory, $matches)
        ) {
            $suffix    = $matches[1];
            $directory = str_replace($matches[0], '', $directory);
        }

        $methodSuffix = 'add' === $method ? 'ToWhitelist' : 'FromWhitelist';
        if ('directory' === $type) {
            $definition->addMethodCall($method.'Directory'.$methodSuffix, [$directory, $suffix, $prefix]);
        }

        if ('file' === $type) {

            $files = [$file];

            if(false !== ($pos=strpos($file, '*'))){

                $files = [];
                foreach(glob($file) as $filename){
                    $files[] = $filename;
                }
            }

            $definition->addMethodCall($method.'File'.$methodSuffix, $files);
        }
    }
}


1		<?php
2
3		/*
4		* This file is part of the doyo/code-coverage project.
5		*
6		* (c) Anthonius Munthi <[email protected]>
7		*
8		* For the full copyright and license information, please view the LICENSE
9		* file that was distributed with this source code.
10		*/
11
12		declare(strict_types=1);
13
14		namespace Doyo\Bridge\CodeCoverage\Compiler;
15
16		use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
17		use Symfony\Component\DependencyInjection\ContainerBuilder;
18		use Symfony\Component\DependencyInjection\Definition;
19		use Symfony\Component\DependencyInjection\Reference;
20
21		class CoveragePass implements CompilerPassInterface
22		{
23	21	public function process(ContainerBuilder $container)
24		{
25	21	$this->processDriver($container);
26	21	$this->processFilter($container);
27		}
28
29	21	private function processDriver(ContainerBuilder $container)
30		{
31	21	$driverClass = $container->get('runtime')->getDriverClass();
32	21	$container->getParameterBag()->set('coverage.driver.class', $driverClass);
33
34	21	$r = new \ReflectionClass($driverClass);
35	21	if ($r->hasMethod('setFilter')) {
36		$definition = $container->getDefinition('coverage.filter');
37		$definition->addMethodCall('setFilter', [new Reference('coverage.filter')]);
38		}
39		}
40
41	21	private function processFilter(ContainerBuilder $container)
42		{
43	21	$config = $container->getParameter('config.filter');
44	21	$definition = $container->getDefinition('coverage.filter');
45
46	21	foreach ($config as $options) {
47	2	$options['basePath'] = '';
48	2	$this->filterWhitelist($definition, $options, 'add');
49	2	$exclude = $options['exclude'];
50	2	foreach ($exclude as $item) {
51		$item['basePath'] = '';
52		$this->filterWhitelist($definition, $item, 'remove');
53		}
54		}
55		}
56
57	2	private function filterWhitelist(Definition $definition, $options, $method)
58		{
59	2	$suffix = $options['suffix'] ?: '.php';
60	2	$prefix = $options['prefix'] ?: '';
61	2	$directory = $options['directory'];
62	2	$file = $options['file'];
63	2	$type = !is_null($options['directory']) ? 'directory' : 'file';
64
65	2	if(!is_null($directory) && "" === $directory){
66		$directory = getcwd();
67		}
68
69		if (
70	2	!is_null($directory)
71	2	&& preg_match('/\/\*(\..+)/', $directory, $matches)
72		) {
73		$suffix = $matches[1];
74		$directory = str_replace($matches[0], '', $directory);
75		}
76
77	2	$methodSuffix = 'add' === $method ? 'ToWhitelist' : 'FromWhitelist';
78	2	if ('directory' === $type) {
79	2	$definition->addMethodCall($method.'Directory'.$methodSuffix, [$directory, $suffix, $prefix]);
80		}
81
82	2	if ('file' === $type) {
83
84		$files = [$file];
85
86		if(false !== ($pos=strpos($file, '*'))){
		0 ignored issues – show Unused Code introduced 2019-06-14 05:22 UTC by Report Bug Copy Issue Report The assignment to `$pos` is dead and can be removed. Loading history...
87		$files = [];
88		foreach(glob($file) as $filename){
89		$files[] = $filename;
90		}
91		}
92
93		$definition->addMethodCall($method.'File'.$methodSuffix, $files);
94		}
95		}
96		}
97

doyolabs / code-coverage

Push — master ( 146be5...1e12ef )

CoveragePass::filterWhitelist() C

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like