jenkins-bot merged this change.
[cleanup] use page_namespaces as a set instead a list
Change-Id: I08733690a8d56e89d2e5dbffab4303e62fc604ab
---
M tests/aspects.py
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/tests/aspects.py b/tests/aspects.py
index 79af4f8..71f14e6 100644
--- a/tests/aspects.py
+++ b/tests/aspects.py
@@ -249,14 +249,14 @@
else:
assert isinstance(namespaces, set)
- page_namespaces = [page.namespace() for page in gen]
+ page_namespaces = {page.namespace() for page in gen}
- if skip and set(page_namespaces) != namespaces:
+ if skip and page_namespaces != namespaces:
raise unittest.SkipTest('Pages in namespaces {!r} not found.'
- .format(list(namespaces
- - set(page_namespaces))))
+ .format(
+ list(namespaces - page_namespaces)))
else:
- self.assertEqual(set(page_namespaces), namespaces)
+ self.assertEqual(page_namespaces, namespaces)
def assertPageTitlesEqual(self, gen, titles, site=None):
"""
To view, visit change 500441. To unsubscribe, or for help writing mail filters, visit settings.