TestResultsParser::getResultsFromOutput() - Code Metrics - Inspection of "Still WIP" - facile-it/paraunit - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — output_parsers_refactor ( 9ffc6c...755930 )

by Alessandro

created 2016-03-12 13:00 UTC

TestResultsParser::getResultsFromOutput() A

↳ Parent: TestResultsParser

Complexity

Conditions	3
Paths	3

Size

Total Lines	18
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	3

Importance

Changes	1
Bugs	1	Features	0

Metric	Value
c	1
b	1
f	0
dl	0
loc	18
ccs	11
cts	11
cp	1
rs	9.4285
cc	3
eloc	10
nc	3
nop	1
crap	3

<?php

namespace Paraunit\Parser;

use Paraunit\Process\ProcessResultInterface;

class TestResultsParser implements JSONParserChainElementInterface
{
    const PHPUNIT_TEST_RESULTS_REGEX = '/(?<=by Sebastian Bergmann and contributors.\n\n)(.*?)+(?=\n\nTime:)/s';
    const PHPUNIT_CONFIG_LOAD = 'Configuration read from ';

    /**
     * @param ProcessResultInterface $process
     *
     * @return bool True if chain should continue
     */
    public function parsingFoundResult(ProcessResultInterface $process)
    {
        $results = $this->getResultsFromOutput($process->getOutput());

        if ($results != '') {
            $cleanResults = preg_replace('/[^WFIESR.]+/', '', $results);
            $process->setTestResults(str_split($cleanResults));

        }

        return true;
    }

    /**
     * @param string $output
     *
     * @return string
     */
    private function getResultsFromOutput($output)
    {
        $matches = array();
        preg_match(self::PHPUNIT_TEST_RESULTS_REGEX, $output, $matches);

        $results = '';

        if (!empty($matches)) {
            // fix per PHPUNIT < 4.7, strip della riga della config
            if (substr($matches[0], 0, 24) == $this::PHPUNIT_CONFIG_LOAD) {
                $results = preg_replace('/^'.preg_quote($this::PHPUNIT_CONFIG_LOAD).'.*\n\n/', '', $matches[0]);
            } else {
                $results = $matches[0];
            }
        }

        return $results;
    }
}


1		<?php
2
3		namespace Paraunit\Parser;
4
5		use Paraunit\Process\ProcessResultInterface;
6
7		class TestResultsParser implements JSONParserChainElementInterface
8		{
9		const PHPUNIT_TEST_RESULTS_REGEX = '/(?<=by Sebastian Bergmann and contributors.\n\n)(.*?)+(?=\n\nTime:)/s';
10		const PHPUNIT_CONFIG_LOAD = 'Configuration read from ';
11
12		/**
13		* @param ProcessResultInterface $process
14		*
15		* @return bool True if chain should continue
16		*/
17	13	public function parsingFoundResult(ProcessResultInterface $process)
18		{
19	13	$results = $this->getResultsFromOutput($process->getOutput());
20
21	13	if ($results != '') {
22	11	$cleanResults = preg_replace('/[^WFIESR.]+/', '', $results);
23	11	$process->setTestResults(str_split($cleanResults));
		0 ignored issues – show Deprecated Code introduced 2016-03-12 01:12 UTC by Report Bug Copy Issue Report The method `Paraunit\Process\Process...rface::setTestResults()` has been deprecated with message: We should use just addTestResult This method has been deprecated. The supplier of the class has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead. Loading history...
24	11	}
25
26	13	return true;
27		}
28
29		/**
30		* @param string $output
31		*
32		* @return string
33		*/
34	13	private function getResultsFromOutput($output)
35		{
36	13	$matches = array();
37	13	preg_match(self::PHPUNIT_TEST_RESULTS_REGEX, $output, $matches);
38
39	13	$results = '';
40
41	13	if (!empty($matches)) {
42		// fix per PHPUNIT < 4.7, strip della riga della config
43	11	if (substr($matches[0], 0, 24) == $this::PHPUNIT_CONFIG_LOAD) {
44	4	$results = preg_replace('/^'.preg_quote($this::PHPUNIT_CONFIG_LOAD).'.*\n\n/', '', $matches[0]);
45	4	} else {
46	7	$results = $matches[0];
47		}
48	11	}
49
50	13	return $results;
51		}
52		}
53

facile-it / paraunit

Push — output_parsers_refactor ( 9ffc6c...755930 )

TestResultsParser::getResultsFromOutput() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like