The test added for PR#164905 doesn't run on Windows.

[jimingham]

No description provided.

[llvmbot]

@llvm/pr-subscribers-lldb

Author: None (jimingham)

Changes

---

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

1 Files Affected:

• (modified) lldb/test/API/driver/stdio_closed/TestDriverWithClosedSTDIO.py (+3)

diff --git a/lldb/test/API/driver/stdio_closed/TestDriverWithClosedSTDIO.py b/lldb/test/API/driver/stdio_closed/TestDriverWithClosedSTDIO.py
index cff97b822db81..45a59f7839964 100644
--- a/lldb/test/API/driver/stdio_closed/TestDriverWithClosedSTDIO.py
+++ b/lldb/test/API/driver/stdio_closed/TestDriverWithClosedSTDIO.py
@@ -20,6 +20,9 @@ class TestDriverWithClosedSTDIO(TestBase):
     # each debug info format.
     NO_DEBUG_INFO_TESTCASE = True
 
+    # Windows doesn't have the fcntl module, so we can't run this
+    # test there.
+    @skipIf(oslist=["windows"])
     def test_run_lldb_and_wait(self):
         """This test forks, closes the stdio channels and exec's lldb.
         Then it waits for it to exit and asserts it did that successfully"""

[jimingham]

The x86_64 linux bot succeeded, and the aarch64 failure was a failure to even get the test run started. Since this is fixing a test failure on the Windows bot, I'm going to push this now rather than waiting for another aarch64 Linux run.