commit 075ddba7c61c8eda4833f12d8c422f2939ccefdb
author Lei Zhang <thestig@chromium.org> Thu May 04 20:26:30 2023 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu May 04 20:26:30 2023 +0000
tree 14a7f35b722596b870e9e813242c85100da30ab3
parent 77b1258e9418f665608c06a4c285259461c54b8b

Teach googletest how to pretty-print WideString

Add a test-only PrintTo() function for WideString.

Change-Id: I923ba171e12bebc1791d37bfaf5ade06bde7db66
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/106710
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: K. Moon <kmoon@chromium.org>

core/fxcrt/string_test_support.cpp

diff --git a/core/fxcrt/string_test_support.cpp b/core/fxcrt/string_test_support.cpp
index 254e37a..cbb1220 100644
--- a/core/fxcrt/string_test_support.cpp
+++ b/core/fxcrt/string_test_support.cpp
@@ -5,6 +5,7 @@
 #include <ostream>
 
 #include "core/fxcrt/bytestring.h"
+#include "core/fxcrt/widestring.h"
 
 namespace fxcrt {
 
@@ -12,4 +13,8 @@
   *os << str;
 }
 
+void PrintTo(const WideString& str, std::ostream* os) {
+  *os << str;
+}
+
 }  // namespace fxcrt

core/fxcrt/widestring.h

diff --git a/core/fxcrt/widestring.h b/core/fxcrt/widestring.h
index cf9edd3..ff291f8 100644
--- a/core/fxcrt/widestring.h
+++ b/core/fxcrt/widestring.h
@@ -304,6 +304,13 @@
 std::wostream& operator<<(std::wostream& os, WideStringView str);
 std::ostream& operator<<(std::ostream& os, WideStringView str);
 
+// This is declared here for use in gtest-based tests but is defined in a test
+// support target. This should not be used in production code. Just use
+// operator<< from above instead.
+// In some cases, gtest will automatically use operator<< as well, but in this
+// case, it needs PrintTo() because WideString looks like a container to gtest.
+void PrintTo(const WideString& str, std::ostream* os);
+
 }  // namespace fxcrt
 
 using WideString = fxcrt::WideString;