M4tx has uploaded a new change for review.
https://gerrit.wikimedia.org/r/186083
Change subject: Make data provider in MediaHandlerTest static
......................................................................
Make data provider in MediaHandlerTest static
Also, improve function names.
Follows-up Ie1cf501a6a0c8e688aca1a5577a293f526398dd3
Change-Id: I5eef5f193192041d7b0514eaa8b779c03e6647c7
---
M tests/phpunit/includes/media/MediaHandlerTest.php
1 file changed, 6 insertions(+), 6 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/83/186083/1
diff --git a/tests/phpunit/includes/media/MediaHandlerTest.php
b/tests/phpunit/includes/media/MediaHandlerTest.php
index 5fa609a..6c9d7b1 100644
--- a/tests/phpunit/includes/media/MediaHandlerTest.php
+++ b/tests/phpunit/includes/media/MediaHandlerTest.php
@@ -19,24 +19,24 @@
"($width, $height, $max) wanted: {$expected}x$y, got:
{z$result}x$y2" );
}
- public function provideTestFitBoxWidth() {
+ public static function provideTestFitBoxWidth() {
return array_merge(
- $this->provideTestFitBoxWidthSingle( 50, 50, array(
+ static::generateTestFitBoxWidthData( 50, 50, array(
50 => 50,
17 => 17,
18 => 18 )
),
- $this->provideTestFitBoxWidthSingle( 366, 300, array(
+ static::generateTestFitBoxWidthData( 366, 300, array(
50 => 61,
17 => 21,
18 => 22 )
),
- $this->provideTestFitBoxWidthSingle( 300, 366, array(
+ static::generateTestFitBoxWidthData( 300, 366, array(
50 => 41,
17 => 14,
18 => 15 )
),
- $this->provideTestFitBoxWidthSingle( 100, 400, array(
+ static::generateTestFitBoxWidthData( 100, 400, array(
50 => 12,
17 => 4,
18 => 4 )
@@ -44,7 +44,7 @@
);
}
- private function provideTestFitBoxWidthSingle( $width, $height, $tests
) {
+ private static function generateTestFitBoxWidthData( $width, $height,
$tests ) {
$result = array();
foreach ( $tests as $max => $expected ) {
$result[] = array( $width, $height, $max, $expected );
--
To view, visit https://gerrit.wikimedia.org/r/186083
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5eef5f193192041d7b0514eaa8b779c03e6647c7
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: M4tx <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
