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

[Zachary Turner via lldb-commits]

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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20151210/cc128eba/attachment-0001.html>