Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(900)

Unified Diff: base/test/gtest_xml_unittest_result_printer.cc

Issue 2638763004: Report CHECK/DCHECK to test launcher summary output. (Closed)
Patch Set: Fix segmentation fault on android. Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: base/test/gtest_xml_unittest_result_printer.cc
diff --git a/base/test/gtest_xml_unittest_result_printer.cc b/base/test/gtest_xml_unittest_result_printer.cc
index 6ce4fdecf3fcddd2487f5e529e56f86d00f1a2e3..9a9e1ac0ea5dcf9504f134067cd9c7d006d5da87 100644
--- a/base/test/gtest_xml_unittest_result_printer.cc
+++ b/base/test/gtest_xml_unittest_result_printer.cc
@@ -35,6 +35,27 @@ bool XmlUnitTestResultPrinter::Initialize(const FilePath& output_file_path) {
return true;
}
+void XmlUnitTestResultPrinter::OnAssert(const char* file,
+ int line,
+ const std::string& summary,
+ const std::string& message) {
+ if (!output_file_)
brettw 2017/04/17 22:20:34 This function seems to duplicate WriteTestPartResu
alex-ac 2017/04/18 06:34:21 Done.
+ return;
+
+ std::string encoded_summary;
+ Base64Encode(summary, &encoded_summary);
+ std::string encoded_message;
+ Base64Encode(message, &encoded_message);
+ fprintf(output_file_,
+ " <x-test-result-part type=\"fatal_failure\" "
+ "file=\"%s\" line=\"%d\">\n"
+ " <summary>%s</summary>\n"
+ " <message>%s</message>\n"
+ " </x-test-result-part>",
+ file, line, encoded_summary.c_str(), encoded_message.c_str());
+ fflush(output_file_);
+}
+
void XmlUnitTestResultPrinter::OnTestCaseStart(
const testing::TestCase& test_case) {
fprintf(output_file_, " <testsuite>\n");

Powered by Google App Engine
This is Rietveld 408576698