jenkins-bot merged this change.

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

[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(-)

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))

To view, visit change 463726. To unsubscribe, or for help writing mail filters, visit settings.