Fix unit test

Author: newtork

foundation-models/openai/src/main/java/com/sap/ai/sdk/foundationmodels/openai/spring/OpenAiChatOptions.java

@@ -64,7 +64,9 @@ private static ChatCompletionTool toOpenAiTool(final ToolCallback toolCallback)
 
   @Override
   public void setInternalToolExecutionEnabled(
-      @Nullable final Boolean internalToolExecutionEnabled) {}
+      @Nullable final Boolean internalToolExecutionEnabled) {
+    this.internalToolExecutionEnabled = internalToolExecutionEnabled;
+  }
 
   @Override
   @Nonnull

sample-code/spring-app/src/test/java/com/sap/ai/sdk/app/controllers/SpringAiOpenAiTest.java

@@ -4,7 +4,9 @@
 
 import com.sap.ai.sdk.app.services.SpringAiOpenAiService;
 import com.sap.ai.sdk.foundationmodels.openai.OpenAiModel;
+import java.util.List;
 import org.junit.jupiter.api.Test;
+import org.springframework.ai.chat.messages.AssistantMessage;
 import org.springframework.ai.chat.model.ChatResponse;
 
 class SpringAiOpenAiTest {
@@ -47,25 +49,25 @@ void testToolCallingWithExecution() {
     assertThat(response.getResult().getOutput().getText()).contains("Potsdam", "Toulouse", "°C");
   }
 
-  // @Test
-  // void testToolCallingWithoutExecution() {
-  // ChatResponse response = service.toolCalling(false);
-  // assertThat(response.getResult().getOutput().getText()).contains("Potsdam", "Toulouse", "°C");
-  // List<AssistantMessage.ToolCall> toolCalls = response.getResult().getOutput().getToolCalls();
-  // assertThat(toolCalls).hasSize(2);
-  // AssistantMessage.ToolCall toolCall1 = toolCalls.get(0);
-  // AssistantMessage.ToolCall toolCall2 = toolCalls.get(1);
-  // assertThat(toolCall1.type()).isEqualTo("function");
-  // assertThat(toolCall2.type()).isEqualTo("function");
-  // assertThat(toolCall1.name()).isEqualTo("getCurrentWeather");
-  // assertThat(toolCall2.name()).isEqualTo("getCurrentWeather");
-  // assertThat(toolCall1.arguments())
-  //      .isEqualTo("{\"arg0\": {\"location\": \"Potsdam\", \"unit\": \"C\"}}");
-  // assertThat(toolCall2.arguments())
-  //      .isEqualTo("{\"arg0\": {\"location\": \"Toulouse\", \"unit\": \"C\"}}");
-  // }
   @Test
-  void testchatMemory() {
+  void testToolCallingWithoutExecution() {
+    ChatResponse response = service.toolCalling(false);
+    List<AssistantMessage.ToolCall> toolCalls = response.getResult().getOutput().getToolCalls();
+    assertThat(toolCalls).hasSize(2);
+    AssistantMessage.ToolCall toolCall1 = toolCalls.get(0);
+    AssistantMessage.ToolCall toolCall2 = toolCalls.get(1);
+    assertThat(toolCall1.type()).isEqualTo("function");
+    assertThat(toolCall2.type()).isEqualTo("function");
+    assertThat(toolCall1.name()).isEqualTo("getCurrentWeather");
+    assertThat(toolCall2.name()).isEqualTo("getCurrentWeather");
+    assertThat(toolCall1.arguments())
+        .isEqualTo("{\"arg0\": {\"location\": \"Potsdam\", \"unit\": \"C\"}}");
+    assertThat(toolCall2.arguments())
+        .isEqualTo("{\"arg0\": {\"location\": \"Toulouse\", \"unit\": \"C\"}}");
+  }
+
+  @Test
+  void testChatMemory() {
     ChatResponse response = service.chatMemory();
     assertThat(response).isNotNull();
     String text = response.getResult().getOutput().getText();