[testsuite] Reintroduce test to check leaking.

It wasn't python leaking, it was lldb. Thanks to Pavel for the
explanation. Pointy-hat to me.

llvm-svn: 324919

1 files changed, 108 insertions, 0 deletions

diff --git a/lldb/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/TestFdLeak.py b/lldb/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/TestFdLeak.py
new file mode 100644
index 00000000000..38848e87cc4
--- /dev/null
+++ b/lldb/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/TestFdLeak.py
@@ -0,0 +1,108 @@
+"""
+Test whether a process started by lldb has no extra file descriptors open.
+"""
+
+from __future__ import print_function
+
+
+import os
+import lldb
+from lldbsuite.test import lldbutil
+from lldbsuite.test.lldbtest import *
+from lldbsuite.test.decorators import *
+
+
+def python_leaky_fd_version(test):
+    import sys
+    # Python random module leaks file descriptors on some versions.
+    if sys.version_info >= (2, 7, 8) and sys.version_info < (2, 7, 10):
+        return "Python random module leaks file descriptors in this python version"
+    return None
+
+
+class AvoidsFdLeakTestCase(TestBase):
+
+    NO_DEBUG_INFO_TESTCASE = True
+
+    mydir = TestBase.compute_mydir(__file__)
+
+    @expectedFailure(python_leaky_fd_version, "bugs.freebsd.org/197376")
+    @expectedFailureAll(
+        oslist=['freebsd'],
+        bugnumber="llvm.org/pr25624 still failing with Python 2.7.10")
+    # The check for descriptor leakage needs to be implemented differently
+    # here.
+    @skipIfWindows
+    @skipIfTargetAndroid()  # Android have some other file descriptors open by the shell
+    @skipIfDarwinEmbedded # <rdar://problem/33888742>  # debugserver on ios has an extra fd open on launch
+    def test_fd_leak_basic(self):
+        self.do_test([])
+
+    @expectedFailure(python_leaky_fd_version, "bugs.freebsd.org/197376")
+    @expectedFailureAll(
+        oslist=['freebsd'],
+        bugnumber="llvm.org/pr25624 still failing with Python 2.7.10")
+    # The check for descriptor leakage needs to be implemented differently
+    # here.
+    @skipIfWindows
+    @skipIfTargetAndroid()  # Android have some other file descriptors open by the shell
+    @skipIfDarwinEmbedded # <rdar://problem/33888742>  # debugserver on ios has an extra fd open on launch
+    def test_fd_leak_log(self):
+        self.do_test(["log enable -f '/dev/null' lldb commands"])
+
+    def do_test(self, commands):
+        self.build()
+        exe = self.getBuildArtifact("a.out")
+
+        for c in commands:
+            self.runCmd(c)
+
+        target = self.dbg.CreateTarget(exe)
+
+        process = target.LaunchSimple(
+            None, None, self.get_process_working_directory())
+        self.assertTrue(process, PROCESS_IS_VALID)
+
+        self.assertTrue(
+            process.GetState() == lldb.eStateExited,
+            "Process should have exited.")
+        self.assertTrue(
+            process.GetExitStatus() == 0,
+            "Process returned non-zero status. Were incorrect file descriptors passed?")
+
+    @expectedFailure(python_leaky_fd_version, "bugs.freebsd.org/197376")
+    @expectedFailureAll(
+        oslist=['freebsd'],
+        bugnumber="llvm.org/pr25624 still failing with Python 2.7.10")
+    # The check for descriptor leakage needs to be implemented differently
+    # here.
+    @skipIfWindows
+    @skipIfTargetAndroid()  # Android have some other file descriptors open by the shell
+    @skipIfDarwinEmbedded # <rdar://problem/33888742>  # debugserver on ios has an extra fd open on launch
+    def test_fd_leak_multitarget(self):
+        self.build()
+        exe = self.getBuildArtifact("a.out")
+
+        target = self.dbg.CreateTarget(exe)
+        breakpoint = target.BreakpointCreateBySourceRegex(
+            'Set breakpoint here', lldb.SBFileSpec("main.c", False))
+        self.assertTrue(breakpoint, VALID_BREAKPOINT)
+
+        process1 = target.LaunchSimple(
+            None, None, self.get_process_working_directory())
+        self.assertTrue(process1, PROCESS_IS_VALID)
+        self.assertTrue(
+            process1.GetState() == lldb.eStateStopped,
+            "Process should have been stopped.")
+
+        target2 = self.dbg.CreateTarget(exe)
+        process2 = target2.LaunchSimple(
+            None, None, self.get_process_working_directory())
+        self.assertTrue(process2, PROCESS_IS_VALID)
+
+        self.assertTrue(
+            process2.GetState() == lldb.eStateExited,
+            "Process should have exited.")
+        self.assertTrue(
+            process2.GetExitStatus() == 0,
+            "Process returned non-zero status. Were incorrect file descriptors passed?")