jenkins-bot submitted this change.

View Change

Approvals: Xqt: Looks good to me, approved jenkins-bot: Verified
[cleanup] Use GeneratorFactory.handle_arg

Bug: T271437
Change-Id: If513c6427b9a2089050a04968b388dc49d0d4b2c
---
M tests/pagegenerators_tests.py
1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/tests/pagegenerators_tests.py b/tests/pagegenerators_tests.py
index 93dfdc9..7d84dbe 100755
--- a/tests/pagegenerators_tests.py
+++ b/tests/pagegenerators_tests.py
@@ -8,6 +8,7 @@
import datetime
import logging
import sys
+import unittest

from contextlib import suppress
from typing import Optional
@@ -28,7 +29,6 @@

from tests import join_data_path, mock
from tests.aspects import (
- unittest,
TestCase,
DeprecationTestCase,
WikidataTestCase,
@@ -715,7 +715,7 @@
"""Test unsupported option."""
gf = pagegenerators.GeneratorFactory(site=self.get_site())
with self.assertRaises(UnknownExtension):
- gf.handleArg('-ql:2')
+ gf.handle_arg('-ql:2')

def test_one_excluded_namespaces(self):
"""Test one excluded namespaces."""
@@ -1039,13 +1039,13 @@
gen = gf.getCombinedGenerator()
self.assertIsNotNone(gen)
with self.assertRaises(ValueError):
- gf.handleArg('-recentchanges:3,2,1')
+ gf.handle_arg('-recentchanges:3,2,1')
with self.assertRaises(ValueError):
- gf.handleArg('-recentchanges:12,-12')
+ gf.handle_arg('-recentchanges:12,-12')
with self.assertRaises(ValueError):
- gf.handleArg('-recentchanges:visualeditor,3,2,1')
+ gf.handle_arg('-recentchanges:visualeditor,3,2,1')
with self.assertRaises(ValueError):
- gf.handleArg('-recentchanges:"mobile edit,-10,20"')
+ gf.handle_arg('-recentchanges:"mobile edit,-10,20"')

def test_recentchanges_rctag(self):
"""Test recentchanges generator with recent changes tag."""
@@ -1254,7 +1254,7 @@
.format(self.site))
gf = pagegenerators.GeneratorFactory(site=self.site)
with self.assertRaises(AssertionError):
- gf.handleArg('-linter:dummy')
+ gf.handle_arg('-linter:dummy')

def test_linter_generator_show(self):
"""Test generator of pages with lint errors."""
@@ -1265,7 +1265,7 @@
self.assertEqual(cm.exception.code, 0)
else:
with self.assertRaises(UnknownExtension):
- gf.handleArg('-linter:show')
+ gf.handle_arg('-linter:show')

def test_querypage_generator_with_valid_page(self):
"""Test generator of pages with lint errors."""
@@ -1283,7 +1283,7 @@
"""Test generator of pages with lint errors."""
gf = pagegenerators.GeneratorFactory(site=self.site)
with self.assertRaises(AssertionError):
- gf.handleArg('-querypage:dummy')
+ gf.handle_arg('-querypage:dummy')

def test_querypage_generator_with_no_page(self):
"""Test generator of pages with lint errors."""
@@ -1509,7 +1509,7 @@
self.assertFalse(gf.handle_arg('-log'))
self.assertFalse(gf.handle_arg('-log:text_here'))
with self.assertRaises(NotImplementedError):
- gf.handleArg('-logevents:anyevent')
+ gf.handle_arg('-logevents:anyevent')
# test that old format log option is not handled by any handler method.
gf_mock = mock.create_autospec(gf)
self.assertFalse(factory.handle_arg(gf_mock, '-anotherlog'))

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

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: If513c6427b9a2089050a04968b388dc49d0d4b2c
Gerrit-Change-Number: 667542
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <info@gno.de>
Gerrit-Reviewer: Xqt <info@gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged