Favor addNewSiteLink() over addSiteLink( new SiteLink() )

Author: thiemowmde

tests/unit/Entity/ItemTest.php

@@ -191,7 +191,7 @@ public function simpleSiteLinksProvider() {
 
 	public function testHasLinkToSiteForFalse() {
 		$item = new Item();
-		$item->getSiteLinkList()->addSiteLink( new SiteLink( 'ENWIKI', 'Wikidata', array( new ItemId( 'Q42' ) ) ) );
+		$item->getSiteLinkList()->addNewSiteLink( 'ENWIKI', 'Wikidata', array( new ItemId( 'Q42' ) ) );
 
 		$this->assertFalse( $item->getSiteLinkList()->hasLinkWithSiteId( 'enwiki' ) );
 		$this->assertFalse( $item->getSiteLinkList()->hasLinkWithSiteId( 'dewiki' ) );
@@ -200,9 +200,9 @@ public function testHasLinkToSiteForFalse() {
 
 	public function testHasLinkToSiteForTrue() {
 		$item = new Item();
-		$item->getSiteLinkList()->addSiteLink( new SiteLink( 'enwiki', 'Wikidata', array( new ItemId( 'Q42' ) ) ) );
-		$item->getSiteLinkList()->addSiteLink( new SiteLink( 'dewiki', 'Wikidata' ) );
-		$item->getSiteLinkList()->addSiteLink( new SiteLink( 'foo bar', 'Wikidata' ) );
+		$item->getSiteLinkList()->addNewSiteLink( 'enwiki', 'Wikidata', array( new ItemId( 'Q42' ) ) );
+		$item->getSiteLinkList()->addNewSiteLink( 'dewiki', 'Wikidata' );
+		$item->getSiteLinkList()->addNewSiteLink( 'foo bar', 'Wikidata' );
 
 		$this->assertTrue( $item->getSiteLinkList()->hasLinkWithSiteId( 'enwiki' ) );
 		$this->assertTrue( $item->getSiteLinkList()->hasLinkWithSiteId( 'dewiki' ) );
@@ -236,7 +236,7 @@ public function testEmptyItemReturnsEmptySiteLinkList() {
 	public function testAddSiteLinkOverridesOldLinks() {
 		$item = new Item();
 
-		$item->getSiteLinkList()->addSiteLink( new SiteLink( 'kittens', 'foo' ) );
+		$item->getSiteLinkList()->addNewSiteLink( 'kittens', 'foo' );
 
 		$newLink = new SiteLink( 'kittens', 'bar' );
 		$item->addSiteLink( $newLink );

tests/unit/Entity/TestItems.php

@@ -36,13 +36,13 @@ public static function getItems() {
 		$items[] = $item;
 
 		$item = new Item();
-		$item->getSiteLinkList()->addSiteLink( new SiteLink( 'enwiki', 'spam' ) );
+		$item->getSiteLinkList()->addNewSiteLink( 'enwiki', 'spam' );
 
 		$items[] = $item;
 
 		$item = new Item();
-		$item->getSiteLinkList()->addSiteLink( new SiteLink( 'enwiki', 'spamz' ) );
-		$item->getSiteLinkList()->addSiteLink( new SiteLink( 'dewiki', 'foobar' ) );
+		$item->getSiteLinkList()->addNewSiteLink( 'enwiki', 'spamz' );
+		$item->getSiteLinkList()->addNewSiteLink( 'dewiki', 'foobar' );
 
 		$item->setDescription( 'en', 'foo' );
 		$item->setLabel( 'en', 'bar' );