ReadOnlyFallbackAdapter - Code Metrics - BePark/flysystem-fallback-to-read-only-adapter - Measure and Improve Code Quality continuously with Scrutinizer

ReadOnlyFallbackAdapter A
last analyzed 2017-10-16 19:45 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	204
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	4
Bugs	1	Features	0

Metric	Value
wmc	8
c	4
b	1
f	0
lcom	1
cbo	2
dl	0
loc	204
rs	10

8 Methods

Rating	Name	Size	Complexity
A	testAdapters()	8	1
B	testHasPath()	33	1
B	testWriteAndWriteStream()	24	1
B	testUpdate()	27	1
A	testDelete()	22	1
A	testRead()	21	1
A	testGetMetadata()	12	1
A	testListContent()	47	1

<?php

namespace tests\units\BePark\Flysystem\ReadOnlyFallback;

use League\Flysystem\AdapterInterface;
use League\Flysystem\Config;

class ReadOnlyFallbackAdapter extends \atoum\test
{
	public function testAdapters(AdapterInterface $mainAdapter, AdapterInterface $readOnlyAdapter)
	{
		$this
			->given($this->newTestedInstance($mainAdapter, $readOnlyAdapter))
			->then
				->object($this->testedInstance->getMainAdapter())->isIdenticalTo($mainAdapter)
				->object($this->testedInstance->getReadOnlyAdapter())->isIdenticalTo($readOnlyAdapter);
	}

	public function testHasPath(AdapterInterface $mainAdapter, AdapterInterface $readOnlyAdapter)
	{
		$this
			->assert('test path existence on readOnly')
			->given(
				$this->newTestedInstance($mainAdapter, $readOnlyAdapter),
				$this->calling($readOnlyAdapter)->has = true,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$this->calling($mainAdapter)->has = false
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			)
			->then
				->boolean($this->testedInstance->has('/foo'))->isTrue
				->mock($readOnlyAdapter)->receive('has')->withIdenticalArguments('/foo')->once
				->mock($mainAdapter)->receive('has')->withIdenticalArguments('/foo')->once;

		$this
			->assert('test path existence on mainAdapter')
			->given($this->calling($mainAdapter)->has = true,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
					$this->calling($readOnlyAdapter)->has = false)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			->then
				->boolean($this->testedInstance->has('/foo'))->isTrue
				->mock($mainAdapter)->receive('has')->withIdenticalArguments('/foo')->once
				->mock($readOnlyAdapter)->call('has')->never;


		$this
			->assert('test none existing path')
			->given($this->calling($mainAdapter)->has = false,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
					$this->calling($readOnlyAdapter)->has = false)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			->then
				->boolean($this->testedInstance->has('/foo'))->isFalse
				->mock($mainAdapter)->receive('has')->withIdenticalArguments('/foo')->once
				->mock($readOnlyAdapter)->receive('has')->withIdenticalArguments('/foo')->once;
	}

	public function testWriteAndWriteStream(AdapterInterface $mainAdapter, AdapterInterface $readOnlyAdapter)
	{
		$this->calling($mainAdapter)->write = true;
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
		$this->calling($readOnlyAdapter)->write = false;
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

		$this->calling($mainAdapter)->writeStream = true;
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
		$this->calling($readOnlyAdapter)->writeStream = false;
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

		$this
			->assert('test write')
			->given($this->newTestedInstance($mainAdapter, $readOnlyAdapter))
			->then
			->boolean($this->testedInstance->write('/foo', 'bar', new Config()))->isTrue
				->mock($readOnlyAdapter)->wasNotCalled
				->mock($mainAdapter)->receive('write')->withAtLeastArguments(['/foo', 'bar'])->once;

		$this
			->assert('test write stream')
			->given($this->newTestedInstance($mainAdapter, $readOnlyAdapter))
			->then
			->boolean($this->testedInstance->writeStream('/foo', 'bar', new Config()))->isTrue
				->mock($readOnlyAdapter)->wasNotCalled
				->mock($mainAdapter)->receive('writeStream')->withAtLeastArguments(['/foo', 'bar'])->once;
	}

	public function testUpdate(AdapterInterface $mainAdapter, AdapterInterface $readOnlyAdapter)
	{
		$this
			->assert('test update only on main if exist on main')
			->given(
				$this->newTestedInstance($mainAdapter, $readOnlyAdapter),
				$this->calling($mainAdapter)->has = true,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$this->calling($readOnlyAdapter)->has = true,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$this->calling($mainAdapter)->update = true
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			)
			->then
				->boolean($this->testedInstance->update('/foo', 'bar', new Config()))->isTrue
				->mock($readOnlyAdapter)->wasNotCalled
				->mock($mainAdapter)->receive('update')->withAtLeastArguments(['/foo', 'bar'])->once;

		$this
			->assert('test update only on main but not exist on main')
			->given(
				$this->calling($mainAdapter)->has = false,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$this->calling($readOnlyAdapter)->has = true,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$this->calling($mainAdapter)->update = true
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			)
			->then
			->boolean($this->testedInstance->update('/foo', 'bar', new Config()))->isTrue
			->mock($readOnlyAdapter)->call('update')->never
			->mock($mainAdapter)->receive('update')->withAtLeastArguments(['/foo', 'bar'])->once;
	}

	public function testDelete(AdapterInterface $mainAdapter, AdapterInterface $readOnlyAdapter)
	{
		$this->newTestedInstance($mainAdapter, $readOnlyAdapter);

		$this->assert('test delete on non existing on main')
			->given(
				$this->calling($mainAdapter)->has = false,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$this->calling($readOnlyAdapter)->has = true,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$this->calling($mainAdapter)->delete = true
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			)
			->boolean($this->testedInstance->delete('foo/bar'))->isTrue
			->mock($mainAdapter)->receive('delete')->never
			->mock($readOnlyAdapter)->receive('delete')->never;

		$this->assert('test delete on existing on main')
			->given(
				$this->calling($mainAdapter)->has = true,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$this->calling($mainAdapter)->delete = true
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			)
			->boolean($this->testedInstance->delete('foo/bar'))->isTrue
			->mock($mainAdapter)->receive('delete')->withIdenticalArguments('foo/bar')->once;
	}

	public function testRead(AdapterInterface $mainAdapter, AdapterInterface $readOnlyAdapter)
	{
		$this->newTestedInstance($mainAdapter, $readOnlyAdapter);
		$this->assert('test read on existing on main')
			->given(
				$this->calling($readOnlyAdapter)->has = true,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$this->calling($mainAdapter)->has = true,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$this->calling($mainAdapter)->read = 'foo'
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			)
			->string($this->testedInstance->read('foo/bar'))->isEqualTo('foo')
			->mock($readOnlyAdapter)->wasNotCalled;

		$this->assert('test read on existing only on read only')
			->given(
				$this->calling($readOnlyAdapter)->has = true,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$this->calling($mainAdapter)->has = false,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$this->calling($readOnlyAdapter)->read = 'baz',
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$this->calling($mainAdapter)->read = false
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			)
			->string($this->testedInstance->read('foo/bar'))->isEqualTo('baz');
	}

	public function testGetMetadata(AdapterInterface $mainAdapter, AdapterInterface $readOnlyAdapter)
	{
		$this->given(
			$this->newTestedInstance($mainAdapter, $readOnlyAdapter),
			$this->calling($mainAdapter)->has = true,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			$this->calling($mainAdapter)->getMetadata = array('metadata')
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
		)
		->then
			->array($this->testedInstance->getMetadata('foo'))->isEqualTo(['metadata'])
			->mock($readOnlyAdapter)->wasNotCalled
			->mock($mainAdapter)->receive('getMetadata')->once;
	}

	public function testListContent(AdapterInterface $mainAdapter, AdapterInterface $readOnlyAdapter)
	{
		$a = [
			[
				'type' => 'file',
				'path' => 'foo',
				'size' => 0,
				'timestamp' => time(),
			],
			[
				'type' => 'file',
				'path' => 'bar',
				'size' => 100,
				'timestamp' => time() - 100,
			],
			[
				'type' => 'dir',
				'path' => 'baz',
				'size' => 0,
				'timestamp' => time() - 200,
			],
	    ];

		$b = [
			[
				'type' => 'file',
				'path' => 'foo',
				'size' => 10,
				'timestamp' => time(),
			],
			[
				'type' => 'file',
				'path' => 'baz',
				'size' => 150,
				'timestamp' => time() - 150,
			]
		];

		$this->given(
			$this->newTestedInstance($mainAdapter, $readOnlyAdapter),
			$this->calling($mainAdapter)->listContents = $a,
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			$this->calling($readOnlyAdapter)->listContents = $a
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
		)
		->then
			->array($this->testedInstance->listContents())
				->isIdenticalTo($a + $b);
	}
}


1			<?php
2
3			namespace tests\units\BePark\Flysystem\ReadOnlyFallback;
4
5			use League\Flysystem\AdapterInterface;
6			use League\Flysystem\Config;
7
8			class ReadOnlyFallbackAdapter extends \atoum\test
9			{
10			public function testAdapters(AdapterInterface $mainAdapter, AdapterInterface $readOnlyAdapter)
11			{
12			$this
13			->given($this->newTestedInstance($mainAdapter, $readOnlyAdapter))
14			->then
15			->object($this->testedInstance->getMainAdapter())->isIdenticalTo($mainAdapter)
16			->object($this->testedInstance->getReadOnlyAdapter())->isIdenticalTo($readOnlyAdapter);
17			}
18
19			public function testHasPath(AdapterInterface $mainAdapter, AdapterInterface $readOnlyAdapter)
20			{
21			$this
22			->assert('test path existence on readOnly')
23			->given(
24			$this->newTestedInstance($mainAdapter, $readOnlyAdapter),
25			$this->calling($readOnlyAdapter)->has = true,
			0 ignored issues – show Documentation introduced 2017-05-06 15:47 UTC by Report Bug Copy Issue Report `$readOnlyAdapter` is of type `object<League\Flysystem\AdapterInterface>`, but the function expects a `object<mageekguy\atoum\mock\aggregator>`. 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...
26			$this->calling($mainAdapter)->has = false
			0 ignored issues – show Documentation introduced 2017-05-06 15:47 UTC by Report Bug Copy Issue Report `$mainAdapter` is of type `object<League\Flysystem\AdapterInterface>`, but the function expects a `object<mageekguy\atoum\mock\aggregator>`. 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...
27			)
28			->then
29			->boolean($this->testedInstance->has('/foo'))->isTrue
30			->mock($readOnlyAdapter)->receive('has')->withIdenticalArguments('/foo')->once
31			->mock($mainAdapter)->receive('has')->withIdenticalArguments('/foo')->once;
32
33			$this
34			->assert('test path existence on mainAdapter')
35			->given($this->calling($mainAdapter)->has = true,
			0 ignored issues – show Documentation introduced 2017-05-06 15:47 UTC by Report Bug Copy Issue Report `$mainAdapter` is of type `object<League\Flysystem\AdapterInterface>`, but the function expects a `object<mageekguy\atoum\mock\aggregator>`. 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...
36			$this->calling($readOnlyAdapter)->has = false)
			0 ignored issues – show Documentation introduced 2017-05-06 15:47 UTC by Report Bug Copy Issue Report `$readOnlyAdapter` is of type `object<League\Flysystem\AdapterInterface>`, but the function expects a `object<mageekguy\atoum\mock\aggregator>`. 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...

BePark / flysystem-fallback-to-read-only-adapter

ReadOnlyFallbackAdapter A last analyzed 2017-10-16 19:45 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

8 Methods

Duplication Side-by-Side

Filter issues like

ReadOnlyFallbackAdapter A
last analyzed 2017-10-16 19:45 UTC