add test

metacat-connector-hive/src/main/java/com/netflix/metacat/connector/hive/converters/HiveTypeConverter.java

@@ -317,11 +317,18 @@ Type getCanonicalType(final TypeInfo typeInfo) {
             case STRUCT:
                 final StructTypeInfo structTypeInfo =
                     TypeUtils.checkType(typeInfo, StructTypeInfo.class, "typeInfo");
-                final List<TypeSignature> fieldTypes = new ArrayList<>();
-                final List<Object> fieldNames = new ArrayList<>();
-                for (String fieldName : structTypeInfo.getAllStructFieldNames()) {
-                    fieldNames.add(fieldName);
-                    final Type fieldType = getCanonicalType(structTypeInfo.getStructFieldTypeInfo(fieldName));
+                // Hive struct type infos
+                final List<String> structFieldNames = structTypeInfo.getAllStructFieldNames();
+                final List<TypeInfo> structFieldTypeInfos = structTypeInfo.getAllStructFieldTypeInfos();
+                int structInfoCounts = structFieldNames.size();
+
+                // Metacat canonical type infos
+                final List<TypeSignature> fieldTypes = new ArrayList<>(structInfoCounts);
+                final List<Object> fieldNames = new ArrayList<>(structInfoCounts);
+
+                for (int i = 0; i < structInfoCounts; i++) {
+                    fieldNames.add(structFieldNames.get(i));
+                    final Type fieldType = getCanonicalType(structFieldTypeInfos.get(i));
                     if (fieldType == null) {
                         return null;
                     }

metacat-functional-tests/src/functionalTest/groovy/com/netflix/metacat/MetacatSmokeSpec.groovy

@@ -267,6 +267,7 @@ class MetacatSmokeSpec extends Specification {
                 ]
             }
             """
+        assert nestedFieldDto.jsonType.toPrettyString() == expectedJsonString
         cleanup:
         api.deleteTable(catalogName, databaseName, tableName)
     }