Merge pull request #222 from erikrose/nth_combination_tests

Add more tests for nth_combination
more-itertools · Jul 15, 2018 · ac8c103 · ac8c103
2 parents 7041f0a + d8d96ca
commit ac8c103
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/more_itertools/tests/test_recipes.py b/more_itertools/tests/test_recipes.py
@@ -590,6 +590,15 @@ def test_long(self):
         expected = (2, 12, 35, 126)
         self.assertEqual(actual, expected)
 
+    def test_invalid_r(self):
+        for r in (-1, 3):
+            with self.assertRaises(ValueError):
+                mi.nth_combination([], r, 0)
+
+    def test_invalid_index(self):
+        with self.assertRaises(IndexError):
+            mi.nth_combination('abcdefg', 3, -36)
+
 
 class PrependTests(TestCase):
     def test_basic(self):