From d461d5c8afff7e3f9bdcd4fa9f0efa0fc359b9cd Mon Sep 17 00:00:00 2001 From: "Amir E. Aharoni" Date: Thu, 1 Oct 2015 23:16:46 +0300 Subject: [PATCH] Make lines short to pass phpcs in WikitextContentTest.php Bug: T102614 Change-Id: Id5f12d4cca1d952580d2bb68fdbcb3cfb2467085 --- .../includes/content/WikitextContentTest.php | 45 ++++++++++++++----- 1 file changed, 35 insertions(+), 10 deletions(-) diff --git a/tests/phpunit/includes/content/WikitextContentTest.php b/tests/phpunit/includes/content/WikitextContentTest.php index b436ebfa97..f0d486ad07 100644 --- a/tests/phpunit/includes/content/WikitextContentTest.php +++ b/tests/phpunit/includes/content/WikitextContentTest.php @@ -90,7 +90,11 @@ more stuff foreach ( $fieldValues as $field => $value ) { $v = $update->$field; # if the field doesn't exist, just crash and burn - $this->assertEquals( $value, $v, "unexpected value for field $field in instance of $class" ); + $this->assertEquals( + $value, + $v, + "unexpected value for field $field in instance of $class" + ); } } @@ -208,10 +212,12 @@ just a test" public static function dataPreloadTransform() { return array( - array( 'hello this is ~~~', + array( + 'hello this is ~~~', "hello this is ~~~", ), - array( 'hello \'\'this\'\' is foobar', + array( + 'hello \'\'this\'\' is foobar', 'hello \'\'this\'\' is bar', ), ); @@ -318,7 +324,10 @@ just a test" $this->assertTrue( $content->matchMagicWord( $mw ), "should have matched magic word" ); $content = $this->newContent( "#REDIRECT [[FOO]]" ); - $this->assertFalse( $content->matchMagicWord( $mw ), "should not have matched magic word" ); + $this->assertFalse( + $content->matchMagicWord( $mw ), + "should not have matched magic word" + ); } /** @@ -340,7 +349,10 @@ just a test" $this->assertFalse( $content->equals( $newContent ), "content should have changed" ); $this->assertTrue( $newContent->isRedirect(), "new content should be a redirect" ); - $this->assertEquals( $target->getFullText(), $newContent->getRedirectTarget()->getFullText() ); + $this->assertEquals( + $target->getFullText(), + $newContent->getRedirectTarget()->getFullText() + ); } /** @@ -394,15 +406,28 @@ just a test" // Test with a redirect page $wikitext = false; $redirectTarget = false; - $content = $this->newContent( "#REDIRECT [[TestRedirectParserOption/redir]]\nhello redirect." ); + $content = $this->newContent( + "#REDIRECT [[TestRedirectParserOption/redir]]\nhello redirect." + ); $options = $content->getContentHandler()->makeParserOptions( 'canonical' ); $content->getParserOutput( $title, null, $options ); - $this->assertEquals( 'hello redirect.', $wikitext, 'Wikitext passed to hook was not as expected' ); - $this->assertNotEquals( null, $redirectTarget, 'Redirect seen in hook was null' ); - $this->assertEquals( 'TestRedirectParserOption/redir', $redirectTarget->getFullText(), + $this->assertEquals( + 'hello redirect.', + $wikitext, + 'Wikitext passed to hook was not as expected' + ); + $this->assertNotEquals( + null, + $redirectTarget, + 'Redirect seen in hook was null' ); + $this->assertEquals( + 'TestRedirectParserOption/redir', + $redirectTarget->getFullText(), 'Redirect seen in hook was not the expected title' ); - $this->assertEquals( null, $options->getRedirectTarget(), + $this->assertEquals( + null, + $options->getRedirectTarget(), 'ParserOptions\' redirectTarget was changed' ); } -- 2.20.1