test: cover the remaining ConfigurationServiceOverrider settings

operator-framework-core/src/test/java/io/javaoperatorsdk/operator/api/config/ConfigurationServiceOverriderTest.java

@@ -17,13 +17,17 @@
 
 import java.time.Duration;
 import java.util.Optional;
+import java.util.Set;
 import java.util.concurrent.Executors;
 import java.util.concurrent.ThreadPoolExecutor;
 
 import org.junit.jupiter.api.Test;
 
+import io.fabric8.kubernetes.api.model.ConfigMap;
 import io.fabric8.kubernetes.api.model.HasMetadata;
+import io.fabric8.kubernetes.api.model.Secret;
 import io.javaoperatorsdk.operator.api.monitoring.Metrics;
+import io.javaoperatorsdk.operator.api.reconciler.dependent.DependentResourceFactory;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.junit.jupiter.api.Assertions.assertNotEquals;
@@ -33,7 +37,7 @@
   private static final Metrics METRICS = new Metrics() {};
 
   private static final LeaderElectionConfiguration LEADER_ELECTION_CONFIGURATION =
       new LeaderElectionConfiguration("foo", "fooNS");
 
   private static final Cloner CLONER =
       new Cloner() {
@@ -85,7 +89,7 @@
             .withConcurrentReconciliationThreads(25)
             .withMetrics(new Metrics() {})
             .withLeaderElectionConfiguration(
                 new LeaderElectionConfiguration("newLease", "newLeaseNS"))
             .withInformerStoppedHandler((informer, ex) -> {})
             .withReconciliationTerminationTimeout(Duration.ofSeconds(30))
             .build();
@@ -98,10 +102,10 @@
     assertNotEquals(config.getExecutorService(), overridden.getExecutorService());
     assertNotEquals(config.getWorkflowExecutorService(), overridden.getWorkflowExecutorService());
     assertNotEquals(config.getMetrics(), overridden.getMetrics());
+    assertNotEquals(config.getResourceCloner(), overridden.getResourceCloner());
     assertNotEquals(
         config.getLeaderElectionConfiguration(), overridden.getLeaderElectionConfiguration());
-    assertNotEquals(
-        config.getInformerStoppedHandler(), overridden.getLeaderElectionConfiguration());
+    assertNotEquals(config.getInformerStoppedHandler(), overridden.getInformerStoppedHandler());
     assertNotEquals(
         config.reconciliationTerminationTimeout(), overridden.reconciliationTerminationTimeout());
   }
@@ -118,4 +122,30 @@
     assertThat(((ThreadPoolExecutor) overridden.getWorkflowExecutorService()).getMaximumPoolSize())
         .isEqualTo(14);
   }
+
+  @SuppressWarnings("rawtypes")
+  @Test
+  void dependentResourceFactoryDefaultsToTheSharedOneAndCanBeOverridden() {
+    final var factory = new DependentResourceFactory() {};
+
+    assertThat(config.dependentResourceFactory()).isSameAs(DependentResourceFactory.DEFAULT);
+    assertThat(
+            new ConfigurationServiceOverrider(config)
+                .withDependentResourceFactory(factory)
+                .build()
+                .dependentResourceFactory())
+        .isSameAs(factory);
+  }
+
+  @Test
+  void defaultNonSSAResourcesDefaultToConfigMapsAndSecretsAndCanBeOverridden() {
+    assertThat(config.defaultNonSSAResources())
+        .containsExactlyInAnyOrder(ConfigMap.class, Secret.class);
+    assertThat(
+            new ConfigurationServiceOverrider(config)
+                .withDefaultNonSSAResource(Set.of())
+                .build()
+                .defaultNonSSAResources())
+        .isEmpty();
+  }
 }