Skip to content

Commit c0cbe6a

Browse files
authored
Fix integration tests compilation in develop (#1283)
1 parent 3583b5c commit c0cbe6a

File tree

3 files changed

+0
-52
lines changed

3 files changed

+0
-52
lines changed

integration-tests/src/jvmTest/kotlin/ai/koog/integration/tests/executor/ExecutorIntegrationTestBase.kt

Lines changed: 0 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -222,42 +222,6 @@ abstract class ExecutorIntegrationTestBase {
222222
}
223223
}
224224

225-
open fun integration_testExecuteStreamingWithTools(model: LLModel) = runTest(timeout = 300.seconds) {
226-
Models.assumeAvailable(model.provider)
227-
assumeTrue(model.capabilities.contains(LLMCapability.Tools), "Model $model does not support tools")
228-
229-
val executor = getExecutor(model)
230-
231-
val prompt = Prompt.build("test-streaming", LLMParams(toolChoice = ToolChoice.Required)) {
232-
system("You are a helpful assistant.")
233-
user("Count three times five")
234-
}
235-
236-
withRetry(times = 3, testName = "integration_testExecuteStreamingWithTools[${model.id}]") {
237-
with(StringBuilder()) {
238-
val endMessages = mutableListOf<StreamFrame.End>()
239-
val toolMessages = mutableListOf<StreamFrame.ToolCall>()
240-
241-
executor.executeStreamAndCollect(
242-
prompt = prompt,
243-
model = model,
244-
tools = listOf(SimpleCalculatorTool.descriptor),
245-
appendable = this,
246-
endMessages = endMessages,
247-
toolMessages = toolMessages
248-
)
249-
250-
toolMessages.shouldNotBeEmpty()
251-
withClue("Expected calculator tool call but got: [$toolMessages]") {
252-
toolMessages.any {
253-
it.name == SimpleCalculatorTool.name &&
254-
it.content.contains(CalculatorOperation.MULTIPLY.name, ignoreCase = true)
255-
} shouldBe true
256-
}
257-
}
258-
}
259-
}
260-
261225
open fun integration_testToolWithRequiredParams(model: LLModel) = runTest(timeout = 300.seconds) {
262226
Models.assumeAvailable(model.provider)
263227
assumeTrue(model.capabilities.contains(LLMCapability.Tools), "Model $model does not support tools")

integration-tests/src/jvmTest/kotlin/ai/koog/integration/tests/executor/MultipleLLMPromptExecutorIntegrationTest.kt

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import ai.koog.integration.tests.utils.getLLMClientForProvider
1010
import ai.koog.prompt.executor.llms.MultiLLMPromptExecutor
1111
import ai.koog.prompt.executor.model.PromptExecutor
1212
import ai.koog.prompt.llm.LLModel
13-
import org.junit.jupiter.api.Disabled
1413
import org.junit.jupiter.params.ParameterizedTest
1514
import org.junit.jupiter.params.provider.Arguments
1615
import org.junit.jupiter.params.provider.MethodSource
@@ -114,13 +113,6 @@ class MultipleLLMPromptExecutorIntegrationTest : ExecutorIntegrationTestBase() {
114113
super.integration_testExecuteStreaming(model)
115114
}
116115

117-
@Disabled("KG-616")
118-
@ParameterizedTest
119-
@MethodSource("allCompletionModels")
120-
override fun integration_testExecuteStreamingWithTools(model: LLModel) {
121-
super.integration_testExecuteStreamingWithTools(model)
122-
}
123-
124116
@ParameterizedTest
125117
@MethodSource("allCompletionModels")
126118
override fun integration_testExecuteStreamingWithTools(model: LLModel) {

integration-tests/src/jvmTest/kotlin/ai/koog/integration/tests/executor/SingleLLMPromptExecutorIntegrationTest.kt

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ import ai.koog.prompt.executor.llms.SingleLLMPromptExecutor
1111
import ai.koog.prompt.executor.model.PromptExecutor
1212
import ai.koog.prompt.llm.LLMProvider
1313
import ai.koog.prompt.llm.LLModel
14-
import org.junit.jupiter.api.Disabled
1514
import org.junit.jupiter.params.ParameterizedTest
1615
import org.junit.jupiter.params.provider.Arguments
1716
import org.junit.jupiter.params.provider.MethodSource
@@ -108,13 +107,6 @@ class SingleLLMPromptExecutorIntegrationTest : ExecutorIntegrationTestBase() {
108107
super.integration_testExecuteStreaming(model)
109108
}
110109

111-
@Disabled("KG-616")
112-
@ParameterizedTest
113-
@MethodSource("allCompletionModels")
114-
override fun integration_testExecuteStreamingWithTools(model: LLModel) {
115-
super.integration_testExecuteStreamingWithTools(model)
116-
}
117-
118110
@ParameterizedTest
119111
@MethodSource("allCompletionModels")
120112
override fun integration_testExecuteStreamingWithTools(model: LLModel) {

0 commit comments

Comments
 (0)