Skip to content

Commit c39e8c2

Browse files
author
Inbal Tako
committed
Disable windows unitest compitability
1 parent 4b5ec32 commit c39e8c2

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

src/test/java/com/securenative/config/ConfigurationManagerTest.java

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,12 @@
1414

1515
import static org.assertj.core.api.Assertions.assertThat;
1616

17+
public class NotRunningOnWindows implements IgnoreCondition {
18+
public boolean isSatisfied() {
19+
return !System.getProperty("os.name").startsWith("Windows");
20+
}
21+
}
22+
1723
@TestMethodOrder(MethodOrderer.OrderAnnotation.class)
1824
public class ConfigurationManagerTest {
1925
@SuppressWarnings({ "unchecked" })
@@ -167,7 +173,7 @@ public void loadDefaultConfigTest() throws SecureNativeConfigException {
167173
@Order(6)
168174
@Timeout(value = 1000, unit = TimeUnit.MILLISECONDS)
169175
@DisplayName("Should get config via env variables")
170-
@DisabledOnOs({OS.WINDOWS})
176+
@ConditionalIgnore(condition = NotRunningOnWindows.class)
171177
public void getConfigFromEnvVariablesTest() throws SecureNativeConfigException, ReflectiveOperationException {
172178
setEnv("SECURENATIVE_API_KEY", "SOME_ENV_API_KEY");
173179
setEnv("SECURENATIVE_API_URL", "SOME_API_URL");
@@ -206,7 +212,7 @@ public void getConfigFromEnvVariablesTest() throws SecureNativeConfigException,
206212
@Order(7)
207213
@Timeout(value = 1000, unit = TimeUnit.MILLISECONDS)
208214
@DisplayName("Should overwrite env variables with vales from config file")
209-
@DisabledOnOs({OS.WINDOWS})
215+
@ConditionalIgnore(condition = NotRunningOnWindows.class)
210216
public void overwriteEnvVariablesWithConfigFileTest() throws SecureNativeConfigException, ReflectiveOperationException {
211217
String config = String.join(System.getProperty("line.separator"),
212218
"SECURENATIVE_API_KEY=API_KEY_FROM_FILE",

0 commit comments

Comments
 (0)