diff --git a/providers/flagd/pom.xml b/providers/flagd/pom.xml index dd5e8ba30..1c6cb90c8 100644 --- a/providers/flagd/pom.xml +++ b/providers/flagd/pom.xml @@ -147,13 +147,13 @@ org.testcontainers testcontainers - 1.21.3 + 2.0.2 test org.testcontainers - junit-jupiter - 1.21.3 + testcontainers-junit-jupiter + 2.0.2 test @@ -162,6 +162,12 @@ 5.5.6 test + + commons-io + commons-io + 2.21.0 + test + org.slf4j diff --git a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/SyncMetadataHookTest.java b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/SyncMetadataHookTest.java index 15cabffd9..92d0398a9 100644 --- a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/SyncMetadataHookTest.java +++ b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/SyncMetadataHookTest.java @@ -1,6 +1,6 @@ package dev.openfeature.contrib.providers.flagd; -import static org.junit.Assert.assertEquals; +import static org.junit.jupiter.api.Assertions.assertEquals; import dev.openfeature.sdk.EvaluationContext; import dev.openfeature.sdk.FlagValueType; @@ -9,7 +9,7 @@ import dev.openfeature.sdk.MutableContext; import java.util.Optional; import java.util.function.Supplier; -import org.junit.Test; +import org.junit.jupiter.api.Test; /** SyncMetadataHookTest */ public class SyncMetadataHookTest { diff --git a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/RunInProcessTest.java b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/RunInProcessTest.java index acb060065..c694aa9ef 100644 --- a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/RunInProcessTest.java +++ b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/RunInProcessTest.java @@ -6,7 +6,6 @@ import dev.openfeature.contrib.providers.flagd.Config; import org.apache.logging.log4j.core.config.Order; -import org.junit.jupiter.api.parallel.Isolated; import org.junit.platform.suite.api.BeforeSuite; import org.junit.platform.suite.api.ConfigurationParameter; import org.junit.platform.suite.api.ExcludeTags; @@ -31,7 +30,6 @@ @IncludeTags("in-process") @ExcludeTags({"unixsocket"}) @Testcontainers -@Isolated public class RunInProcessTest { @BeforeSuite diff --git a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/ContextSteps.java b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/ContextSteps.java index 9de541e23..42e466a35 100644 --- a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/ContextSteps.java +++ b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/ContextSteps.java @@ -7,9 +7,7 @@ import io.cucumber.java.en.Given; import java.util.HashMap; import java.util.Map; -import org.junit.jupiter.api.parallel.Isolated; -@Isolated() public class ContextSteps extends AbstractSteps { public ContextSteps(State state) { diff --git a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/EventSteps.java b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/EventSteps.java index 32eed5b9d..c31517320 100644 --- a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/EventSteps.java +++ b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/EventSteps.java @@ -11,9 +11,7 @@ import java.util.LinkedList; import lombok.extern.slf4j.Slf4j; import org.jetbrains.annotations.NotNull; -import org.junit.jupiter.api.parallel.Isolated; -@Isolated() @Slf4j public class EventSteps extends AbstractSteps { diff --git a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/FlagSteps.java b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/FlagSteps.java index ab946f882..462fa1381 100644 --- a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/FlagSteps.java +++ b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/FlagSteps.java @@ -15,10 +15,8 @@ import java.util.List; import java.util.Map; import lombok.extern.slf4j.Slf4j; -import org.junit.jupiter.api.parallel.Isolated; @Slf4j -@Isolated() public class FlagSteps extends AbstractSteps { public FlagSteps(State state) { diff --git a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/ProviderSteps.java b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/ProviderSteps.java index e8760909c..27806f955 100644 --- a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/ProviderSteps.java +++ b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/e2e/steps/ProviderSteps.java @@ -21,14 +21,12 @@ import java.nio.file.Paths; import java.time.Duration; import lombok.extern.slf4j.Slf4j; +import org.apache.commons.io.FileUtils; import org.apache.commons.lang3.RandomStringUtils; import org.apache.commons.lang3.StringUtils; -import org.junit.jupiter.api.parallel.Isolated; import org.testcontainers.containers.ComposeContainer; import org.testcontainers.containers.wait.strategy.Wait; -import org.testcontainers.shaded.org.apache.commons.io.FileUtils; -@Isolated() @Slf4j public class ProviderSteps extends AbstractSteps { diff --git a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/resolver/process/storage/FlagStoreTest.java b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/resolver/process/storage/FlagStoreTest.java index 7f3fa99da..86ca298e3 100644 --- a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/resolver/process/storage/FlagStoreTest.java +++ b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/resolver/process/storage/FlagStoreTest.java @@ -17,7 +17,6 @@ import java.util.concurrent.BlockingQueue; import java.util.concurrent.LinkedBlockingQueue; import java.util.stream.Collectors; -import org.junit.Assert; import org.junit.jupiter.api.Test; class FlagStoreTest { @@ -100,7 +99,7 @@ public void changedFlags() throws Exception { FlagParser.parseString(getFlagsFromResource(VALID_LONG), true).getFlags(); expectedChangedFlags.remove("myBoolFlag"); // flags changed from initial VALID_SIMPLE flag, as a set because we don't care about order - Assert.assertEquals( + assertEquals( expectedChangedFlags.keySet().stream().collect(Collectors.toSet()), new HashSet<>(storageStateDTOS.take().getChangedFlagsKeys())); } diff --git a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/resolver/process/storage/connector/sync/SyncStreamQueueSourceTest.java b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/resolver/process/storage/connector/sync/SyncStreamQueueSourceTest.java index ef632a829..9116b8142 100644 --- a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/resolver/process/storage/connector/sync/SyncStreamQueueSourceTest.java +++ b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/resolver/process/storage/connector/sync/SyncStreamQueueSourceTest.java @@ -1,9 +1,9 @@ package dev.openfeature.contrib.providers.flagd.resolver.process.storage.connector.sync; -import static org.junit.Assert.assertNotNull; -import static org.junit.Assert.assertNull; -import static org.junit.Assert.assertTrue; import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertNotNull; +import static org.junit.jupiter.api.Assertions.assertNull; +import static org.junit.jupiter.api.Assertions.assertTrue; import static org.mockito.ArgumentMatchers.any; import static org.mockito.ArgumentMatchers.anyLong; import static org.mockito.Mockito.doAnswer; diff --git a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/resolver/rpc/EventStreamObserverTest.java b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/resolver/rpc/EventStreamObserverTest.java index d5f73c188..046bee3b8 100644 --- a/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/resolver/rpc/EventStreamObserverTest.java +++ b/providers/flagd/src/test/java/dev/openfeature/contrib/providers/flagd/resolver/rpc/EventStreamObserverTest.java @@ -1,6 +1,6 @@ package dev.openfeature.contrib.providers.flagd.resolver.rpc; -import static org.junit.Assert.assertNotNull; +import static org.junit.jupiter.api.Assertions.assertNotNull; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when;