[PATCH] D139093: [include-cleaner] Use RAV instead of ASTMatchers in LocateSymbolTest

[Kadir Cetinkaya via Phabricator via cfe-commits]

kadircet marked an inline comment as done.
kadircet added inline comments.


================
Comment at: clang-tools-extra/include-cleaner/unittests/LocateSymbolTest.cpp:60
+          return true;
+        Out = ND;
+        return false;
----------------
sammccall wrote:
> you might want to EXPECT_EQ(Out, nullptr) first to ensure only a single match
> 
> (A single match for a name may not generalize well, but the matching would have to be made more specific regardless in that case)
we expect to see multiple declarations for the same entity anyway though, so instead of asserting null, i'll assert same canonical or null.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139093/new/

https://reviews.llvm.org/D139093