From 1ff77a20c42591ea7df1a0897826166e96b39779 Mon Sep 17 00:00:00 2001 From: Feiyang Li Date: Thu, 29 Jan 2026 17:58:50 +0800 Subject: [PATCH] test: Rename TypeTest to VisitTypeTest to avoid naming conflict --- src/iceberg/test/visit_type_test.cc | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/iceberg/test/visit_type_test.cc b/src/iceberg/test/visit_type_test.cc index 2e9247c53..786e1fd2c 100644 --- a/src/iceberg/test/visit_type_test.cc +++ b/src/iceberg/test/visit_type_test.cc @@ -210,15 +210,15 @@ const static std::array kNestedTypes = {{ } // namespace -class TypeTest : public ::testing::TestWithParam {}; +class VisitTypeTest : public ::testing::TestWithParam {}; -INSTANTIATE_TEST_SUITE_P(Primitive, TypeTest, ::testing::ValuesIn(kPrimitiveTypes), +INSTANTIATE_TEST_SUITE_P(Primitive, VisitTypeTest, ::testing::ValuesIn(kPrimitiveTypes), TypeTestCaseToString); -INSTANTIATE_TEST_SUITE_P(Nested, TypeTest, ::testing::ValuesIn(kNestedTypes), +INSTANTIATE_TEST_SUITE_P(Nested, VisitTypeTest, ::testing::ValuesIn(kNestedTypes), TypeTestCaseToString); -TEST_P(TypeTest, VisitTypePrintToString) { +TEST_P(VisitTypeTest, VisitTypePrintToString) { TypeNameVisitor visitor; std::ostringstream oss; const auto& test_case = GetParam(); @@ -226,7 +226,7 @@ TEST_P(TypeTest, VisitTypePrintToString) { ASSERT_EQ(oss.str(), test_case.repr); } -TEST_P(TypeTest, VisitTypeReturnNestedTypeId) { +TEST_P(VisitTypeTest, VisitTypeReturnNestedTypeId) { auto visitor = [&](auto&& type) -> Result { using Type = std::remove_cvref_t; // Check if the type is a nested type