[Lldb-commits] [lldb] Add warning message to `session save` when transcript isn't saved. (PR #109020)

via lldb-commits lldb-commits at lists.llvm.org
Tue Sep 17 10:35:16 PDT 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 2d13302d3851747a023ffecc6388585e888cd0e9...60045b710e1102d6f220dfd4367f997b73bb64df lldb/test/API/commands/session/save/TestSessionSave.py
``````````

</details>

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

``````````diff
--- TestSessionSave.py	2024-09-17 16:56:09.000000 +0000
+++ TestSessionSave.py	2024-09-17 17:34:46.082913 +0000
@@ -111,18 +111,21 @@
 
         for command in commands:
             res = lldb.SBCommandReturnObject()
             interpreter.HandleCommand(command, res)
 
-        output_file = self.getBuildArtifact('my-session')
+        output_file = self.getBuildArtifact("my-session")
 
         res = lldb.SBCommandReturnObject()
         interpreter.HandleCommand("session save " + output_file, res)
         self.assertTrue(res.Succeeded())
         # We should warn about the setting being false.
         self.assertIn("interpreter.save-transcript is set to false", res.GetError())
-        self.assertTrue(os.path.getsize(output_file) == 0, "Output file should be empty since we didn't save the transcript.")
+        self.assertTrue(
+            os.path.getsize(output_file) == 0,
+            "Output file should be empty since we didn't save the transcript.",
+        )
 
     @no_debug_info_test
     def test_session_save_on_quit(self):
         raw = ""
         interpreter = self.dbg.GetCommandInterpreter()

``````````

</details>


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


More information about the lldb-commits mailing list