fix: only read 'properties' if present

Author: bevzzz

src/main/java/io/weaviate/client6/v1/api/collections/CollectionConfig.java

@@ -264,16 +264,19 @@ public void write(JsonWriter out, CollectionConfig value) throws IOException {
         public CollectionConfig read(JsonReader in) throws IOException {
           var jsonObject = JsonParser.parseReader(in).getAsJsonObject();
 
-          var mixedProperties = jsonObject.get("properties").getAsJsonArray();
           var references = new JsonArray();
           var properties = new JsonArray();
 
-          for (var property : mixedProperties) {
-            var dataTypes = property.getAsJsonObject().get("dataType").getAsJsonArray();
-            if (dataTypes.size() == 1 && DataType.KNOWN_TYPES.contains(dataTypes.get(0).getAsString())) {
-              properties.add(property);
-            } else {
-              references.add(property);
+          if (jsonObject.has("properties") && jsonObject.get("properties").isJsonArray()) {
+            var mixedProperties = jsonObject.get("properties").getAsJsonArray();
+
+            for (var property : mixedProperties) {
+              var dataTypes = property.getAsJsonObject().get("dataType").getAsJsonArray();
+              if (dataTypes.size() == 1 && DataType.KNOWN_TYPES.contains(dataTypes.get(0).getAsString())) {
+                properties.add(property);
+              } else {
+                references.add(property);
+              }
             }
           }
 

src/test/java/io/weaviate/client6/v1/internal/json/JSONTest.java

@@ -400,7 +400,7 @@ public void test_ReferenceAddManyResponse_CustomDeserializer() {
             "result": { "status": "SUCCESS", "errors": {} }
           },
           {
-            "result": { "status": "FAILED", "errors": {
+            "result": { "status": "FAILED", "errors": jsonObject.has("properties") && {
               "error": [ "oops" ]
             }}
           }
@@ -413,4 +413,14 @@ public void test_ReferenceAddManyResponse_CustomDeserializer() {
         .as("response contains 1 error")
         .hasSize(1);
   }
+
+  @Test
+  public void test_CollectionConfig_read_empty() {
+    var json = """
+        { "class": "BarebonesCollection" }
+        """;
+    Assertions.assertThatCode(() -> JSON.deserialize(json, CollectionConfig.class))
+        .as("deserialize CollectionConfig with no properties")
+        .doesNotThrowAnyException();
+  }
 }