blob: feb7ae913f9968487ebfbe82d3e12d44132e6cb1 [file] [log] [blame]
Daniel Jasperd07c8402013-07-29 08:19:24 +00001#include "ClangTidyTest.h"
Alexander Kornienko72f1e752014-06-18 09:33:46 +00002#include "google/ExplicitConstructorCheck.h"
Samuel Benzaquen59c8aa92015-02-11 21:21:05 +00003#include "google/GlobalNamesInHeadersCheck.h"
Alexander Kornienko09887162014-02-27 14:28:02 +00004#include "gtest/gtest.h"
Daniel Jasperd07c8402013-07-29 08:19:24 +00005
Alexander Kornienkoed824e02015-03-05 13:46:14 +00006using namespace clang::tidy::google;
7
Daniel Jasperd07c8402013-07-29 08:19:24 +00008namespace clang {
9namespace tidy {
Alexander Kornienko09887162014-02-27 14:28:02 +000010namespace test {
Daniel Jasperd07c8402013-07-29 08:19:24 +000011
Alexander Kornienko09887162014-02-27 14:28:02 +000012TEST(ExplicitConstructorCheckTest, SingleArgumentConstructorsOnly) {
13 EXPECT_NO_CHANGES(ExplicitConstructorCheck, "class C { C(); };");
14 EXPECT_NO_CHANGES(ExplicitConstructorCheck, "class C { C(int i, int j); };");
Alexander Kornienkof65a5492014-03-20 09:39:36 +000015 EXPECT_NO_CHANGES(ExplicitConstructorCheck, "class C { C(const C&); };");
16 EXPECT_NO_CHANGES(ExplicitConstructorCheck, "class C { C(C&&); };");
Alexander Kornienkof9a7b462014-03-19 12:48:22 +000017 EXPECT_NO_CHANGES(ExplicitConstructorCheck,
18 "class C { C(const C&) = delete; };");
Alexander Kornienkof65a5492014-03-20 09:39:36 +000019 EXPECT_NO_CHANGES(ExplicitConstructorCheck,
20 "class C { C(int) = delete; };");
Daniel Jasperd07c8402013-07-29 08:19:24 +000021}
22
Alexander Kornienko09887162014-02-27 14:28:02 +000023TEST(ExplicitConstructorCheckTest, Basic) {
Daniel Jasperd07c8402013-07-29 08:19:24 +000024 EXPECT_EQ("class C { explicit C(int i); };",
Alexander Kornienko09887162014-02-27 14:28:02 +000025 runCheckOnCode<ExplicitConstructorCheck>("class C { C(int i); };"));
Daniel Jasperd07c8402013-07-29 08:19:24 +000026}
27
Alexander Kornienko09887162014-02-27 14:28:02 +000028TEST(ExplicitConstructorCheckTest, DefaultParameters) {
Daniel Jasperd07c8402013-07-29 08:19:24 +000029 EXPECT_EQ("class C { explicit C(int i, int j = 0); };",
Alexander Kornienko09887162014-02-27 14:28:02 +000030 runCheckOnCode<ExplicitConstructorCheck>(
31 "class C { C(int i, int j = 0); };"));
Daniel Jasperd07c8402013-07-29 08:19:24 +000032}
33
Alexander Kornienko09887162014-02-27 14:28:02 +000034TEST(ExplicitConstructorCheckTest, OutOfLineDefinitions) {
Alexander Kornienko32eaa372014-02-13 10:11:48 +000035 EXPECT_EQ("class C { explicit C(int i); }; C::C(int i) {}",
Alexander Kornienko09887162014-02-27 14:28:02 +000036 runCheckOnCode<ExplicitConstructorCheck>(
37 "class C { C(int i); }; C::C(int i) {}"));
Alexander Kornienko32eaa372014-02-13 10:11:48 +000038}
39
Alexander Kornienko014225e2014-04-29 15:07:08 +000040TEST(ExplicitConstructorCheckTest, RemoveExplicit) {
41 EXPECT_EQ("class A { A(const A&); };\n"
Alexander Kornienko3b268492014-04-29 15:50:14 +000042 "class B { /*asdf*/ B(B&&); };\n"
43 "class C { /*asdf*/ C(const C&, int i = 0); };",
Alexander Kornienko014225e2014-04-29 15:07:08 +000044 runCheckOnCode<ExplicitConstructorCheck>(
45 "class A { explicit A(const A&); };\n"
Alexander Kornienko3b268492014-04-29 15:50:14 +000046 "class B { explicit /*asdf*/ B(B&&); };\n"
47 "class C { explicit/*asdf*/ C(const C&, int i = 0); };"));
Alexander Kornienko014225e2014-04-29 15:07:08 +000048}
49
Alexander Kornienko4aab5792014-05-05 18:49:31 +000050TEST(ExplicitConstructorCheckTest, RemoveExplicitWithMacros) {
51 EXPECT_EQ(
52 "#define A(T) class T##Bar { explicit T##Bar(const T##Bar &b) {} };\n"
53 "A(Foo);",
54 runCheckOnCode<ExplicitConstructorCheck>(
55 "#define A(T) class T##Bar { explicit T##Bar(const T##Bar &b) {} };\n"
56 "A(Foo);"));
57}
58
Samuel Benzaquen59c8aa92015-02-11 21:21:05 +000059class GlobalNamesInHeadersCheckTest : public ::testing::Test {
60protected:
61 bool runCheckOnCode(const std::string &Code, const std::string &Filename) {
62 static const char *const Header = "namespace std {\n"
63 "class string {};\n"
64 "} // namespace std\n"
65 "\n"
66 "#define SOME_MACRO(x) using x\n";
67 std::vector<ClangTidyError> Errors;
68 std::vector<std::string> Args;
69 if (!StringRef(Filename).endswith(".cpp")) {
70 Args.emplace_back("-xc++-header");
71 }
72 test::runCheckOnCode<readability::GlobalNamesInHeadersCheck>(
73 Header + Code, &Errors, Filename, Args);
74 if (Errors.empty())
75 return false;
76 assert(Errors.size() == 1);
77 assert(
78 Errors[0].Message.Message ==
79 "using declarations in the global namespace in headers are prohibited");
80 return true;
81 }
82};
83
84TEST_F(GlobalNamesInHeadersCheckTest, UsingDeclarations) {
85 EXPECT_TRUE(runCheckOnCode("using std::string;", "foo.h"));
86 EXPECT_FALSE(runCheckOnCode("using std::string;", "foo.cpp"));
87 EXPECT_FALSE(runCheckOnCode("namespace my_namespace {\n"
88 "using std::string;\n"
89 "} // my_namespace\n",
90 "foo.h"));
91 EXPECT_FALSE(runCheckOnCode("SOME_MACRO(std::string);", "foo.h"));
92}
93
94TEST_F(GlobalNamesInHeadersCheckTest, UsingDirectives) {
95 EXPECT_TRUE(runCheckOnCode("using namespace std;", "foo.h"));
96 EXPECT_FALSE(runCheckOnCode("using namespace std;", "foo.cpp"));
97 EXPECT_FALSE(runCheckOnCode("namespace my_namespace {\n"
98 "using namespace std;\n"
99 "} // my_namespace\n",
100 "foo.h"));
101 EXPECT_FALSE(runCheckOnCode("SOME_MACRO(namespace std);", "foo.h"));
102}
103
Samuel Benzaquen3199b9a2015-03-24 15:21:45 +0000104TEST_F(GlobalNamesInHeadersCheckTest, RegressionAnonymousNamespace) {
105 EXPECT_FALSE(runCheckOnCode("namespace {}", "foo.h"));
106}
107
Alexander Kornienko09887162014-02-27 14:28:02 +0000108} // namespace test
Daniel Jasperd07c8402013-07-29 08:19:24 +0000109} // namespace tidy
110} // namespace clang