mirror of
				https://github.com/python/cpython.git
				synced 2025-11-03 23:21:29 +00:00 
			
		
		
		
	I like tests.
The new split functions use a preallocated list. Added tests which exceed the preallocation size, to exercise list appends/resizes. Also added more edge case tests.
This commit is contained in:
		
							parent
							
								
									fe98f9613b
								
							
						
					
					
						commit
						005aee2c39
					
				
					 1 changed files with 32 additions and 0 deletions
				
			
		| 
						 | 
					@ -243,6 +243,8 @@ def test_split(self):
 | 
				
			||||||
        self.checkequal(['a', 'b', 'c d'], 'a b c d', 'split', None, 2)
 | 
					        self.checkequal(['a', 'b', 'c d'], 'a b c d', 'split', None, 2)
 | 
				
			||||||
        self.checkequal(['a', 'b', 'c', 'd'], 'a b c d', 'split', None, 3)
 | 
					        self.checkequal(['a', 'b', 'c', 'd'], 'a b c d', 'split', None, 3)
 | 
				
			||||||
        self.checkequal(['a', 'b', 'c', 'd'], 'a b c d', 'split', None, 4)
 | 
					        self.checkequal(['a', 'b', 'c', 'd'], 'a b c d', 'split', None, 4)
 | 
				
			||||||
 | 
					        self.checkequal(['a', 'b', 'c', 'd'], 'a b c d', 'split', None,
 | 
				
			||||||
 | 
					                        sys.maxint-1)
 | 
				
			||||||
        self.checkequal(['a b c d'], 'a b c d', 'split', None, 0)
 | 
					        self.checkequal(['a b c d'], 'a b c d', 'split', None, 0)
 | 
				
			||||||
        self.checkequal(['a', 'b', 'c  d'], 'a  b  c  d', 'split', None, 2)
 | 
					        self.checkequal(['a', 'b', 'c  d'], 'a  b  c  d', 'split', None, 2)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					@ -253,27 +255,53 @@ def test_split(self):
 | 
				
			||||||
        self.checkequal(['a', 'b   c   '], '  a    b   c   ', 'split', None, 1)
 | 
					        self.checkequal(['a', 'b   c   '], '  a    b   c   ', 'split', None, 1)
 | 
				
			||||||
        self.checkequal(['a', 'b', 'c   '], '  a    b   c   ', 'split', None, 2)
 | 
					        self.checkequal(['a', 'b', 'c   '], '  a    b   c   ', 'split', None, 2)
 | 
				
			||||||
        self.checkequal(['a', 'b'], '\n\ta \t\r b \v ', 'split')
 | 
					        self.checkequal(['a', 'b'], '\n\ta \t\r b \v ', 'split')
 | 
				
			||||||
 | 
					        aaa = ' a '*20
 | 
				
			||||||
 | 
					        self.checkequal(['a']*20, aaa, 'split')
 | 
				
			||||||
 | 
					        self.checkequal(['a'] + [aaa[4:]], aaa, 'split', None, 1)
 | 
				
			||||||
 | 
					        self.checkequal(['a']*19 + ["a "], aaa, 'split', None, 19)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        # by a char
 | 
					        # by a char
 | 
				
			||||||
        self.checkequal(['a', 'b', 'c', 'd'], 'a|b|c|d', 'split', '|')
 | 
					        self.checkequal(['a', 'b', 'c', 'd'], 'a|b|c|d', 'split', '|')
 | 
				
			||||||
 | 
					        self.checkequal(['a|b|c|d'], 'a|b|c|d', 'split', '|', 0)
 | 
				
			||||||
        self.checkequal(['a', 'b|c|d'], 'a|b|c|d', 'split', '|', 1)
 | 
					        self.checkequal(['a', 'b|c|d'], 'a|b|c|d', 'split', '|', 1)
 | 
				
			||||||
        self.checkequal(['a', 'b', 'c|d'], 'a|b|c|d', 'split', '|', 2)
 | 
					        self.checkequal(['a', 'b', 'c|d'], 'a|b|c|d', 'split', '|', 2)
 | 
				
			||||||
        self.checkequal(['a', 'b', 'c', 'd'], 'a|b|c|d', 'split', '|', 3)
 | 
					        self.checkequal(['a', 'b', 'c', 'd'], 'a|b|c|d', 'split', '|', 3)
 | 
				
			||||||
        self.checkequal(['a', 'b', 'c', 'd'], 'a|b|c|d', 'split', '|', 4)
 | 
					        self.checkequal(['a', 'b', 'c', 'd'], 'a|b|c|d', 'split', '|', 4)
 | 
				
			||||||
 | 
					        self.checkequal(['a', 'b', 'c', 'd'], 'a|b|c|d', 'split', '|',
 | 
				
			||||||
 | 
					                        sys.maxint-2)
 | 
				
			||||||
        self.checkequal(['a|b|c|d'], 'a|b|c|d', 'split', '|', 0)
 | 
					        self.checkequal(['a|b|c|d'], 'a|b|c|d', 'split', '|', 0)
 | 
				
			||||||
        self.checkequal(['a', '', 'b||c||d'], 'a||b||c||d', 'split', '|', 2)
 | 
					        self.checkequal(['a', '', 'b||c||d'], 'a||b||c||d', 'split', '|', 2)
 | 
				
			||||||
        self.checkequal(['endcase ', ''], 'endcase |', 'split', '|')
 | 
					        self.checkequal(['endcase ', ''], 'endcase |', 'split', '|')
 | 
				
			||||||
 | 
					        self.checkequal(['', ' startcase'], '| startcase', 'split', '|')
 | 
				
			||||||
 | 
					        self.checkequal(['', 'bothcase', ''], '|bothcase|', 'split', '|')
 | 
				
			||||||
        self.checkequal(['a', '', 'b\x00c\x00d'], 'a\x00\x00b\x00c\x00d', 'split', '\x00', 2)
 | 
					        self.checkequal(['a', '', 'b\x00c\x00d'], 'a\x00\x00b\x00c\x00d', 'split', '\x00', 2)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					        self.checkequal(['a']*20, ('a|'*20)[:-1], 'split', '|')
 | 
				
			||||||
 | 
					        self.checkequal(['a']*15 +['a|a|a|a|a'],
 | 
				
			||||||
 | 
					                                   ('a|'*20)[:-1], 'split', '|', 15)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        # by string
 | 
					        # by string
 | 
				
			||||||
        self.checkequal(['a', 'b', 'c', 'd'], 'a//b//c//d', 'split', '//')
 | 
					        self.checkequal(['a', 'b', 'c', 'd'], 'a//b//c//d', 'split', '//')
 | 
				
			||||||
        self.checkequal(['a', 'b//c//d'], 'a//b//c//d', 'split', '//', 1)
 | 
					        self.checkequal(['a', 'b//c//d'], 'a//b//c//d', 'split', '//', 1)
 | 
				
			||||||
        self.checkequal(['a', 'b', 'c//d'], 'a//b//c//d', 'split', '//', 2)
 | 
					        self.checkequal(['a', 'b', 'c//d'], 'a//b//c//d', 'split', '//', 2)
 | 
				
			||||||
        self.checkequal(['a', 'b', 'c', 'd'], 'a//b//c//d', 'split', '//', 3)
 | 
					        self.checkequal(['a', 'b', 'c', 'd'], 'a//b//c//d', 'split', '//', 3)
 | 
				
			||||||
        self.checkequal(['a', 'b', 'c', 'd'], 'a//b//c//d', 'split', '//', 4)
 | 
					        self.checkequal(['a', 'b', 'c', 'd'], 'a//b//c//d', 'split', '//', 4)
 | 
				
			||||||
 | 
					        self.checkequal(['a', 'b', 'c', 'd'], 'a//b//c//d', 'split', '//',
 | 
				
			||||||
 | 
					                        sys.maxint-10)
 | 
				
			||||||
        self.checkequal(['a//b//c//d'], 'a//b//c//d', 'split', '//', 0)
 | 
					        self.checkequal(['a//b//c//d'], 'a//b//c//d', 'split', '//', 0)
 | 
				
			||||||
        self.checkequal(['a', '', 'b////c////d'], 'a////b////c////d', 'split', '//', 2)
 | 
					        self.checkequal(['a', '', 'b////c////d'], 'a////b////c////d', 'split', '//', 2)
 | 
				
			||||||
        self.checkequal(['endcase ', ''], 'endcase test', 'split', 'test')
 | 
					        self.checkequal(['endcase ', ''], 'endcase test', 'split', 'test')
 | 
				
			||||||
 | 
					        self.checkequal(['', ''], 'aaa', 'split', 'aaa')
 | 
				
			||||||
 | 
					        self.checkequal(['aaa'], 'aaa', 'split', 'aaa', 0)
 | 
				
			||||||
 | 
					        self.checkequal(['ab', 'ab'], 'abbaab', 'split', 'ba')
 | 
				
			||||||
 | 
					        self.checkequal(['aaaa'], 'aaaa', 'split', 'aab')
 | 
				
			||||||
 | 
					        self.checkequal([''], '', 'split', 'aaa')
 | 
				
			||||||
 | 
					        self.checkequal(['aa'], 'aa', 'split', 'aaa')
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					        self.checkequal(['a']*20, ('aBLAH'*20)[:-4], 'split', 'BLAH')
 | 
				
			||||||
 | 
					        self.checkequal(['a']*20, ('aBLAH'*20)[:-4], 'split', 'BLAH', 19)
 | 
				
			||||||
 | 
					        self.checkequal(['a']*18 + ['aBLAHa'], ('aBLAH'*20)[:-4],
 | 
				
			||||||
 | 
					                        'split', 'BLAH', 18)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        # mixed use of str and unicode
 | 
					        # mixed use of str and unicode
 | 
				
			||||||
        self.checkequal([u'a', u'b', u'c d'], 'a b c d', 'split', u' ', 2)
 | 
					        self.checkequal([u'a', u'b', u'c d'], 'a b c d', 'split', u' ', 2)
 | 
				
			||||||
| 
						 | 
					@ -281,6 +309,10 @@ def test_split(self):
 | 
				
			||||||
        # argument type
 | 
					        # argument type
 | 
				
			||||||
        self.checkraises(TypeError, 'hello', 'split', 42, 42, 42)
 | 
					        self.checkraises(TypeError, 'hello', 'split', 42, 42, 42)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					        # null case
 | 
				
			||||||
 | 
					        self.checkraises(ValueError, 'hello', 'split', '')
 | 
				
			||||||
 | 
					        self.checkraises(ValueError, 'hello', 'split', '', 0)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    def test_rsplit(self):
 | 
					    def test_rsplit(self):
 | 
				
			||||||
        self.checkequal(['this', 'is', 'the', 'rsplit', 'function'],
 | 
					        self.checkequal(['this', 'is', 'the', 'rsplit', 'function'],
 | 
				
			||||||
                         'this is the rsplit function', 'rsplit')
 | 
					                         'this is the rsplit function', 'rsplit')
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue