Change some assertEquals to assertSame

lucaswerkmeister · lucaswerkmeister · commit 799be75f15d6 · 2020-10-16T15:28:52.000+02:00
This is a good practice in general, but also has the added benefit of
pushing one long line just below 120 characters, avoiding a phpcs
warning.

These changes include some assertions where the values being compared
are objects, i.e. we now compare their identity. This is certainly
debatable, but I think that in some cases, it makes sense that a
“container” object, like a StatementList, the objects coming out of the
container are identical, not just equal, to the ones put in. On the
other hand, this is certainly a bit fuzzy and debatable (they’re not
generic containers – why shouldn’t they be allowed to store their
contents in a more efficient form and return new but equal objects
later?), and I doubt I’ve been fully consistent here either, so this can
also be changed later.

Bug: T253636
diff --git a/tests/unit/Entity/EntityIdTest.php b/tests/unit/Entity/EntityIdTest.php
@@ -72,7 +72,7 @@ public function testSerializationStability() {
 		$serialization = 'C:32:"Wikibase\DataModel\Entity\ItemId":4:{Q123}';
 		$id = new ItemId( 'q123' );
 
-		$this->assertEquals(
+		$this->assertSame(
 			serialize( $id ),
 			$serialization
 		);
diff --git a/tests/unit/Entity/EntityIdValueTest.php b/tests/unit/Entity/EntityIdValueTest.php
@@ -62,7 +62,7 @@ public function instanceProvider() {
 	 * @dataProvider instanceProvider
 	 */
 	public function testGetType( EntityIdValue $id ) {
-		$this->assertEquals( 'wikibase-entityid', $id->getType() );
+		$this->assertSame( 'wikibase-entityid', $id->getType() );
 	}
 
 	/**
diff --git a/tests/unit/Entity/EntityRedirectTest.php b/tests/unit/Entity/EntityRedirectTest.php
@@ -80,10 +80,10 @@ public function equalsProvider() {
 	 * @param bool $expected
 	 */
 	public function testEquals( EntityRedirect $a, $b, $expected ) {
-		$this->assertEquals( $expected, $a->equals( $b ), '$a->equals( $b )' );
+		$this->assertSame( $expected, $a->equals( $b ), '$a->equals( $b )' );
 
 		if ( $b instanceof EntityRedirect ) {
-			$this->assertEquals( $expected, $b->equals( $a ), '$b->equals( $a )' );
+			$this->assertSame( $expected, $b->equals( $a ), '$b->equals( $a )' );
 		}
 	}
 
diff --git a/tests/unit/Entity/ItemIdSetTest.php b/tests/unit/Entity/ItemIdSetTest.php
@@ -27,7 +27,7 @@ public function testGetIterator() {
 	 */
 	public function testGetSerializations( array $itemIds, array $expected ) {
 		$set = new ItemIdSet( $itemIds );
-		$this->assertEquals( $expected, $set->getSerializations() );
+		$this->assertSame( $expected, $set->getSerializations() );
 	}
 
 	public function serializationsProvider() {
@@ -48,7 +48,7 @@ public function testGivenSetWithTwoItems_countReturnsTwo() {
 			new ItemId( 'Q1' ),
 			new ItemId( 'Q2' ),
 		] );
-		$this->assertEquals( 2, $set->count() );
+		$this->assertSame( 2, $set->count() );
 	}
 
 	public function testGivenNotSetId_hasReturnsFalse() {
diff --git a/tests/unit/Entity/ItemIdTest.php b/tests/unit/Entity/ItemIdTest.php
@@ -23,7 +23,7 @@ class ItemIdTest extends \PHPUnit\Framework\TestCase {
 	public function testCanConstructId( $idSerialization, $normalizedIdSerialization ) {
 		$id = new ItemId( $idSerialization );
 
-		$this->assertEquals(
+		$this->assertSame(
 			$normalizedIdSerialization,
 			$id->getSerialization()
 		);
@@ -123,7 +123,7 @@ public function serializationProvider() {
 	 */
 	public function testNewFromNumber( $number ) {
 		$id = ItemId::newFromNumber( $number );
-		$this->assertEquals( 'Q' . $number, $id->getSerialization() );
+		$this->assertSame( 'Q' . $number, $id->getSerialization() );
 	}
 
 	public function numericIdProvider() {
diff --git a/tests/unit/Entity/PropertyIdTest.php b/tests/unit/Entity/PropertyIdTest.php
@@ -23,7 +23,7 @@ class PropertyIdTest extends \PHPUnit\Framework\TestCase {
 	public function testCanConstructId( $idSerialization, $normalizedIdSerialization ) {
 		$id = new PropertyId( $idSerialization );
 
-		$this->assertEquals(
+		$this->assertSame(
 			$normalizedIdSerialization,
 			$id->getSerialization()
 		);
@@ -123,7 +123,7 @@ public function serializationProvider() {
 	 */
 	public function testNewFromNumber( $number ) {
 		$id = PropertyId::newFromNumber( $number );
-		$this->assertEquals( 'P' . $number, $id->getSerialization() );
+		$this->assertSame( 'P' . $number, $id->getSerialization() );
 	}
 
 	public function numericIdProvider() {
diff --git a/tests/unit/Entity/PropertyTest.php b/tests/unit/Entity/PropertyTest.php
@@ -617,7 +617,7 @@ public function testClear( Property $property ) {
 			$property->getId(),
 			'cleared Property should keep its id'
 		);
-		$this->assertEquals(
+		$this->assertSame(
 			$clone->getDataTypeId(),
 			$property->getDataTypeId(),
 			'cleared Property should keep its data type'
diff --git a/tests/unit/Internal/MapValueHasherTest.php b/tests/unit/Internal/MapValueHasherTest.php
@@ -40,10 +40,10 @@ public function testHash() {
 		unset( $map1['foo'] );
 		$map1[] = $map0['foo'];
 
-		$this->assertEquals( $hash, $hasher->hash( $map1 ) );
+		$this->assertSame( $hash, $hasher->hash( $map1 ) );
 
 		$map4 = new ArrayObject( $map0 );
-		$this->assertEquals( $hash, $hasher->hash( $map4 ) );
+		$this->assertSame( $hash, $hasher->hash( $map4 ) );
 
 		$map2 = $map0;
 		unset( $map2['foo'] );
diff --git a/tests/unit/ReferenceTest.php b/tests/unit/ReferenceTest.php
@@ -92,15 +92,15 @@ public function testGetHashReturnsString( Reference $reference ) {
 	 * @dataProvider instanceProvider
 	 */
 	public function testGetHashIsStable( Reference $reference ) {
-		$this->assertEquals( $reference->getHash(), $reference->getHash() );
+		$this->assertSame( $reference->getHash(), $reference->getHash() );
 	}
 
 	/**
 	 * @dataProvider instanceProvider
 	 */
 	public function testGetHashIsTheSameForInstanceWithSameValue( Reference $reference ) {
 		$newRef = unserialize( serialize( $reference ) );
-		$this->assertEquals( $newRef->getHash(), $reference->getHash() );
+		$this->assertSame( $newRef->getHash(), $reference->getHash() );
 	}
 
 	/**
@@ -174,7 +174,7 @@ public function unorderedReferenceProvider() {
 	 * @dataProvider unorderedReferenceProvider
 	 */
 	public function testUnorderedReference( Reference $unorderedReference, Reference $orderedReference ) {
-		$this->assertEquals( $unorderedReference->getHash(), $orderedReference->getHash() );
+		$this->assertSame( $unorderedReference->getHash(), $orderedReference->getHash() );
 	}
 
 	public function testReferenceEqualsItself() {
diff --git a/tests/unit/SiteLinkTest.php b/tests/unit/SiteLinkTest.php
@@ -31,7 +31,7 @@ public function testCanConstruct() {
 	 */
 	public function testGetSiteId( $siteId ) {
 		$siteLink = new SiteLink( $siteId, 'Wikidata' );
-		$this->assertEquals( $siteId, $siteLink->getSiteId() );
+		$this->assertSame( $siteId, $siteLink->getSiteId() );
 	}
 
 	public function siteIdProvider() {
@@ -67,7 +67,7 @@ public function invalidStringIdentifierProvider() {
 	 */
 	public function testGetPageName( $pageName ) {
 		$siteLink = new SiteLink( 'enwiki', $pageName );
-		$this->assertEquals( $pageName, $siteLink->getPageName() );
+		$this->assertSame( $pageName, $siteLink->getPageName() );
 	}
 
 	public function pageNameProvider() {
diff --git a/tests/unit/Snak/DerivedPropertyValueSnakTest.php b/tests/unit/Snak/DerivedPropertyValueSnakTest.php
@@ -127,7 +127,7 @@ public function testSerializationDoesNotContainDerivedValues() {
 			[ 'foo' => new StringValue( 'foo' ), 'bar' => new StringValue( 'bar' ) ]
 		);
 
-		$this->assertEquals(
+		$this->assertSame(
 			'a:2:{i:0;s:5:"P9001";i:1;C:22:"DataValues\StringValue":2:{bc}}',
 			$snak->serialize()
 		);
diff --git a/tests/unit/Snak/PropertyNoValueSnakTest.php b/tests/unit/Snak/PropertyNoValueSnakTest.php
@@ -65,7 +65,7 @@ public function testGetHash() {
 		$snak = new PropertyNoValueSnak( new PropertyId( 'P1' ) );
 		$hash = $snak->getHash();
 		$this->assertIsString( $hash );
-		$this->assertEquals( 40, strlen( $hash ) );
+		$this->assertSame( 40, strlen( $hash ) );
 	}
 
 	/**
diff --git a/tests/unit/Snak/PropertySomeValueSnakTest.php b/tests/unit/Snak/PropertySomeValueSnakTest.php
@@ -65,7 +65,7 @@ public function testGetHash() {
 		$snak = new PropertySomeValueSnak( new PropertyId( 'P1' ) );
 		$hash = $snak->getHash();
 		$this->assertIsString( $hash );
-		$this->assertEquals( 40, strlen( $hash ) );
+		$this->assertSame( 40, strlen( $hash ) );
 	}
 
 	/**
diff --git a/tests/unit/Snak/PropertyValueSnakTest.php b/tests/unit/Snak/PropertyValueSnakTest.php
@@ -76,7 +76,7 @@ public function testGetHash() {
 		$snak = new PropertyValueSnak( new PropertyId( 'P1' ), new StringValue( 'a' ) );
 		$hash = $snak->getHash();
 		$this->assertIsString( $hash );
-		$this->assertEquals( 40, strlen( $hash ) );
+		$this->assertSame( 40, strlen( $hash ) );
 	}
 
 	/**
diff --git a/tests/unit/Snak/SnakListTest.php b/tests/unit/Snak/SnakListTest.php
@@ -125,7 +125,7 @@ public function testRemoveSnak( SnakList $array ) {
 			}
 
 			$this->assertFalse( $array->hasSnak( $element ) );
-			$this->assertEquals( --$elementCount, $array->count() );
+			$this->assertSame( --$elementCount, $array->count() );
 		}
 
 		$element = new PropertyNoValueSnak( new PropertyId( 'P42' ) );
@@ -149,13 +149,13 @@ public function testAddSnak( SnakList $array ) {
 			++$elementCount;
 		}
 
-		$this->assertEquals( !$array->hasSnak( $element ), $array->addSnak( $element ) );
+		$this->assertSame( !$array->hasSnak( $element ), $array->addSnak( $element ) );
 
-		$this->assertEquals( $elementCount, $array->count() );
+		$this->assertSame( $elementCount, $array->count() );
 
 		$this->assertFalse( $array->addSnak( $element ) );
 
-		$this->assertEquals( $elementCount, $array->count() );
+		$this->assertSame( $elementCount, $array->count() );
 	}
 
 	public function orderByPropertyProvider() {
diff --git a/tests/unit/Snak/SnakTest.php b/tests/unit/Snak/SnakTest.php
@@ -85,7 +85,7 @@ public function testSerialize( Snak $snak ) {
 
 		$this->assertEquals( $snak, $newInstance );
 		$this->assertEquals( $snak->getPropertyId(), $newInstance->getPropertyId() );
-		$this->assertEquals( $snak->getType(), $newInstance->getType() );
+		$this->assertSame( $snak->getType(), $newInstance->getType() );
 	}
 
 	/**
@@ -95,7 +95,7 @@ public function testSerialize( Snak $snak ) {
 	public function testGetHash( Snak $snak ) {
 		$hash = $snak->getHash();
 		$this->assertIsString( $hash );
-		$this->assertEquals( $hash, $snak->getHash() );
+		$this->assertSame( $hash, $snak->getHash() );
 	}
 
 	/**
diff --git a/tests/unit/Snak/TypedSnakTest.php b/tests/unit/Snak/TypedSnakTest.php
@@ -25,7 +25,7 @@ public function testGettersReturnCorrectValues() {
 		$typedSnak = new TypedSnak( $snak, $dataTypeId );
 
 		$this->assertEquals( $snak, $typedSnak->getSnak() );
-		$this->assertEquals( $dataTypeId, $typedSnak->getDataTypeId() );
+		$this->assertSame( $dataTypeId, $typedSnak->getDataTypeId() );
 	}
 
 }
diff --git a/tests/unit/Statement/StatementByGuidMapTest.php b/tests/unit/Statement/StatementByGuidMapTest.php
@@ -63,7 +63,7 @@ public function testGivenGuidOfPresentStatement_getStatementByGuidReturnsStateme
 
 		$statements = new StatementByGuidMap( [ $statement ] );
 
-		$this->assertEquals( $statement, $statements->getStatementByGuid( 'some guid' ) );
+		$this->assertSame( $statement, $statements->getStatementByGuid( 'some guid' ) );
 	}
 
 	public function testGivenGuidOfNotPresentStatement_getStatementByGuidReturnsNull() {
@@ -160,7 +160,7 @@ public function testMapCanBeIteratedOver() {
 			'other guid' => $statement2
 		];
 
-		$this->assertEquals( $expectedStatements, $iteratedStatements );
+		$this->assertSame( $expectedStatements, $iteratedStatements );
 	}
 
 	public function testGivenNotPresentStatement_addStatementAddsIt() {
@@ -179,7 +179,7 @@ public function testGivenStatementWithPresentGuid_addStatementReplacesThePresent
 
 		$statements->addStatement( $statement2 );
 
-		$this->assertEquals( $statement2, $statements->getStatementByGuid( 'some guid' ) );
+		$this->assertSame( $statement2, $statements->getStatementByGuid( 'some guid' ) );
 	}
 
 	public function testToArray() {
@@ -193,7 +193,7 @@ public function testToArray() {
 			'other guid' => $statement2
 		];
 
-		$this->assertEquals( $expectedStatements, $statementMap->toArray() );
+		$this->assertSame( $expectedStatements, $statementMap->toArray() );
 	}
 
 }
diff --git a/tests/unit/Statement/StatementGuidTest.php b/tests/unit/Statement/StatementGuidTest.php
@@ -26,7 +26,7 @@ public function testConstructor( EntityId $entityId, $guid, $expected ) {
 
 		$this->assertSame( $expected, $statementGuid->getSerialization() );
 		$this->assertEquals( $entityId, $statementGuid->getEntityId() );
-		$this->assertEquals( $guid, $statementGuid->getGuidPart() );
+		$this->assertSame( $guid, $statementGuid->getGuidPart() );
 	}
 
 	public function provideConstructionData() {
diff --git a/tests/unit/Statement/StatementListTest.php b/tests/unit/Statement/StatementListTest.php
@@ -138,7 +138,7 @@ public function testCanIterate() {
 		$list = new StatementList( $statement );
 
 		foreach ( $list as $statementFormList ) {
-			$this->assertEquals( $statement, $statementFormList );
+			$this->assertSame( $statement, $statementFormList );
 		}
 	}
 
@@ -340,7 +340,7 @@ public function testCanConstructWithTraversableContainingOnlyStatements() {
 		$object = new ArrayObject( $statementArray );
 		$list = new StatementList( $object );
 
-		$this->assertEquals(
+		$this->assertSame(
 			$statementArray,
 			array_values( $list->toArray() )
 		);
diff --git a/tests/unit/Statement/StatementTest.php b/tests/unit/Statement/StatementTest.php
@@ -64,7 +64,7 @@ public function validConstructorArgumentsProvider() {
 	 */
 	public function testSetGuid( Statement $statement ) {
 		$statement->setGuid( 'foo-bar-baz' );
-		$this->assertEquals( 'foo-bar-baz', $statement->getGuid() );
+		$this->assertSame( 'foo-bar-baz', $statement->getGuid() );
 	}
 
 	/**
@@ -73,10 +73,10 @@ public function testSetGuid( Statement $statement ) {
 	public function testGetGuid( Statement $statement ) {
 		$guid = $statement->getGuid();
 		$this->assertTrue( $guid === null || is_string( $guid ) );
-		$this->assertEquals( $guid, $statement->getGuid() );
+		$this->assertSame( $guid, $statement->getGuid() );
 
 		$statement->setGuid( 'foobar' );
-		$this->assertEquals( 'foobar', $statement->getGuid() );
+		$this->assertSame( 'foobar', $statement->getGuid() );
 	}
 
 	public function testHashStability() {
@@ -110,7 +110,7 @@ public function testSetAndGetQualifiers() {
 	public function testSerialize( Statement $statement ) {
 		$copy = unserialize( serialize( $statement ) );
 
-		$this->assertEquals( $statement->getHash(), $copy->getHash(), 'Serialization roundtrip should not affect hash' );
+		$this->assertSame( $statement->getHash(), $copy->getHash(), 'Serialization roundtrip should not affect hash' );
 	}
 
 	public function testGuidDoesNotAffectHash() {
@@ -120,7 +120,7 @@ public function testGuidDoesNotAffectHash() {
 		$statement1 = new Statement( new PropertyNoValueSnak( 42 ) );
 		$statement1->setGuid( 'statement1' );
 
-		$this->assertEquals( $statement0->getHash(), $statement1->getHash() );
+		$this->assertSame( $statement0->getHash(), $statement1->getHash() );
 	}
 
 	/**
@@ -206,7 +206,7 @@ public function testSetReferences( Statement $statement ) {
 
 		$statement->setReferences( $references );
 
-		$this->assertEquals( $references, $statement->getReferences() );
+		$this->assertSame( $references, $statement->getReferences() );
 	}
 
 	/**
@@ -250,7 +250,7 @@ public function testGetRank( Statement $statement ) {
 	 */
 	public function testSetRank( Statement $statement ) {
 		$statement->setRank( Statement::RANK_DEPRECATED );
-		$this->assertEquals( Statement::RANK_DEPRECATED, $statement->getRank() );
+		$this->assertSame( Statement::RANK_DEPRECATED, $statement->getRank() );
 	}
 
 	/**
@@ -265,7 +265,7 @@ public function testSetInvalidRank( Statement $statement ) {
 	 * @dataProvider instanceProvider
 	 */
 	public function testGetPropertyId( Statement $statement ) {
-		$this->assertEquals(
+		$this->assertSame(
 			$statement->getMainSnak()->getPropertyId(),
 			$statement->getPropertyId()
 		);
diff --git a/tests/unit/Term/TermListTest.php b/tests/unit/Term/TermListTest.php

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ public function instanceProvider() {`
`62`	`62`	`* @dataProvider instanceProvider`
`63`	`63`	`*/`
`64`	`64`	`public function testGetType( EntityIdValue $id ) {`
`65`		`- $this->assertEquals( 'wikibase-entityid', $id->getType() );`
	`65`	`+ $this->assertSame( 'wikibase-entityid', $id->getType() );`
`66`	`66`	`}`
`67`	`67`
`68`	`68`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -80,10 +80,10 @@ public function equalsProvider() {`
`80`	`80`	`* @param bool $expected`
`81`	`81`	`*/`
`82`	`82`	`public function testEquals( EntityRedirect $a, $b, $expected ) {`
`83`		`- $this->assertEquals( $expected, $a->equals( $b ), '$a->equals( $b )' );`
	`83`	`+ $this->assertSame( $expected, $a->equals( $b ), '$a->equals( $b )' );`
`84`	`84`
`85`	`85`	`if ( $b instanceof EntityRedirect ) {`
`86`		`- $this->assertEquals( $expected, $b->equals( $a ), '$b->equals( $a )' );`
	`86`	`+ $this->assertSame( $expected, $b->equals( $a ), '$b->equals( $a )' );`
`87`	`87`	`}`
`88`	`88`	`}`
`89`	`89`