BaseRunner - Code Metrics - Inspection of "Attempt at fixing failed build on Travis CI" - brianium/paratest - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#204)

unknown

created 2016-05-08 15:16 UTC

BaseRunner A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	204
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	9

Test Coverage

Coverage

84.72%

Importance

Changes	3
Bugs	1	Features	0

Metric	Value
wmc	25
c	3
b	1
f	0
lcom	1
cbo	9
dl	0
loc	204
ccs	61
cts	72
cp	0.8472
rs	10

12 Methods

Rating	Name	Size	Complexity
A	getExitCode()	4	1
A	hasCoverage()	4	1
A	getCoverage()	4	1
A	addCoverageFromFile()	13	4
A	__construct()	6	1
A	load()	10	3
A	log()	9	2
A	logCoverage()	20	4
A	initCoverage()	8	2
A	getCoverageObject()	11	2
A	run()	7	1
A	verifyConfiguration()	7	3

<?php
namespace ParaTest\Runners\PHPUnit;

use ParaTest\Coverage\CoverageMerger;
use ParaTest\Logging\LogInterpreter;
use ParaTest\Logging\JUnit\Writer;
use PHP_CodeCoverage_Report_Clover;
use PHP_CodeCoverage_Report_HTML;
use PHP_CodeCoverage_Report_PHP;
use RuntimeException;

abstract class BaseRunner
{
    const PHPUNIT_FATAL_ERROR = 255;

    /**
     * @var Options
     */
    protected $options;

    /**
     * @var \ParaTest\Logging\LogInterpreter
     */
    protected $interpreter;

    /**
     * @var ResultPrinter
     */
    protected $printer;

    /**
     * A collection of pending ExecutableTest objects that have
     * yet to run
     *
     * @var array
     */
    protected $pending = array();

    /**
     * A collection of ExecutableTest objects that have processes
     * currently running
     *
     * @var array
     */
    protected $running = array();

    /**
     * A tallied exit code that returns the highest exit
     * code returned out of the entire collection of tests
     *
     * @var int
     */
    protected $exitcode = -1;

    /**
     * CoverageMerger to hold track of the accumulated coverage
     *
     * @var CoverageMerger
     */
    protected $coverage = null;


    public function __construct($opts = array())
    {
        $this->options = new Options($opts);
        $this->interpreter = new LogInterpreter();
        $this->printer = new ResultPrinter($this->interpreter);
    }

    public function run()
    {
        $this->verifyConfiguration();
        $this->initCoverage();
        $this->load();
        $this->printer->start($this->options);
    }

    /**
     * Ensures a valid configuration was supplied. If not
     * causes ParaTest to print the error message and exit immediately
     * with an exit code of 1
     */
    protected function verifyConfiguration()
    {
        if (isset($this->options->filtered['configuration']) && !file_exists($this->options->filtered['configuration']->getPath())) {
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
            $this->printer->println(sprintf('Could not read "%s".', $this->options->filtered['configuration']));
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
            exit(1);

        }
    }

    /**
     * Builds the collection of pending ExecutableTest objects
     * to run. If functional mode is enabled $this->pending will
     * contain a collection of TestMethod objects instead of Suite
     * objects
     */
    protected function load()
    {
        $loader = new SuiteLoader($this->options);
        $loader->load($this->options->path);
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        $executables = $this->options->functional ? $loader->getTestMethods() : $loader->getSuites();
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        $this->pending = array_merge($this->pending, $executables);
        foreach ($this->pending as $pending) {
            $this->printer->addTest($pending);
        }
    }

    /**
     * Returns the highest exit code encountered
     * throughout the course of test execution
     *
     * @return int
     */
    public function getExitCode()
    {
        return $this->exitcode;
    }

    /**
     * Write output to JUnit format if requested
     */
    protected function log()
    {
        if (!isset($this->options->filtered['log-junit'])) {
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
            return;
        }
        $output = $this->options->filtered['log-junit'];
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        $writer = new Writer($this->interpreter, $this->options->path);
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        $writer->write($output);
    }

    /**
     * Write coverage to file if requested
     */
    protected function logCoverage()
    {
        if (!$this->hasCoverage()) {
            return;
        }

        $filteredOptions = $this->options->filtered;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        if (isset($filteredOptions['coverage-clover'])) {
            $clover = new PHP_CodeCoverage_Report_Clover();
            $clover->process($this->getCoverage()->getCoverage(), $filteredOptions['coverage-clover']);
        }

        if (isset($filteredOptions['coverage-html'])) {
            $html = new PHP_CodeCoverage_Report_HTML();
            $html->process($this->getCoverage()->getCoverage(), $filteredOptions['coverage-html']);
        }

        $php = new PHP_CodeCoverage_Report_PHP();
        $php->process($this->getCoverage()->getCoverage(), $filteredOptions['coverage-php']);
    }

    protected function initCoverage()
    {
        if (!isset($this->options->filtered['coverage-php'])) {
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
            return;
        }

        $this->coverage = new CoverageMerger();
    }

    protected function hasCoverage()
    {
        return $this->getCoverage() != null;
    }

    /**
     * @return CoverageMerger
     */
    public function getCoverage()
    {
        return $this->coverage;
    }

    /**
     * Returns coverage object from file.
     *
     * @param string $coverageFile Coverage file.
     *
     * @return \PHP_CodeCoverage
     */
    protected function getCoverageObject($coverageFile)
    {
        $coverage = file_get_contents($coverageFile);

        if (substr($coverage, 0, 5) === '<?php') {
            return include $coverageFile;
        }
        
        // the PHPUnit 3.x and below
        return unserialize($coverage);
    }

    /**
     * Adds the coverage contained in $coverageFile and deletes the file afterwards
     * @param $coverageFile
     * @throws RuntimeException
     */
    protected function addCoverageFromFile($coverageFile)
    {
        if ($coverageFile === null || !file_exists($coverageFile)) {
            return;
        }

        if (filesize($coverageFile) == 0) {
            throw new RuntimeException("Coverage file $coverageFile is empty. This means a PHPUnit process has crashed.");
        }

        $this->getCoverage()->addCoverage($this->getCoverageObject($coverageFile));
        unlink($coverageFile);
    }
}


1		<?php
2		namespace ParaTest\Runners\PHPUnit;
3
4		use ParaTest\Coverage\CoverageMerger;
5		use ParaTest\Logging\LogInterpreter;
6		use ParaTest\Logging\JUnit\Writer;
7		use PHP_CodeCoverage_Report_Clover;
8		use PHP_CodeCoverage_Report_HTML;
9		use PHP_CodeCoverage_Report_PHP;
10		use RuntimeException;
11
12		abstract class BaseRunner
13		{
14		const PHPUNIT_FATAL_ERROR = 255;
15
16		/**
17		* @var Options
18		*/
19		protected $options;
20
21		/**
22		* @var \ParaTest\Logging\LogInterpreter
23		*/
24		protected $interpreter;
25
26		/**
27		* @var ResultPrinter
28		*/
29		protected $printer;
30
31		/**
32		* A collection of pending ExecutableTest objects that have
33		* yet to run
34		*
35		* @var array
36		*/
37		protected $pending = array();
38
39		/**
40		* A collection of ExecutableTest objects that have processes
41		* currently running
42		*
43		* @var array
44		*/
45		protected $running = array();
46
47		/**
48		* A tallied exit code that returns the highest exit
49		* code returned out of the entire collection of tests
50		*
51		* @var int
52		*/
53		protected $exitcode = -1;
54
55		/**
56		* CoverageMerger to hold track of the accumulated coverage
57		*
58		* @var CoverageMerger
59		*/
60		protected $coverage = null;
61
62
63	8	public function __construct($opts = array())
64		{
65	8	$this->options = new Options($opts);
66	8	$this->interpreter = new LogInterpreter();
67	8	$this->printer = new ResultPrinter($this->interpreter);
68	8	}
69
70	2	public function run()
71	2	{
72	2	$this->verifyConfiguration();
73	2	$this->initCoverage();
74	2	$this->load();
75	2	$this->printer->start($this->options);
76	2	}
77
78		/**
79		* Ensures a valid configuration was supplied. If not
80		* causes ParaTest to print the error message and exit immediately
81		* with an exit code of 1
82		*/
83	2	protected function verifyConfiguration()
84		{
85	2	if (isset($this->options->filtered['configuration']) && !file_exists($this->options->filtered['configuration']->getPath())) {
		0 ignored issues – show Documentation introduced 2015-12-02 18:19 UTC by Report Bug Copy Issue Report The property `$filtered` is declared protected in `ParaTest\Runners\PHPUnit\Options`. Since you implemented `__get()`, maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
86		$this->printer->println(sprintf('Could not read "%s".', $this->options->filtered['configuration']));
		0 ignored issues – show Documentation introduced 2015-12-02 18:19 UTC by Report Bug Copy Issue Report The property `$filtered` is declared protected in `ParaTest\Runners\PHPUnit\Options`. Since you implemented `__get()`, maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
87		exit(1);
		0 ignored issues – show Coding Style Compatibility introduced 2015-12-02 18:19 UTC by Report Bug Copy Issue Report The method `verifyConfiguration()` contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
88		}
89	2	}
90
91		/**
92		* Builds the collection of pending ExecutableTest objects
93		* to run. If functional mode is enabled $this->pending will
94		* contain a collection of TestMethod objects instead of Suite
95		* objects
96		*/
97	2	protected function load()
98		{
99	2	$loader = new SuiteLoader($this->options);
100	2	$loader->load($this->options->path);
		0 ignored issues – show Documentation introduced 2015-12-02 18:19 UTC by Report Bug Copy Issue Report The property `$path` is declared protected in `ParaTest\Runners\PHPUnit\Options`. Since you implemented `__get()`, maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
101	2	$executables = $this->options->functional ? $loader->getTestMethods() : $loader->getSuites();
		0 ignored issues – show Documentation introduced 2015-12-02 18:19 UTC by Report Bug Copy Issue Report The property `$functional` is declared protected in `ParaTest\Runners\PHPUnit\Options`. Since you implemented `__get()`, maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
102	2	$this->pending = array_merge($this->pending, $executables);
103	2	foreach ($this->pending as $pending) {
104	2	$this->printer->addTest($pending);
105	2	}
106	2	}
107
108		/**
109		* Returns the highest exit code encountered
110		* throughout the course of test execution
111		*
112		* @return int
113		*/
114	1	public function getExitCode()
115		{
116	1	return $this->exitcode;
117		}
118
119		/**
120		* Write output to JUnit format if requested
121		*/
122	2	protected function log()
123		{
124	2	if (!isset($this->options->filtered['log-junit'])) {
		0 ignored issues – show Documentation introduced 2015-12-02 18:19 UTC by Report Bug Copy Issue Report The property `$filtered` is declared protected in `ParaTest\Runners\PHPUnit\Options`. Since you implemented `__get()`, maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
125	1	return;
126		}
127	1	$output = $this->options->filtered['log-junit'];
		0 ignored issues – show Documentation introduced 2015-12-02 18:19 UTC by Report Bug Copy Issue Report The property `$filtered` is declared protected in `ParaTest\Runners\PHPUnit\Options`. Since you implemented `__get()`, maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
128	1	$writer = new Writer($this->interpreter, $this->options->path);
		0 ignored issues – show Documentation introduced 2015-12-02 18:19 UTC by Report Bug Copy Issue Report The property `$path` is declared protected in `ParaTest\Runners\PHPUnit\Options`. Since you implemented `__get()`, maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
129	1	$writer->write($output);
130	1	}
131
132		/**
133		* Write coverage to file if requested
134		*/
135	2	protected function logCoverage()
136		{
137	2	if (!$this->hasCoverage()) {
138		return;
139		}
140
141	2	$filteredOptions = $this->options->filtered;
		0 ignored issues – show Documentation introduced 2015-12-02 18:19 UTC by Report Bug Copy Issue Report The property `$filtered` is declared protected in `ParaTest\Runners\PHPUnit\Options`. Since you implemented `__get()`, maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
142	2	if (isset($filteredOptions['coverage-clover'])) {
143	2	$clover = new PHP_CodeCoverage_Report_Clover();
144		$clover->process($this->getCoverage()->getCoverage(), $filteredOptions['coverage-clover']);
145	2	}
146
147	2	if (isset($filteredOptions['coverage-html'])) {
148		$html = new PHP_CodeCoverage_Report_HTML();
149		$html->process($this->getCoverage()->getCoverage(), $filteredOptions['coverage-html']);
150		}
151
152	2	$php = new PHP_CodeCoverage_Report_PHP();
153	2	$php->process($this->getCoverage()->getCoverage(), $filteredOptions['coverage-php']);
154	2	}
155
156	2	protected function initCoverage()
157		{
158	2	if (!isset($this->options->filtered['coverage-php'])) {
		0 ignored issues – show Documentation introduced 2015-12-02 18:19 UTC by Report Bug Copy Issue Report The property `$filtered` is declared protected in `ParaTest\Runners\PHPUnit\Options`. Since you implemented `__get()`, maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
159		return;
160		}
161
162	2	$this->coverage = new CoverageMerger();
163	2	}
164
165	2	protected function hasCoverage()
166		{
167	2	return $this->getCoverage() != null;
168		}
169
170		/**
171		* @return CoverageMerger
172		*/
173	2	public function getCoverage()
174		{
175	2	return $this->coverage;
176		}
177
178		/**
179		* Returns coverage object from file.
180		*
181		* @param string $coverageFile Coverage file.
182		*
183		* @return \PHP_CodeCoverage
184		*/
185	2	protected function getCoverageObject($coverageFile)
186		{
187	2	$coverage = file_get_contents($coverageFile);
188
189	2	if (substr($coverage, 0, 5) === '<?php') {
190		return include $coverageFile;
191		}
192
193		// the PHPUnit 3.x and below
194	2	return unserialize($coverage);
195		}
196
197		/**
198		* Adds the coverage contained in $coverageFile and deletes the file afterwards
199		* @param $coverageFile
200		* @throws RuntimeException
201		*/
202	2	protected function addCoverageFromFile($coverageFile)
203		{
204	2	if ($coverageFile === null \|\| !file_exists($coverageFile)) {
205		return;
206		}
207
208	2	if (filesize($coverageFile) == 0) {
209		throw new RuntimeException("Coverage file $coverageFile is empty. This means a PHPUnit process has crashed.");
210		}
211
212	2	$this->getCoverage()->addCoverage($this->getCoverageObject($coverageFile));
213	2	unlink($coverageFile);
214	2	}
215		}
216

brianium / paratest

Pull Request — master (#204)

BaseRunner A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

12 Methods

Duplication Side-by-Side

Filter issues like