Tobias Gritschacher has submitted this change and it was merged.

Change subject: Fix LangLinkHandlerTest to use spaces, not underscores
......................................................................


Fix LangLinkHandlerTest to use spaces, not underscores

- follow up to change 9a21a0625

Change-Id: Iadaf13e7fe9309f77f9d1c91f03976e6b1b9b403
---
M client/tests/phpunit/includes/LangLinkHandlerTest.php
1 file changed, 21 insertions(+), 21 deletions(-)

Approvals:
  Tobias Gritschacher: Verified; Looks good to me, approved
  jenkins-bot: Verified



diff --git a/client/tests/phpunit/includes/LangLinkHandlerTest.php 
b/client/tests/phpunit/includes/LangLinkHandlerTest.php
index 0ec3516..03c6aec 100644
--- a/client/tests/phpunit/includes/LangLinkHandlerTest.php
+++ b/client/tests/phpunit/includes/LangLinkHandlerTest.php
@@ -49,18 +49,18 @@
                        'id' => 1,
                        'label' => array( 'en' => 'Foo' ),
                        'links' => array(
-                               'dewiki' => 'Foo_de',
-                               'enwiki' => 'Foo_en',
-                               'srwiki' => 'Foo_sr',
+                               'dewiki' => 'Foo de',
+                               'enwiki' => 'Foo en',
+                               'srwiki' => 'Foo sr',
                        )
                ),
                array( // matches, but not in a namespace with external 
langlinks enabled
                        'id' => 2,
                        'label' => array( 'en' => 'Talk:Foo' ),
                        'links' => array(
-                               'dewiki' => 'Talk:Foo_de',
-                               'enwiki' => 'Talk:Foo_en',
-                               'srwiki' => 'Talk:Foo_sr',
+                               'dewiki' => 'Talk:Foo de',
+                               'enwiki' => 'Talk:Foo en',
+                               'srwiki' => 'Talk:Foo sr',
                        )
                )
        );
@@ -101,9 +101,9 @@
                        array( // #1
                                'Foo_sr', // page
                                array( // expected links
-                                       'dewiki' => 'Foo_de',
-                                       'enwiki' => 'Foo_en',
-                                       'srwiki' => 'Foo_sr',
+                                       'dewiki' => 'Foo de',
+                                       'enwiki' => 'Foo en',
+                                       'srwiki' => 'Foo sr',
                                )
                        ),
                );
@@ -244,20 +244,20 @@
                        array( // #0: local overrides remote
                                'Foo_sr', // title
                                array( // langlinks
-                                       'de' => 'Xoo_de',
-                                       'nl' => 'Foo_nl',
+                                       'de' => 'Xoo de',
+                                       'nl' => 'Foo nl',
                                ),
                                array( // noexternallinks
                                ),
                                array( // expectedLinks
-                                       'enwiki' => 'Foo_en',
+                                       'enwiki' => 'Foo en',
                                )
                        ),
                        array( // #1: namespace not covered
                                'Talk:Foo_sr', // title
                                array( // langlinks
-                                       'de' => 'Talk:Foo_de',
-                                       'nl' => 'Talk:Foo_nl',
+                                       'de' => 'Talk:Foo de',
+                                       'nl' => 'Talk:Foo nl',
                                ),
                                array( // noexternallinks
                                ),
@@ -267,8 +267,8 @@
                        array( // #2: disabled
                                'Foo_sr', // title
                                array( // langlinks
-                                       'de' => 'Foo_de',
-                                       'nl' => 'Foo_nl',
+                                       'de' => 'Foo de',
+                                       'nl' => 'Foo nl',
                                ),
                                array( // noexternallinks
                                        '*'
@@ -279,8 +279,8 @@
                        array( // #3: suppressed
                                'Foo_sr', // title
                                array( // langlinks
-                                       'de' => 'Foo_de',
-                                       'nl' => 'Foo_nl',
+                                       'de' => 'Foo de',
+                                       'nl' => 'Foo nl',
                                ),
                                array( // noexternallinks
                                        'en'
@@ -291,14 +291,14 @@
                        array( // #4: suppressed redundantly
                                'Foo_sr', // title
                                array( // langlinks
-                                       'de' => 'Foo_de',
-                                       'nl' => 'Foo_nl',
+                                       'de' => 'Foo de',
+                                       'nl' => 'Foo nl',
                                ),
                                array( // noexternallinks
                                        'de'
                                ),
                                array( // expectedLinks
-                                       'enwiki' => 'Foo_en',
+                                       'enwiki' => 'Foo en',
                                )
                        ),
                );

-- 
To view, visit https://gerrit.wikimedia.org/r/61979
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iadaf13e7fe9309f77f9d1c91f03976e6b1b9b403
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Aude <aude.w...@gmail.com>
Gerrit-Reviewer: Daniel Kinzler <daniel.kinz...@wikimedia.de>
Gerrit-Reviewer: Tobias Gritschacher <tobias.gritschac...@wikimedia.de>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to