Skip to content

Commit 4c51921

Browse files
authored
Merge pull request #1866 from fedinskiy/fix/robust-tests
Make OpenAI tests more predictable, when working with real LLM
2 parents 3b5ddfd + 322dcc5 commit 4c51921

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

integration-tests/openai/src/main/java/org/acme/example/openai/chat/QuarkusOpenAiClientChatResource.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ public String sync() {
3838
@GET
3939
@Path("async")
4040
public Uni<String> async() {
41-
var request = MessageUtil.createChatCompletionRequest("Write a short 1 paragraph funny poem about Scrum");
41+
var request = MessageUtil.createChatCompletionRequest("Write a short definition of Scrum");
4242
return Uni.createFrom().emitter(emitter -> {
4343
quarkusOpenAiClient.chatCompletion(request)
4444
.onResponse(res -> emitter.complete(res.content()))

integration-tests/openai/src/test/java/org/acme/example/openai/aiservices/AssistantResourceWithMetricsTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ public void noMicrometerAnnotations() throws InterruptedException {
4141
.get("a1")
4242
.then()
4343
.statusCode(200)
44-
.body(TestUtils.containsStringOrMock("test"));
44+
.body(TestUtils.containsStringOrMock("today"));
4545

4646
waitForMeters(
4747
registry.find("langchain4j.aiservices.timed")
@@ -66,7 +66,7 @@ public void micrometerAnnotationOnClass() throws InterruptedException {
6666
.get("a2")
6767
.then()
6868
.statusCode(200)
69-
.body(TestUtils.containsStringOrMock("test"));
69+
.body(TestUtils.containsStringOrMock("today"));
7070

7171
waitForMeters(
7272
registry.find("langchain4j.aiservices.timed")

integration-tests/openai/src/test/java/org/acme/example/openai/aiservices/AssistantResourceWithToolsTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ public class AssistantResourceWithToolsTest {
2222
public void get() {
2323
given()
2424
.baseUri(url.toString())
25-
.queryParam("message", "This is a test")
25+
.queryParam("message", "This is a test. Answer with word 'test', if you get this message")
2626
.get()
2727
.then()
2828
.statusCode(200)

0 commit comments

Comments
 (0)