[Lldb-commits] [lldb] r255310 - test result details now print module.class.test_name in verbose mode.

[Todd Fiala via lldb-commits]

Sure thing :-)

On Thu, Dec 10, 2015 at 3:20 PM, Zachary Turner <zturner at google.com> wrote:

> That probably got turned on by default by me on accident when I removed
> some of the command line options that mucked with the verbosity setting.
> Thanks!
>
> On Thu, Dec 10, 2015 at 3:17 PM Todd Fiala via lldb-commits <
> lldb-commits at lists.llvm.org> wrote:
>
>> Author: tfiala
>> Date: Thu Dec 10 17:14:24 2015
>> New Revision: 255310
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=255310&view=rev
>> Log:
>> test result details now print module.class.test_name in verbose mode.
>>
>> And, turns off verbose mode by default.  This must have been switched
>> on as the default when somebody was testing.
>>
>> Modified:
>>     lldb/trunk/packages/Python/lldbsuite/test/basic_results_formatter.py
>>     lldb/trunk/packages/Python/lldbsuite/test/configuration.py
>>
>> Modified:
>> lldb/trunk/packages/Python/lldbsuite/test/basic_results_formatter.py
>> URL:
>> http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/basic_results_formatter.py?rev=255310&r1=255309&r2=255310&view=diff
>>
>> ==============================================================================
>> --- lldb/trunk/packages/Python/lldbsuite/test/basic_results_formatter.py
>> (original)
>> +++ lldb/trunk/packages/Python/lldbsuite/test/basic_results_formatter.py
>> Thu Dec 10 17:14:24 2015
>> @@ -13,6 +13,7 @@ from __future__ import print_function
>>  import os
>>
>>  # Our imports
>> +from . import configuration
>>  from . import result_formatter
>>  from .result_formatter import EventBuilder
>>  import lldbsuite
>> @@ -272,6 +273,13 @@ class BasicResultsFormatter(result_forma
>>                              extra_info,
>>                              event["test_filename"]))
>>                  else:
>> +                    # Figure out the identity we will use for this test.
>> +                    if configuration.verbose and ("test_class" in event):
>> +                        test_id = "{}.{}".format(
>> +                            event["test_class"], event["test_name"])
>> +                    else:
>> +                        test_id = event["test_name"]
>> +
>>                      # Test-method events have richer detail, use that
>> here.
>>                      test_relative_path = os.path.relpath(
>>                          os.path.realpath(event["test_filename"]),
>> @@ -279,7 +287,7 @@ class BasicResultsFormatter(result_forma
>>                      self.out_file.write("{}: {}{} ({})\n".format(
>>                          detail_label,
>>                          extra_info,
>> -                        event["test_name"],
>> +                        test_id,
>>                          test_relative_path))
>>
>>      def _finish_output_no_lock(self):
>>
>> Modified: lldb/trunk/packages/Python/lldbsuite/test/configuration.py
>> URL:
>> http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/configuration.py?rev=255310&r1=255309&r2=255310&view=diff
>>
>> ==============================================================================
>> --- lldb/trunk/packages/Python/lldbsuite/test/configuration.py (original)
>> +++ lldb/trunk/packages/Python/lldbsuite/test/configuration.py Thu Dec 10
>> 17:14:24 2015
>> @@ -120,7 +120,7 @@ sdir_has_content = False
>>  svn_info = ''
>>
>>  # Default verbosity is 0.
>> -verbose = 1
>> +verbose = 0
>>
>>  # By default, search from the script directory.
>>  # We can't use sys.path[0] to determine the script directory
>>
>>
>> _______________________________________________
>> lldb-commits mailing list
>> lldb-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
>>
>


-- 
-Todd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20151211/a844f836/attachment.html>