Merge pull request #70 from brettcannon/remove_idempotent_redundancy

Remove explicit idempotent tests

Author: takluyver

tests/test_fix_filter.py

@@ -35,14 +35,6 @@
     pass
 """)
 
-FILTER_SIX_ALREADY = ("""\
-from six.moves import filter
-filter(None, [1])
-""", """\
-from six.moves import filter
-filter(None, [1])
-""")
-
 
 def test_filter_call():
     check_on_input(*FILTER_CALL)
@@ -55,9 +47,3 @@ def test_filter_too_many_args():
 
 def test_filter_kwargs():
     check_on_input(*FILTER_KWARGS)
-
-def test_filter_iterator_context():
-    check_on_input(*FILTER_ITERATOR_CONTEXT)
-
-def test_filter_six_already():
-    check_on_input(*FILTER_SIX_ALREADY)

tests/test_fix_map.py

@@ -49,14 +49,6 @@
     pass
 """)
 
-MAP_SIX_ALREADY = ("""\
-from six.moves import map
-map(x, [1])
-""", """\
-from six.moves import map
-map(x, [1])
-""")
-
 
 def test_map_2_args():
     check_on_input(*MAP_2_ARGS)
@@ -78,6 +70,3 @@ def test_map_ref():
 
 def test_map_iterator_context():
     check_on_input(*MAP_ITERATOR_CONTEXT)
-
-def test_map_six_already():
-    check_on_input(*MAP_SIX_ALREADY)

tests/test_fix_xrange_six.py

@@ -1,38 +1,20 @@
 from utils import check_on_input
 
 
-# range()
 RANGE = ("""\
 x = range(1)
 """, """\
 from six.moves import range
 x = list(range(1))
 """)
 
-RANGE_IDEMPOTENT = ("""\
-x = list(range(1))
-""", """\
-from six.moves import range
-x = list(range(1))
-""")
-
-# xrange()
 XRANGE = ("""\
 xrange(1)
 """, """\
 from six.moves import range
 range(1)
 """)
 
-XRANGE_IDEMPOTENT = ("""\
-from six.moves import range
-range(1)
-""", """\
-from six.moves import range
-range(1)
-""")
-
-# Both
 XRANGE_RANGE = ("""\
 x = xrange(1)
 y = range(1)
@@ -46,14 +28,8 @@
 def test_range():
     check_on_input(*RANGE)
 
-def test_range_idempotent():
-    check_on_input(*RANGE_IDEMPOTENT)
-
 def test_xrange():
     check_on_input(*XRANGE)
 
-def test_xrange_idempotent():
-    check_on_input(*XRANGE_IDEMPOTENT)
-
 def test_xrange_range():
     check_on_input(*XRANGE_RANGE)

tests/test_fix_zip.py

@@ -46,14 +46,6 @@
     pass
 """)
 
-ZIP_SIX_ALREADY = ("""\
-from six.moves import zip
-zip(x)
-""", """\
-from six.moves import zip
-zip(x)
-""")
-
 
 def test_zip_call_no_args():
     check_on_input(*ZIP_CALL_NO_ARGS)
@@ -72,6 +64,3 @@ def test_zip_kwargs():
 
 def test_zip_iterator_context():
     check_on_input(*ZIP_ITERATOR_CONTEXT)
-
-def test_zip_six_already():
-    check_on_input(*ZIP_SIX_ALREADY)