Skip to content

Commit 2fda82b

Browse files
committed
log test issue
1 parent 0072b5d commit 2fda82b

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

caffein-bounded-cache-support/src/test/java/io/javaoperatorsdk/operator/processing/event/source/cache/BoundedCacheTestBase.java

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,8 @@
44
import java.util.stream.IntStream;
55

66
import org.junit.jupiter.api.Test;
7+
import org.slf4j.Logger;
8+
import org.slf4j.LoggerFactory;
79

810
import io.fabric8.kubernetes.api.model.ConfigMap;
911
import io.fabric8.kubernetes.client.CustomResource;
@@ -17,6 +19,8 @@
1719

1820
public abstract class BoundedCacheTestBase<P extends CustomResource<BoundedCacheTestSpec, BoundedCacheTestStatus>> {
1921

22+
private static final Logger log = LoggerFactory.getLogger(BoundedCacheTestBase.class);
23+
2024
public static final int NUMBER_OF_RESOURCE_TO_TEST = 3;
2125
public static final String RESOURCE_NAME_PREFIX = "test-";
2226
public static final String INITIAL_DATA_PREFIX = "data-";
@@ -38,7 +42,7 @@ void reconciliationWorksWithLimitedCache() {
3842
}
3943

4044
private void assertConfigMapsDeleted() {
41-
await().atMost(Duration.ofSeconds(20))
45+
await().atMost(Duration.ofSeconds(30))
4246
.untilAsserted(() -> IntStream.range(0, NUMBER_OF_RESOURCE_TO_TEST).forEach(i -> {
4347
var cm = extension().get(ConfigMap.class, RESOURCE_NAME_PREFIX + i);
4448
assertThat(cm).isNull();
@@ -48,7 +52,10 @@ private void assertConfigMapsDeleted() {
4852
private void deleteTestResources() {
4953
IntStream.range(0, NUMBER_OF_RESOURCE_TO_TEST).forEach(i -> {
5054
var cm = extension().get(customResourceClass(), RESOURCE_NAME_PREFIX + i);
51-
extension().delete(cm);
55+
var deleted = extension().delete(cm);
56+
if (!deleted) {
57+
log.warn("Custom resource might not be deleted: {}", cm);
58+
}
5259
});
5360
}
5461

caffein-bounded-cache-support/src/test/java/io/javaoperatorsdk/operator/processing/event/source/cache/sample/AbstractTestReconciler.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ public abstract class AbstractTestReconciler<P extends CustomResource<BoundedCac
3434
LoggerFactory.getLogger(BoundedCacheClusterScopeTestReconciler.class);
3535

3636
public static final String DATA_KEY = "dataKey";
37-
public static final String DEFAULT_NAMESPACE = "default";
3837

3938
protected KubernetesClient client;
4039

0 commit comments

Comments
 (0)