[Lldb-commits] [lldb] [test] Switch LLDB API tests from vendored unittest2 to unittest (PR #79945)

[via lldb-commits]

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 6b330470df7db34ee1b894e0e2edf8270d5c51da...edd3cfa6559acb37331945e45887d2d5069d5179 lldb/packages/Python/lldbsuite/test/configuration.py lldb/packages/Python/lldbsuite/test/decorators.py lldb/packages/Python/lldbsuite/test/dotest.py lldb/packages/Python/lldbsuite/test/lldbtest.py lldb/packages/Python/lldbsuite/test/test_result.py lldb/test/API/commands/expression/test/TestExprs.py lldb/test/API/functionalities/breakpoint/thread_plan_user_breakpoint/TestThreadPlanUserBreakpoint.py lldb/test/API/functionalities/jitloader_gdb/TestJITLoaderGDB.py lldb/test/API/functionalities/thread/state/TestThreadStates.py lldb/test/API/functionalities/tty/TestTerminal.py lldb/test/API/lang/c/shared_lib/TestSharedLib.py lldb/test/API/lang/c/shared_lib_stripped_symbols/TestSharedLibStrippedSymbols.py lldb/test/API/lang/cpp/namespace/TestNamespaceLookup.py lldb/test/API/lang/cpp/reference-to-outer-type/TestCppReferenceToOuterClass.py lldb/test/API/lang/cpp/thread_local/TestThreadLocal.py lldb/test/API/lang/objc/hidden-ivars/TestHiddenIvars.py lldb/test/API/lldbtest.py lldb/test/API/macosx/universal/TestUniversal.py lldb/test/API/tools/lldb-server/libraries-svr4/TestGdbRemoteLibrariesSvr4Support.py lldb/test/API/tools/lldb-server/test/test_lldbgdbserverutils.py
``````````

</details>

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

``````````diff
--- packages/Python/lldbsuite/test/dotest.py	2024-01-30 03:07:36.000000 +0000
+++ packages/Python/lldbsuite/test/dotest.py	2024-01-30 05:10:45.742493 +0000
@@ -667,13 +667,11 @@
 
     if not filtered:
         # Add the entire file's worth of tests since we're not filtered.
         # Also the fail-over case when the filterspec branch
         # (base, filterspec) combo doesn't make sense.
-        configuration.suite.addTests(
-            unittest.defaultTestLoader.loadTestsFromName(base)
-        )
+        configuration.suite.addTests(unittest.defaultTestLoader.loadTestsFromName(base))
 
 
 def visit(prefix, dir, names):
     """Visitor function for os.path.walk(path, visit, arg)."""
 

``````````

</details>


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