speed up the tests (extreme facepalm) 🤦🏻

Author: ThoSap

Makefile

@@ -19,7 +19,7 @@ test:
 	./gradlew --console=colored :operator:clean :operator:test --rerun-tasks
 
 test-pg18:
-	./gradlew --console=colored :operator:clean :operator:test --rerun-tasks -Dquarkus.test.profile=test-pg18
+	./gradlew --console=colored :operator:clean :operator:test --fail-fast --rerun-tasks -Dquarkus.test.profile=test-pg18
 
 test-pg17:
 	./gradlew --console=colored :operator:clean :operator:test --rerun-tasks -Dquarkus.test.profile=test-pg17

operator/src/main/java/it/aboutbits/postgresql/crd/database/DatabaseReconciler.java

@@ -119,7 +119,7 @@ public DeleteControl cleanup(
             context.getClient().resource(resource).patchStatus();
 
             return DeleteControl.noFinalizerRemoval()
-                    .rescheduleAfter(1, TimeUnit.SECONDS);
+                    .rescheduleAfter(0, TimeUnit.SECONDS);
         }
 
         // We do not actually delete the database if the reclaimPolicy is set to RETAIN, we only delete the CR instance

operator/src/main/java/it/aboutbits/postgresql/crd/defaultprivilege/DefaultPrivilegeReconciler.java

@@ -141,7 +141,7 @@ public DeleteControl cleanup(
             context.getClient().resource(resource).patchStatus();
 
             return DeleteControl.noFinalizerRemoval()
-                    .rescheduleAfter(1, TimeUnit.SECONDS);
+                    .rescheduleAfter(0, TimeUnit.SECONDS);
         }
 
         var clusterRef = spec.getClusterRef();

operator/src/main/java/it/aboutbits/postgresql/crd/grant/GrantReconciler.java

@@ -146,7 +146,7 @@ public DeleteControl cleanup(
             context.getClient().resource(resource).patchStatus();
 
             return DeleteControl.noFinalizerRemoval()
-                    .rescheduleAfter(1, TimeUnit.SECONDS);
+                    .rescheduleAfter(0, TimeUnit.SECONDS);
         }
 
         var clusterRef = spec.getClusterRef();

operator/src/main/java/it/aboutbits/postgresql/crd/role/RoleReconciler.java

@@ -149,7 +149,7 @@ public DeleteControl cleanup(
             context.getClient().resource(resource).patchStatus();
 
             return DeleteControl.noFinalizerRemoval()
-                    .rescheduleAfter(1, TimeUnit.SECONDS);
+                    .rescheduleAfter(0, TimeUnit.SECONDS);
         }
 
         var clusterRef = spec.getClusterRef();

operator/src/main/java/it/aboutbits/postgresql/crd/schema/SchemaReconciler.java

@@ -121,7 +121,7 @@ public DeleteControl cleanup(
             context.getClient().resource(resource).patchStatus();
 
             return DeleteControl.noFinalizerRemoval()
-                    .rescheduleAfter(1, TimeUnit.SECONDS);
+                    .rescheduleAfter(0, TimeUnit.SECONDS);
         }
 
         // We do not actually delete the schema if the reclaimPolicy is set to RETAIN, we only delete the CR instance

operator/src/test/java/it/aboutbits/postgresql/crd/defaultprivilege/DefaultPrivilegeReconcilerTest.java

@@ -14,7 +14,6 @@
 import lombok.RequiredArgsConstructor;
 import org.jspecify.annotations.NullMarked;
 import org.junit.jupiter.api.BeforeEach;
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Nested;
 import org.junit.jupiter.api.Test;
@@ -62,7 +61,6 @@ void resetEnvironment() {
     class CRDValidation {
         @Nested
         class FieldSize {
-            @Disabled("Test is way too slow with all combinations")
             @ParameterizedTest
             @BlankSource
             @DisplayName("Should fail when the database is a blank or empty String (CEL rule)")
@@ -81,7 +79,6 @@ void failWhenDatabaseIsBlankOrEmptyString(
                 ).withMessageContaining("The DefaultPrivilege database must not be empty.");
             }
 
-            @Disabled("Test is way too slow with all combinations")
             @ParameterizedTest
             @BlankSource
             @DisplayName("Should fail when the role is a blank or empty String (CEL rule)")
@@ -100,7 +97,6 @@ void failWhenRoleIsBlankOrEmptyString(
                 ).withMessageContaining("The DefaultPrivilege role must not be empty.");
             }
 
-            @Disabled("Test is way too slow with all combinations")
             @ParameterizedTest
             @BlankSource
             @DisplayName("Should fail when the owner is a blank or empty String (CEL rule)")
@@ -119,7 +115,6 @@ void failWhenOwnerIsBlankOrEmptyString(
                 ).withMessageContaining("The DefaultPrivilege owner must not be empty.");
             }
 
-            @Disabled("Test is way too slow with all combinations")
             @ParameterizedTest
             @BlankSource
             @DisplayName("Should fail when the schema is a blank or empty String (CEL rule)")

operator/src/test/java/it/aboutbits/postgresql/crd/grant/GrantReconcilerTest.java

@@ -15,7 +15,6 @@
 import org.jooq.impl.SQLDataType;
 import org.jspecify.annotations.NullMarked;
 import org.junit.jupiter.api.BeforeEach;
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Nested;
 import org.junit.jupiter.api.Test;
@@ -67,7 +66,6 @@ void resetEnvironment() {
     class CRDValidation {
         @Nested
         class FieldSize {
-            @Disabled("Test is way too slow with all combinations")
             @ParameterizedTest
             @BlankSource
             @DisplayName("Should fail when the database is a blank or empty String (CEL rule)")
@@ -86,7 +84,6 @@ void failWhenDatabaseIsBlankOrEmptyString(
                 ).withMessageContaining("The Grant database must not be empty.");
             }
 
-            @Disabled("Test is way too slow with all combinations")
             @ParameterizedTest
             @BlankSource
             @DisplayName("Should fail when the role is a blank or empty String (CEL rule)")
@@ -105,7 +102,6 @@ void failWhenRoleIsBlankOrEmptyString(
                 ).withMessageContaining("The Grant role must not be empty.");
             }
 
-            @Disabled("Test is way too slow with all combinations")
             @ParameterizedTest
             @BlankSource
             @DisplayName("Should fail when the schema is a blank or empty String (CEL rule)")