Skip to content

Commit 9533d73

Browse files
committed
Upgrade LangChain4j 1.5.0; Fix deprecated warnings
Signed-off-by: Ricardo Zanini <ricardozanini@gmail.com>
1 parent 94f41da commit 9533d73

File tree

6 files changed

+15
-18
lines changed

6 files changed

+15
-18
lines changed

api/src/main/java/io/serverlessworkflow/api/ObjectMapperFactory.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,11 +34,11 @@ class ObjectMapperFactory {
3434
private static final ObjectMapper yamlMapper =
3535
configure(new ObjectMapper(new YAMLFactory().enable(Feature.MINIMIZE_QUOTES)));
3636

37-
public static final ObjectMapper jsonMapper() {
37+
public static ObjectMapper jsonMapper() {
3838
return jsonMapper;
3939
}
4040

41-
public static final ObjectMapper yamlMapper() {
41+
public static ObjectMapper yamlMapper() {
4242
return yamlMapper;
4343
}
4444

@@ -49,7 +49,7 @@ private static ObjectMapper configure(ObjectMapper mapper) {
4949
validationModule.setDeserializerModifier(new BeanDeserializerModifierWithValidation());
5050

5151
return mapper
52-
.setSerializationInclusion(Include.NON_NULL)
52+
.setDefaultPropertyInclusion(Include.NON_NULL)
5353
.configure(SerializationFeature.INDENT_OUTPUT, true)
5454
.configure(SerializationFeature.WRITE_EMPTY_JSON_ARRAYS, false)
5555
.configure(SerializationFeature.WRITE_NULL_MAP_VALUES, false)

impl/core/src/main/java/io/serverlessworkflow/impl/WorkflowModelFactory.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ default WorkflowModel fromAny(WorkflowModel prev, Object obj) {
5151
return fromAny(obj);
5252
}
5353

54+
@SuppressWarnings("unchecked")
5455
default WorkflowModel fromAny(Object obj) {
5556
if (obj == null) {
5657
return fromNull();
@@ -70,10 +71,6 @@ default WorkflowModel fromAny(Object obj) {
7071
return from((Map<String, Object>) obj);
7172
} else if (obj instanceof WorkflowModel model) {
7273
return model;
73-
} else if (obj instanceof CloudEventData ce) {
74-
return from(ce);
75-
} else if (obj instanceof CloudEvent ce) {
76-
return from(ce);
7774
} else {
7875
return fromOther(obj);
7976
}

impl/jackson/src/main/java/io/serverlessworkflow/impl/jackson/JsonUtils.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,9 @@ public static JsonNode fromString(String value) {
162162

163163
private static Object toJavaValue(ObjectNode node) {
164164
Map<String, Object> result = new HashMap<>();
165-
node.fields().forEachRemaining(iter -> result.put(iter.getKey(), toJavaValue(iter.getValue())));
165+
node.properties()
166+
.iterator()
167+
.forEachRemaining(iter -> result.put(iter.getKey(), toJavaValue(iter.getValue())));
166168
return result;
167169
}
168170

impl/jackson/src/main/java/io/serverlessworkflow/impl/jackson/MergeUtils.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
import com.fasterxml.jackson.databind.node.ArrayNode;
2020
import com.fasterxml.jackson.databind.node.ObjectNode;
2121
import java.util.HashSet;
22-
import java.util.Iterator;
2322
import java.util.Map;
2423
import java.util.Set;
2524

@@ -56,9 +55,7 @@ public static JsonNode merge(JsonNode src, JsonNode target, boolean mergeArray)
5655

5756
private static ObjectNode mergeObject(JsonNode src, ObjectNode target, boolean mergeArray) {
5857
if (src.isObject()) {
59-
Iterator<Map.Entry<String, JsonNode>> mergedIterator = src.fields();
60-
while (mergedIterator.hasNext()) {
61-
Map.Entry<String, JsonNode> entry = mergedIterator.next();
58+
for (Map.Entry<String, JsonNode> entry : src.properties()) {
6259
JsonNode found = target.get(entry.getKey());
6360
target.set(
6461
entry.getKey(),

impl/test/src/test/java/io/serverlessworkflow/impl/test/OpenIDCHTTPWorkflowDefinitionTest.java

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717

1818
import static io.serverlessworkflow.api.WorkflowReader.readWorkflowFromClasspath;
1919
import static io.serverlessworkflow.impl.test.OAuthHTTPWorkflowDefinitionTest.fakeAccessToken;
20-
import static org.junit.Assert.assertEquals;
20+
import static org.junit.jupiter.api.Assertions.assertEquals;
2121
import static org.junit.jupiter.api.Assertions.assertTrue;
2222

2323
import com.fasterxml.jackson.databind.ObjectMapper;
@@ -30,6 +30,7 @@
3030
import okhttp3.mockwebserver.MockWebServer;
3131
import okhttp3.mockwebserver.RecordedRequest;
3232
import org.junit.jupiter.api.AfterEach;
33+
import org.junit.jupiter.api.Assertions;
3334
import org.junit.jupiter.api.BeforeEach;
3435
import org.junit.jupiter.api.Test;
3536

@@ -502,8 +503,9 @@ public void testOpenIDCJSONWithArgsNoEndPointWorkflowExecution() throws Exceptio
502503

503504
RecordedRequest tokenRequest = authServer.takeRequest();
504505
assertEquals("POST", tokenRequest.getMethod());
505-
assertEquals("/realms/test-realm/protocol/openid-connect/token", tokenRequest.getPath());
506-
assertEquals("application/json", tokenRequest.getHeader("Content-Type"));
506+
Assertions.assertEquals(
507+
"/realms/test-realm/protocol/openid-connect/token", tokenRequest.getPath());
508+
Assertions.assertEquals("application/json", tokenRequest.getHeader("Content-Type"));
507509

508510
String tokenRequestBody = tokenRequest.getBody().readUtf8();
509511
Map<String, Object> asJson = MAPPER.readValue(tokenRequestBody, Map.class);

pom.xml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@
5252
<properties>
5353
<java.version>17</java.version>
5454
<maven.compiler.source>${java.version}</maven.compiler.source>
55-
<maven.compiler.target>${java.version}</maven.compiler.target>
5655
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
5756
<version.maven>3.9.11</version.maven>
5857

@@ -94,9 +93,9 @@
9493
<version.org.hibernate.validator>9.0.1.Final</version.org.hibernate.validator>
9594
<version.org.glassfish.expressly>6.0.0</version.org.glassfish.expressly>
9695
<!-- Experimental modules from langchain4j -->
97-
<version.dev.langchain4j.beta>1.4.1-beta10</version.dev.langchain4j.beta>
96+
<version.dev.langchain4j.beta>1.5.0-beta11</version.dev.langchain4j.beta>
9897
<!-- Base langchain4j version -->
99-
<version.dev.langchain4j>1.4.0</version.dev.langchain4j>
98+
<version.dev.langchain4j>1.5.0</version.dev.langchain4j>
10099

101100
<!-- Checkstyle props -->
102101
<checkstyle.failOnViolation>true</checkstyle.failOnViolation>

0 commit comments

Comments
 (0)