Alexander Kornienko | 9eb8c92 | 2014-04-08 12:27:49 +0000 | [diff] [blame] | 1 | #include "ClangTidy.h" |
| 2 | #include "ClangTidyTest.h" |
| 3 | #include "gtest/gtest.h" |
| 4 | |
| 5 | namespace clang { |
| 6 | namespace tidy { |
| 7 | namespace test { |
| 8 | |
| 9 | class TestCheck : public ClangTidyCheck { |
| 10 | public: |
| 11 | void registerMatchers(ast_matchers::MatchFinder *Finder) override { |
| 12 | Finder->addMatcher(ast_matchers::varDecl().bind("var"), this); |
| 13 | } |
| 14 | void check(const ast_matchers::MatchFinder::MatchResult &Result) override { |
| 15 | const VarDecl *Var = Result.Nodes.getNodeAs<VarDecl>("var"); |
| 16 | // Add diagnostics in the wrong order. |
| 17 | diag(Var->getLocation(), "variable"); |
| 18 | diag(Var->getTypeSpecStartLoc(), "type specifier"); |
| 19 | } |
| 20 | }; |
| 21 | |
| 22 | TEST(ClangTidyDiagnosticConsumer, SortsErrors) { |
Alexander Kornienko | 826b5ad | 2014-05-09 12:24:09 +0000 | [diff] [blame] | 23 | std::vector<ClangTidyError> Errors; |
| 24 | runCheckOnCode<TestCheck>("int a;", &Errors); |
Alexander Kornienko | 9eb8c92 | 2014-04-08 12:27:49 +0000 | [diff] [blame] | 25 | EXPECT_EQ(2ul, Errors.size()); |
| 26 | // FIXME: Remove " []" once the check name is removed from the message text. |
| 27 | EXPECT_EQ("type specifier []", Errors[0].Message.Message); |
| 28 | EXPECT_EQ("variable []", Errors[1].Message.Message); |
| 29 | } |
| 30 | |
Alexander Kornienko | b3d331d | 2014-08-06 11:49:10 +0000 | [diff] [blame^] | 31 | TEST(GlobList, Empty) { |
| 32 | GlobList Filter(""); |
Alexander Kornienko | 23fe959 | 2014-05-15 14:27:36 +0000 | [diff] [blame] | 33 | |
Alexander Kornienko | b3d331d | 2014-08-06 11:49:10 +0000 | [diff] [blame^] | 34 | EXPECT_TRUE(Filter.contains("")); |
| 35 | EXPECT_FALSE(Filter.contains("aaa")); |
Alexander Kornienko | 23fe959 | 2014-05-15 14:27:36 +0000 | [diff] [blame] | 36 | } |
| 37 | |
Alexander Kornienko | b3d331d | 2014-08-06 11:49:10 +0000 | [diff] [blame^] | 38 | TEST(GlobList, Nothing) { |
| 39 | GlobList Filter("-*"); |
Alexander Kornienko | 23fe959 | 2014-05-15 14:27:36 +0000 | [diff] [blame] | 40 | |
Alexander Kornienko | b3d331d | 2014-08-06 11:49:10 +0000 | [diff] [blame^] | 41 | EXPECT_FALSE(Filter.contains("")); |
| 42 | EXPECT_FALSE(Filter.contains("a")); |
| 43 | EXPECT_FALSE(Filter.contains("-*")); |
| 44 | EXPECT_FALSE(Filter.contains("-")); |
| 45 | EXPECT_FALSE(Filter.contains("*")); |
Alexander Kornienko | 23fe959 | 2014-05-15 14:27:36 +0000 | [diff] [blame] | 46 | } |
| 47 | |
Alexander Kornienko | b3d331d | 2014-08-06 11:49:10 +0000 | [diff] [blame^] | 48 | TEST(GlobList, Everything) { |
| 49 | GlobList Filter("*"); |
Alexander Kornienko | 23fe959 | 2014-05-15 14:27:36 +0000 | [diff] [blame] | 50 | |
Alexander Kornienko | b3d331d | 2014-08-06 11:49:10 +0000 | [diff] [blame^] | 51 | EXPECT_TRUE(Filter.contains("")); |
| 52 | EXPECT_TRUE(Filter.contains("aaaa")); |
| 53 | EXPECT_TRUE(Filter.contains("-*")); |
| 54 | EXPECT_TRUE(Filter.contains("-")); |
| 55 | EXPECT_TRUE(Filter.contains("*")); |
Alexander Kornienko | 23fe959 | 2014-05-15 14:27:36 +0000 | [diff] [blame] | 56 | } |
| 57 | |
Alexander Kornienko | b3d331d | 2014-08-06 11:49:10 +0000 | [diff] [blame^] | 58 | TEST(GlobList, Simple) { |
| 59 | GlobList Filter("aaa"); |
Alexander Kornienko | 23fe959 | 2014-05-15 14:27:36 +0000 | [diff] [blame] | 60 | |
Alexander Kornienko | b3d331d | 2014-08-06 11:49:10 +0000 | [diff] [blame^] | 61 | EXPECT_TRUE(Filter.contains("aaa")); |
| 62 | EXPECT_FALSE(Filter.contains("")); |
| 63 | EXPECT_FALSE(Filter.contains("aa")); |
| 64 | EXPECT_FALSE(Filter.contains("aaaa")); |
| 65 | EXPECT_FALSE(Filter.contains("bbb")); |
Alexander Kornienko | 23fe959 | 2014-05-15 14:27:36 +0000 | [diff] [blame] | 66 | } |
| 67 | |
Alexander Kornienko | b3d331d | 2014-08-06 11:49:10 +0000 | [diff] [blame^] | 68 | TEST(GlobList, Complex) { |
| 69 | GlobList Filter("*,-a.*,-b.*,a.1.*,-a.1.A.*,-..,-...,-..+,-*$,-*qwe*"); |
Alexander Kornienko | 23fe959 | 2014-05-15 14:27:36 +0000 | [diff] [blame] | 70 | |
Alexander Kornienko | b3d331d | 2014-08-06 11:49:10 +0000 | [diff] [blame^] | 71 | EXPECT_TRUE(Filter.contains("aaa")); |
| 72 | EXPECT_TRUE(Filter.contains("qqq")); |
| 73 | EXPECT_FALSE(Filter.contains("a.")); |
| 74 | EXPECT_FALSE(Filter.contains("a.b")); |
| 75 | EXPECT_FALSE(Filter.contains("b.")); |
| 76 | EXPECT_FALSE(Filter.contains("b.b")); |
| 77 | EXPECT_TRUE(Filter.contains("a.1.b")); |
| 78 | EXPECT_FALSE(Filter.contains("a.1.A.a")); |
| 79 | EXPECT_FALSE(Filter.contains("qwe")); |
| 80 | EXPECT_FALSE(Filter.contains("asdfqweasdf")); |
| 81 | EXPECT_TRUE(Filter.contains("asdfqwEasdf")); |
Alexander Kornienko | 23fe959 | 2014-05-15 14:27:36 +0000 | [diff] [blame] | 82 | } |
| 83 | |
Alexander Kornienko | 9eb8c92 | 2014-04-08 12:27:49 +0000 | [diff] [blame] | 84 | } // namespace test |
| 85 | } // namespace tidy |
| 86 | } // namespace clang |