fixed all tests

Author: CharlesDuboisSAP

orchestration/src/test/java/com/sap/ai/sdk/orchestration/OrchestrationConvenienceUnitTest.java

@@ -1,5 +1,6 @@
 package com.sap.ai.sdk.orchestration;
 
+import static com.sap.ai.sdk.orchestration.model.UserChatMessage.RoleEnum.USER;
 import static org.assertj.core.api.Assertions.assertThat;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
@@ -130,9 +131,7 @@ void testConfigWithResponseSchema() {
   void testTemplateConstruction() {
     List<ChatMessage> templateMessages =
         List.of(
-            UserChatMessage.create()
-                .content(UserChatMessageContent.create("message"))
-                .role(UserChatMessage.RoleEnum.USER));
+            UserChatMessage.create().content(UserChatMessageContent.create("message")).role(USER));
     var defaults = Map.of("key", "value");
     var tools =
         List.of(
@@ -212,11 +211,10 @@ void testTemplateFromLocalFileWithJsonSchemaAndTools() throws IOException {
                     SystemChatMessage.create()
                         .role(RoleEnum.SYSTEM)
                         .content(ChatMessageContent.create("You are a language translator.")),
-                    (ChatMessage)
-                        UserChatMessage.create()
-                            .content(
-                                UserChatMessageContent.create(
-                                    "Whats {{ ?word }} in {{ ?language }}?"))))
+                    UserChatMessage.create()
+                        .content(
+                            UserChatMessageContent.create("Whats {{ ?word }} in {{ ?language }}?"))
+                        .role(USER)))
             .withDefaults(Map.of("word", "apple"))
             .withJsonSchemaResponse(
                 ResponseJsonSchema.fromMap(schema, "translation-schema")
@@ -274,11 +272,10 @@ void testTemplateFromLocalFileWithJsonObject() throws IOException {
                     SystemChatMessage.create()
                         .role(RoleEnum.SYSTEM)
                         .content(ChatMessageContent.create("You are a language translator.")),
-                    (ChatMessage)
-                        UserChatMessage.create()
-                            .content(
-                                UserChatMessageContent.create(
-                                    "Whats {{ ?word }} in {{ ?language }}?"))))
+                    UserChatMessage.create()
+                        .content(
+                            UserChatMessageContent.create("Whats {{ ?word }} in {{ ?language }}?"))
+                        .role(USER)))
             .withDefaults(Map.of("word", "apple"))
             .withJsonResponse();
     assertThat(templateWithJsonObject).isEqualTo(expectedTemplateWithJsonObject);

orchestration/src/test/resources/toolCallsRequest2.json

@@ -10,42 +10,43 @@
         "template": [
           {
             "role": "user",
-            "content": [ {
-              "type" : "text",
-              "text" : "What is the weather in Potsdam and in Toulouse?"
-            } ]
+            "content": [
+              {
+                "type": "text",
+                "text": "What is the weather in Potsdam and in Toulouse?"
+              }
+            ]
           },
           {
             "role": "assistant",
-            "content": "",
             "tool_calls": [
               {
+                "id": "call_LOyP7EVdeqFlGEmVzmPdCVey",
+                "type": "function",
                 "function": {
                   "name": "getCurrentWeather",
                   "arguments": "{\"arg0\": {\"location\": \"Potsdam\", \"unit\": \"C\"}}"
-                },
-                "id": "call_LOyP7EVdeqFlGEmVzmPdCVey",
-                "type": "function"
+                }
               },
               {
+                "id": "call_bwFjnXCfCO4N3f0bMtFMlNSg",
+                "type": "function",
                 "function": {
                   "name": "getCurrentWeather",
                   "arguments": "{\"arg0\": {\"location\": \"Toulouse\", \"unit\": \"C\"}}"
-                },
-                "id": "call_bwFjnXCfCO4N3f0bMtFMlNSg",
-                "type": "function"
+                }
               }
             ]
           },
           {
             "role": "tool",
-            "content": "{\"temp\":30.0,\"unit\":\"C\"}",
-            "tool_call_id": "call_LOyP7EVdeqFlGEmVzmPdCVey"
+            "tool_call_id": "call_LOyP7EVdeqFlGEmVzmPdCVey",
+            "content": "{\"temp\":30.0,\"unit\":\"C\"}"
           },
           {
             "role": "tool",
-            "content": "{\"temp\":30.0,\"unit\":\"C\"}",
-            "tool_call_id": "call_bwFjnXCfCO4N3f0bMtFMlNSg"
+            "tool_call_id": "call_bwFjnXCfCO4N3f0bMtFMlNSg",
+            "content": "{\"temp\":30.0,\"unit\":\"C\"}"
           }
         ],
         "defaults": {},