Tamslo has submitted this change and it was merged.
Change subject: Fix for PHP 5.3
......................................................................
Fix for PHP 5.3
Change-Id: I91f650bf5ac0d05039e6d4c93c7d17047c1f4326
---
M tests/phpunit/Violations/CheckResultToViolationTranslatorTest.php
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Tamslo: Verified; Looks good to me, approved
diff --git a/tests/phpunit/Violations/CheckResultToViolationTranslatorTest.php
b/tests/phpunit/Violations/CheckResultToViolationTranslatorTest.php
index dc5e1cb..bebc810 100644
--- a/tests/phpunit/Violations/CheckResultToViolationTranslatorTest.php
+++ b/tests/phpunit/Violations/CheckResultToViolationTranslatorTest.php
@@ -135,7 +135,8 @@
$this->assertEquals(
'wbqc|P1$aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeeeRangemandatory',
$violation->getConstraintId() );
$this->assertEquals( $checkResult->getConstraintName(),
$violation->getConstraintTypeEntityId() );
$this->assertEquals( 42, $violation->getRevisionId() );
- $this->assertEquals( 'mandatory',
$violation->getAdditionalInfo()['constraint_status'][0] );
+ $additionalInfo = $violation->getAdditionalInfo();
+ $this->assertEquals( 'mandatory',
$additionalInfo['constraint_status'][0] );
}
public function testMultipleCheckResults() {
--
To view, visit https://gerrit.wikimedia.org/r/217272
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I91f650bf5ac0d05039e6d4c93c7d17047c1f4326
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikidataQualityConstraints
Gerrit-Branch: master
Gerrit-Owner: Tamslo <tamaraslosa...@gmail.com>
Gerrit-Reviewer: Tamslo <tamaraslosa...@gmail.com>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
