[cfe-commits] [PATCH] Add unit tests for source locations of AST nodes

[Philip Craig]

================
Comment at: unittests/AST/SourceLocationTest.cpp:24
@@ +23,3 @@
+/// \brief Base class for verifying some property of nodes found by a matcher.
+class MatchVerifier : public MatchFinder::MatchCallback {
+public:
----------------
Manuel Klimek wrote:
> I see overlap in the implementation with both TestVisitor.h and RecursiveASTVisitorTest.cpp (which already has location testing).
> 
> Do you think we can pull out something common?
There's bits of overlap between all of these unit tests, and I'm not sure yet which bits should be common.

I started using TestVisitor.h, but the structure of that code doesn't lend itself well to adding in range checking. I couldn't see a way to add it without either rewriting it all, or duplicating a lot of code.

The secondary issue is that TestVisitor.h doesn't use ASTMatchers, and I think they are quite useful for this sort of testing. I'm assuming that ASTMatchers shouldn't be used for testing RAV, since it's essentially another lay on top of RAV, so there may be things you can't test that way.


http://llvm-reviews.chandlerc.com/D72