Skip to content

Commit 96f2cba

Browse files
committed
Update tests
1 parent ddc4b21 commit 96f2cba

File tree

2 files changed

+52
-27
lines changed

2 files changed

+52
-27
lines changed
Lines changed: 52 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,52 @@
1+
package com.sap.ai.sdk.orchestration;
2+
3+
import static com.sap.ai.sdk.orchestration.AzureModerationPolicy.ALLOW_SAFE_LOW_MEDIUM;
4+
import static org.assertj.core.api.Assertions.assertThat;
5+
import static org.assertj.core.api.Assertions.assertThatThrownBy;
6+
7+
import com.sap.ai.sdk.orchestration.client.model.LLMModuleConfig;
8+
import java.util.Map;
9+
import org.junit.jupiter.api.BeforeEach;
10+
import org.junit.jupiter.api.Test;
11+
12+
class OrchestrationModuleConfigTest {
13+
14+
private OrchestrationModuleConfig config;
15+
16+
@BeforeEach
17+
void setUp() {
18+
final var llmModuleConfig =
19+
new LLMModuleConfig().modelName("gpt-35-turbo").modelParams(Map.of());
20+
config = new OrchestrationModuleConfig().withLlmConfig(llmModuleConfig);
21+
}
22+
23+
@Test
24+
void testStackingInputAndOutputFilter() {
25+
final var filter =
26+
new AzureContentFilter()
27+
.hate(ALLOW_SAFE_LOW_MEDIUM)
28+
.selfHarm(ALLOW_SAFE_LOW_MEDIUM)
29+
.sexual(ALLOW_SAFE_LOW_MEDIUM)
30+
.violence(ALLOW_SAFE_LOW_MEDIUM);
31+
32+
final var configWithInputFirst = config.withInputFiltering(filter).withOutputFiltering(filter);
33+
assertThat(configWithInputFirst.getFilteringConfig()).isNotNull();
34+
assertThat(configWithInputFirst.getFilteringConfig().getInput()).isNotNull();
35+
36+
final var configWithOutputFirst = config.withOutputFiltering(filter).withInputFiltering(filter);
37+
assertThat(configWithOutputFirst.getFilteringConfig()).isNotNull();
38+
assertThat(configWithOutputFirst.getFilteringConfig().getOutput()).isNotNull();
39+
}
40+
41+
@Test
42+
void testThrowOnEmptyFilterConfig() {
43+
final var filter = new AzureContentFilter();
44+
45+
assertThatThrownBy(() -> config.withInputFiltering(filter))
46+
.isInstanceOf(IllegalArgumentException.class)
47+
.hasMessage("At least one filter moderation policy must be set");
48+
assertThatThrownBy(() -> config.withOutputFiltering(filter))
49+
.isInstanceOf(IllegalArgumentException.class)
50+
.hasMessage("At least one filter moderation policy must be set");
51+
}
52+
}

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

Lines changed: 0 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -393,31 +393,4 @@ void testEmptyChoicesResponse() {
393393

394394
assertThat(result.getContent()).isEmpty();
395395
}
396-
397-
@Test
398-
void testStackingInputAndOutputFilter() {
399-
final var filter =
400-
new AzureContentFilter()
401-
.hate(ALLOW_SAFE_LOW_MEDIUM)
402-
.selfHarm(ALLOW_SAFE_LOW_MEDIUM)
403-
.sexual(ALLOW_SAFE_LOW_MEDIUM)
404-
.violence(ALLOW_SAFE_LOW_MEDIUM);
405-
406-
final var configWithInputFirst = config.withInputFiltering(filter).withOutputFiltering(filter);
407-
assertThat(configWithInputFirst.getFilteringConfig()).isNotNull();
408-
assertThat(configWithInputFirst.getFilteringConfig().getInput()).isNotNull();
409-
410-
final var configWithOutputFirst = config.withOutputFiltering(filter).withInputFiltering(filter);
411-
assertThat(configWithOutputFirst.getFilteringConfig()).isNotNull();
412-
assertThat(configWithOutputFirst.getFilteringConfig().getOutput()).isNotNull();
413-
}
414-
415-
@Test
416-
void testThrowOnEmptyFilterConfig() {
417-
final var filter = new AzureContentFilter();
418-
419-
assertThatThrownBy(filter::createConfig)
420-
.isInstanceOf(IllegalArgumentException.class)
421-
.hasMessage("At least one filter moderation policy must be set");
422-
}
423396
}

0 commit comments

Comments
 (0)