Fix PutIndexTemplateRequest deserialization (opensearch-project#765)

* Fix PutIndexTemplateRequest deserialization Signed-off-by: Matthew Bogner <matt@ibogner.net> * CHANGELOG.md Signed-off-by: Matthew Bogner <matt@ibogner.net> * spotless Signed-off-by: Matthew Bogner <matt@ibogner.net> --------- Signed-off-by: Matthew Bogner <matt@ibogner.net>
reta · Dec 12, 2023 · d4aade4 · d4aade4
1 parent 1725fc9
commit d4aade4
Show file tree

Hide file tree

Showing 3 changed files with 33 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -29,6 +29,7 @@ This section is for maintaining a changelog for all breaking changes for the cli
 ### Fixed
 - Fix version and build ([#254](https://github.com/opensearch-project/opensearch-java/pull/254))
 - Fix PutTemplateRequest field deserialization ([#723](https://github.com/opensearch-project/opensearch-java/pull/723))
+- Fix PutIndexTemplateRequest field deserialization ([#765](https://github.com/opensearch-project/opensearch-java/pull/765))
 
 ### Security
 

diff --git a/...lient/src/main/java/org/opensearch/client/opensearch/indices/PutIndexTemplateRequest.java b/...lient/src/main/java/org/opensearch/client/opensearch/indices/PutIndexTemplateRequest.java
@@ -406,6 +406,7 @@ protected static void setupPutIndexTemplateRequestDeserializer(ObjectDeserialize
         op.add(Builder::priority, JsonpDeserializer.integerDeserializer(), "priority");
         op.add(Builder::template, IndexTemplateMapping._DESERIALIZER, "template");
         op.add(Builder::version, JsonpDeserializer.longDeserializer(), "version");
+        op.add(Builder::name, JsonpDeserializer.stringDeserializer(), "name");
 
     }
 

diff --git a/...ient/src/test/java/org/opensearch/client/opensearch/core/PutIndexTemplateRequestTest.java b/...ient/src/test/java/org/opensearch/client/opensearch/core/PutIndexTemplateRequestTest.java
@@ -0,0 +1,31 @@
+package org.opensearch.client.opensearch.core;
+
+import com.fasterxml.jackson.core.JsonProcessingException;
+import com.fasterxml.jackson.databind.ObjectMapper;
+import java.io.StringReader;
+import java.util.List;
+import java.util.Map;
+import org.junit.Assert;
+import org.junit.Test;
+import org.opensearch.client.json.JsonpMapper;
+import org.opensearch.client.json.jsonb.JsonbJsonpMapper;
+import org.opensearch.client.opensearch.indices.PutIndexTemplateRequest;
+
+public class PutIndexTemplateRequestTest extends Assert {
+
+    @Test
+    public void deserialize_validFieldsIncluded_RequestIsBuilt() throws JsonProcessingException {
+        final JsonpMapper mapper = new JsonbJsonpMapper();
+        final Map<String, Object> indexTemplateMap = Map.of("name", "test", "index_patterns", "*", "create", true, "priority", 1);
+
+        final String indexTemplate = new ObjectMapper().writeValueAsString(indexTemplateMap);
+        final var parser = mapper.jsonProvider().createParser(new StringReader(indexTemplate));
+
+        final PutIndexTemplateRequest putIndexTemplateRequest = PutIndexTemplateRequest._DESERIALIZER.deserialize(parser, mapper);
+
+        assertEquals(putIndexTemplateRequest.name(), "test");
+        assertEquals(putIndexTemplateRequest.indexPatterns(), List.of("*"));
+        assertEquals((int) putIndexTemplateRequest.priority(), 1);
+    }
+
+}