[Lldb-commits] [lldb] r221402 - Allow inline test case to register actually useful teardown hooks by allowing a hook to be passed back the test instance, were it not to be already bound to self. Use this ability to make the reversal of escape-non-printables a teardown hook for added reliability of the testing logic

[Ed Maste]

On 5 November 2014 16:31, Enrico Granata <egranata at apple.com> wrote:
> Author: enrico
> Date: Wed Nov  5 15:31:57 2014
> New Revision: 221402
>
> URL: http://llvm.org/viewvc/llvm-project?rev=221402&view=rev
> Log:
> Allow inline test case to register actually useful teardown hooks by allowing a hook to be passed back the test instance, were it not to be already bound to self. Use this ability to make the reversal of escape-non-printables a teardown hook for added reliability of the testing logic

I have a couple of tests failing after this change like so:

======================================================================
ERROR: test_with_dwarf_and_attach_to_process_with_name_api
(TestHelloWorld.HelloWorldTestCase)
   Create target, spawn a process, and attach to it with process name.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tank/emaste/src/llvm/tools/lldb/test/lldbtest.py", line 1659,
in tearDown
    Base.tearDown(self)
  File "/tank/emaste/src/llvm/tools/lldb/test/lldbtest.py", line 1043,
in tearDown
    hook(self)
TypeError: cleanupSubprocesses() takes exactly 1 argument (2 given)
Config=amd64-/usr/bin/clang
----------------------------------------------------------------------
Ran 1 test in 1.484s

FAILED (errors=1)