<div dir="ltr">No issues on Windows, looks good over here.</div><br><div class="gmail_quote"><div dir="ltr">On Wed, May 11, 2016 at 6:03 PM Todd Fiala <<a href="mailto:todd.fiala@gmail.com">todd.fiala@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tfiala created this revision.<br>
tfiala added reviewers: labath, zturner.<br>
tfiala added subscribers: lldb-commits, zturner.<br>
<br>
This change enhances the LLDB test infrastructure to convert<br>
load-time exceptions in a given Python test module into errors.<br>
Before this change, specifying a non-existent test decorator,<br>
or otherwise having some load-time error in a python test module,<br>
would not get flagged as an error.<br>
<br>
With this change, typos and other load-time errors in a python<br>
test file get converted to errors and reported by the<br>
test runner.<br>
<br>
This change also includes test infrastructure tests that include<br>
covering the new work here.  I'm going to wait until we have<br>
these infrastructure tests runnable on the main platforms before<br>
I try to work that into all the normal testing workflows.<br>
<br>
The test infrastructure tests can be run by using the standard python module testing practice of doing the following:<br>
<br>
cd packages/Python/lldbsuite/test_event<br>
python -m unittest discover -s test/src -p 'Test*.py'<br>
<br>
Those tests run the dotest inferior with a known broken test and verify that the errors are caught.  These tests did not pass until I modified dotest.py to capture them properly.<br>
<br>
@zturner, if you have the chance, if you could try those steps above (the python -m unittest ... line) on Windows, that would be great if we can address any python2/3/Windows bits there.  I don't think there's anything fancy, but I didn't want to hook it into test flow until I know it works there.<br>
<br>
I'll be slowly adding more tests that cover some of the other breakage I've occasionally seen that didn't get collected as part of the summarization.  This is the biggest one I'm aware of.<br>
<br>
<a href="http://reviews.llvm.org/D20193" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20193</a><br>
<br>
Files:<br>
  packages/Python/lldbsuite/test/dotest.py<br>
  packages/Python/lldbsuite/test/issue_verification/TestInvalidDecorator.py.park<br>
  packages/Python/lldbsuite/test_event/event_builder.py<br>
  packages/Python/lldbsuite/test_event/formatter/pickled.py<br>
  packages/Python/lldbsuite/test_event/test/resources/invalid_decorator/TestInvalidDecorator.py<br>
  packages/Python/lldbsuite/test_event/test/src/TestCatchInvalidDecorator.py<br>
  packages/Python/lldbsuite/test_event/test/src/event_collector.py<br>
<br>
</blockquote></div>