aboutsummaryrefslogtreecommitdiff
path: root/lldb/test/API/python_api/sbdata
diff options
context:
space:
mode:
authorDave Lee <davelee.com@gmail.com>2022-02-11 21:23:16 -0800
committerDave Lee <davelee.com@gmail.com>2022-02-14 08:31:17 -0800
commit779bbbf27fe631154bdfaac7a443f198d4654688 (patch)
treefde0e9527bc0d5a37f487bffc9964738222b3418 /lldb/test/API/python_api/sbdata
parent3c8d2aa87c1701ca16e13f06aea484637e03d005 (diff)
downloadllvm-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.py24
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()