<div dir="ltr">Hi Ted!<div><br></div><div>As noted in the copied code, *running is async notification and it only must be shown in async mode. The test checks that it's not being shown in sync mode. So the test is correct.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 19, 2016 at 12:56 AM, Ted Woodward via lldb-dev <span dir="ltr"><<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-US" link="#0563C1" vlink="#954F72"><div><p class="MsoNormal">Packages/Python/lldbsuite/test/tools/lldb-mi/TestMiGdbSetShow.py, in test_lldbmi_gdb_set_target_async_off we have this code:<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">        self.runCmd("-gdb-set target-async off")<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">…<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">        self.runCmd("-exec-run")<u></u><u></u></p><p class="MsoNormal">        unexpected = [ "\*running" ] # "\*running" is async notification<u></u><u></u></p><p class="MsoNormal">        it = self.expect(unexpected + [ "@\"argc=1\\\\r\\\\n" ])<u></u><u></u></p><p class="MsoNormal">        if it < len(unexpected):<u></u><u></u></p><p class="MsoNormal">            self.fail("unexpected found: %s" % unexpected[it])<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">But lldb-mi does the right thing, expect won’t match “running”, so the self.expect command fails, which causes the test to error out. Shouldn’t the self.expect be in a try, with an except being a pass?<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">--<u></u><u></u></p><p class="MsoNormal">Qualcomm Innovation Center, Inc.<u></u><u></u></p><p class="MsoNormal">The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p></div></div><br>_______________________________________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@lists.llvm.org">lldb-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">- Ilia</div></div>
</div>