<div dir="ltr"><span style="font-size:12.8px">> Now there is no equivelent to "dosep.py -s", and:</span><br style="font-size:12.8px"><span style="font-size:12.8px">>    ./dotest.py -v --executable $INSTALLDIR/bin/lldb</span><br style="font-size:12.8px"><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">All of dosep's args moved over to dotest.  I think if you wanted dosep.py -s behavior, you only need to use --output-on-success (long form of what used to be -s in dosep I believe).  The reason I couldn't maintain it as -s is that dotest.py already had a -s abbreviation (for session dir specification).</span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 4, 2015 at 12:02 PM,  <span dir="ltr"><<a href="mailto:dawn@burble.org" target="_blank">dawn@burble.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, Sep 04, 2015 at 06:25:44PM +0000, Todd Fiala wrote:<br>
> tfiala added a comment.<br>
><br>
> Dawn, the output supported by the command shouldn't have been changed.  I'll have a peek at what may have caused that to happen.  All this change was trying to do was stop the need to call dosep and roll that behavior into dotest.<br>
<br>
</span>Both of these commands:<br>
    ./dosep.py -s --options "-v --executable $INSTALLDIR/bin/lldb"<br>
    ./dotest.py -v --executable $INSTALLDIR/bin/lldb<br>
<br>
used to print results like the following:<br>
<span class="">    Collected 6 tests<br>
<br>
    1: test_double_type_from_block_with_dsym<br>
    (TestFloatTypes.FloatTypesTestCase)<br>
    Test that double-type variables are displayed correctly from a<br>
    block. ... ok<br>
    2: test_double_type_with_dsym (TestFloatTypes.FloatTypesTestCase)<br>
    Test that double-type variables are displayed correctly. ... ok<br>
    3: test_double_type_with_dwarf (TestFloatTypes.FloatTypesTestCase)<br>
    Test that double-type variables are displayed correctly. ... ok<br>
    4: test_float_type_from_block_with_dsym<br>
    (TestFloatTypes.FloatTypesTestCase)<br>
    Test that float-type variables are displayed correctly from a block.<br>
    ... ok<br>
    5: test_float_type_with_dsym (TestFloatTypes.FloatTypesTestCase)<br>
    Test that float-type variables are displayed correctly. ... ok<br>
    6: test_float_type_with_dwarf (TestFloatTypes.FloatTypesTestCase)<br>
    Test that float-type variables are displayed correctly. ... ok<br>
<br>
    ----------------------------------------------------------------------<br>
    Ran 6 tests in 6.844s<br>
<br>
    RESULT: PASSED (6 passes, 0 failures, 0 errors, 0 skipped, 0<br>
            expected failures, 0 unexpected successes)<br>
<br>
    [TestFloatTypes.py PASSED]<br>
<br>
</span>Now there is no equivelent to "dosep.py -s", and:<br>
    ./dotest.py -v --executable $INSTALLDIR/bin/lldb<br>
<br>
prints results like the following:<br>
    Testing: 395 test suites, 8 threads<br>
<br>
    0 out of 395 test suites processed - ['./dotest.py', '-v',<br>
    '--executable',<br>
    '/Users/testuser/build/workspace/LLVM-Clang-LLDB_master_release_OSX/llvm/build_ninja/lldb_redist/usr/local/bin/lldb']<br>
<br>
        1 out of 395 test suites processed - TestCompileRunToBreakpointTurnaround.py<br>
        [...]<br>
    395 out of 395 test suites processed - TestRecursiveTypes.py<br>
    Ran 395 test suites (0 failed) (0.000000%)<br>
    Ran 826 test cases (0 failed) (0.000000%)<br>
<br>
So we no longer have info on what tests ran, can't figure out what<br>
tests were skipped, can't calculate the correct totals, etc.<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">-Todd</div></div>
</div>