Set limit for number of failed EXPECTs per test in test launcher output.

base/test/gtest_xml_unittest_result_printer.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/test/gtest_xml_unittest_result_printer.h"
 
 #include "base/base64.h"
 #include "base/files/file_util.h"
 #include "base/logging.h"
 #include "base/time/time.h"
 
 namespace base {
 
-XmlUnitTestResultPrinter::XmlUnitTestResultPrinter() : output_file_(NULL) {
+namespace {
+const int kDefaultTestPartResultsLimit = 10;
 }
 
+XmlUnitTestResultPrinter::XmlUnitTestResultPrinter()
+    : output_file_(NULL),
+      test_part_results_limit_(kDefaultTestPartResultsLimit) {}
+
 XmlUnitTestResultPrinter::~XmlUnitTestResultPrinter() {
   if (output_file_) {
     fprintf(output_file_, "</testsuites>\n");
     fflush(output_file_);
     CloseFile(output_file_);
   }
 }
 
 bool XmlUnitTestResultPrinter::Initialize(const FilePath& output_file_path) {
   DCHECK(!output_file_);
@@ skipping 39 matching lines @@
           "    <testcase name=\"%s\" status=\"run\" time=\"%.3f\""
           " classname=\"%s\">\n",
           test_info.name(),
           static_cast<double>(test_info.result()->elapsed_time()) /
               Time::kMillisecondsPerSecond,
           test_info.test_case_name());
   if (test_info.result()->Failed()) {
     fprintf(output_file_,
             "      <failure message=\"\" type=\"\"></failure>\n");
   }
-  for (int i = 0; i < test_info.result()->total_part_count(); ++i) {
+
+  int limit = test_info.result()->total_part_count();
+  if (has_test_part_results_limit())
+    limit = std::min(limit, test_part_results_limit());
+
+  for (int i = 0; i < limit; ++i) {

[Paweł Hajdan Jr., 2017/05/05 17:33:03]

We need to add something in case the limit is exceeded, to avoid a
silent-failure-like condition.

[alex-ac, 2017/05/07 12:07:24]

Done.

     const auto& test_part_result = test_info.result()->GetTestPartResult(i);
     WriteTestPartResult(test_part_result.file_name(),
                         test_part_result.line_number(), test_part_result.type(),
                         test_part_result.summary(), test_part_result.message());
   }
+
   fprintf(output_file_, "    </testcase>\n");
   fflush(output_file_);
 }
 
 void XmlUnitTestResultPrinter::OnTestCaseEnd(
     const testing::TestCase& test_case) {
   fprintf(output_file_, "  </testsuite>\n");
   fflush(output_file_);
 }
 
@@ skipping 22 matching lines @@
   fprintf(output_file_,
           "      <x-test-result-part type=\"%s\" file=\"%s\" line=\"%d\">\n"
           "        <summary>%s</summary>\n"
           "        <message>%s</message>\n"
           "      </x-test-result-part>\n",
           type, file, line, summary_encoded.c_str(), message_encoded.c_str());
   fflush(output_file_);
 }
 
 }  // namespace base