UnwindOfflineTest: replace "> >" with ">>"

Accept either output from the C++ demangler (__cxa_demangle):

 * Foo<Bar<Baz> >
 * Foo<Bar<Baz>>

Bug: 175635923
Bug: 329940638
Test: libunwindstack_unit_test
Change-Id: I4277e73e8a3f7010b6e72d7ab30a6931d767e0eb

1 files changed, 10 insertions, 0 deletions

diff --git a/libunwindstack/tests/UnwindOfflineTest.cpp b/libunwindstack/tests/UnwindOfflineTest.cpp
index 7f7dee3..f8ee566 100644
--- a/libunwindstack/tests/UnwindOfflineTest.cpp
+++ b/libunwindstack/tests/UnwindOfflineTest.cpp
@@ -16,6 +16,7 @@
 
 #include <sys/mman.h>
 
+#include <android-base/strings.h>
 #include <gtest/gtest.h>
 
 #include <cstddef>
@@ -88,6 +89,15 @@ class UnwindOfflineTest : public ::testing::Test {
 
       std::string actual_frame_info = DumpFrames(unwinder);
       ASSERT_EQ(expected_num_frames, unwinder.NumFrames()) << "Unwind:\n" << actual_frame_info;
+
+      // The old demangler in external/libcxx adds an extra space when closing
+      // multiple sets of template args (e.g. "Foo<Bar<Baz> >"), but the new
+      // libc++ demangler omits it. Replace "> >" with ">>" to work with either
+      // demangler until libc++ is upgraded.
+      // TODO(b/329940638). Remove this workaround and fix the tests.
+      actual_frame_info = android::base::StringReplace(actual_frame_info, "> >", ">>", true);
+      expected_frame_info = android::base::StringReplace(expected_frame_info, "> >", ">>", true);
+
       EXPECT_EQ(expected_frame_info, actual_frame_info);
     }
   }