<div dir="ltr">Hi Vince,<div><br></div><div>Can you put back the @expectedFailure decorator next to the @skipIfDarwin decorator? Having both will have the same effect as @skipIfDarwin only but makes it easier when we grep for XFAIL-s (we do it for several test case which times out on the Linux build bot).</div><div><br></div><div>Thanks,</div><div>Tamas</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 12, 2015 at 6:18 AM, Vince Harron <span dir="ltr"><<a href="mailto:vince@nethacker.com" target="_blank">vince@nethacker.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: vharron<br>
Date: Tue May 12 00:18:06 2015<br>
New Revision: 237089<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=237089&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=237089&view=rev</a><br>
Log:<br>
TestPublicAPIHeaders.py - Changed expectedFailureDarwin to skipIfDarwin<br>
<br>
expectedFailure doesn't work if the failure is in a teardown step.<br>
<br>
<br>
Modified:<br>
    lldb/trunk/test/api/check_public_api_headers/TestPublicAPIHeaders.py<br>
<br>
Modified: lldb/trunk/test/api/check_public_api_headers/TestPublicAPIHeaders.py<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/lldb/trunk/test/api/check_public_api_headers/TestPublicAPIHeaders.py?rev=237089&r1=237088&r2=237089&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/lldb/trunk/test/api/check_public_api_headers/TestPublicAPIHeaders.py?rev=237089&r1=237088&r2=237089&view=diff</a><br>
==============================================================================<br>
--- lldb/trunk/test/api/check_public_api_headers/TestPublicAPIHeaders.py (original)<br>
+++ lldb/trunk/test/api/check_public_api_headers/TestPublicAPIHeaders.py Tue May 12 00:18:06 2015<br>
@@ -21,7 +21,8 @@ class SBDirCheckerCase(TestBase):<br>
         self.exe_name = 'a.out'<br>
<br>
     @skipIfNoSBHeaders<br>
-    @expectedFailureDarwin # test passes but teardown command 'settings remove target.env-vars DYLD_LIBRARY_PATH' fails<br>
+    @skipIfDarwin # test passes but teardown command 'settings remove target.env-vars DYLD_LIBRARY_PATH' fails<br>
+    # (expectedFailureDarwin doesn't work for teardown failures)<br>
     def test_sb_api_directory(self):<br>
         """Test the SB API directory and make sure there's no unwanted stuff."""<br>
<br>
<br>
<br>
_______________________________________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
</blockquote></div><br></div>