RedirectResolvingEntityLookupTest - Code Metrics - wmde/WikibaseDataModelServices - Measure and Improve Code Quality continuously with Scrutinizer

RedirectResolvingEntityLookupTest A
last analyzed 2021-06-01 10:00 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	106
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes

Metric	Value
dl	0
loc	106
c	0
b	0
f	0
wmc	14
lcom	1
cbo	8
rs	10

9 Methods

Rating	Name	Size	Complexity
A	getEntity()	14	5
A	getLookupDouble()	15	1
A	getEntityProvider()	6	1
A	testGetEntity()	11	2
A	testGetEntity_missing()	6	1
A	testGetEntity_brokenRedirect()	6	1
A	testGetEntity_doubleRedirect()	8	1
A	hasEntityProvider()	8	1
A	testHasEntity()	5	1

<?php

namespace Wikibase\DataModel\Services\Tests\Lookup;

use PHPUnit\Framework\TestCase;
use Wikibase\DataModel\Entity\EntityId;
use Wikibase\DataModel\Entity\Item;
use Wikibase\DataModel\Entity\ItemId;
use Wikibase\DataModel\Services\Lookup\EntityLookup;
use Wikibase\DataModel\Services\Lookup\RedirectResolvingEntityLookup;
use Wikibase\DataModel\Services\Lookup\UnresolvedEntityRedirectException;

/**
 * @covers \Wikibase\DataModel\Services\Lookup\RedirectResolvingEntityLookup
 *
 * @license GPL-2.0-or-later
 * @author Daniel Kinzler
 */
class RedirectResolvingEntityLookupTest extends TestCase {

	/**
	 * @param EntityId $id
	 *
	 * @return null|Item
	 * @throws UnresolvedEntityRedirectException
	 */
	public function getEntity( EntityId $id ) {
		switch ( $id->getSerialization() ) {
			case 'Q10':
				return new Item( $id );
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			case 'Q11':
				throw new UnresolvedEntityRedirectException( new ItemId( 'Q11' ), new ItemId( 'Q10' ) );
			case 'Q12':
				throw new UnresolvedEntityRedirectException( new ItemId( 'Q12' ), new ItemId( 'Q11' ) );
			case 'Q21':
				throw new UnresolvedEntityRedirectException( new ItemId( 'Q21' ), new ItemId( 'Q20' ) );
			default:
				return null;
		}
	}

	/**
	 * @return EntityLookup
	 */
	public function getLookupDouble() {
		$mock = $this->createMock( EntityLookup::class );

		$mock->expects( $this->any() )
			->method( 'getEntity' )
			->will( $this->returnCallback( [ $this, 'getEntity' ] ) );

		$mock->expects( $this->any() )
			->method( 'hasEntity' )
			->will( $this->returnCallback( function ( EntityId $id ) {
				return $this->getEntity( $id ) !== null;
			} ) );

		return $mock;
	}

	public function getEntityProvider() {
		return [
			'no redirect' => [ new ItemId( 'Q10' ), new ItemId( 'Q10' ) ],
			'one redirect' => [ new ItemId( 'Q11' ), new ItemId( 'Q10' ) ],
		];
	}

	/**
	 * @dataProvider getEntityProvider
	 */
	public function testGetEntity( EntityId $id, EntityId $expected ) {
		$lookup = new RedirectResolvingEntityLookup( $this->getLookupDouble() );
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

		$entity = $lookup->getEntity( $id );

		if ( $expected === null ) {
			$this->assertNull( $entity );
		} else {
			$this->assertTrue( $expected->equals( $entity->getId() ) );
		}
	}

	public function testGetEntity_missing() {
		$lookup = new RedirectResolvingEntityLookup( $this->getLookupDouble() );
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

		$id = new ItemId( 'Q7' ); // entity Q7 is not known
		$this->assertNull( $lookup->getEntity( $id ) );
	}

	public function testGetEntity_brokenRedirect() {
		$lookup = new RedirectResolvingEntityLookup( $this->getLookupDouble() );
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

		$id = new ItemId( 'Q20' ); // Q20 is a broken redirect
		$this->assertNull( $lookup->getEntity( $id ) );
	}

	public function testGetEntity_doubleRedirect() {
		$lookup = new RedirectResolvingEntityLookup( $this->getLookupDouble() );
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

		$id = new ItemId( 'Q12' ); // Q12 is a double redirect

		$this->expectException( UnresolvedEntityRedirectException::class );
		$lookup->getEntity( $id );
	}

	public function hasEntityProvider() {
		return [
			'unknown entity' => [ new ItemId( 'Q7' ), false ],
			'no redirect' => [ new ItemId( 'Q10' ), true ],
			'one redirect' => [ new ItemId( 'Q11' ), true ],
			'broken redirect' => [ new ItemId( 'Q21' ), false ],
		];
	}

	/**
	 * @dataProvider hasEntityProvider
	 */
	public function testHasEntity( EntityId $id, $exists ) {
		$lookup = new RedirectResolvingEntityLookup( $this->getLookupDouble() );
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

		$this->assertEquals( $exists, $lookup->hasEntity( $id ) );
	}

}


1			<?php
2
3			namespace Wikibase\DataModel\Services\Tests\Lookup;
4
5			use PHPUnit\Framework\TestCase;
6			use Wikibase\DataModel\Entity\EntityId;
7			use Wikibase\DataModel\Entity\Item;
8			use Wikibase\DataModel\Entity\ItemId;
9			use Wikibase\DataModel\Services\Lookup\EntityLookup;
10			use Wikibase\DataModel\Services\Lookup\RedirectResolvingEntityLookup;
11			use Wikibase\DataModel\Services\Lookup\UnresolvedEntityRedirectException;
12
13			/**
14			* @covers \Wikibase\DataModel\Services\Lookup\RedirectResolvingEntityLookup
15			*
16			* @license GPL-2.0-or-later
17			* @author Daniel Kinzler
18			*/
19			class RedirectResolvingEntityLookupTest extends TestCase {
20
21			/**
22			* @param EntityId $id
23			*
24			* @return null\|Item
25			* @throws UnresolvedEntityRedirectException
26			*/
27			public function getEntity( EntityId $id ) {
28			switch ( $id->getSerialization() ) {
29			case 'Q10':
30			return new Item( $id );
			0 ignored issues – show Documentation introduced 2015-11-23 08:52 UTC by Report Bug Copy Issue Report `$id` is of type `object<Wikibase\DataModel\Entity\EntityId>`, but the function expects a `null\|object<Wikibase\DataModel\Entity\ItemId>`. 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...
31			case 'Q11':
32			throw new UnresolvedEntityRedirectException( new ItemId( 'Q11' ), new ItemId( 'Q10' ) );
33			case 'Q12':
34			throw new UnresolvedEntityRedirectException( new ItemId( 'Q12' ), new ItemId( 'Q11' ) );
35			case 'Q21':
36			throw new UnresolvedEntityRedirectException( new ItemId( 'Q21' ), new ItemId( 'Q20' ) );
37			default:
38			return null;
39			}
40			}
41
42			/**
43			* @return EntityLookup
44			*/
45			public function getLookupDouble() {
46			$mock = $this->createMock( EntityLookup::class );
47
48			$mock->expects( $this->any() )
49			->method( 'getEntity' )
50			->will( $this->returnCallback( [ $this, 'getEntity' ] ) );
51
52			$mock->expects( $this->any() )
53			->method( 'hasEntity' )
54			->will( $this->returnCallback( function ( EntityId $id ) {
55			return $this->getEntity( $id ) !== null;
56			} ) );
57
58			return $mock;
59			}
60
61			public function getEntityProvider() {
62			return [
63			'no redirect' => [ new ItemId( 'Q10' ), new ItemId( 'Q10' ) ],
64			'one redirect' => [ new ItemId( 'Q11' ), new ItemId( 'Q10' ) ],
65			];
66			}
67
68			/**
69			* @dataProvider getEntityProvider
70			*/
71			public function testGetEntity( EntityId $id, EntityId $expected ) {
72			$lookup = new RedirectResolvingEntityLookup( $this->getLookupDouble() );
			0 ignored issues – show Documentation introduced 2020-01-24 17:31 UTC by Report Bug Copy Issue Report `$this->getLookupDouble()` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<Wikibase\DataMode...es\Lookup\EntityLookup>`. 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...
73
74			$entity = $lookup->getEntity( $id );
75
76			if ( $expected === null ) {
77			$this->assertNull( $entity );
78			} else {
79			$this->assertTrue( $expected->equals( $entity->getId() ) );
80			}
81			}
82
83			public function testGetEntity_missing() {
84			$lookup = new RedirectResolvingEntityLookup( $this->getLookupDouble() );
			0 ignored issues – show Documentation introduced 2020-01-24 17:31 UTC by Report Bug Copy Issue Report `$this->getLookupDouble()` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<Wikibase\DataMode...es\Lookup\EntityLookup>`. 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...
85
86			$id = new ItemId( 'Q7' ); // entity Q7 is not known
87			$this->assertNull( $lookup->getEntity( $id ) );
88			}
89
90			public function testGetEntity_brokenRedirect() {
91			$lookup = new RedirectResolvingEntityLookup( $this->getLookupDouble() );
			0 ignored issues – show Documentation introduced 2020-01-24 17:31 UTC by Report Bug Copy Issue Report `$this->getLookupDouble()` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<Wikibase\DataMode...es\Lookup\EntityLookup>`. 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...
92
93			$id = new ItemId( 'Q20' ); // Q20 is a broken redirect
94			$this->assertNull( $lookup->getEntity( $id ) );
95			}
96
97			public function testGetEntity_doubleRedirect() {
98			$lookup = new RedirectResolvingEntityLookup( $this->getLookupDouble() );
			0 ignored issues – show Documentation introduced 2020-01-24 17:31 UTC by Report Bug Copy Issue Report `$this->getLookupDouble()` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<Wikibase\DataMode...es\Lookup\EntityLookup>`. 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...
99
100			$id = new ItemId( 'Q12' ); // Q12 is a double redirect
101
102			$this->expectException( UnresolvedEntityRedirectException::class );
103			$lookup->getEntity( $id );
104			}
105
106			public function hasEntityProvider() {
107			return [
108			'unknown entity' => [ new ItemId( 'Q7' ), false ],
109			'no redirect' => [ new ItemId( 'Q10' ), true ],
110			'one redirect' => [ new ItemId( 'Q11' ), true ],
111			'broken redirect' => [ new ItemId( 'Q21' ), false ],
112			];
113			}
114
115			/**
116			* @dataProvider hasEntityProvider
117			*/
118			public function testHasEntity( EntityId $id, $exists ) {
119			$lookup = new RedirectResolvingEntityLookup( $this->getLookupDouble() );
			0 ignored issues – show Documentation introduced 2020-01-24 17:31 UTC by Report Bug Copy Issue Report `$this->getLookupDouble()` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<Wikibase\DataMode...es\Lookup\EntityLookup>`. 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...
120
121			$this->assertEquals( $exists, $lookup->hasEntity( $id ) );
122			}
123
124			}
125

wmde / WikibaseDataModelServices

RedirectResolvingEntityLookupTest A last analyzed 2021-06-01 10:00 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

9 Methods

Duplication Side-by-Side

Filter issues like

RedirectResolvingEntityLookupTest A
last analyzed 2021-06-01 10:00 UTC