fix handling of greenmail.sieve.ignore.detail

greenmail-mail-test · Nov 26, 2024 · 43535a8 · 43535a8
1 parent cf514ee
commit 43535a8
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/...va/com/icegreen/greenmail/configuration/PropertiesBasedGreenMailConfigurationBuilder.java b/...va/com/icegreen/greenmail/configuration/PropertiesBasedGreenMailConfigurationBuilder.java
@@ -80,7 +80,7 @@ public GreenMailConfiguration build(Properties properties) {
         }
 
         String sieveIgnoreDetail = properties.getProperty(GREENMAIL_SIEVE_IGNORE_DETAIL, "false");
-        if (null != sieveIgnoreDetail) {
+        if (Boolean.TRUE.toString().equalsIgnoreCase(sieveIgnoreDetail)) {
             configuration.withSieveIgnoreDetail();
         }
 

diff --git a/...om/icegreen/greenmail/configuration/PropertiesBasedGreenMailConfigurationBuilderTest.java b/...om/icegreen/greenmail/configuration/PropertiesBasedGreenMailConfigurationBuilderTest.java
@@ -60,7 +60,7 @@ public void testBuildWithSieveIgnoreDetailEnabledSetting() {
         GreenMailConfiguration config = new PropertiesBasedGreenMailConfigurationBuilder().build(props);
 
         assertThat(config).isNotNull();
-        assertThat(config.isSieveIgnoreDetailEnabled()).isTrue();
+        assertThat(config.isSieveIgnoreDetailEnabled()).isFalse();
     }
 
     @Test

diff --git a/...dalone/src/test/java/com/icegreen/greenmail/standalone/GreenMailStandaloneRunnerTest.java b/...dalone/src/test/java/com/icegreen/greenmail/standalone/GreenMailStandaloneRunnerTest.java
@@ -81,7 +81,7 @@ public void testApi() {
             "{\"port\":3143,\"address\":\"127.0.0.1\",\"protocol\":\"imap\",\"isSecure\":false,\"readTimeout\":-1," +
             "\"writeTimeout\":-1,\"connectionTimeout\":-1,\"serverStartupTimeout\":2000,\"isDynamicPort\":false}]," +
             "\"authenticationDisabled\":false," +
-            "\"sieveIgnoreDetail\":true," +
+            "\"sieveIgnoreDetail\":false," +
             "\"preloadDirectory\":null" +
             "}");