This is an automated email from the ASF dual-hosted git repository.
gangwu pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/iceberg-cpp.git
The following commit(s) were added to refs/heads/main by this push:
new 5dda5a39 test: Rename TypeTest to VisitTypeTest to avoid naming
conflict (#546)
5dda5a39 is described below
commit 5dda5a39aadd412b45d2b01d96d4dc6826e030bc
Author: Feiyang Li <[email protected]>
AuthorDate: Thu Jan 29 22:04:31 2026 +0800
test: Rename TypeTest to VisitTypeTest to avoid naming conflict (#546)
---
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 2e9247c5..786e1fd2 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<TypeTestCase, 4> kNestedTypes = {{
} // namespace
-class TypeTest : public ::testing::TestWithParam<TypeTestCase> {};
+class VisitTypeTest : public ::testing::TestWithParam<TypeTestCase> {};
-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<TypeId> {
using Type = std::remove_cvref_t<decltype(type)>;
// Check if the type is a nested type
