Move tests into anonymous namespace.

This CL moves all test files to the anonymous namespace.

Change-Id: Ie5ae0efe907c03eeb441e90f31c976844172b594
Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/17900
Reviewed-by: Ryan Harrison <rharrison@chromium.org>

src/ast/array_accessor_expression_test.cc

@@ -19,6 +19,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using ArrayAccessorExpressionTest = testing::Test;
 
@@ -101,5 +102,6 @@ TEST_F(ArrayAccessorExpressionTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/as_expression_test.cc

@@ -20,6 +20,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using AsExpressionTest = testing::Test;
 
@@ -94,5 +95,6 @@ TEST_F(AsExpressionTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/assignment_statement_test.cc

@@ -19,6 +19,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using AssignmentStatementTest = testing::Test;
 
@@ -105,5 +106,6 @@ TEST_F(AssignmentStatementTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/binding_decoration_test.cc

@@ -18,6 +18,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using BindingDecorationTest = testing::Test;
 
@@ -42,5 +43,6 @@ TEST_F(BindingDecorationTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/bool_literal_test.cc

@@ -18,6 +18,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using BoolLiteralTest = testing::Test;
 
@@ -51,5 +52,6 @@ TEST_F(BoolLiteralTest, ToStr) {
   EXPECT_EQ(f.to_str(), "false");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/break_statement_test.cc

@@ -20,6 +20,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using BreakStatementTest = testing::Test;
 
@@ -109,5 +110,6 @@ TEST_F(BreakStatementTest, ToStr_WithCondition) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/builtin_decoration_test.cc

@@ -18,6 +18,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using BuiltinDecorationTest = testing::Test;
 
@@ -42,5 +43,6 @@ TEST_F(BuiltinDecorationTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/call_expression_test.cc

@@ -19,6 +19,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using CallExpressionTest = testing::Test;
 
@@ -127,5 +128,6 @@ TEST_F(CallExpressionTest, ToStr_WithParams) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/case_statement_test.cc

@@ -21,6 +21,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using CaseStatementTest = testing::Test;
 
@@ -121,5 +122,6 @@ TEST_F(CaseStatementTest, ToStr_WithoutCondition) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/cast_expression_test.cc

@@ -20,6 +20,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using CastExpressionTest = testing::Test;
 
@@ -92,5 +93,6 @@ TEST_F(CastExpressionTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/const_initializer_expression_test.cc

@@ -19,6 +19,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using ConstInitializerExpressionTest = testing::Test;
 
@@ -57,5 +58,6 @@ TEST_F(ConstInitializerExpressionTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/continue_statement_test.cc

@@ -20,6 +20,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using ContinueStatementTest = testing::Test;
 
@@ -109,5 +110,6 @@ TEST_F(ContinueStatementTest, ToStr_WithCondition) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/decorated_variable_test.cc

@@ -25,6 +25,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using DecoratedVariableTest = testing::Test;
 
@@ -93,5 +94,6 @@ TEST_F(DecoratedVariableTest, to_str) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/else_statement_test.cc

@@ -22,6 +22,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using ElseStatementTest = testing::Test;
 
@@ -135,5 +136,6 @@ TEST_F(ElseStatementTest, ToStr_NoCondition) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/entry_point_test.cc

@@ -20,6 +20,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using EntryPointTest = testing::Test;
 
@@ -95,5 +96,6 @@ TEST_F(EntryPointTest, ToStr_NoName) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/fallthrough_statement_test.cc

@@ -18,6 +18,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using FallthroughStatementTest = testing::Test;
 
@@ -52,5 +53,6 @@ TEST_F(FallthroughStatementTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/float_literal_test.cc

@@ -18,6 +18,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using FloatLiteralTest = testing::Test;
 
@@ -41,5 +42,6 @@ TEST_F(FloatLiteralTest, ToStr) {
   EXPECT_EQ(f.to_str(), "42.1");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/function_test.cc

@@ -23,6 +23,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using FunctionTest = testing::Test;
 
@@ -225,5 +226,6 @@ TEST_F(FunctionTest, TypeName_WithParams) {
   EXPECT_EQ(f.type_name(), "__func__void__i32__f32");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/identifier_expression_test.cc

@@ -18,6 +18,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using IdentifierExpressionTest = testing::Test;
 
@@ -80,5 +81,6 @@ TEST_F(IdentifierExpressionTest, ToStr_WithNamespace) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/if_statement_test.cc

@@ -21,6 +21,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using IfStatementTest = testing::Test;
 
@@ -361,5 +362,6 @@ TEST_F(IfStatementTest, ToStr_WithPremerge) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/import_test.cc

@@ -20,6 +20,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using ImportTest = testing::Test;
 
@@ -87,5 +88,6 @@ TEST_F(ImportTest, IsValid_InvalidEndingCharacter) {
   EXPECT_FALSE(i.IsValid());
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/int_literal_test.cc

@@ -18,6 +18,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using IntLiteralTest = testing::Test;
 
@@ -41,5 +42,6 @@ TEST_F(IntLiteralTest, ToStr) {
   EXPECT_EQ(i.to_str(), "-42");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/kill_statement_test.cc

@@ -20,6 +20,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using KillStatementTest = testing::Test;
 
@@ -53,5 +54,6 @@ TEST_F(KillStatementTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/location_decoration_test.cc

@@ -20,6 +20,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using LocationDecorationTest = testing::Test;
 
@@ -44,5 +45,6 @@ TEST_F(LocationDecorationTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/loop_statement_test.cc

@@ -23,6 +23,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using LoopStatementTest = testing::Test;
 
@@ -183,5 +184,6 @@ TEST_F(LoopStatementTest, ToStr_WithContinuing) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/member_accessor_expression_test.cc

@@ -21,6 +21,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using MemberAccessorExpressionTest = testing::Test;
 
@@ -105,5 +106,6 @@ TEST_F(MemberAccessorExpressionTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/module_test.cc

@@ -26,6 +26,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using ModuleTest = testing::Test;
 
@@ -184,5 +185,6 @@ TEST_F(ModuleTest, IsValid_Invalid_Function) {
   EXPECT_FALSE(m.IsValid());
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/nop_statement_test.cc

@@ -20,6 +20,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using NopStatementTest = testing::Test;
 
@@ -53,5 +54,6 @@ TEST_F(NopStatementTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/regardless_statement_test.cc

@@ -23,6 +23,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using RegardlessStatementTest = testing::Test;
 
@@ -122,5 +123,6 @@ TEST_F(RegardlessStatementTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/relational_expression_test.cc

@@ -21,6 +21,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using RelationalExpressionTest = testing::Test;
 
@@ -118,5 +119,6 @@ TEST_F(RelationalExpressionTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/return_statement_test.cc

@@ -21,6 +21,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using ReturnStatementTest = testing::Test;
 
@@ -93,5 +94,6 @@ TEST_F(ReturnStatementTest, ToStr_WithoutValue) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/set_decoration_test.cc

@@ -18,6 +18,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using SetDecorationTest = testing::Test;
 
@@ -42,5 +43,6 @@ TEST_F(SetDecorationTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/struct_member_offset_decoration_test.cc

@@ -18,6 +18,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using StructMemberOffsetDecorationTest = testing::Test;
 
@@ -31,5 +32,6 @@ TEST_F(StructMemberOffsetDecorationTest, Is) {
   EXPECT_TRUE(d.IsOffset());
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/struct_member_test.cc

@@ -23,6 +23,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using StructMemberTest = testing::Test;
 
@@ -98,5 +99,6 @@ TEST_F(StructMemberTest, ToStrNoDecorations) {
   EXPECT_EQ(out.str(), "  StructMember{a: __i32}\n");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/struct_test.cc

@@ -24,6 +24,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using StructTest = testing::Test;
 
@@ -97,5 +98,6 @@ TEST_F(StructTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/switch_statement_test.cc

@@ -23,6 +23,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using SwitchStatementTest = testing::Test;
 
@@ -150,5 +151,6 @@ TEST_F(SwitchStatementTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/type/alias_type_test.cc

@@ -21,6 +21,7 @@
 namespace tint {
 namespace ast {
 namespace type {
+namespace {
 
 using AliasTypeTest = testing::Test;
 
@@ -53,6 +54,7 @@ TEST_F(AliasTypeTest, TypeName) {
   EXPECT_EQ(at.type_name(), "__alias_Particle__i32");
 }
 
+}  // namespace
 }  // namespace type
 }  // namespace ast
 }  // namespace tint

src/ast/type/array_type_test.cc

@@ -21,6 +21,7 @@
 namespace tint {
 namespace ast {
 namespace type {
+namespace {
 
 using ArrayTypeTest = testing::Test;
 
@@ -64,6 +65,7 @@ TEST_F(ArrayTypeTest, TypeName) {
   EXPECT_EQ(arr.type_name(), "__array__i32_3");
 }
 
+}  // namespace
 }  // namespace type
 }  // namespace ast
 }  // namespace tint

src/ast/type/bool_type_test.cc

@@ -19,6 +19,7 @@
 namespace tint {
 namespace ast {
 namespace type {
+namespace {
 
 using BoolTypeTest = testing::Test;
 
@@ -41,6 +42,7 @@ TEST_F(BoolTypeTest, TypeName) {
   EXPECT_EQ(b.type_name(), "__bool");
 }
 
+}  // namespace
 }  // namespace type
 }  // namespace ast
 }  // namespace tint

src/ast/type/f32_type_test.cc

@@ -19,6 +19,7 @@
 namespace tint {
 namespace ast {
 namespace type {
+namespace {
 
 using F32TypeTest = testing::Test;
 
@@ -41,6 +42,7 @@ TEST_F(F32TypeTest, TypeName) {
   EXPECT_EQ(f.type_name(), "__f32");
 }
 
+}  // namespace
 }  // namespace type
 }  // namespace ast
 }  // namespace tint

src/ast/type/i32_type_test.cc

@@ -19,6 +19,7 @@
 namespace tint {
 namespace ast {
 namespace type {
+namespace {
 
 using I32TypeTest = testing::Test;
 
@@ -41,6 +42,7 @@ TEST_F(I32TypeTest, TypeName) {
   EXPECT_EQ(i.type_name(), "__i32");
 }
 
+}  // namespace
 }  // namespace type
 }  // namespace ast
 }  // namespace tint

src/ast/type/matrix_type_test.cc

@@ -20,6 +20,7 @@
 namespace tint {
 namespace ast {
 namespace type {
+namespace {
 
 using MatrixTypeTest = testing::Test;
 
@@ -52,6 +53,7 @@ TEST_F(MatrixTypeTest, TypeName) {
   EXPECT_EQ(m.type_name(), "__mat_2_3__i32");
 }
 
+}  // namespace
 }  // namespace type
 }  // namespace ast
 }  // namespace tint

src/ast/type/pointer_type_test.cc

@@ -20,6 +20,7 @@
 namespace tint {
 namespace ast {
 namespace type {
+namespace {
 
 using PointerTypeTest = testing::Test;
 
@@ -51,6 +52,7 @@ TEST_F(PointerTypeTest, TypeName) {
   EXPECT_EQ(p.type_name(), "__ptr_workgroup__i32");
 }
 
+}  // namespace
 }  // namespace type
 }  // namespace ast
 }  // namespace tint

src/ast/type/struct_type_test.cc

@@ -22,6 +22,7 @@
 namespace tint {
 namespace ast {
 namespace type {
+namespace {
 
 using StructTypeTest = testing::Test;
 
@@ -54,6 +55,7 @@ TEST_F(StructTypeTest, TypeName) {
   EXPECT_EQ(s.type_name(), "__struct_my_struct");
 }
 
+}  // namespace
 }  // namespace type
 }  // namespace ast
 }  // namespace tint

src/ast/type/u32_type_test.cc

@@ -19,6 +19,7 @@
 namespace tint {
 namespace ast {
 namespace type {
+namespace {
 
 using U32TypeTest = testing::Test;
 
@@ -41,6 +42,7 @@ TEST_F(U32TypeTest, TypeName) {
   EXPECT_EQ(u.type_name(), "__u32");
 }
 
+}  // namespace
 }  // namespace type
 }  // namespace ast
 }  // namespace tint

src/ast/type/vector_type_test.cc

@@ -20,6 +20,7 @@
 namespace tint {
 namespace ast {
 namespace type {
+namespace {
 
 using VectorTypeTest = testing::Test;
 
@@ -51,6 +52,7 @@ TEST_F(VectorTypeTest, TypeName) {
   EXPECT_EQ(v.type_name(), "__vec_3__i32");
 }
 
+}  // namespace
 }  // namespace type
 }  // namespace ast
 }  // namespace tint

src/ast/type_initializer_expression_test.cc

@@ -23,6 +23,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using TypeInitializerExpressionTest = testing::Test;
 
@@ -119,5 +120,6 @@ TEST_F(TypeInitializerExpressionTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/uint_literal_test.cc

@@ -18,6 +18,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using UintLiteralTest = testing::Test;
 
@@ -41,5 +42,6 @@ TEST_F(UintLiteralTest, ToStr) {
   EXPECT_EQ(i.to_str(), "42");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/unary_derivative_expression_test.cc

@@ -21,6 +21,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using UnaryDerivativeExpressionTest = testing::Test;
 
@@ -84,5 +85,6 @@ TEST_F(UnaryDerivativeExpressionTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/unary_method_expression_test.cc

@@ -21,6 +21,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using UnaryMethodExpressionTest = testing::Test;
 
@@ -96,5 +97,6 @@ TEST_F(UnaryMethodExpressionTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/unary_op_expression_test.cc

@@ -21,6 +21,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using UnaryOpExpressionTest = testing::Test;
 
@@ -76,5 +77,6 @@ TEST_F(UnaryOpExpressionTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/unless_statement_test.cc

@@ -21,6 +21,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using UnlessStatementTest = testing::Test;
 
@@ -120,5 +121,6 @@ TEST_F(UnlessStatementTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/variable_statement_test.cc

@@ -20,6 +20,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using VariableStatementTest = testing::Test;
 
@@ -83,5 +84,6 @@ TEST_F(VariableStatementTest, ToStr) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/ast/variable_test.cc

@@ -21,6 +21,7 @@
 
 namespace tint {
 namespace ast {
+namespace {
 
 using VariableTest = testing::Test;
 
@@ -113,5 +114,6 @@ TEST_F(VariableTest, to_str) {
 )");
 }
 
+}  // namespace
 }  // namespace ast
 }  // namespace tint

src/reader/spirv/parser_impl_test.cc

@@ -24,7 +24,6 @@
 namespace tint {
 namespace reader {
 namespace spirv {
-
 namespace {
 
 using ::testing::HasSubstr;
@@ -49,7 +48,6 @@ TEST_F(SpvParserTest, Impl_InvalidModuleFails) {
 // TODO(dneto): uint32 vec, valid SPIR-V
 
 }  // namespace
-
 }  // namespace spirv
 }  // namespace reader
 }  // namespace tint

src/reader/spirv/parser_test.cc

@@ -23,8 +23,8 @@
 namespace tint {
 namespace reader {
 namespace spirv {
-
 namespace {
+
 using ParserTest = testing::Test;
 
 TEST_F(ParserTest, Uint32VecEmpty) {
@@ -39,7 +39,6 @@ TEST_F(ParserTest, Uint32VecEmpty) {
 // TODO(dneto): uint32 vec, invalid SPIR-V
 
 }  // namespace
-
 }  // namespace spirv
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/lexer_test.cc

@@ -21,6 +21,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 using LexerTest = testing::Test;
 
@@ -526,6 +527,7 @@ INSTANTIATE_TEST_SUITE_P(LexerTest,
                                          "u16",
                                          "u64"));
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_additive_expression_test.cc

@@ -23,6 +23,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, AdditiveExpression_Parses_Plus) {
   auto p = parser("a + true");
@@ -91,6 +92,7 @@ TEST_F(ParserImplTest, AdditiveExpression_NoOr_ReturnsLHS) {
   ASSERT_TRUE(e->IsIdentifier());
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_and_expression_test.cc

@@ -23,6 +23,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, AndExpression_Parses) {
   auto p = parser("a & true");
@@ -69,6 +70,7 @@ TEST_F(ParserImplTest, AndExpression_NoOr_ReturnsLHS) {
   ASSERT_TRUE(e->IsIdentifier());
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_argument_expression_list_test.cc

@@ -26,6 +26,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, ArgumentExpressionList_Parses) {
   auto p = parser("a");
@@ -61,6 +62,7 @@ TEST_F(ParserImplTest, ArgumentExpressionList_HandlesInvalidExpression) {
   EXPECT_EQ(p->error(), "1:1: unable to parse argument expression");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_assignment_stmt_test.cc

@@ -26,6 +26,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, AssignmentStmt_Parses_ToVariable) {
   auto p = parser("a = 123");
@@ -130,6 +131,7 @@ TEST_F(ParserImplTest, AssignmentStmt_InvalidRHS) {
   EXPECT_EQ(p->error(), "1:14: unable to parse right side of assignment");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_body_stmt_test.cc

@@ -19,6 +19,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, BodyStmt) {
   auto p = parser(R"({
@@ -55,6 +56,7 @@ TEST_F(ParserImplTest, BodyStmt_MissingRightParen) {
   EXPECT_EQ(p->error(), "1:9: missing }");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_break_stmt_test.cc

@@ -22,6 +22,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, BreakStmt) {
   auto p = parser("break");
@@ -71,6 +72,7 @@ TEST_F(ParserImplTest, BreakStmt_MissingRHS) {
   EXPECT_EQ(p->error(), "1:9: expected (");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_builtin_decoration_test.cc

@@ -54,8 +54,6 @@ class BuiltinTest : public testing::TestWithParam<BuiltinData> {
   TypeManager tm_;
 };
 
-}  // namespace
-
 TEST_P(BuiltinTest, Parses) {
   auto params = GetParam();
   auto p = parser(params.input);
@@ -94,6 +92,7 @@ TEST_F(ParserImplTest, BuiltinDecoration_NoMatch) {
   EXPECT_EQ(t.to_str(), "not");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_case_body_test.cc

@@ -19,6 +19,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, CaseBody_Empty) {
   auto p = parser("");
@@ -63,6 +64,7 @@ TEST_F(ParserImplTest, CaseBody_Fallthrough_MissingSemicolon) {
   EXPECT_EQ(p->error(), "1:12: missing ;");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_const_expr_test.cc

@@ -24,6 +24,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, ConstExpr_TypeDecl) {
   auto p = parser("vec2<f32>(1., 2.)");
@@ -121,6 +122,7 @@ TEST_F(ParserImplTest, ConstExpr_ConstLiteral_Invalid) {
   EXPECT_EQ(p->error(), "1:1: unknown type alias 'invalid'");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_const_literal_test.cc

@@ -23,6 +23,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, ConstLiteral_Int) {
   auto p = parser("-234");
@@ -82,6 +83,7 @@ TEST_F(ParserImplTest, ConstLiteral_NoMatch) {
   ASSERT_EQ(c, nullptr);
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_continue_stmt_test.cc

@@ -22,6 +22,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, ContinueStmt) {
   auto p = parser("continue");
@@ -71,6 +72,7 @@ TEST_F(ParserImplTest, ContinueStmt_MissingRHS) {
   EXPECT_EQ(p->error(), "1:12: expected (");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_continuing_stmt_test.cc

@@ -19,6 +19,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, ContinuingStmt) {
   auto p = parser("continuing { nop; }");
@@ -36,6 +37,7 @@ TEST_F(ParserImplTest, ContinuingStmt_InvalidBody) {
   EXPECT_EQ(p->error(), "1:18: missing ;");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_derivative_modifier_test.cc

@@ -56,8 +56,6 @@ class DerivativeModifierTest
   TypeManager tm_;
 };
 
-}  // namespace
-
 TEST_P(DerivativeModifierTest, Parses) {
   auto params = GetParam();
   auto p = parser(params.input);
@@ -86,6 +84,7 @@ TEST_F(ParserImplTest, DerivativeModifier_NoMatch) {
   EXPECT_EQ(t.to_str(), "not");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_else_stmt_test.cc

@@ -20,6 +20,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, ElseStmt) {
   auto p = parser("else { a = b; c = d; }");
@@ -47,6 +48,7 @@ TEST_F(ParserImplTest, ElseStmt_MissingBody) {
   EXPECT_EQ(p->error(), "1:5: missing {");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_elseif_stmt_test.cc

@@ -20,6 +20,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, ElseIfStmt) {
   auto p = parser("elseif (a == 4) { a = b; c = d; }");
@@ -64,6 +65,7 @@ TEST_F(ParserImplTest, ElseIfStmt_MissingBody) {
   EXPECT_EQ(p->error(), "1:14: missing {");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_entry_point_decl_test.cc

@@ -20,6 +20,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, EntryPoint_Parses) {
   auto p = parser("entry_point fragment = main");
@@ -115,6 +116,7 @@ TEST_F(ParserImplTest, EntryPoint_InvalidStage) {
   EXPECT_EQ(p->error(), "1:13: missing pipeline stage for entry point");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_equality_expression_test.cc

@@ -23,6 +23,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, EqualityExpression_Parses_Equal) {
   auto p = parser("a == true");
@@ -91,6 +92,7 @@ TEST_F(ParserImplTest, EqualityExpression_NoOr_ReturnsLHS) {
   ASSERT_TRUE(e->IsIdentifier());
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_exclusive_or_expression_test.cc

@@ -23,6 +23,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, ExclusiveOrExpression_Parses) {
   auto p = parser("a ^ true");
@@ -69,6 +70,7 @@ TEST_F(ParserImplTest, ExclusiveOrExpression_NoOr_ReturnsLHS) {
   ASSERT_TRUE(e->IsIdentifier());
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_function_decl_test.cc

@@ -21,6 +21,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, FunctionDecl) {
   auto p = parser("fn main(a : i32, b : f32) -> void { return; }");
@@ -59,6 +60,7 @@ TEST_F(ParserImplTest, FunctionDecl_InvalidBody) {
   EXPECT_EQ(p->error(), "1:28: missing ;");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_function_header_test.cc

@@ -21,6 +21,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, FunctionHeader) {
   auto p = parser("fn main(a : i32, b: f32) -> void");
@@ -99,6 +100,7 @@ TEST_F(ParserImplTest, FunctionHeader_MissingReturnType) {
   EXPECT_EQ(p->error(), "1:13: unable to determine function return type");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_function_type_decl_test.cc

@@ -25,6 +25,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, FunctionTypeDecl_Void) {
   auto v = tm()->Get(std::make_unique<ast::type::VoidType>());
@@ -53,6 +54,7 @@ TEST_F(ParserImplTest, FunctionTypeDecl_InvalidType) {
   EXPECT_EQ(p->error(), "1:6: unknown type alias 'invalid'");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_global_constant_decl_test.cc

@@ -21,6 +21,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, GlobalConstantDecl) {
   auto p = parser("const a : f32 = 1.");
@@ -69,6 +70,7 @@ TEST_F(ParserImplTest, GlobalConstantDecl_MissingExpression) {
   EXPECT_EQ(p->error(), "1:15: unable to parse const literal");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_global_decl_test.cc

@@ -19,6 +19,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, GlobalDecl_Semicolon) {
   auto p = parser(";");
@@ -172,6 +173,7 @@ TEST_F(ParserImplTest, GlobalDecl_Function_Invalid) {
   EXPECT_EQ(p->error(), "1:14: unable to determine function return type");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_global_variable_decl_test.cc

@@ -21,6 +21,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, GlobalVariableDecl_WithoutInitializer) {
   auto p = parser("var<out> a : f32");
@@ -100,6 +101,7 @@ TEST_F(ParserImplTest, GlobalVariableDecl_InvalidVariableDecl) {
   EXPECT_EQ(p->error(), "1:5: invalid storage class for variable decoration");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_if_stmt_test.cc

@@ -21,6 +21,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, IfStmt) {
   auto p = parser("if (a == 4) { a = b; c = d; }");
@@ -137,6 +138,7 @@ TEST_F(ParserImplTest, IfStmt_InvalidPremerge) {
   EXPECT_EQ(p->error(), "1:30: missing }");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_import_decl_test.cc

@@ -19,6 +19,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, ImportDecl_Import) {
   auto p = parser(R"(import "GLSL.std.450" as glsl)");
@@ -89,6 +90,7 @@ TEST_F(ParserImplTest, ImportDecl_Invalid_MissingAs) {
   EXPECT_EQ(p->error(), "1:23: missing 'as' for import");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_inclusive_or_expression_test.cc

@@ -23,6 +23,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, InclusiveOrExpression_Parses) {
   auto p = parser("a | true");
@@ -69,6 +70,7 @@ TEST_F(ParserImplTest, InclusiveOrExpression_NoOr_ReturnsLHS) {
   ASSERT_TRUE(e->IsIdentifier());
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_logical_and_expression_test.cc

@@ -23,6 +23,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, LogicalAndExpression_Parses) {
   auto p = parser("a && true");
@@ -69,6 +70,7 @@ TEST_F(ParserImplTest, LogicalAndExpression_NoOr_ReturnsLHS) {
   ASSERT_TRUE(e->IsIdentifier());
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_logical_or_expression_test.cc

@@ -23,6 +23,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, LogicalOrExpression_Parses) {
   auto p = parser("a || true");
@@ -69,6 +70,7 @@ TEST_F(ParserImplTest, LogicalOrExpression_NoOr_ReturnsLHS) {
   ASSERT_TRUE(e->IsIdentifier());
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_loop_stmt_test.cc

@@ -19,6 +19,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, LoopStmt_BodyNoContinuing) {
   auto p = parser("loop { nop; }");
@@ -96,6 +97,7 @@ TEST_F(ParserImplTest, LoopStmt_InvalidContinuing) {
   EXPECT_EQ(p->error(), "1:26: missing ;");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_multiplicative_expression_test.cc

@@ -23,6 +23,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, MultiplicativeExpression_Parses_Multiply) {
   auto p = parser("a * true");
@@ -113,6 +114,7 @@ TEST_F(ParserImplTest, MultiplicativeExpression_NoOr_ReturnsLHS) {
   ASSERT_TRUE(e->IsIdentifier());
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_param_list_test.cc

@@ -26,6 +26,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, ParamList_Single) {
   auto i32 = tm()->Get(std::make_unique<ast::type::I32Type>());
@@ -73,6 +74,7 @@ TEST_F(ParserImplTest, ParamList_HangingComma) {
   EXPECT_EQ(p->error(), "1:8: found , but no variable declaration");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_paren_rhs_stmt_test.cc

@@ -19,6 +19,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, ParenRhsStmt) {
   auto p = parser("(a + b)");
@@ -60,6 +61,7 @@ TEST_F(ParserImplTest, ParenRhsStmt_MissingExpression) {
   EXPECT_EQ(p->error(), "1:2: unable to parse expression");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_pipeline_stage_test.cc

@@ -54,8 +54,6 @@ class PipelineStageTest : public testing::TestWithParam<PipelineStageData> {
   TypeManager tm_;
 };
 
-}  // namespace
-
 TEST_P(PipelineStageTest, Parses) {
   auto params = GetParam();
   auto p = parser(params.input);
@@ -85,6 +83,7 @@ TEST_F(ParserImplTest, PipelineStage_NoMatch) {
   EXPECT_EQ(t.to_str(), "not");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_postfix_expression_test.cc

@@ -28,6 +28,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, PostfixExpression_Array_ConstantIndex) {
   auto p = parser("a[1]");
@@ -194,6 +195,7 @@ TEST_F(ParserImplTest, PostfixExpression_NonMatch_returnLHS) {
   ASSERT_TRUE(e->IsIdentifier());
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_premerge_stmt_test.cc

@@ -19,6 +19,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, PremergeStmt) {
   auto p = parser("premerge { nop; }");
@@ -36,6 +37,7 @@ TEST_F(ParserImplTest, PremergeStmt_InvalidBody) {
   EXPECT_EQ(p->error(), "1:16: missing ;");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_primary_expression_test.cc

@@ -33,6 +33,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, PrimaryExpression_Ident) {
   auto p = parser("a");
@@ -323,6 +324,7 @@ TEST_F(ParserImplTest, PrimaryExpression_As_InvalidExpression) {
   EXPECT_EQ(p->error(), "1:9: unable to parse expression");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_regardless_stmt_test.cc

@@ -19,6 +19,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, RegardlessStmt) {
   auto p = parser("regardless (a) { kill; }");
@@ -56,6 +57,7 @@ TEST_F(ParserImplTest, RegardlessStmt_InvalidBody) {
   EXPECT_EQ(p->error(), "1:38: missing ;");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_relational_expression_test.cc

@@ -23,6 +23,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, RelationalExpression_Parses_LessThan) {
   auto p = parser("a < true");
@@ -135,6 +136,7 @@ TEST_F(ParserImplTest, RelationalExpression_NoOr_ReturnsLHS) {
   ASSERT_TRUE(e->IsIdentifier());
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_shift_expression_test.cc

@@ -23,6 +23,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, ShiftExpression_Parses_ShiftLeft) {
   auto p = parser("a << true");
@@ -113,6 +114,7 @@ TEST_F(ParserImplTest, ShiftExpression_NoOr_ReturnsLHS) {
   ASSERT_TRUE(e->IsIdentifier());
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_statement_test.cc

@@ -21,6 +21,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, Statement) {
   auto p = parser("return;");
@@ -284,6 +285,7 @@ TEST_F(ParserImplTest, Statement_Nop_MissingSemicolon) {
   EXPECT_EQ(p->error(), "1:4: missing ;");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_statements_test.cc

@@ -20,6 +20,7 @@
 namespace tint {
 namespace reader {
 namespace wgsl {
+namespace {
 
 TEST_F(ParserImplTest, Statements) {
   auto p = parser("nop; kill; return;");
@@ -38,6 +39,7 @@ TEST_F(ParserImplTest, Statements_Empty) {
   ASSERT_EQ(e.size(), 0);
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint

src/reader/wgsl/parser_impl_storage_class_test.cc

@@ -54,8 +54,6 @@ class StorageClassTest : public testing::TestWithParam<StorageClassData> {
   TypeManager tm_;
 };
 
-}  // namespace
-
 TEST_P(StorageClassTest, Parses) {
   auto params = GetParam();
   auto p = parser(params.input);
@@ -93,6 +91,7 @@ TEST_F(ParserImplTest, StorageClass_NoMatch) {
   EXPECT_EQ(t.to_str(), "not");
 }
 
+}  // namespace
 }  // namespace wgsl
 }  // namespace reader
 }  // namespace tint