Compare `6909c3c ... +0 ... 0b1ad6e`

     */
    public function isColored()
    {
        return isset($this->config['color_support'])
            ? $this->config['color_support']
        return $this->config['color_support'] ??
            // @codeCoverageIgnoreStart
            : (DIRECTORY_SEPARATOR === '\\'
            (DIRECTORY_SEPARATOR === '\\'
                ? false !== getenv('ANSICON') ||
                'ON' === getenv('ConEmuANSI') ||
                false !== getenv('BABUN_HOME')

@@ -76,7 +76,7 @@

        $arguments = array_filter($arguments, function ($argument) {
            return $argument !== '--verbose' && $argument !== '-v';
        });
        $this->configFile = isset($arguments[1]) ? $arguments[1] : $multiTester->getMultiTesterFile();
        $this->configFile = $arguments[1] ?? $multiTester->getMultiTesterFile();
        $this->addProjects();

        if (!file_exists($this->configFile)) {

@@ -157,7 +157,7 @@

            unset($config['config']);
        }

        $this->projects = isset($config['projects']) ? $config['projects'] : $config;
        $this->projects = $config['projects'] ?? $config;
    }

    /**

@@ -168,10 +168,13 @@

        if (!file_exists($this->composerFile)) {
            throw new MultiTesterException("Set the 'directory' entry to a path containing a composer.json file.");
        }

        $this->data = new File($this->composerFile);

        if (!isset($this->data['name'])) {
            throw new MultiTesterException("The composer.json file must contains a 'name' entry.");
        }

        $this->packageName = $this->data['name'];
    }
}

@@ -174,7 +174,7 @@

            $package = $this->getPackage();
            $tester = $this->getConfig()->getTester();
            $composerSettings = $tester->getComposerSettings($package);
            $version = $this->filterVersion($settings['version'], array_keys($composerSettings ?? []));
            $version = $this->filterVersion($settings['version'], array_keys($composerSettings ?: []));

            $settings['source'] = isset($composerSettings[$version]['source'])
                ? $composerSettings[$version]['source']

@@ -81,6 +81,7 @@

    protected function copyItem($source, $destination, $file, &$success)
    {
        $path = "$source/$file";

        if (@is_dir($path)) {
            if (!(new static($path))->copy("$destination/$file")) {
                $success = false;

@@ -17,6 +17,7 @@

    use StorageDirectory;
    use ProcStreams;
    use Verbose;

    /**
     * @var array|File Composer package settings cache.
     */

@@ -29,7 +30,7 @@


    public function __construct($storageDirectory = null)
    {
        $this->storageDirectory = $storageDirectory ?? sys_get_temp_dir();
        $this->storageDirectory = $storageDirectory ?: sys_get_temp_dir();
    }

    public function exec($command, $quiet = false)

@@ -43,9 +44,7 @@

    {
        if (!isset($this->composerSettings[$package])) {
            $this->composerSettings[$package] = new File("https://repo.packagist.org/p/$package.json");
            $this->composerSettings[$package] = isset($this->composerSettings[$package]['packages'], $this->composerSettings[$package]['packages'][$package])
                ? $this->composerSettings[$package]['packages'][$package]
                : null;
            $this->composerSettings[$package] = ($this->composerSettings[$package]['packages'] ?? [])[$package] ?? null;
        }

        return $this->composerSettings[$package];

@@ -55,6 +54,7 @@

    {
        $streams = $this->getProcStreams();
        $stdout = is_array($streams) && isset($streams[1]) ? $streams[1] : null;

        if (is_array($stdout) && $stdout[0] === 'file') {
            $file = fopen($stdout[1], $stdout[2]);
            fwrite($file, $text);

@@ -197,7 +197,7 @@


    protected function extractVersion(&$package, &$settings)
    {
        list($package, $version) = explode(':', "$package:");
        [$package, $version] = explode(':', "$package:");

        if ($version !== '' && !isset($settings['version'])) {
            $settings['version'] = $version;

@@ -226,7 +226,7 @@

        } catch (TestFailedException $exception) {
            $state = false;

            if (isset($config->config['stop_on_failure']) && $config->config['stop_on_failure']) {
            if ($config->config['stop_on_failure'] ?? false) {
                $this->error($exception);
            }
        } catch (MultiTesterException $exception) {

Everything is accounted for!

No changes detected that need to be reviewed.

What changes does Codecov check for?

Lines, not adjusted in diff, that have changed coverage data.

Files that introduced coverage data that had none before.

Files that have missing coverage data that once were tracked.

100.00% 100.00% ø 188 -3

Upgrade to PHP 7 syntax

kylekatarnls

0b1ad6e

CI Passed

-3

100.00% 100.00% ø 191 -6

Test enhancement (#7)

peter279k

6909c3c

CI Passed

Files		•	•	•	Complexity	Coverage
src/MultiTester	-3	-3	ø	ø	ø	100.00%
Project Totals (12 files)	442	442	0	0	188	100.00%

157	157		unset($config['config']);
158	158		}
159	159
160		-	$this->projects = isset($config['projects']) ? $config['projects'] : $config;
	160	+	$this->projects = $config['projects'] ?? $config;
161	161		}
162	162
163	163		/**

168	168		if (!file_exists($this->composerFile)) {
169	169		throw new MultiTesterException("Set the 'directory' entry to a path containing a composer.json file.");
170	170		}
	171	+
171	172		$this->data = new File($this->composerFile);
	173	+
172	174		if (!isset($this->data['name'])) {
173	175		throw new MultiTesterException("The composer.json file must contains a 'name' entry.");
174	176		}
	177	+
175	178		$this->packageName = $this->data['name'];
176	179		}
177	180		}

174	174		$package = $this->getPackage();
175	175		$tester = $this->getConfig()->getTester();
176	176		$composerSettings = $tester->getComposerSettings($package);
177		-	$version = $this->filterVersion($settings['version'], array_keys($composerSettings ?? []));
	177	+	$version = $this->filterVersion($settings['version'], array_keys($composerSettings ?: []));
178	178
179	179		$settings['source'] = isset($composerSettings[$version]['source'])
180	180		? $composerSettings[$version]['source']

81	81		protected function copyItem($source, $destination, $file, &$success)
82	82		{
83	83		$path = "$source/$file";
	84	+
84	85		if (@is_dir($path)) {
85	86		if (!(new static($path))->copy("$destination/$file")) {
86	87		$success = false;

29	30
30	31		public function __construct($storageDirectory = null)
31	32		{
32		-	$this->storageDirectory = $storageDirectory ?? sys_get_temp_dir();
	33	+	$this->storageDirectory = $storageDirectory ?: sys_get_temp_dir();
33	34		}
34	35
35	36		public function exec($command, $quiet = false)

25	25		*/
26	26		public function isColored()
27	27		{
28		-	return isset($this->config['color_support'])
29		-	? $this->config['color_support']
	28	+	return $this->config['color_support'] ??
30	29		// @codeCoverageIgnoreStart
31		-	: (DIRECTORY_SEPARATOR === '\\'
	30	+	(DIRECTORY_SEPARATOR === '\\'
32	31		? false !== getenv('ANSICON') \|\|
33	32		'ON' === getenv('ConEmuANSI') \|\|
34	33		false !== getenv('BABUN_HOME')

76	76		$arguments = array_filter($arguments, function ($argument) {
77	77		return $argument !== '--verbose' && $argument !== '-v';
78	78		});
79		-	$this->configFile = isset($arguments[1]) ? $arguments[1] : $multiTester->getMultiTesterFile();
	79	+	$this->configFile = $arguments[1] ?? $multiTester->getMultiTesterFile();
80	80		$this->addProjects();
81	81
82	82		if (!file_exists($this->configFile)) {

17	17		use StorageDirectory;
18	18		use ProcStreams;
19	19		use Verbose;
	20	+
20	21		/**
21	22		* @var array\|File Composer package settings cache.
22	23		*/

43	44		{
44	45		if (!isset($this->composerSettings[$package])) {
45	46		$this->composerSettings[$package] = new File("https://repo.packagist.org/p/$package.json");
46		-	$this->composerSettings[$package] = isset($this->composerSettings[$package]['packages'], $this->composerSettings[$package]['packages'][$package])
47		-	? $this->composerSettings[$package]['packages'][$package]
48		-	: null;
	47	+	$this->composerSettings[$package] = ($this->composerSettings[$package]['packages'] ?? [])[$package] ?? null;
49	48		}
50	49
51	50		return $this->composerSettings[$package];

55	54		{
56	55		$streams = $this->getProcStreams();
57	56		$stdout = is_array($streams) && isset($streams[1]) ? $streams[1] : null;
	57	+
58	58		if (is_array($stdout) && $stdout[0] === 'file') {
59	59		$file = fopen($stdout[1], $stdout[2]);
60	60		fwrite($file, $text);

197	197
198	198		protected function extractVersion(&$package, &$settings)
199	199		{
200		-	list($package, $version) = explode(':', "$package:");
	200	+	[$package, $version] = explode(':', "$package:");
201	201
202	202		if ($version !== '' && !isset($settings['version'])) {
203	203		$settings['version'] = $version;

226	226		} catch (TestFailedException $exception) {
227	227		$state = false;
228	228
229		-	if (isset($config->config['stop_on_failure']) && $config->config['stop_on_failure']) {
	229	+	if ($config->config['stop_on_failure'] ?? false) {
230	230		$this->error($exception);
231	231		}
232	232		} catch (MultiTesterException $exception) {

Navigation	Overlay
`t` Navigate files	`h` Toggle hits
`y` Change url to tip of branch	`m` Toggle misses
`b / v` Jump to prev/next hit line	`p` Toggle partial
`z / x` Jump to prev/next missed or partial line	`1..9` Toggle flags
`shift + o` Open current page in GitHub	`a` Toggle all on
`/ or ?` Show keyboard shortcuts dialog	`c` Toggle context lines or commits

Compare 6909c3c ... +0 ... 0b1ad6e

No flags found

Everything is accounted for!

What changes does Codecov check for?

2 Commits

Compare `6909c3c ... +0 ... 0b1ad6e`