Repository navigation
Commit 856bb54
gh-131357: Add a set of asserts to test.test_capi.test_bytearray (GH-131554)
add a set of asserts to test.test_capi.test_bytearray
1. Assert empty bytearray object for PyByteArray_Check.
2. Assert empty bytearray object for PyByteArray_CheckExact.
3. Assert 0-size bytearray object for PyByteArray_Size.
4. Assert empty bytearray object for PyByteArray_AsString.
5. Assert concatenation of the bytearray object with itself for PyByteArray_Concat.
(cherry picked from commit f3bf304)
Co-authored-by: Nybblista <170842536+nybblista@users.noreply.github.com>
1 parent eedaa4e commit 856bb54
1 file changed
Lines changed: 5 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -20,6 +20,7 @@ class CAPITest(unittest.TestCase): | |||
| 20 | 20 | def test_check(self): | |
| 21 | 21 | # Test PyByteArray_Check() | |
| 22 | 22 | check = _testlimitedcapi.bytearray_check | |
| 23 | + self.assertTrue(check(bytearray(b''))) | ||
| 23 | 24 | self.assertTrue(check(bytearray(b'abc'))) | |
| 24 | 25 | self.assertFalse(check(b'abc')) | |
| 25 | 26 | self.assertTrue(check(ByteArraySubclass(b'abc'))) | |
@@ -33,6 +34,7 @@ def test_check(self): | |||
| 33 | 34 | def test_checkexact(self): | |
| 34 | 35 | # Test PyByteArray_CheckExact() | |
| 35 | 36 | check = _testlimitedcapi.bytearray_checkexact | |
| 37 | + self.assertTrue(check(bytearray(b''))) | ||
| 36 | 38 | self.assertTrue(check(bytearray(b'abc'))) | |
| 37 | 39 | self.assertFalse(check(b'abc')) | |
| 38 | 40 | self.assertFalse(check(ByteArraySubclass(b'abc'))) | |
@@ -78,7 +80,7 @@ def test_fromobject(self): | |||
| 78 | 80 | def test_size(self): | |
| 79 | 81 | # Test PyByteArray_Size() | |
| 80 | 82 | size = _testlimitedcapi.bytearray_size | |
| 81 | - | ||
| 83 | + self.assertEqual(size(bytearray(b'')), 0) | ||
| 82 | 84 | self.assertEqual(size(bytearray(b'abc')), 3) | |
| 83 | 85 | self.assertEqual(size(ByteArraySubclass(b'abc')), 3) | |
| 84 | 86 | ||
@@ -89,7 +91,7 @@ def test_size(self): | |||
| 89 | 91 | def test_asstring(self): | |
| 90 | 92 | """Test PyByteArray_AsString()""" | |
| 91 | 93 | asstring = _testlimitedcapi.bytearray_asstring | |
| 92 | - | ||
| 94 | + self.assertEqual(asstring(bytearray(b''), 1), b'\0') | ||
| 93 | 95 | self.assertEqual(asstring(bytearray(b'abc'), 4), b'abc\0') | |
| 94 | 96 | self.assertEqual(asstring(ByteArraySubclass(b'abc'), 4), b'abc\0') | |
| 95 | 97 | self.assertEqual(asstring(bytearray(b'abc\0def'), 8), b'abc\0def\0') | |
@@ -105,6 +107,7 @@ def test_concat(self): | |||
| 105 | 107 | ba = bytearray(b'abc') | |
| 106 | 108 | self.assertEqual(concat(ba, b'def'), bytearray(b'abcdef')) | |
| 107 | 109 | self.assertEqual(ba, b'abc') | |
| 110 | + self.assertEqual(concat(ba, ba), bytearray(b'abcabc')) | ||
| 108 | 111 | ||
| 109 | 112 | self.assertEqual(concat(b'abc', b'def'), bytearray(b'abcdef')) | |
| 110 | 113 | self.assertEqual(concat(b'a\0b', b'c\0d'), bytearray(b'a\0bc\0d')) | |
0 commit comments