[Lldb-commits] [lldb] [LLDB]Skip Summary Statistics Tests for Windows (PR #108079)

via lldb-commits lldb-commits at lists.llvm.org
Tue Sep 10 13:11:33 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: Jacob Lalonde (Jlalond)

<details>
<summary>Changes</summary>

Follow up to #<!-- -->102708, the tests are failing for windows. There is a large variance in these tests between summary strings and built in types. I'm disabling these test for windows, and will add windows specific tests as a follow up to this.

---
Full diff: https://github.com/llvm/llvm-project/pull/108079.diff


1 Files Affected:

- (modified) lldb/test/API/commands/statistics/basic/TestStats.py (+2) 


``````````diff
diff --git a/lldb/test/API/commands/statistics/basic/TestStats.py b/lldb/test/API/commands/statistics/basic/TestStats.py
index 03ec1693447051..a0a9eeb6493207 100644
--- a/lldb/test/API/commands/statistics/basic/TestStats.py
+++ b/lldb/test/API/commands/statistics/basic/TestStats.py
@@ -921,6 +921,7 @@ def test_order_of_options_do_not_matter(self):
                 f"The order of options '{options[0]}' and '{options[1]}' should not matter",
             )
 
+    @skipIfWindows
     def test_summary_statistics_providers(self):
         """
         Test summary timing statistics is included in statistics dump when
@@ -960,6 +961,7 @@ def test_summary_statistics_providers(self):
         self.assertIn("'totalTime':", summary_provider_str)
         self.assertIn("'type': 'python'", summary_provider_str)
 
+    @skipIfWindows
     def test_summary_statistics_providers_vec(self):
         """
         Test summary timing statistics is included in statistics dump when

``````````

</details>


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


More information about the lldb-commits mailing list