Inspection of "Check for installed FlySystem in tests" - aimeos/ai-filesystem - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 3aaa57...3b1bb7 )

by Aimeos

created 2016-01-16 17:02 UTC

Status

Spacing +5 added lines, -5 removed lines patch added patch discarded remove patch

@@ -7,17 +7,17 @@
 block discarded – undo
 {
 	protected function setUp()
 	{
-		if( !interface_exists( '\\League\\Flysystem\\FilesystemInterface' ) ) {
-			$this->markTestSkipped( 'Install Flysystem first' );
+		if (!interface_exists('\\League\\Flysystem\\FilesystemInterface')) {
+			$this->markTestSkipped('Install Flysystem first');
 		}
 	}
 
 
 	public function testGetProvider()
 	{
-		$object = new FlyFtp( array( 'host' => 'ftp.kernel.org' ) );
-		$this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
+		$object = new FlyFtp(array('host' => 'ftp.kernel.org'));
+		$this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
 
-		$object->has( 'test' );
+		$object->has('test');
 	}
 }

Please login to merge, or discard this patch.

lib/custom/tests/MW/Filesystem/FlyRackspaceTest.php 1 patch

Spacing +12 added lines, -12 removed lines patch added patch discarded remove patch

@@ -7,32 +7,32 @@
 block discarded – undo
 {
 	protected function setUp()
 	{
-		if( !interface_exists( '\\League\\Flysystem\\FilesystemInterface' ) ) {
-			$this->markTestSkipped( 'Install Flysystem first' );
+		if (!interface_exists('\\League\\Flysystem\\FilesystemInterface')) {
+			$this->markTestSkipped('Install Flysystem first');
 		}
 	}
 
 
 	public function testGetProvider()
 	{
-		$object = new FlyRackspace( array() );
-		$this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
+		$object = new FlyRackspace(array());
+		$this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$object->has( 'test' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$object->has('test');
 	}
 
 
 	public function testGetProviderContainer()
 	{
-		if( !class_exists( '\League\Flysystem\Rackspace\RackspaceAdapter' ) ) {
-			$this->markTestSkipped( 'Install Flysystem Rackspace adapter' );
+		if (!class_exists('\League\Flysystem\Rackspace\RackspaceAdapter')) {
+			$this->markTestSkipped('Install Flysystem Rackspace adapter');
 		}
 
-		$object = new FlyRackspace( array( 'container' => 'test' ) );
-		$this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
+		$object = new FlyRackspace(array('container' => 'test'));
+		$this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
 
-		$this->setExpectedException( '\OpenCloud\Common\Exceptions\CredentialError' );
-		$object->has( 'test' );
+		$this->setExpectedException('\OpenCloud\Common\Exceptions\CredentialError');
+		$object->has('test');
 	}
 }

Please login to merge, or discard this patch.

lib/custom/tests/MW/Filesystem/FlyLocalTest.php 1 patch

Spacing +9 added lines, -9 removed lines patch added patch discarded remove patch

@@ -7,27 +7,27 @@
 block discarded – undo
 {
 	protected function setUp()
 	{
-		if( !interface_exists( '\\League\\Flysystem\\FilesystemInterface' ) ) {
-			$this->markTestSkipped( 'Install Flysystem first' );
+		if (!interface_exists('\\League\\Flysystem\\FilesystemInterface')) {
+			$this->markTestSkipped('Install Flysystem first');
 		}
 	}
 
 
 	public function testGetProvider()
 	{
-		$object = new FlyLocal( array( 'basedir' => dirname( dirname( __DIR__ ) ) ) );
-		$this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
+		$object = new FlyLocal(array('basedir' => dirname(dirname(__DIR__))));
+		$this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
 
-		$object->has( 'test' );
+		$object->has('test');
 	}
 
 
 	public function testGetProviderNoBasedir()
 	{
-		$object = new FlyLocal( array() );
-		$this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
+		$object = new FlyLocal(array());
+		$this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
 
-		$this->setExpectedException( 'Aimeos\MW\Filesystem\Exception' );
-		$object->has( 'test' );
+		$this->setExpectedException('Aimeos\MW\Filesystem\Exception');
+		$object->has('test');
 	}
 }

Please login to merge, or discard this patch.

lib/custom/tests/MW/Filesystem/FlyAwsS3Test.php 1 patch

Spacing +12 added lines, -12 removed lines patch added patch discarded remove patch

@@ -7,32 +7,32 @@
 block discarded – undo
 {
 	protected function setUp()
 	{
-		if( !interface_exists( '\\League\\Flysystem\\FilesystemInterface' ) ) {
-			$this->markTestSkipped( 'Install Flysystem first' );
+		if (!interface_exists('\\League\\Flysystem\\FilesystemInterface')) {
+			$this->markTestSkipped('Install Flysystem first');
 		}
 	}
 
 
 	public function testGetProvider()
 	{
-		if( !class_exists( '\League\Flysystem\AwsS3v3\AwsS3Adapter' ) ) {
-			$this->markTestSkipped( 'Install Flysystem AwsS3v3 adapter' );
+		if (!class_exists('\League\Flysystem\AwsS3v3\AwsS3Adapter')) {
+			$this->markTestSkipped('Install Flysystem AwsS3v3 adapter');
 		}
 
-		$object = new FlyAwsS3( array( 'bucket' => 'test' ) );
-		$this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
+		$object = new FlyAwsS3(array('bucket' => 'test'));
+		$this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
 
-		$this->setExpectedException( 'InvalidArgumentException' );
-		$object->has( 'test' );
+		$this->setExpectedException('InvalidArgumentException');
+		$object->has('test');
 	}
 
 
 	public function testGetProviderNoBucket()
 	{
-		$object = new FlyAwsS3( array() );
-		$this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
+		$object = new FlyAwsS3(array());
+		$this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
 
-		$this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
-		$object->has( 'test' );
+		$this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
+		$object->has('test');
 	}
 }

Please login to merge, or discard this patch.

lib/custom/tests/MW/Filesystem/FlyReplicateTest.php 1 patch

Spacing +18 added lines, -18 removed lines patch added patch discarded remove patch

@@ -7,27 +7,27 @@  discard block
 block discarded – undo
 {
 	protected function setUp()
 	{
-		if( !interface_exists( '\\League\\Flysystem\\FilesystemInterface' ) ) {
-			$this->markTestSkipped( 'Install Flysystem first' );
+		if (!interface_exists('\\League\\Flysystem\\FilesystemInterface')) {
+			$this->markTestSkipped('Install Flysystem first');
 		}
 	}
 
 
 	public function testGetProvider()
 	{
-		if( !class_exists( '\League\Flysystem\Replicate\ReplicateAdapter' ) ) {
-			$this->markTestSkipped( 'Install Flysystem Replicate adapter' );
+		if (!class_exists('\League\Flysystem\Replicate\ReplicateAdapter')) {
+			$this->markTestSkipped('Install Flysystem Replicate adapter');
 		}
 
 		$config = array(
 			'adapter' => 'Replicate',
-			'source' => array( 'adapter' => 'FlyMemory' ),
-			'replica' => array( 'adapter' => 'FlyMemory' ),
+			'source' => array('adapter' => 'FlyMemory'),
+			'replica' => array('adapter' => 'FlyMemory'),
 		);
-		$object = new FlyReplicate( $config );
-		$this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
+		$object = new FlyReplicate($config);
+		$this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
 
-		$object->has( 'test' );
+		$object->has('test');
 	}
 
 
@@ -36,11 +36,11 @@  discard block
 block discarded – undo
 		$config = array(
 			'adapter' => 'Replicate',
 		);
-		$object = new FlyReplicate( $config );
-		$this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
+		$object = new FlyReplicate($config);
+		$this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
 
-		$this->setExpectedException( 'Aimeos\MW\Filesystem\Exception' );
-		$object->has( 'test' );
+		$this->setExpectedException('Aimeos\MW\Filesystem\Exception');
+		$object->has('test');
 	}
 
 
@@ -48,12 +48,12 @@  discard block
 block discarded – undo
 	{
 		$config = array(
 			'adapter' => 'Replicate',
-			'source' => array( 'adapter' => 'FlyMemory' ),
+			'source' => array('adapter' => 'FlyMemory'),
 		);
-		$object = new FlyReplicate( $config );
-		$this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
+		$object = new FlyReplicate($config);
+		$this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
 
-		$this->setExpectedException( 'Aimeos\MW\Filesystem\Exception' );
-		$object->has( 'test' );
+		$this->setExpectedException('Aimeos\MW\Filesystem\Exception');
+		$object->has('test');
 	}
 }

Please login to merge, or discard this patch.

		@@ -7,17 +7,17 @@
		block discarded – undo
7	7	{
8	8	protected function setUp()
9	9	{
10		- if( !interface_exists( '\\League\\Flysystem\\FilesystemInterface' ) ) {
11		- $this->markTestSkipped( 'Install Flysystem first' );
	10	+ if (!interface_exists('\\League\\Flysystem\\FilesystemInterface')) {
	11	+ $this->markTestSkipped('Install Flysystem first');
12	12	}
13	13	}
14	14
15	15
16	16	public function testGetProvider()
17	17	{
18		- $object = new FlyFtp( array( 'host' => 'ftp.kernel.org' ) );
19		- $this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
	18	+ $object = new FlyFtp(array('host' => 'ftp.kernel.org'));
	19	+ $this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
20	20
21		- $object->has( 'test' );
	21	+ $object->has('test');
22	22	}
23	23	}

		@@ -7,32 +7,32 @@
		block discarded – undo
7	7	{
8	8	protected function setUp()
9	9	{
10		- if( !interface_exists( '\\League\\Flysystem\\FilesystemInterface' ) ) {
11		- $this->markTestSkipped( 'Install Flysystem first' );
	10	+ if (!interface_exists('\\League\\Flysystem\\FilesystemInterface')) {
	11	+ $this->markTestSkipped('Install Flysystem first');
12	12	}
13	13	}
14	14
15	15
16	16	public function testGetProvider()
17	17	{
18		- $object = new FlyRackspace( array() );
19		- $this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
	18	+ $object = new FlyRackspace(array());
	19	+ $this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
20	20
21		- $this->setExpectedException( '\Aimeos\MW\Filesystem\Exception' );
22		- $object->has( 'test' );
	21	+ $this->setExpectedException('\Aimeos\MW\Filesystem\Exception');
	22	+ $object->has('test');
23	23	}
24	24
25	25
26	26	public function testGetProviderContainer()
27	27	{
28		- if( !class_exists( '\League\Flysystem\Rackspace\RackspaceAdapter' ) ) {
29		- $this->markTestSkipped( 'Install Flysystem Rackspace adapter' );
	28	+ if (!class_exists('\League\Flysystem\Rackspace\RackspaceAdapter')) {
	29	+ $this->markTestSkipped('Install Flysystem Rackspace adapter');
30	30	}
31	31
32		- $object = new FlyRackspace( array( 'container' => 'test' ) );
33		- $this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
	32	+ $object = new FlyRackspace(array('container' => 'test'));
	33	+ $this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
34	34
35		- $this->setExpectedException( '\OpenCloud\Common\Exceptions\CredentialError' );
36		- $object->has( 'test' );
	35	+ $this->setExpectedException('\OpenCloud\Common\Exceptions\CredentialError');
	36	+ $object->has('test');
37	37	}
38	38	}

		@@ -7,27 +7,27 @@
		block discarded – undo
7	7	{
8	8	protected function setUp()
9	9	{
10		- if( !interface_exists( '\\League\\Flysystem\\FilesystemInterface' ) ) {
11		- $this->markTestSkipped( 'Install Flysystem first' );
	10	+ if (!interface_exists('\\League\\Flysystem\\FilesystemInterface')) {
	11	+ $this->markTestSkipped('Install Flysystem first');
12	12	}
13	13	}
14	14
15	15
16	16	public function testGetProvider()
17	17	{
18		- $object = new FlyLocal( array( 'basedir' => dirname( dirname( __DIR__ ) ) ) );
19		- $this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
	18	+ $object = new FlyLocal(array('basedir' => dirname(dirname(__DIR__))));
	19	+ $this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
20	20
21		- $object->has( 'test' );
	21	+ $object->has('test');
22	22	}
23	23
24	24
25	25	public function testGetProviderNoBasedir()
26	26	{
27		- $object = new FlyLocal( array() );
28		- $this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
	27	+ $object = new FlyLocal(array());
	28	+ $this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
29	29
30		- $this->setExpectedException( 'Aimeos\MW\Filesystem\Exception' );
31		- $object->has( 'test' );
	30	+ $this->setExpectedException('Aimeos\MW\Filesystem\Exception');
	31	+ $object->has('test');
32	32	}
33	33	}

		@@ -7,27 +7,27 @@ discard block
		block discarded – undo
7	7	{
8	8	protected function setUp()
9	9	{
10		- if( !interface_exists( '\\League\\Flysystem\\FilesystemInterface' ) ) {
11		- $this->markTestSkipped( 'Install Flysystem first' );
	10	+ if (!interface_exists('\\League\\Flysystem\\FilesystemInterface')) {
	11	+ $this->markTestSkipped('Install Flysystem first');
12	12	}
13	13	}
14	14
15	15
16	16	public function testGetProvider()
17	17	{
18		- if( !class_exists( '\League\Flysystem\Replicate\ReplicateAdapter' ) ) {
19		- $this->markTestSkipped( 'Install Flysystem Replicate adapter' );
	18	+ if (!class_exists('\League\Flysystem\Replicate\ReplicateAdapter')) {
	19	+ $this->markTestSkipped('Install Flysystem Replicate adapter');
20	20	}
21	21
22	22	$config = array(
23	23	'adapter' => 'Replicate',
24		- 'source' => array( 'adapter' => 'FlyMemory' ),
25		- 'replica' => array( 'adapter' => 'FlyMemory' ),
	24	+ 'source' => array('adapter' => 'FlyMemory'),
	25	+ 'replica' => array('adapter' => 'FlyMemory'),
26	26	);
27		- $object = new FlyReplicate( $config );
28		- $this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
	27	+ $object = new FlyReplicate($config);
	28	+ $this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
29	29
30		- $object->has( 'test' );
	30	+ $object->has('test');
31	31	}
32	32
33	33
		@@ -36,11 +36,11 @@ discard block
		block discarded – undo
36	36	$config = array(
37	37	'adapter' => 'Replicate',
38	38	);
39		- $object = new FlyReplicate( $config );
40		- $this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
	39	+ $object = new FlyReplicate($config);
	40	+ $this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
41	41
42		- $this->setExpectedException( 'Aimeos\MW\Filesystem\Exception' );
43		- $object->has( 'test' );
	42	+ $this->setExpectedException('Aimeos\MW\Filesystem\Exception');
	43	+ $object->has('test');
44	44	}
45	45
46	46
		@@ -48,12 +48,12 @@ discard block
		block discarded – undo
48	48	{
49	49	$config = array(
50	50	'adapter' => 'Replicate',
51		- 'source' => array( 'adapter' => 'FlyMemory' ),
	51	+ 'source' => array('adapter' => 'FlyMemory'),
52	52	);
53		- $object = new FlyReplicate( $config );
54		- $this->assertInstanceof( '\Aimeos\MW\Filesystem\Iface', $object );
	53	+ $object = new FlyReplicate($config);
	54	+ $this->assertInstanceof('\Aimeos\MW\Filesystem\Iface', $object);
55	55
56		- $this->setExpectedException( 'Aimeos\MW\Filesystem\Exception' );
57		- $object->has( 'test' );
	56	+ $this->setExpectedException('Aimeos\MW\Filesystem\Exception');
	57	+ $object->has('test');
58	58	}
59	59	}

aimeos / ai-filesystem

Push — master ( 3aaa57...3b1bb7 )

Status

Category

Spacing +5 added lines, -5 removed lines patch added patch discarded remove patch

Spacing +12 added lines, -12 removed lines patch added patch discarded remove patch

Spacing +9 added lines, -9 removed lines patch added patch discarded remove patch

Spacing +12 added lines, -12 removed lines patch added patch discarded remove patch

Spacing +18 added lines, -18 removed lines patch added patch discarded remove patch