jenkins-bot has submitted this change and it was merged. Change subject: Cleanup SetClaimTest ......................................................................
Cleanup SetClaimTest Change-Id: I586674730f196806d9fe7a1a1691f0d640499230 --- M repo/tests/phpunit/includes/api/SetClaimTest.php 1 file changed, 40 insertions(+), 26 deletions(-) Approvals: Addshore: Looks good to me, approved jenkins-bot: Verified diff --git a/repo/tests/phpunit/includes/api/SetClaimTest.php b/repo/tests/phpunit/includes/api/SetClaimTest.php index 1d09874..08b7732 100644 --- a/repo/tests/phpunit/includes/api/SetClaimTest.php +++ b/repo/tests/phpunit/includes/api/SetClaimTest.php @@ -2,10 +2,24 @@ namespace Wikibase\Test\Api; use Wikibase\Claim; +use Wikibase\Claims; use Wikibase\DataModel\Entity\PropertyId; -use Wikibase\EntityId; +use Wikibase\DataModel\Entity\EntityId; use Wikibase\Property; use Wikibase\PropertyContent; +use Wikibase\Lib\Serializers\SerializerFactory; +use Wikibase\Repo\WikibaseRepo; +use Wikibase\Statement; +use Wikibase\Reference; +use Wikibase\Snak; +use Wikibase\SnakList; +use Wikibase\PropertyValueSnak; +use Wikibase\PropertyNoValueSnak; +use Wikibase\PropertySomeValueSnak; +use Wikibase\Item; +use Wikibase\ItemContent; +use Wikibase\Lib\ClaimGuidGenerator; + /** * Unit tests for the Wikibase\Repo\Api\ApSetClaim class. @@ -30,7 +44,7 @@ class SetClaimTest extends WikibaseApiTestCase { /** - * @return \Wikibase\Snak[] + * @return Snak[] */ protected function snakProvider() { static $hasProperties = false; @@ -60,9 +74,9 @@ $snaks = array(); - $snaks[] = new \Wikibase\PropertyNoValueSnak( $prop42 ); - $snaks[] = new \Wikibase\PropertySomeValueSnak( $prop9001 ); - $snaks[] = new \Wikibase\PropertyValueSnak( $prop7201010, new \DataValues\StringValue( 'o_O' ) ); + $snaks[] = new PropertyNoValueSnak( $prop42 ); + $snaks[] = new PropertySomeValueSnak( $prop9001 ); + $snaks[] = new PropertyValueSnak( $prop7201010, new \DataValues\StringValue( 'o_O' ) ); return $snaks; } @@ -75,35 +89,35 @@ $snaks = $this->snakProvider(); $mainSnak = $snaks[0]; - $statement = new \Wikibase\Statement( $mainSnak ); + $statement = new Statement( $mainSnak ); $statements[] = $statement; foreach ( $snaks as $snak ) { $statement = clone $statement; - $snaks = new \Wikibase\SnakList( array( $snak ) ); - $statement->getReferences()->addReference( new \Wikibase\Reference( $snaks ) ); + $snaks = new SnakList( array( $snak ) ); + $statement->getReferences()->addReference( new Reference( $snaks ) ); $statements[] = $statement; } $statement = clone $statement; - $snaks = new \Wikibase\SnakList( $this->snakProvider() ); - $statement->getReferences()->addReference( new \Wikibase\Reference( $snaks ) ); + $snaks = new SnakList( $this->snakProvider() ); + $statement->getReferences()->addReference( new Reference( $snaks ) ); $statements[] = $statement; $statement = clone $statement; - $snaks = new \Wikibase\SnakList( $this->snakProvider() ); + $snaks = new SnakList( $this->snakProvider() ); $statement->setQualifiers( $snaks ); - $statement->getReferences()->addReference( new \Wikibase\Reference( $snaks ) ); + $statement->getReferences()->addReference( new Reference( $snaks ) ); $statements[] = $statement; $ranks = array( - \Wikibase\Statement::RANK_DEPRECATED, - \Wikibase\Statement::RANK_NORMAL, - \Wikibase\Statement::RANK_PREFERRED + Statement::RANK_DEPRECATED, + Statement::RANK_NORMAL, + Statement::RANK_PREFERRED ); /** - * @var \Wikibase\Statement[] $statements + * @var Statement[] $statements */ foreach ( $statements as &$statement ) { $statement->setRank( $ranks[array_rand( $ranks )] ); @@ -114,11 +128,11 @@ public function testAddClaim() { foreach ( $this->claimProvider() as $claim ) { - $item = \Wikibase\Item::newEmpty(); - $content = new \Wikibase\ItemContent( $item ); + $item = Item::newEmpty(); + $content = new ItemContent( $item ); $content->save( '', null, EDIT_NEW ); - $guidGenerator = new \Wikibase\Lib\ClaimGuidGenerator( $item->getId() ); + $guidGenerator = new ClaimGuidGenerator( $item->getId() ); $guid = $guidGenerator->newGuid(); $claim->setGuid( $guid ); @@ -131,7 +145,7 @@ // Simply reorder the qualifiers by putting the first qualifier to the end. This is // supposed to be done in the serialized representation since changing the actual // object might apply intrinsic sorting. - $serializerFactory = new \Wikibase\Lib\Serializers\SerializerFactory(); + $serializerFactory = new SerializerFactory(); $serializer = $serializerFactory->newSerializerForObject( $claim ); $serializedClaim = $serializer->getSerialized( $claim ); $firstPropertyId = array_shift( $serializedClaim['qualifiers-order'] ); @@ -139,7 +153,7 @@ $this->makeRequest( $serializedClaim, $item->getId(), 1 ); } - $claim = new \Wikibase\Statement( new \Wikibase\PropertyNoValueSnak( 9001 ) ); + $claim = new Statement( new PropertyNoValueSnak( 9001 ) ); $claim->setGuid( $guid ); // Update request @@ -148,12 +162,12 @@ } /** - * @param \Wikibase\Claim|array $claim Native or serialized claim object. + * @param Claim|array $claim Native or serialized claim object. * @param EntityId $entityId * @param $claimCount */ - protected function makeRequest( $claim, \Wikibase\EntityId $entityId, $claimCount ) { - $serializerFactory = new \Wikibase\Lib\Serializers\SerializerFactory(); + protected function makeRequest( $claim, EntityId $entityId, $claimCount ) { + $serializerFactory = new SerializerFactory(); if( is_a( $claim, '\Wikibase\Claim' ) ) { $serializer = $serializerFactory->newSerializerForObject( $claim ); @@ -171,11 +185,11 @@ $this->makeValidRequest( $params ); - $content = \Wikibase\EntityContentFactory::singleton()->getFromId( $entityId ); + $content = WikibaseRepo::getDefaultInstance()->getEntityContentFactory()->getFromId( $entityId ); $this->assertInstanceOf( '\Wikibase\EntityContent', $content ); - $claims = new \Wikibase\Claims( $content->getEntity()->getClaims() ); + $claims = new Claims( $content->getEntity()->getClaims() ); $this->assertTrue( $claims->hasClaim( $claim ) ); -- To view, visit https://gerrit.wikimedia.org/r/84508 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I586674730f196806d9fe7a1a1691f0d640499230 Gerrit-PatchSet: 3 Gerrit-Project: mediawiki/extensions/Wikibase Gerrit-Branch: master Gerrit-Owner: Tobias Gritschacher <tobias.gritschac...@wikimedia.de> Gerrit-Reviewer: Addshore <addshorew...@gmail.com> Gerrit-Reviewer: Daniel Kinzler <daniel.kinz...@wikimedia.de> Gerrit-Reviewer: jenkins-bot _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits