<div dir="ltr">looks good</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Jan 15, 2016 at 12:44 PM 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 updated this revision to Diff 45023.<br>
amccarth added a comment.<br>
<br>
Used the more explicit assertion methods.<br>
<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.assertGreater(len(log_lines), 0, "Something was written to the log file.")<br>
<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.assertEquals(contents.find("bacon"), -1)<br>
<br>
     # Check that lldb can append to a log file<br>
     @no_debug_info_test<br>
@@ -104,4 +104,4 @@<br>
             contents = f.read ()<br>
<br>
         # check that it is still there<br>
-        self.assertTrue(string.find(contents, "bacon") == 0)<br>
+        self.assertEquals(contents.find("bacon"), 0)<br>
<br>
<br>
</blockquote></div>