jenkins-bot has submitted this change and it was merged.
Change subject: Improve snak testToArrayRoundtrip test.
......................................................................
Improve snak testToArrayRoundtrip test.
The ->equals() method basically justchecks
to see thet the classtype is correct and
the hashes are the same. Having these checks
in the test allows failure output to be more
usefull
This will naturally not pass unitl the issue is
fixed in Change: Ic66442813528bb30
Change-Id: Idaa7bc41f1aee70b8e2897fdda4de73ea353a520
---
M tests/phpunit/Snak/SnakTest.php
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Aude: Looks good to me, approved
jenkins-bot: Verified
diff --git a/tests/phpunit/Snak/SnakTest.php b/tests/phpunit/Snak/SnakTest.php
index e8656bf..73c95f6 100644
--- a/tests/phpunit/Snak/SnakTest.php
+++ b/tests/phpunit/Snak/SnakTest.php
@@ -132,6 +132,7 @@
$copy = \Wikibase\SnakObject::newFromArray( $data );
$this->assertInstanceOf( '\Wikibase\Snak', $copy,
'newFromArray should return object implementing Snak' );
+ $this->assertEquals( $snak->getHash(),
$copy->getHash(), 'newFromArray should return object with same Hash used
previously' );
$this->assertTrue( $snak->equals( $copy ), 'getArray
newFromArray roundtrip should work' );
}
--
To view, visit https://gerrit.wikimedia.org/r/80795
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Idaa7bc41f1aee70b8e2897fdda4de73ea353a520
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/WikibaseDataModel
Gerrit-Branch: master
Gerrit-Owner: Addshore <[email protected]>
Gerrit-Reviewer: Addshore <[email protected]>
Gerrit-Reviewer: Anja Jentzsch <[email protected]>
Gerrit-Reviewer: Ataherivand <[email protected]>
Gerrit-Reviewer: Aude <[email protected]>
Gerrit-Reviewer: Daniel Kinzler <[email protected]>
Gerrit-Reviewer: Daniel Werner <[email protected]>
Gerrit-Reviewer: Denny Vrandecic <[email protected]>
Gerrit-Reviewer: Henning Snater <[email protected]>
Gerrit-Reviewer: Hoo man <[email protected]>
Gerrit-Reviewer: Jens Ohlig <[email protected]>
Gerrit-Reviewer: Jeroen De Dauw <[email protected]>
Gerrit-Reviewer: John Erling Blad <[email protected]>
Gerrit-Reviewer: Liangent <[email protected]>
Gerrit-Reviewer: Lydia Pintscher <[email protected]>
Gerrit-Reviewer: Markus Kroetzsch <[email protected]>
Gerrit-Reviewer: Nikola Smolenski <[email protected]>
Gerrit-Reviewer: Nilesh <[email protected]>
Gerrit-Reviewer: Tobias Gritschacher <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
