[Lldb-commits] [PATCH] D86792: [lldb] Improve test failure reporting for expect()

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Sep 2 07:17:55 PDT 2020

labath added a comment.

I like this. And a big thank-you for writing the tests. We don't usually bother to write self-tests for the test infrastructure, but we definitely should be doing that. The place is slightly weird, but I think it will do for now -- it's easy to move this around if we find a better place for it.

Comment at: lldb/test/API/assert_messages_test/TestAssertMessages.py:37-38
+        # Will stop at startstr fail
+        e = self.catch_assert_fail("help", startstr="dog", endstr="cat")
+        self.expect_assert_msg(e, [dedent("""\
+                Ran command:
These two calls always come in pairs, right? Might be nice to make this a single function which gets both the command to run, and the error message it is expected to fail with.

  rG LLVM Github Monorepo



More information about the lldb-commits mailing list