Switch to error for empty content filter configs

Author: MatKuhr

orchestration/src/main/java/com/sap/ai/sdk/orchestration/AzureContentFilter.java

@@ -48,6 +48,11 @@ public enum Sensitivity {
   @Nonnull
   com.sap.ai.sdk.orchestration.client.model.FilterConfig toFilterConfigDTO() {
     val dto = AzureContentSafety.create();
+    if (hate == null && selfHarm == null && sexual == null && violence == null) {
+      throw new IllegalStateException(
+          "When configuring an azure content filter, at least one filter category must be set");
+    }
+
     if (hate != null) {
       dto.hate(fromValue(hate.value));
     }
@@ -60,6 +65,7 @@ com.sap.ai.sdk.orchestration.client.model.FilterConfig toFilterConfigDTO() {
     if (violence != null) {
       dto.violence(fromValue(violence.value));
     }
+
     return com.sap.ai.sdk.orchestration.client.model.FilterConfig.create()
         .type(com.sap.ai.sdk.orchestration.client.model.FilterConfig.TypeEnum.AZURE_CONTENT_SAFETY)
         .config(dto);

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

@@ -1,6 +1,7 @@
 package com.sap.ai.sdk.orchestration;
 
 import static com.sap.ai.sdk.orchestration.AzureContentFilter.Sensitivity.HIGH;
+import static com.sap.ai.sdk.orchestration.AzureContentFilter.Sensitivity.LOW;
 import static com.sap.ai.sdk.orchestration.ModuleConfigFactory.toModuleConfigDTO;
 import static com.sap.ai.sdk.orchestration.client.model.FilterConfig.TypeEnum.AZURE_CONTENT_SAFETY;
 import static org.assertj.core.api.Assertions.assertThat;
@@ -138,8 +139,8 @@ void testOutputFilter() {
 
   @Test
   void testInputAndOutputFilter() {
-    var inputFilter = new AzureContentFilter();
-    var outputFilter = new AzureContentFilter();
+    var inputFilter = new AzureContentFilter().hate(HIGH);
+    var outputFilter = new AzureContentFilter().violence(LOW);
     config.withInputContentFilter(inputFilter);
     config.withOutputContentFilter(outputFilter);
 
@@ -149,6 +150,15 @@ void testInputAndOutputFilter() {
     assertThat(result.getOutput().getFilters()).isNotEmpty();
   }
 
+  @Test
+  void testEmptyFilter() {
+    var inputFilter = new AzureContentFilter();
+    assertThatThrownBy(inputFilter::toFilterConfigDTO)
+        .isInstanceOf(IllegalStateException.class)
+        .hasMessageContaining(
+            "When configuring an azure content filter, at least one filter category must be set");
+  }
+
   @Test
   void testMasking() {
     var maskingConfig = DpiMaskingConfig.anonymization().withEntities(DPIEntities.ADDRESS);