unit test fixes

elastic · Dec 17, 2024 · 61d01cc · 61d01cc
1 parent ebbf1d8
commit 61d01cc
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/...src/test/java/co/elastic/clients/elasticsearch/_helpers/esql/EsqlAdapterEndToEndTest.java b/...src/test/java/co/elastic/clients/elasticsearch/_helpers/esql/EsqlAdapterEndToEndTest.java
@@ -134,7 +134,11 @@ public void objectsTest() throws Exception {
         {
             EmpData emp = it.next();
             assertEquals("10042", emp.empNo);
-            assertEquals(Arrays.asList("Architect", "Business Analyst", "Internship", "Junior Developer"), emp.jobPositions);
+            assertEquals(4, emp.jobPositions.size());
+            assertTrue(emp.jobPositions.contains("Architect"));
+            assertTrue(emp.jobPositions.contains("Business Analyst"));
+            assertTrue(emp.jobPositions.contains("Internship"));
+            assertTrue(emp.jobPositions.contains("Junior Developer"));
 
             assertEquals("1993-03-21T00:00:00Z[UTC]",
                 DateTimeFormatter.ISO_DATE_TIME.format(emp.hireDate.toInstant().atZone(ZoneId.of("UTC")))
@@ -172,7 +176,11 @@ public void asyncObjects() throws Exception {
                 {
                     EmpData emp = it.next();
                     assertEquals("10042", emp.empNo);
-                    assertEquals(Arrays.asList("Architect", "Business Analyst", "Internship", "Junior Developer"), emp.jobPositions);
+                    assertEquals(4, emp.jobPositions.size());
+                    assertTrue(emp.jobPositions.contains("Architect"));
+                    assertTrue(emp.jobPositions.contains("Business Analyst"));
+                    assertTrue(emp.jobPositions.contains("Internship"));
+                    assertTrue(emp.jobPositions.contains("Junior Developer"));
 
                     assertEquals("1993-03-21T00:00:00Z[UTC]",
                         DateTimeFormatter.ISO_DATE_TIME.format(emp.hireDate.toInstant().atZone(ZoneId.of("UTC")))

diff --git a/java-client/src/test/java/co/elastic/clients/elasticsearch/spec_issues/SpecIssuesTest.java b/java-client/src/test/java/co/elastic/clients/elasticsearch/spec_issues/SpecIssuesTest.java
@@ -115,12 +115,12 @@ public void i0297_mappingSettings() {
             .index("i0297")
             .settings(s -> s
                 // This is "mapping" and not "mappings"
-                .mapping(m -> m.totalFields(totalFields -> totalFields.limit(1001L)))
+                .mapping(m -> m.totalFields(totalFields -> totalFields.limit("1001")))
                 .otherSettings("foo", JsonData.of("bar"))
             )
         );
 
-        assertEquals("{\"settings\":{\"foo\":\"bar\",\"mapping\":{\"total_fields\":{\"limit\":1001}}}}", toJson(request));
+        assertEquals("{\"settings\":{\"foo\":\"bar\",\"mapping\":{\"total_fields\":{\"limit\":\"1001\"}}}}", toJson(request));
     }
 
     @Test