diff options
| author | Eric Liu <ioeric@google.com> | 2016-12-02 11:23:07 +0000 |
|---|---|---|
| committer | Eric Liu <ioeric@google.com> | 2016-12-02 11:23:07 +0000 |
| commit | 4bf67af2d59862f123c7d50170bdd6fd65da7da1 (patch) | |
| tree | 724cddd956d93f490ba41c7c81a60d84fb107744 | |
| parent | e705c8b5e606de939f34adccd493aabb219f944f (diff) | |
| download | bcm5719-llvm-4bf67af2d59862f123c7d50170bdd6fd65da7da1.tar.gz bcm5719-llvm-4bf67af2d59862f123c7d50170bdd6fd65da7da1.zip | |
Fix a buildbot failure in include-fixer.
llvm-svn: 288495
| -rw-r--r-- | clang-tools-extra/unittests/include-fixer/IncludeFixerTest.cpp | 11 |
1 files 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 47e7d041703..756c2586283 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 <string>\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 <string>\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 <string>\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 <string>\nstd::string foo;\n", runIncludeFixer("#include \"foo.h\"\nstd::string foo;\n")); |
