llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-format Author: Owen Pan (owenca) <details> <summary>Changes</summary> --- Full diff: https://github.com/llvm/llvm-project/pull/136662.diff 1 Files Affected: - (modified) clang/unittests/Format/FormatTestJS.cpp (+10-6) ``````````diff diff --git a/clang/unittests/Format/FormatTestJS.cpp b/clang/unittests/Format/FormatTestJS.cpp index 1cfacc060d944..91577b9a49167 100644 --- a/clang/unittests/Format/FormatTestJS.cpp +++ b/clang/unittests/Format/FormatTestJS.cpp @@ -48,18 +48,22 @@ class FormatTestJS : public testing::Test { static void verifyFormat( StringRef Code, const FormatStyle &Style = getGoogleStyle(FormatStyle::LK_JavaScript)) { - EXPECT_EQ(Code.str(), format(Code, Style)) << "Expected code is not stable"; - std::string Result = format(test::messUp(Code), Style); - EXPECT_EQ(Code.str(), Result) << "Formatted:\n" << Result; + auto Result = format(test::messUp(Code), Style); + EXPECT_EQ(Code, Result) << "Formatted:\n" << Result; + if (Code != Result) + return; + EXPECT_EQ(Code, format(Code, Style)) << "Expected code is not stable"; } static void verifyFormat( StringRef Expected, StringRef Code, const FormatStyle &Style = getGoogleStyle(FormatStyle::LK_JavaScript)) { - EXPECT_EQ(Expected.str(), format(Expected, Style)) + auto Result = format(Code, Style); + EXPECT_EQ(Expected, Result) << "Formatted:\n" << Result; + if (Expected != Result) + return; + EXPECT_EQ(Expected, format(Expected, Style)) << "Expected code is not stable"; - std::string Result = format(Code, Style); - EXPECT_EQ(Expected.str(), Result) << "Formatted:\n" << Result; } }; `````````` </details> https://github.com/llvm/llvm-project/pull/136662 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
