Better test example and confirm test failure

Signed-off-by: Thomas Farr <tsfarr@amazon.com>

Author: Xtansia

java-client/src/main/java/org/opensearch/client/json/JsonpUtils.java

@@ -43,8 +43,6 @@
 import jakarta.json.stream.JsonParser.Event;
 import jakarta.json.stream.JsonParsingException;
 import java.io.StringReader;
-import java.math.BigDecimal;
-import java.math.BigInteger;
 import java.util.AbstractMap;
 import java.util.Map;
 import java.util.stream.Collectors;
@@ -80,26 +78,26 @@ public JsonProvider jsonProvider() {
         public <T> void serialize(T value, JsonGenerator generator) {
             if (value instanceof JsonpSerializable) {
                 ((JsonpSerializable) value).serialize(generator, this);
-            } else if (value instanceof JsonValue) {
-                generator.write((JsonValue) value);
-            } else if (value instanceof String) {
-                generator.write((String) value);
-            } else if (value instanceof BigDecimal) {
-                generator.write((BigDecimal) value);
-            } else if (value instanceof BigInteger) {
-                generator.write((BigInteger) value);
-            } else if (value instanceof Short) {
-                generator.write((Short) value);
-            } else if (value instanceof Integer) {
-                generator.write((Integer) value);
-            } else if (value instanceof Long) {
-                generator.write((Long) value);
-            } else if (value instanceof Float) {
-                generator.write((Float) value);
-            } else if (value instanceof Double) {
-                generator.write((Double) value);
-            } else if (value instanceof Boolean) {
-                generator.write((Boolean) value);
+                // } else if (value instanceof JsonValue) {
+                // generator.write((JsonValue) value);
+                // } else if (value instanceof String) {
+                // generator.write((String) value);
+                // } else if (value instanceof BigDecimal) {
+                // generator.write((BigDecimal) value);
+                // } else if (value instanceof BigInteger) {
+                // generator.write((BigInteger) value);
+                // } else if (value instanceof Short) {
+                // generator.write((Short) value);
+                // } else if (value instanceof Integer) {
+                // generator.write((Integer) value);
+                // } else if (value instanceof Long) {
+                // generator.write((Long) value);
+                // } else if (value instanceof Float) {
+                // generator.write((Float) value);
+                // } else if (value instanceof Double) {
+                // generator.write((Double) value);
+                // } else if (value instanceof Boolean) {
+                // generator.write((Boolean) value);
             } else {
                 throw new JsonException(
                     "Cannot find a serializer for type " + value.getClass().getName() + ". Consider using a full-featured JsonpMapper."

java-client/src/test/java/org/opensearch/client/opensearch/_types/aggregations/ExtendedBoundsTest.java

@@ -0,0 +1,21 @@
+/*
+ * SPDX-License-Identifier: Apache-2.0
+ *
+ * The OpenSearch Contributors require contributions made to
+ * this file be licensed under the Apache-2.0 license or a
+ * compatible open source license.
+ */
+
+package org.opensearch.client.opensearch._types.aggregations;
+
+import org.junit.Test;
+import org.opensearch.client.opensearch.model.ModelTestCase;
+
+public class ExtendedBoundsTest extends ModelTestCase {
+    @Test
+    public void testToJsonString() {
+        ExtendedBounds<Double> bounds = ExtendedBounds.of(eb -> eb.min(0.0).max(4096.0));
+
+        assertEquals("{\"max\":4096.0,\"min\":0.0}", bounds.toJsonString());
+    }
+}

java-client/src/test/java/org/opensearch/client/opensearch/core/IndexRequestTest.java

@@ -28,10 +28,4 @@ static class MyDoc {
             this.key = key;
         }
     }
-
-    @Test
-    public void indexRequestToJson() {
-        IndexRequest<Object> indexRequest = IndexRequest.of(i -> i.index("index").document("document"));
-        assertNotNull(indexRequest.toJsonString());
-    }
 }