Moving created methods and tests to v2.

Author: n-o-u-r-h-a-n

sample-code/spring-app/src/main/java/com/sap/ai/sdk/app/controllers/OpenAiController.java

@@ -41,19 +41,6 @@ Object chatCompletion(
     return response.getContent();
   }
 
-  @GetMapping("/messagesHistory")
-  @Nonnull
-  Object messagesHistory(
-      @Nullable @RequestParam(value = "format", required = false) final String format,
-      @Nonnull @RequestParam(value = "previousMessage", defaultValue = "What is the capital of France?")
-          final String previousMessage) {
-    final var response = service.messagesHistory(previousMessage);
-    if ("json".equals(format)) {
-      return response;
-    }
-    return response.getContent();
-  }
-
   @SuppressWarnings("unused") // The end-to-end test doesn't use this method
   @GetMapping("/streamChatCompletionDeltas")
   @Nonnull

sample-code/spring-app/src/main/java/com/sap/ai/sdk/app/services/OpenAiService.java

@@ -56,21 +56,6 @@ public OpenAiChatCompletionOutput chatCompletion(@Nonnull final String prompt) {
    * @param previousMessage The request to send to the assistant
    * @return the assistant message response
    */
-  @Nonnull
-  public OpenAiChatCompletionResponse messagesHistory(@Nonnull final String previousMessage) {
-    val messagesList = new ArrayList<OpenAiMessage>();
-    messagesList.add(OpenAiMessage.user(previousMessage));
-
-    final OpenAiChatCompletionResponse result =
-        OpenAiClient.forModel(GPT_4O_MINI)
-            .chatCompletion(new OpenAiChatCompletionRequest(messagesList));
-
-    messagesList.add(result.getMessage());
-    messagesList.add(OpenAiMessage.user("What is the typical food there?"));
-
-    return OpenAiClient.forModel(GPT_4O_MINI)
-        .chatCompletion(new OpenAiChatCompletionRequest(messagesList));
-  }
 
   /**
    * Asynchronous stream of an OpenAI chat request

sample-code/spring-app/src/main/java/com/sap/ai/sdk/app/services/OpenAiServiceV2.java

@@ -19,6 +19,7 @@
 import java.util.stream.Stream;
 import javax.annotation.Nonnull;
 import lombok.extern.slf4j.Slf4j;
+import lombok.val;
 import org.springframework.stereotype.Service;
 
 /** Service class for OpenAI service using latest convenience api */
@@ -37,6 +38,22 @@ public OpenAiChatCompletionResponse chatCompletion(@Nonnull final String prompt)
         .chatCompletion(new OpenAiChatCompletionRequest(prompt));
   }
 
+  @Nonnull
+  public OpenAiChatCompletionResponse messagesHistory(@Nonnull final String previousMessage) {
+    val messagesList = new ArrayList<OpenAiMessage>();
+    messagesList.add(OpenAiMessage.user(previousMessage));
+
+    final OpenAiChatCompletionResponse result =
+        OpenAiClient.forModel(GPT_4O_MINI)
+            .chatCompletion(new OpenAiChatCompletionRequest(messagesList));
+
+    messagesList.add(result.getMessage());
+    messagesList.add(OpenAiMessage.user("What is the typical food there?"));
+
+    return OpenAiClient.forModel(GPT_4O_MINI)
+        .chatCompletion(new OpenAiChatCompletionRequest(messagesList));
+  }
+
   /**
    * Asynchronous stream of an OpenAI chat request
    *

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

@@ -31,11 +31,6 @@ void chatCompletion() {
     assertThat(message.getContent()).isNotEmpty();
   }
 
-  @Test
-  void testMessagesHistory() {
-    assertThat(service.messagesHistory("What is the capital of France?").getContent()).isNotEmpty();
-  }
-
   @Test
   void chatCompletionImage() {
     final var completion =

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

@@ -33,6 +33,11 @@ void chatCompletion() {
     assertThat(completion.getContent()).isNotEmpty();
   }
 
+  @Test
+  void testMessagesHistory() {
+    assertThat(service.messagesHistory("What is the capital of France?").getContent()).isNotEmpty();
+  }
+
   @Test
   void chatCompletionImage() {
     final var completion =