[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

[Enrico Granata]

Sending        lldbtest.py
Transmitting file data .
Committed revision 221624.

This should fix it

> On Nov 10, 2014, at 8:20 AM, Ed Maste <emaste at freebsd.org> wrote:
> 
> 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)

Thanks,
- Enrico
📩 egranata@.com ☎️ 27683




-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20141110/c64f0a5c/attachment.html>