jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/404735 )

Change subject: tests: Quick fix for unit tests asserting the shape of OOUI ids
......................................................................


tests: Quick fix for unit tests asserting the shape of OOUI ids

Longer term, these tests probably shouldn't rely on this detail, but this
unbreaks the tests for now.

Change-Id: Id1a794249d9b5ee3ca6200cb661e37b319155696
---
M tests/phpunit/HTMLFeatureFieldTest.php
M tests/phpunit/NewHTMLCheckFieldTest.php
2 files changed, 4 insertions(+), 4 deletions(-)

Approvals:
  Bartosz Dziewoński: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/tests/phpunit/HTMLFeatureFieldTest.php 
b/tests/phpunit/HTMLFeatureFieldTest.php
index bbbf8c8..3266e7f 100644
--- a/tests/phpunit/HTMLFeatureFieldTest.php
+++ b/tests/phpunit/HTMLFeatureFieldTest.php
@@ -49,11 +49,11 @@
                        ],
                        [
                                'pattern' => "#<input type='checkbox' 
tabindex='0' aria-disabled='false' " .
-                                       "name='wpblahblahblah' value='1' 
id='ooui-\d+' class='oo-ui-inputWidget-input' />#",
+                                       "name='wpblahblahblah' value='1' 
id='ooui-php-\d+' class='oo-ui-inputWidget-input' />#",
                                'message' => 'Styled checkbox input not found.',
                        ],
                        [
-                               'pattern' => "#<label for='ooui-\d+' 
class='oo-ui-labelElement-label'>" .
+                               'pattern' => "#<label for='ooui-php-\d+' 
class='oo-ui-labelElement-label'>" .
                                        "blah blah blah</label>#",
                                'message' => 'Checkbox label not found.',
                        ],
diff --git a/tests/phpunit/NewHTMLCheckFieldTest.php 
b/tests/phpunit/NewHTMLCheckFieldTest.php
index f0671de..0f07b15 100644
--- a/tests/phpunit/NewHTMLCheckFieldTest.php
+++ b/tests/phpunit/NewHTMLCheckFieldTest.php
@@ -47,12 +47,12 @@
                );
                $this->assertRegExp(
                        "#<input type='checkbox' tabindex='0' 
aria-disabled='false' name='wpblahblahblah' " .
-                               "value='1' checked='checked' id='ooui-\d+' 
class='oo-ui-inputWidget-input' />#",
+                               "value='1' checked='checked' id='ooui-php-\d+' 
class='oo-ui-inputWidget-input' />#",
                        $html,
                        'Styled input not found.'
                );
                $this->assertRegExp(
-                       "#<label for='ooui-\d+' 
class='oo-ui-labelElement-label'>blah blah blah</label>#",
+                       "#<label for='ooui-php-\d+' 
class='oo-ui-labelElement-label'>blah blah blah</label>#",
                        $html,
                        'Checkbox label not found.'
                );

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id1a794249d9b5ee3ca6200cb661e37b319155696
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BetaFeatures
Gerrit-Branch: master
Gerrit-Owner: Jforrester <jforres...@wikimedia.org>
Gerrit-Reviewer: Bartosz Dziewoński <matma....@gmail.com>
Gerrit-Reviewer: Legoktm <lego...@member.fsf.org>
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