[clang-tools-extra] r270199 - [find-all-symbol] Try to fix the failure windows unittest.

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Fri May 20 03:36:54 PDT 2016


Author: hokein
Date: Fri May 20 05:36:53 2016
New Revision: 270199

URL: http://llvm.org/viewvc/llvm-project?rev=270199&view=rev
Log:
[find-all-symbol] Try to fix the failure windows unittest.

Modified:
    clang-tools-extra/trunk/unittests/include-fixer/find-all-symbols/FindAllSymbolsTests.cpp

Modified: clang-tools-extra/trunk/unittests/include-fixer/find-all-symbols/FindAllSymbolsTests.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/unittests/include-fixer/find-all-symbols/FindAllSymbolsTests.cpp?rev=270199&r1=270198&r2=270199&view=diff
==============================================================================
--- clang-tools-extra/trunk/unittests/include-fixer/find-all-symbols/FindAllSymbolsTests.cpp (original)
+++ clang-tools-extra/trunk/unittests/include-fixer/find-all-symbols/FindAllSymbolsTests.cpp Fri May 20 05:36:53 2016
@@ -50,6 +50,10 @@ public:
     return false;
   }
 
+  const std::vector<SymbolInfo>& getSymbols() const {
+    return Symbols;
+  }
+
 private:
   std::vector<SymbolInfo> Symbols;
 };
@@ -125,7 +129,7 @@ public:
     return true;
   }
 
-private:
+protected:
   TestSymbolReporter Reporter;
 };
 
@@ -410,6 +414,8 @@ TEST_F(FindAllSymbolsTest, MacroTestWith
   runFindAllSymbols(Code);
   SymbolInfo Symbol =
       SymbolInfo("X", SymbolInfo::SymbolKind::Macro, "bar.h", 3, {});
+  EXPECT_EQ(3, Reporter.getSymbols().size());
+  EXPECT_EQ("bar.h", Reporter.getSymbols().front().getFilePath());
   EXPECT_TRUE(hasSymbol(Symbol));
 
   Symbol = SymbolInfo("Y", SymbolInfo::SymbolKind::Macro, "bar.h", 4, {});




More information about the cfe-commits mailing list