Author: davide Date: Fri Feb 9 08:06:39 2018 New Revision: 324743 URL: http://llvm.org/viewvc/llvm-project?rev=324743&view=rev Log: [Testsuite] Remove leak tests, it's not useful anymore.
This only worked on MacOS, which now ships a newer version of python without this bug. As such, we don't leak the fd, and this test is not needed anymore (as it also hardcoded the python version in the check). Removed: lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/Makefile lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/TestFdLeak.py lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/main.c Removed: lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/Makefile URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/Makefile?rev=324742&view=auto ============================================================================== --- lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/Makefile (original) +++ lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/Makefile (removed) @@ -1,5 +0,0 @@ -LEVEL = ../../make - -C_SOURCES := main.c - -include $(LEVEL)/Makefile.rules Removed: lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/TestFdLeak.py URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/TestFdLeak.py?rev=324742&view=auto ============================================================================== --- lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/TestFdLeak.py (original) +++ lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/TestFdLeak.py (removed) @@ -1,108 +0,0 @@ -""" -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?") Removed: lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/main.c URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/main.c?rev=324742&view=auto ============================================================================== --- lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/main.c (original) +++ lldb/trunk/packages/Python/lldbsuite/test/functionalities/avoids-fd-leak/main.c (removed) @@ -1,28 +0,0 @@ -#include <sys/types.h> -#include <sys/stat.h> -#include <unistd.h> -#include <errno.h> -#include <stdio.h> - -int -main (int argc, char const **argv) -{ - struct stat buf; - int i, rv = 0; // Set breakpoint here. - - // Make sure stdin/stdout/stderr exist. - for (i = 0; i <= 2; ++i) { - if (fstat(i, &buf) != 0) - return 1; - } - - // Make sure no other file descriptors are open. - for (i = 3; i <= 256; ++i) { - if (fstat(i, &buf) == 0 || errno != EBADF) { - fprintf(stderr, "File descriptor %d is open.\n", i); - rv = 2; - } - } - - return rv; -} _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits