[Lldb-commits] [lldb] [lldb][test] Add tests for target.max-string-summary-length setting (PR #77920)

via lldb-commits lldb-commits at lists.llvm.org
Fri Jan 12 04:54:10 PST 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {darker}-->


:warning: Python code formatter, darker found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
darker --check --diff -r 786cf76f434d2691878067dedb8b1eb99472810b...4de3716b55a22dc8b7dda621889089f390026739 lldb/test/API/functionalities/data-formatter/data-formatter-advanced/TestDataFormatterAdv.py
``````````

</details>

<details>
<summary>
View the diff from darker here.
</summary>

``````````diff
--- TestDataFormatterAdv.py	2024-01-12 12:34:29.000000 +0000
+++ TestDataFormatterAdv.py	2024-01-12 12:54:01.521293 +0000
@@ -294,25 +294,25 @@
             matching=False,
             substrs=["e_2", "n_2", "r_2", "i_2", "k_2", "o_2"],
         )
 
         self.runCmd("settings set target.max-string-summary-length 5")
-        some_string = self.frame().FindVariable('some_string')
+        some_string = self.frame().FindVariable("some_string")
         some_string_summary = some_string.GetSummary()
         self.assertEqual(some_string_summary, '"01234"...')
 
-        some_carr = self.frame().FindVariable('some_carr')
+        some_carr = self.frame().FindVariable("some_carr")
         some_carr_summary = some_carr.GetSummary()
         self.assertEqual(some_carr_summary, '"01234"...')
 
         # FIXME: c-strings should honor the target.max-string-summary-length
         # setting. Currently a C-string will be truncated at 64 (an internal
         # implementation detail) instead of the value specified in the setting.
-        some_cstring = self.frame().FindVariable('some_cstring')
+        some_cstring = self.frame().FindVariable("some_cstring")
         some_cstring_summary = some_cstring.GetSummary()
-        self.assertEqual(len(some_cstring_summary), 66) # 64 + 2 (for quotation marks)
-        self.assertFalse(some_cstring_summary.endswith('...'))
+        self.assertEqual(len(some_cstring_summary), 66)  # 64 + 2 (for quotation marks)
+        self.assertFalse(some_cstring_summary.endswith("..."))
 
         # override the cap
         self.expect(
             "frame variable a_long_guy --show-all-children",
             matching=True,

``````````

</details>


https://github.com/llvm/llvm-project/pull/77920


More information about the lldb-commits mailing list