Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(5)

Delta Between Two Patch Sets: Lib/test/test_range.py

Issue 28376: assertion failure in rangeobject.c
Left Patch Set: Created 3 years, 4 months ago
Right Patch Set: Created 3 years, 4 months ago
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « no previous file | Objects/rangeobject.c » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 # Python test set -- built-in functions 1 # Python test set -- built-in functions
2 2
3 import unittest 3 import unittest
4 import sys 4 import sys
5 import pickle 5 import pickle
6 import itertools 6 import itertools
7 import test.support 7 import test.support
8 8
9 # pure Python implementations (3 args only), for comparison 9 # pure Python implementations (3 args only), for comparison
10 def pyrange(start, stop, step): 10 def pyrange(start, stop, step):
(...skipping 481 matching lines...) Expand 10 before | Expand all | Expand 10 after
492 iter1 = reversed(range(start, end, step)) 492 iter1 = reversed(range(start, end, step))
493 iter2 = pyrange_reversed(start, end, step) 493 iter2 = pyrange_reversed(start, end, step)
494 test_id = "reversed(range({}, {}, {}))".format(start, end, step) 494 test_id = "reversed(range({}, {}, {}))".format(start, end, step)
495 self.assert_iterators_equal(iter1, iter2, test_id, limit=100) 495 self.assert_iterators_equal(iter1, iter2, test_id, limit=100)
496 496
497 @test.support.cpython_only 497 @test.support.cpython_only
498 def test_range_iterator_invocation(self): 498 def test_range_iterator_invocation(self):
499 import _testcapi 499 import _testcapi
500 rangeiter_type = type(iter(range(0))) 500 rangeiter_type = type(iter(range(0)))
501 501
502 # rangeiter_new doesn't take keyword arguments 502 self.assertWarns(DeprecationWarning, rangeiter_type, 1, 3, 1)
503 with self.assertRaises(TypeError): 503
504 rangeiter_type(a=1) 504 with test.support.check_warnings(('', DeprecationWarning)):
505 505 # rangeiter_new doesn't take keyword arguments
506 # rangeiter_new takes exactly 3 arguments 506 with self.assertRaises(TypeError):
507 self.assertRaises(TypeError, rangeiter_type) 507 rangeiter_type(a=1)
508 self.assertRaises(TypeError, rangeiter_type, 1) 508
509 self.assertRaises(TypeError, rangeiter_type, 1, 1) 509 # rangeiter_new takes exactly 3 arguments
510 self.assertRaises(TypeError, rangeiter_type, 1, 1, 1, 1) 510 self.assertRaises(TypeError, rangeiter_type)
511 511 self.assertRaises(TypeError, rangeiter_type, 1)
512 # start, stop and stop must fit in C long 512 self.assertRaises(TypeError, rangeiter_type, 1, 1)
513 for good_val in [_testcapi.LONG_MAX, _testcapi.LONG_MIN]: 513 self.assertRaises(TypeError, rangeiter_type, 1, 1, 1, 1)
514 rangeiter_type(good_val, good_val, good_val) 514
515 for bad_val in [_testcapi.LONG_MAX + 1, _testcapi.LONG_MIN - 1]: 515 # start, stop and stop must fit in C long
516 self.assertRaises(OverflowError, 516 for good_val in [_testcapi.LONG_MAX, _testcapi.LONG_MIN]:
517 rangeiter_type, bad_val, 1, 1) 517 rangeiter_type(good_val, good_val, good_val)
518 self.assertRaises(OverflowError, 518 for bad_val in [_testcapi.LONG_MAX + 1, _testcapi.LONG_MIN - 1]:
519 rangeiter_type, 1, bad_val, 1) 519 self.assertRaises(OverflowError,
520 self.assertRaises(OverflowError, 520 rangeiter_type, bad_val, 1, 1)
521 rangeiter_type, 1, 1, bad_val) 521 self.assertRaises(OverflowError,
522 522 rangeiter_type, 1, bad_val, 1)
523 # step mustn't be zero 523 self.assertRaises(OverflowError,
524 self.assertRaises(ValueError, rangeiter_type, 1, 1, 0) 524 rangeiter_type, 1, 1, bad_val)
525
526 # step mustn't be zero
527 self.assertRaises(ValueError, rangeiter_type, 1, 1, 0)
525 528
526 def test_slice(self): 529 def test_slice(self):
527 def check(start, stop, step=None): 530 def check(start, stop, step=None):
528 i = slice(start, stop, step) 531 i = slice(start, stop, step)
529 self.assertEqual(list(r[i]), list(r)[i]) 532 self.assertEqual(list(r[i]), list(r)[i])
530 self.assertEqual(len(r[i]), len(list(r)[i])) 533 self.assertEqual(len(r[i]), len(list(r)[i]))
531 for r in [range(10), 534 for r in [range(10),
532 range(0), 535 range(0),
533 range(1, 9, 3), 536 range(1, 9, 3),
534 range(8, 0, -3), 537 range(8, 0, -3),
(...skipping 137 matching lines...) Expand 10 before | Expand all | Expand 10 after
672 675
673 with self.assertRaises(AttributeError): 676 with self.assertRaises(AttributeError):
674 del rangeobj.start 677 del rangeobj.start
675 with self.assertRaises(AttributeError): 678 with self.assertRaises(AttributeError):
676 del rangeobj.stop 679 del rangeobj.stop
677 with self.assertRaises(AttributeError): 680 with self.assertRaises(AttributeError):
678 del rangeobj.step 681 del rangeobj.step
679 682
680 if __name__ == "__main__": 683 if __name__ == "__main__":
681 unittest.main() 684 unittest.main()
LEFTRIGHT

RSS Feeds Recent Issues | This issue
This is Rietveld 894c83f36cb7+