Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 1 | #include "ClangTidyOptions.h" |
| 2 | #include "gtest/gtest.h" |
Alexander Kornienko | bb56052 | 2016-08-23 14:48:29 +0000 | [diff] [blame] | 3 | #include "llvm/ADT/StringExtras.h" |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 4 | |
| 5 | namespace clang { |
| 6 | namespace tidy { |
| 7 | namespace test { |
| 8 | |
| 9 | TEST(ParseLineFilter, EmptyFilter) { |
Alexander Kornienko | a469522 | 2014-06-05 13:31:45 +0000 | [diff] [blame] | 10 | ClangTidyGlobalOptions Options; |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 11 | EXPECT_FALSE(parseLineFilter("", Options)); |
| 12 | EXPECT_TRUE(Options.LineFilter.empty()); |
| 13 | EXPECT_FALSE(parseLineFilter("[]", Options)); |
| 14 | EXPECT_TRUE(Options.LineFilter.empty()); |
| 15 | } |
| 16 | |
| 17 | TEST(ParseLineFilter, InvalidFilter) { |
Alexander Kornienko | a469522 | 2014-06-05 13:31:45 +0000 | [diff] [blame] | 18 | ClangTidyGlobalOptions Options; |
Daniel Jasper | 136f4b3 | 2015-03-02 18:07:00 +0000 | [diff] [blame] | 19 | EXPECT_TRUE(!!parseLineFilter("asdf", Options)); |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 20 | EXPECT_TRUE(Options.LineFilter.empty()); |
| 21 | |
Rafael Espindola | f145137 | 2014-06-03 04:40:55 +0000 | [diff] [blame] | 22 | EXPECT_TRUE(!!parseLineFilter("[{}]", Options)); |
| 23 | EXPECT_TRUE(!!parseLineFilter("[{\"name\":\"\"}]", Options)); |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 24 | EXPECT_TRUE( |
Rafael Espindola | f145137 | 2014-06-03 04:40:55 +0000 | [diff] [blame] | 25 | !!parseLineFilter("[{\"name\":\"test\",\"lines\":[[1]]}]", Options)); |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 26 | EXPECT_TRUE( |
Rafael Espindola | f145137 | 2014-06-03 04:40:55 +0000 | [diff] [blame] | 27 | !!parseLineFilter("[{\"name\":\"test\",\"lines\":[[1,2,3]]}]", Options)); |
NAKAMURA Takumi | 98247b3 | 2014-05-24 17:22:01 +0000 | [diff] [blame] | 28 | EXPECT_TRUE( |
Rafael Espindola | f145137 | 2014-06-03 04:40:55 +0000 | [diff] [blame] | 29 | !!parseLineFilter("[{\"name\":\"test\",\"lines\":[[1,-1]]}]", Options)); |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 30 | } |
| 31 | |
| 32 | TEST(ParseLineFilter, ValidFilter) { |
Alexander Kornienko | a469522 | 2014-06-05 13:31:45 +0000 | [diff] [blame] | 33 | ClangTidyGlobalOptions Options; |
Rafael Espindola | 15c5784 | 2014-06-12 13:32:11 +0000 | [diff] [blame] | 34 | std::error_code Error = parseLineFilter( |
NAKAMURA Takumi | 98247b3 | 2014-05-24 17:22:01 +0000 | [diff] [blame] | 35 | "[{\"name\":\"file1.cpp\",\"lines\":[[3,15],[20,30],[42,42]]}," |
| 36 | "{\"name\":\"file2.h\"}," |
| 37 | "{\"name\":\"file3.cc\",\"lines\":[[100,1000]]}]", |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 38 | Options); |
| 39 | EXPECT_FALSE(Error); |
| 40 | EXPECT_EQ(3u, Options.LineFilter.size()); |
| 41 | EXPECT_EQ("file1.cpp", Options.LineFilter[0].Name); |
| 42 | EXPECT_EQ(3u, Options.LineFilter[0].LineRanges.size()); |
| 43 | EXPECT_EQ(3u, Options.LineFilter[0].LineRanges[0].first); |
| 44 | EXPECT_EQ(15u, Options.LineFilter[0].LineRanges[0].second); |
| 45 | EXPECT_EQ(20u, Options.LineFilter[0].LineRanges[1].first); |
| 46 | EXPECT_EQ(30u, Options.LineFilter[0].LineRanges[1].second); |
| 47 | EXPECT_EQ(42u, Options.LineFilter[0].LineRanges[2].first); |
| 48 | EXPECT_EQ(42u, Options.LineFilter[0].LineRanges[2].second); |
| 49 | EXPECT_EQ("file2.h", Options.LineFilter[1].Name); |
| 50 | EXPECT_EQ(0u, Options.LineFilter[1].LineRanges.size()); |
| 51 | EXPECT_EQ("file3.cc", Options.LineFilter[2].Name); |
| 52 | EXPECT_EQ(1u, Options.LineFilter[2].LineRanges.size()); |
| 53 | EXPECT_EQ(100u, Options.LineFilter[2].LineRanges[0].first); |
| 54 | EXPECT_EQ(1000u, Options.LineFilter[2].LineRanges[0].second); |
| 55 | } |
| 56 | |
Alexander Kornienko | a469522 | 2014-06-05 13:31:45 +0000 | [diff] [blame] | 57 | TEST(ParseConfiguration, ValidConfiguration) { |
Alexander Kornienko | e995154 | 2014-09-24 18:36:03 +0000 | [diff] [blame] | 58 | llvm::ErrorOr<ClangTidyOptions> Options = |
| 59 | parseConfiguration("Checks: \"-*,misc-*\"\n" |
| 60 | "HeaderFilterRegex: \".*\"\n" |
Roman Lebedev | 6e76a1b | 2018-05-08 23:15:58 +0000 | [diff] [blame] | 61 | "AnalyzeTemporaryDtors: true\n" |
Alexander Kornienko | e995154 | 2014-09-24 18:36:03 +0000 | [diff] [blame] | 62 | "User: some.user"); |
| 63 | EXPECT_TRUE(!!Options); |
| 64 | EXPECT_EQ("-*,misc-*", *Options->Checks); |
| 65 | EXPECT_EQ(".*", *Options->HeaderFilterRegex); |
Alexander Kornienko | e995154 | 2014-09-24 18:36:03 +0000 | [diff] [blame] | 66 | EXPECT_EQ("some.user", *Options->User); |
Alexander Kornienko | a469522 | 2014-06-05 13:31:45 +0000 | [diff] [blame] | 67 | } |
| 68 | |
Alexander Kornienko | bb56052 | 2016-08-23 14:48:29 +0000 | [diff] [blame] | 69 | TEST(ParseConfiguration, MergeConfigurations) { |
| 70 | llvm::ErrorOr<ClangTidyOptions> Options1 = parseConfiguration(R"( |
| 71 | Checks: "check1,check2" |
| 72 | HeaderFilterRegex: "filter1" |
Roman Lebedev | 6e76a1b | 2018-05-08 23:15:58 +0000 | [diff] [blame] | 73 | AnalyzeTemporaryDtors: true |
Alexander Kornienko | bb56052 | 2016-08-23 14:48:29 +0000 | [diff] [blame] | 74 | User: user1 |
| 75 | ExtraArgs: ['arg1', 'arg2'] |
| 76 | ExtraArgsBefore: ['arg-before1', 'arg-before2'] |
| 77 | )"); |
| 78 | ASSERT_TRUE(!!Options1); |
| 79 | llvm::ErrorOr<ClangTidyOptions> Options2 = parseConfiguration(R"( |
| 80 | Checks: "check3,check4" |
| 81 | HeaderFilterRegex: "filter2" |
Roman Lebedev | 6e76a1b | 2018-05-08 23:15:58 +0000 | [diff] [blame] | 82 | AnalyzeTemporaryDtors: false |
Alexander Kornienko | bb56052 | 2016-08-23 14:48:29 +0000 | [diff] [blame] | 83 | User: user2 |
| 84 | ExtraArgs: ['arg3', 'arg4'] |
| 85 | ExtraArgsBefore: ['arg-before3', 'arg-before4'] |
| 86 | )"); |
| 87 | ASSERT_TRUE(!!Options2); |
| 88 | ClangTidyOptions Options = Options1->mergeWith(*Options2); |
| 89 | EXPECT_EQ("check1,check2,check3,check4", *Options.Checks); |
| 90 | EXPECT_EQ("filter2", *Options.HeaderFilterRegex); |
Alexander Kornienko | bb56052 | 2016-08-23 14:48:29 +0000 | [diff] [blame] | 91 | EXPECT_EQ("user2", *Options.User); |
| 92 | ASSERT_TRUE(Options.ExtraArgs.hasValue()); |
| 93 | EXPECT_EQ("arg1,arg2,arg3,arg4", llvm::join(Options.ExtraArgs->begin(), |
| 94 | Options.ExtraArgs->end(), ",")); |
| 95 | ASSERT_TRUE(Options.ExtraArgsBefore.hasValue()); |
| 96 | EXPECT_EQ("arg-before1,arg-before2,arg-before3,arg-before4", |
| 97 | llvm::join(Options.ExtraArgsBefore->begin(), |
| 98 | Options.ExtraArgsBefore->end(), ",")); |
| 99 | } |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 100 | } // namespace test |
| 101 | } // namespace tidy |
| 102 | } // namespace clang |