format code

Author: MisterRaindrop

src/iceberg/avro/avro_schema_util.cc

@@ -236,7 +236,8 @@ Status ToAvroNodeVisitor::Visit(const StructType& type, ::avro::NodePtr* node) {
 
     std::string origFieldName = std::string(sub_field.name());
     bool isValidFieldName = ValidAvroName(origFieldName);
-    std::string fieldName = isValidFieldName ? origFieldName : SanitizeFieldName(origFieldName);
+    std::string fieldName =
+        isValidFieldName ? origFieldName : SanitizeFieldName(origFieldName);
 
     (*node)->addName(fieldName);
     (*node)->addLeaf(field_node);

test/avro_schema_test.cc

@@ -275,10 +275,13 @@ TEST(ToAvroNodeVisitorTest, StructTypeWithFieldNames) {
 
   ASSERT_EQ(node->names(), 6);
 
-  EXPECT_EQ(node->nameAt(0), "user_x2Dname");        // "user-name" -> "user_x2Dname"
-  EXPECT_EQ(node->nameAt(2), "email_x2Eaddress");    // "email.address" -> "email_x2Eaddress"
-  EXPECT_EQ(node->nameAt(4), "_123field");           // "123field" -> "_123field"
-  EXPECT_EQ(node->nameAt(5), "field_x20with_x20spaces");  // "field with spaces" -> "field_x20with_x20spaces"
+  EXPECT_EQ(node->nameAt(0), "user_x2Dname");  // "user-name" -> "user_x2Dname"
+  EXPECT_EQ(node->nameAt(2),
+            "email_x2Eaddress");            // "email.address" -> "email_x2Eaddress"
+  EXPECT_EQ(node->nameAt(4), "_123field");  // "123field" -> "_123field"
+  EXPECT_EQ(
+      node->nameAt(5),
+      "field_x20with_x20spaces");  // "field with spaces" -> "field_x20with_x20spaces"
 
   EXPECT_EQ(node->nameAt(1), "valid_field");
   EXPECT_EQ(node->nameAt(3), "AnotherField");
@@ -296,10 +299,14 @@ TEST(ToAvroNodeVisitorTest, StructTypeWithFieldNames) {
   EXPECT_FALSE(attrs1.getAttribute("iceberg-field-name").has_value());
   EXPECT_FALSE(attrs3.getAttribute("iceberg-field-name").has_value());
 
-  ASSERT_NO_FATAL_FAILURE(CheckIcebergFieldName(node, /*index=*/0, /*original_name=*/"user-name"));
-  ASSERT_NO_FATAL_FAILURE(CheckIcebergFieldName(node, /*index=*/2, /*original_name=*/"email.address"));
-  ASSERT_NO_FATAL_FAILURE(CheckIcebergFieldName(node, /*index=*/4, /*original_name=*/"123field"));
-  ASSERT_NO_FATAL_FAILURE(CheckIcebergFieldName(node, /*index=*/5, /*original_name=*/"field with spaces"));
+  ASSERT_NO_FATAL_FAILURE(
+      CheckIcebergFieldName(node, /*index=*/0, /*original_name=*/"user-name"));
+  ASSERT_NO_FATAL_FAILURE(
+      CheckIcebergFieldName(node, /*index=*/2, /*original_name=*/"email.address"));
+  ASSERT_NO_FATAL_FAILURE(
+      CheckIcebergFieldName(node, /*index=*/4, /*original_name=*/"123field"));
+  ASSERT_NO_FATAL_FAILURE(
+      CheckIcebergFieldName(node, /*index=*/5, /*original_name=*/"field with spaces"));
 }
 
 
@@ -1558,4 +1565,4 @@ TEST_F(NameMappingAvroSchemaTest, MissingFieldIdError) {
   auto result = MakeAvroNodeWithFieldIds(avro_schema.root(), *name_mapping);
   ASSERT_THAT(result, IsError(ErrorKind::kInvalidSchema));
 }
-}  // namespace iceberg::avro
+}  // namespace iceberg::avro