XZise has uploaded a new change for review.
https://gerrit.wikimedia.org/r/185185
Change subject: [IMPROV] Thread tests: Simplified intersections
......................................................................
[IMPROV] Thread tests: Simplified intersections
Instead of using itertools.product it is using sets and their
'intersection' method.
Change-Id: I162573d122e09aba2b3dd0f2a12f78fb2b3018bf
---
M tests/thread_tests.py
1 file changed, 3 insertions(+), 12 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/pywikibot/core
refs/changes/85/185185/1
diff --git a/tests/thread_tests.py b/tests/thread_tests.py
index 3222a8f..9fabb04 100644
--- a/tests/thread_tests.py
+++ b/tests/thread_tests.py
@@ -7,12 +7,6 @@
#
__version__ = '$Id$'
-import itertools
-import sys
-
-if sys.version_info[0] == 2:
- from future_builtins import filter
-
from tests.aspects import unittest, TestCase
from pywikibot.tools import ThreadedGenerator, intersect_generators
@@ -50,17 +44,14 @@
# first otherwise the generator is empty the second time.
datasets = [list(gen) for gen in gens]
- itertools_result = set(
- [item[0] for item in filter(
- lambda lst: all([x == lst[0] for x in lst]),
- itertools.product(*datasets))
- ])
+ set_result = set(datasets[0]).intersection(*[set(gen)
+ for gen in datasets[1:]])
result = list(intersect_generators(datasets))
self.assertEqual(len(set(result)), len(result))
- self.assertCountEqual(result, itertools_result)
+ self.assertCountEqual(result, set_result)
class BasicGeneratorIntersectTestCase(GeneratorIntersectTestCase):
--
To view, visit https://gerrit.wikimedia.org/r/185185
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I162573d122e09aba2b3dd0f2a12f78fb2b3018bf
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: XZise <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
