Merge pull request #132 from dadoonet/dependabot/maven/co.elastic.clients-elasticsearch-java-8.16.1

Bump co.elastic.clients:elasticsearch-java from 8.16.0 to 8.16.1

Author: dadoonet

README.md

@@ -11,7 +11,7 @@ I believe it could be beneficial for many, so I've made the code available here.
 
 You're welcome to contribute your own examples if you'd like.
 
-This repository is tested against Elasticsearch 8.16.0.
+This repository is tested against Elasticsearch 8.16.1.
 
 We automatically start a Docker image using the [Elasticsearch module for TestContainers](https://www.testcontainers.org/modules/elasticsearch/).
 

pom.xml

@@ -10,7 +10,7 @@
 
     <properties>
         <!-- Main dependencies -->
-        <elasticsearch.version>8.16.0</elasticsearch.version>
+        <elasticsearch.version>8.16.1</elasticsearch.version>
         <jackson.version>2.18.1</jackson.version>
         <log4j.version>2.24.2</log4j.version>
         <slf4j.version>2.0.16</slf4j.version>

src/test/java/fr/pilato/test/elasticsearch/hlclient/EsClientIT.java

@@ -938,26 +938,18 @@ void esql() throws IOException, SQLException {
         }
 
         // Using the JDBC ResultSet ES|QL API
-        // And this is now failing because of https://github.com/elastic/elasticsearch-java/pull/903
         try (ResultSet resultSet = client.esql().query(ResultSetEsqlAdapter.INSTANCE, query)) {
-            fail("https://github.com/elastic/elasticsearch-java/pull/903 have been fixed. Update the code.");
             assertTrue(resultSet.next());
             assertEquals("David", resultSet.getString(1));
         } catch (JsonpMappingException e) {
             // This is expected as we have this issue https://github.com/elastic/elasticsearch-java/pull/903
         }
 
         // Using the Object ES|QL API
-        // And this is now failing because of https://github.com/elastic/elasticsearch-java/pull/903
-        try {
-            Iterable<Person> persons = client.esql().query(ObjectsEsqlAdapter.of(Person.class), query);
-            fail("https://github.com/elastic/elasticsearch-java/pull/903 have been fixed. Update the code.");
-            for (Person person : persons) {
-                assertNull(person.getId());
-                assertNotNull(person.getName());
-            }
-        } catch (JsonpMappingException e) {
-            // This is expected as we have this issue https://github.com/elastic/elasticsearch-java/pull/903
+        Iterable<Person> persons = client.esql().query(ObjectsEsqlAdapter.of(Person.class), query);
+        for (Person person : persons) {
+            assertNull(person.getId());
+            assertNotNull(person.getName());
         }
     }