[Lldb-commits] [lldb] r324743 - [Testsuite] Remove leak tests, it's not useful anymore.
Davide Italiano via lldb-commits
lldb-commits at lists.llvm.org
Fri Feb 9 08:06:39 PST 2018
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;
-}
More information about the lldb-commits
mailing list