[SyntaxTree] Split tests related to Namespace

[gribozavr2]

clang/unittests/Tooling/Syntax/BuildTreeTest.cpp

2563

I think grouping this test under Namespace_ is not the best choice, because what's being tested here is the using, not the namespace.

2583

Ditto.

[eduucaldas]

answer comments

clang/unittests/Tooling/Syntax/BuildTreeTest.cpp

`-SimpleDeclaration
   | `-)
   |-)
   |-SimpleDeclarator
   | `-size_t
   `-;
 )txt"));
 }
 
-TEST_P(SyntaxTreeTest, Namespaces) {
+TEST_P(SyntaxTreeTest, Namespace_Nested) {
   if (!GetParam().isCXX()) {
     return;
   }
   EXPECT_TRUE(treeDumpEqual(
       R"cpp(
 namespace a { namespace b {} }
 namespace a::b {}
-namespace {}
-
-namespace foo = a;
 )cpp",
       R"txt(
 *: TranslationUnit
 |-NamespaceDefinition
 | |-namespace
 | |-a
 | |-{
 | |-NamespaceDefinition
 | | |-namespace
 | | |-b
 | | |-{
 | | `-}
 | `-}
-|-NamespaceDefinition
-| |-namespace
-| |-a
-| |-::
-| |-b
-| |-{
-| `-}
-|-NamespaceDefinition
-| |-namespace
-| |-{
-| `-}
-`-NamespaceAliasDefinition
-  |-namespace
-  |-foo
-  |-=
-  |-a
-  `-;
-)txt"));
+`-NamespaceDefinition
+  |-namespace
+  |-a
+  |-::
+  |-b
+  |-{
+  `-}
+)txt"));
+}
+
+TEST_P(SyntaxTreeTest, Namespace_Unnamed) {
+  if (!GetParam().isCXX()) {
+    return;
+  }
+  EXPECT_TRUE(treeDumpEqual(
+      R"cpp(
+namespace {}
+)cpp",
+      R"txt(
+*: TranslationUnit
+`-NamespaceDefinition
+  |-namespace
+  |-{
+  `-}
+)txt"));
+}
+
+TEST_P(SyntaxTreeTest, Namespace_Alias) {
+  if (!GetParam().isCXX()) {
+    return;
+  }
+  EXPECT_TRUE(treeDumpEqualOnAnnotations(
+      R"cpp(
+namespace a {}
+[[namespace foo = a;]]
+)cpp",
+      {R"txt(
+NamespaceAliasDefinition
+)txt"}));
 }
 
 TEST_P(SyntaxTreeTest, UsingDirective) {
   if (!GetParam().isCXX()) {
     return;
   }
-  EXPECT_TRUE(treeDumpEqual(
+  EXPECT_TRUE(treeDumpEqualOnAnnotations(
       R"cpp(
 namespace ns {}
-using namespace ::ns;
+[[using namespace ::ns;]]
 )cpp",
-      R"txt(
-*: TranslationUnit
-|-NamespaceDefinition
-| |-namespace
-| |-ns
-| |-{
-| `-}
-`-UsingNamespaceDirective
-  |-using
-  |-namespace
-  |-NestedNameSpecifier
-  | `-::
-  |-ns
-  `-;
-)txt"));
+      {R"txt(
+UsingNamespaceDirective
+)txt"}));
 }
 
 TEST_P(SyntaxTreeTest, UsingDeclaration) {
   if (!GetParam().isCXX()) {
     return;
   }
-  EXPECT_TRUE(treeDumpEqual(
+  EXPECT_TRUE(treeDumpEqualOnAnnotations(
       R"cpp(
 namespace ns { int a; }
-using ns::a;
+[[using ns::a;]]
 )cpp",
-      R"txt(
-*: TranslationUnit
-|-NamespaceDefinition
-| |-namespace
-| |-ns
-| |-{
-| |-SimpleDeclaration
-| | |-int
-| | |-SimpleDeclarator
-| | | `-a
-| | `-;
-| `-}
-`-UsingDeclaration
-  |-using
-  |-NestedNameSpecifier
-  | |-IdentifierNameSpecifier
-  | | `-ns
-  | `-::
-  |-a
-  `-;
-)txt"));
+      {R"txt(
+UsingDeclaration
+)txt"}));
 }
 
 TEST_P(SyntaxTreeTest, FreeStandingClasses) {
   // Free-standing classes, must live inside a SimpleDeclaration.
   EXPECT_TRUE(treeDumpEqual(
       R"cpp(
 struct X;
 struct X {};
 
 struct Y *y1;
 struct Y {} *y2;
 
 struct {} *a1;
 )cpp",
       R"txt(
 *: TranslationUnit
 |-SimpleDeclaration
 | |-struct
 | |-X
 | `-;
 |-SimpleDeclaration
 | |-struct
 | |-X
 | |-{
 | |-}
 | `-;
 |-SimpleDeclaration
 | |-struct
 | |-Y