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

Change subject: [bugfix] Fix failing tests of TestPageBotMayEdit.test_bot_may_edit_general ......................................................................

[bugfix] Fix failing tests of TestPageBotMayEdit.test_bot_may_edit_general

This partly reverts I2afc60004d9b14460ef465b09a46d61cdd719fc3

Bug: T205838 Change-Id: I2d0795d32d5fceedfa4819c89807079789948086 --- M tests/page_tests.py 1 file changed, 3 insertions(+), 3 deletions(-)

Approvals: D3r1ck01: Looks good to me, but someone else must approve Dvorapa: Looks good to me, approved jenkins-bot: Verified

diff --git a/tests/page_tests.py b/tests/page_tests.py index d471d66..908706f 100644 --- a/tests/page_tests.py +++ b/tests/page_tests.py @@ -782,7 +782,7 @@ .format(page.text, page.botMayEdit(), user))

# Ban all compliant bots not in the list, syntax for de wp. - page.text = '{{nobots|{}, HagermanBot,Werdnabot}}'.format(user) + page.text = '{{nobots|%s, HagermanBot,Werdnabot}}' % user self.assertFalse(page.botMayEdit(), '{}: {} but user={}' .format(page.text, page.botMayEdit(), user)) @@ -811,13 +811,13 @@ .format(page.text, page.botMayEdit(), user))

# Ban all compliant bots not in the list. - page.text = '{{bots|allow={}, HagermanBot}}'.format(user) + page.text = '{{bots|allow=%s, HagermanBot}}' % user self.assertTrue(page.botMayEdit(), '{}: {} but user={}' .format(page.text, page.botMayEdit(), user))

# Ban all compliant bots in the list. - page.text = '{{bots|deny={}, HagermanBot}}'.format(user) + page.text = '{{bots|deny=%s, HagermanBot}}' % user self.assertFalse(page.botMayEdit(), '{}: {} but user={}' .format(page.text, page.botMayEdit(), user))