diff options
| author | Dave Lee <davelee.com@gmail.com> | 2022-02-11 21:23:16 -0800 |
|---|---|---|
| committer | Dave Lee <davelee.com@gmail.com> | 2022-02-14 08:31:17 -0800 |
| commit | 779bbbf27fe631154bdfaac7a443f198d4654688 (patch) | |
| tree | fde0e9527bc0d5a37f487bffc9964738222b3418 /lldb/test/API/python_api/sbdata | |
| parent | 3c8d2aa87c1701ca16e13f06aea484637e03d005 (diff) | |
| download | llvm-779bbbf27fe631154bdfaac7a443f198d4654688.zip llvm-779bbbf27fe631154bdfaac7a443f198d4654688.tar.gz llvm-779bbbf27fe631154bdfaac7a443f198d4654688.tar.bz2 | |
[lldb] Replace asserts on .Success() with assertSuccess()
Replace forms of `assertTrue(err.Success())` with `assertSuccess(err)` (added in D82759).
* `assertSuccess` prints out the error's message
* `assertSuccess` expresses explicit higher level semantics, both to the reader and for test failure output
* `assertSuccess` seems not to be well known, using it where possible will help spread knowledge
* `assertSuccess` statements are more succinct
Differential Revision: https://reviews.llvm.org/D119616
Diffstat (limited to 'lldb/test/API/python_api/sbdata')
| -rw-r--r-- | lldb/test/API/python_api/sbdata/TestSBData.py | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/lldb/test/API/python_api/sbdata/TestSBData.py b/lldb/test/API/python_api/sbdata/TestSBData.py index 1b20cf2..aced0da 100644 --- a/lldb/test/API/python_api/sbdata/TestSBData.py +++ b/lldb/test/API/python_api/sbdata/TestSBData.py @@ -83,10 +83,10 @@ class SBDataAPICase(TestBase): self.assert_data(data.GetUnsignedInt32, offset, 1) offset += 4 low = data.GetSignedInt16(error, offset) - self.assertTrue(error.Success()) + self.assertSuccess(error) offset += 2 high = data.GetSignedInt16(error, offset) - self.assertTrue(error.Success()) + self.assertSuccess(error) offset += 2 self.assertTrue( (low == 9 and high == 0) or ( @@ -99,7 +99,7 @@ class SBDataAPICase(TestBase): offset) - 3.14) < 1, 'foo[0].c == 3.14') - self.assertTrue(error.Success()) + self.assertSuccess(error) offset += 4 self.assert_data(data.GetUnsignedInt32, offset, 8) offset += 4 @@ -172,7 +172,7 @@ class SBDataAPICase(TestBase): offset) - 3.14) < 1, 'foo[1].c == 3.14') - self.assertTrue(error.Success()) + self.assertSuccess(error) data = new_foobar.GetData() @@ -188,7 +188,7 @@ class SBDataAPICase(TestBase): offset) - 6.28) < 1, 'foo[1].c == 6.28') - self.assertTrue(error.Success()) + self.assertSuccess(error) self.runCmd("n") @@ -207,7 +207,7 @@ class SBDataAPICase(TestBase): offset) - 3) < 1, 'barfoo[0].c == 3') - self.assertTrue(error.Success()) + self.assertSuccess(error) offset += 4 self.assert_data(data.GetUnsignedInt32, offset, 4) offset += 4 @@ -220,7 +220,7 @@ class SBDataAPICase(TestBase): offset) - 6) < 1, 'barfoo[1].c == 6') - self.assertTrue(error.Success()) + self.assertSuccess(error) new_object = barfoo.CreateValueFromData( "new_object", data, barfoo.GetType().GetBasicType( @@ -239,7 +239,7 @@ class SBDataAPICase(TestBase): 'A\0\0\0', data.GetByteOrder(), data.GetAddressByteSize()) - self.assertTrue(error.Success()) + self.assertSuccess(error) data2 = lldb.SBData() data2.SetData( @@ -247,7 +247,7 @@ class SBDataAPICase(TestBase): 'BCD', data.GetByteOrder(), data.GetAddressByteSize()) - self.assertTrue(error.Success()) + self.assertSuccess(error) data.Append(data2) @@ -401,7 +401,7 @@ class SBDataAPICase(TestBase): 0) - 3.14) < 0.5, 'double data2[0] = 3.14') - self.assertTrue(error.Success()) + self.assertSuccess(error) self.assertTrue( fabs( data2.GetDouble( @@ -409,7 +409,7 @@ class SBDataAPICase(TestBase): 8) - 6.28) < 0.5, 'double data2[1] = 6.28') - self.assertTrue(error.Success()) + self.assertSuccess(error) self.assertTrue( fabs( data2.GetDouble( @@ -417,7 +417,7 @@ class SBDataAPICase(TestBase): 16) - 2.71) < 0.5, 'double data2[2] = 2.71') - self.assertTrue(error.Success()) + self.assertSuccess(error) data2 = lldb.SBData() |
