From 4bf67af2d59862f123c7d50170bdd6fd65da7da1 Mon Sep 17 00:00:00 2001 From: Eric Liu Date: Fri, 2 Dec 2016 11:23:07 +0000 Subject: [PATCH] Fix a buildbot failure in include-fixer. llvm-svn: 288495 --- .../unittests/include-fixer/IncludeFixerTest.cpp | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/clang-tools-extra/unittests/include-fixer/IncludeFixerTest.cpp b/clang-tools-extra/unittests/include-fixer/IncludeFixerTest.cpp index 47e7d04..756c258 100644 --- a/clang-tools-extra/unittests/include-fixer/IncludeFixerTest.cpp +++ b/clang-tools-extra/unittests/include-fixer/IncludeFixerTest.cpp @@ -111,13 +111,10 @@ TEST(IncludeFixer, Typo) { EXPECT_EQ("#include \nstd::string foo;\n", runIncludeFixer("std::string foo;\n")); - // FIXME: the current version of include-fixer does not get this test case - // right - header should be inserted before definition. - EXPECT_EQ( - "// comment\n#include \"foo.h\"\nstd::string foo;\n" - "#include \"dir/bar.h\"\n#include \n", - runIncludeFixer("// comment\n#include \"foo.h\"\nstd::string foo;\n" - "#include \"dir/bar.h\"\n")); + EXPECT_EQ("// comment\n#include \"foo.h\"\n#include \n" + "std::string foo;\n#include \"dir/bar.h\"\n", + runIncludeFixer("// comment\n#include \"foo.h\"\nstd::string foo;\n" + "#include \"dir/bar.h\"\n")); EXPECT_EQ("#include \"foo.h\"\n#include \nstd::string foo;\n", runIncludeFixer("#include \"foo.h\"\nstd::string foo;\n")); -- 2.7.4