Issues in RestoreTest.php - Fixed Issues - Inspection of "Update dependencies" - redaxscript/redaxscript - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 6661e3...771ea1 )

by Henry

created 2018-01-23 22:57 UTC

tests/phpunit/Console/Command/RestoreTest.php (1 issue)

Labels

Documentation 1

Severity

Minor 1

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php
namespace Redaxscript\Tests\Console\Command;

use Redaxscript\Console\Command;
use Redaxscript\Tests\TestCaseAbstract;
use org\bovigo\vfs\vfsStream as Stream;

/**
 * RestoreTest
 *
 * @since 3.0.0
 *
 * @package Redaxscript
 * @category Tests
 * @author Henry Ruhs
 *
 * @requires OS Linux
 */

class RestoreTest extends TestCaseAbstract
{
	/**
	 * setUp
	 *
	 * @since 3.0.0
	 */

	public function setUp()
	{
		parent::setUp();
		Stream::setup('root', 0777, $this->getProvider('tests/provider/Console/console_setup.json'));
	}

	/**
	 * tearDown
	 *
	 * @since 3.0.0
	 */

	public function tearDown()
	{
		$this->_request->setServer('argv', null);
	}

	/**
	 * testNoArgument
	 *
	 * @since 3.0.0
	 */

	public function testNoArgument()
	{
		/* setup */

		$restoreCommand = new Command\Restore($this->_registry, $this->_request, $this->_language, $this->_config);

		/* expect and actual */

		$expect = $restoreCommand->getHelp();
		$actual = $restoreCommand->run('cli');

		/* compare */

		$this->assertEquals($expect, $actual);
	}

	/**
	 * testDatabase
	 *
	 * @since 3.0.0
	 */

	public function testDatabase()
	{
		/* setup */

		$this->_request->setServer('argv',
		[
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			'console.php',
			'restore',
			'database',
			'--directory',
			Stream::url('root/build'),
			'--file',
			'test.sql'
		]);
		$restoreCommand = new Command\Restore($this->_registry, $this->_request, $this->_language, $this->_config);

		/* actual */

		$actual = $restoreCommand->run('cli');

		/* compare */

		$this->assertTrue($actual);
	}

	/**
	 * testDatabaseInvalid
	 *
	 * @since 3.0.0
	 */

	public function testDatabaseInvalid()
	{
		/* setup */

		$this->_request->setServer('argv',
		[
			'console.php',
			'restore',
			'database',
			'--no-interaction'
		]);
		$restoreCommand = new Command\Restore($this->_registry, $this->_request, $this->_language, $this->_config);

		/* actual */

		$actual = $restoreCommand->run('cli');

		/* compare */

		$this->assertFalse($actual);
	}
}


1			<?php
2			namespace Redaxscript\Tests\Console\Command;
3
4			use Redaxscript\Console\Command;
5			use Redaxscript\Tests\TestCaseAbstract;
6			use org\bovigo\vfs\vfsStream as Stream;
7
8			/**
9			* RestoreTest
10			*
11			* @since 3.0.0
12			*
13			* @package Redaxscript
14			* @category Tests
15			* @author Henry Ruhs
16			*
17			* @requires OS Linux
18			*/
19
20			class RestoreTest extends TestCaseAbstract
21			{
22			/**
23			* setUp
24			*
25			* @since 3.0.0
26			*/
27
28			public function setUp()
29			{
30			parent::setUp();
31			Stream::setup('root', 0777, $this->getProvider('tests/provider/Console/console_setup.json'));
32			}
33
34			/**
35			* tearDown
36			*
37			* @since 3.0.0
38			*/
39
40			public function tearDown()
41			{
42			$this->_request->setServer('argv', null);
43			}
44
45			/**
46			* testNoArgument
47			*
48			* @since 3.0.0
49			*/
50
51			public function testNoArgument()
52			{
53			/* setup */
54
55			$restoreCommand = new Command\Restore($this->_registry, $this->_request, $this->_language, $this->_config);
56
57			/* expect and actual */
58
59			$expect = $restoreCommand->getHelp();
60			$actual = $restoreCommand->run('cli');
61
62			/* compare */
63
64			$this->assertEquals($expect, $actual);
65			}
66
67			/**
68			* testDatabase
69			*
70			* @since 3.0.0
71			*/
72
73			public function testDatabase()
74			{
75			/* setup */
76
77			$this->_request->setServer('argv',
78			[
			0 ignored issues – show Documentation introduced 2017-10-31 20:04 UTC by Report Bug Copy Issue Report Show Similar Issues like this `array('console.php', 're..., '--file', 'test.sql')` is of type `array<integer,?,{"0":"st..."string","6":"string"}>`, but the function expects a `string\|boolean\|null`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
79			'console.php',
80			'restore',
81			'database',
82			'--directory',
83			Stream::url('root/build'),
84			'--file',
85			'test.sql'
86			]);
87			$restoreCommand = new Command\Restore($this->_registry, $this->_request, $this->_language, $this->_config);
88
89			/* actual */
90
91			$actual = $restoreCommand->run('cli');
92
93			/* compare */
94
95			$this->assertTrue($actual);
96			}
97
98			/**
99			* testDatabaseInvalid
100			*
101			* @since 3.0.0
102			*/
103
104			public function testDatabaseInvalid()
105			{
106			/* setup */
107
108			$this->_request->setServer('argv',
109			[
110			'console.php',
111			'restore',
112			'database',
113			'--no-interaction'
114			]);
115			$restoreCommand = new Command\Restore($this->_registry, $this->_request, $this->_language, $this->_config);
116
117			/* actual */
118
119			$actual = $restoreCommand->run('cli');
120
121			/* compare */
122
123			$this->assertFalse($actual);
124			}
125			}
126

redaxscript / redaxscript

Push — master ( 6661e3...771ea1 )

tests/phpunit/Console/Command/RestoreTest.php (1 issue)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like