<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Jan 15, 2016 at 11:57 AM Adrian McCarthy <<a href="mailto:amccarth@google.com">amccarth@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">amccarth created this revision.<br>
amccarth added a reviewer: zturner.<br>
amccarth added a subscriber: lldb-commits.<br>
<br>
Tested on Windows with Python 3.5.<br>
<br>
<a href="http://reviews.llvm.org/D16237" rel="noreferrer" target="_blank">http://reviews.llvm.org/D16237</a><br>
<br>
Files:<br>
  packages/Python/lldbsuite/test/logging/TestLogging.py<br>
<br>
Index: packages/Python/lldbsuite/test/logging/TestLogging.py<br>
===================================================================<br>
--- packages/Python/lldbsuite/test/logging/TestLogging.py<br>
+++ packages/Python/lldbsuite/test/logging/TestLogging.py<br>
@@ -61,7 +61,7 @@<br>
         f.close ()<br>
         os.remove (log_file)<br>
<br>
-        self.assertTrue(log_lines > 0, "Something was written to the log file.")<br>
+        self.assertTrue(len(log_lines) > 0, "Something was written to the log file.")<br></blockquote><div>Can you use self.assertGreater </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
     # Check that lldb truncates its log files<br>
     @no_debug_info_test<br>
@@ -83,7 +83,7 @@<br>
             contents = f.read ()<br>
<br>
         # check that it got removed<br>
-        self.assertTrue(string.find(contents, "bacon") == -1)<br>
+        self.assertTrue(contents.find("bacon") == -1)<br></blockquote><div>and self.assertEqual</div></div></div>