disable tests due to too many false alarams (#21865)

This commit is contained in:
William Cheng 2025-09-02 16:21:10 +08:00 committed by GitHub
parent efd06f5719
commit 20d5126b17
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -8,6 +8,7 @@ import java.util.Properties;
import static org.assertj.core.api.Assertions.assertThat;
import static org.assertj.core.api.Assertions.assertThatNoException;
import org.junit.jupiter.api.Disabled;
import org.slf4j.LoggerFactory;
import org.testng.annotations.BeforeClass;
import org.testng.annotations.Test;
@ -28,11 +29,13 @@ public class GlobalSettingsTest {
System.getProperties().putAll(props);
}
@Test
@Test @Disabled
// comment out the following tests as it generates false alarms from time to time
// also using system property will eventually be decommissioned
public void testNonStringSystemProperties() {
assertThat(GlobalSettings.getProperty("345")).isEqualTo("test2");
assertThat(GlobalSettings.getProperty("test1")).isEqualTo("789");
assertThatNoException().isThrownBy(GlobalSettings::log);
}
}
}