fix: rollback issue (#3)

Author: davidfrigolet

build.gradle.kts

@@ -22,7 +22,7 @@ allprojects {
     val declaredVersion = "1.3.2-SNAPSHOT"
     version = VersionManager.resolveVersion(declaredVersion, project.hasProperty("release"))
 
-    extra["templateApiVersion"] = "1.3.4"
+    extra["templateApiVersion"] = "1.3.2"
     extra["flamingockVersion"] = "1.2.1"//for tests
 
     repositories {

flamingock-sql-template/src/main/java/io/flamingock/template/sql/SqlTemplate.java

@@ -53,7 +53,8 @@
  * executed via {@link java.sql.Statement#execute(String)}.
  *
  * <p>The {@code rollback} payload is optional ({@code rollbackPayloadRequired = false}).
- * If absent or blank, {@link #rollback(Connection)} is a no-op.
+ * If absent, the framework skips the rollback method entirely. If present, the framework's
+ * {@code TemplateString} validation rejects null or blank values at pipeline load time.
  *
  * @see SqlTemplateConfig for configuration options
  */
@@ -69,10 +70,7 @@ public void apply(Connection connection) {
 
     @RollbackTemplate
     public void rollback(Connection connection) {
-        if (rollbackPayload != null && rollbackPayload.getValue() != null
-                && !rollbackPayload.getValue().trim().isEmpty()) {
-            execute(connection, rollbackPayload.getValue());
-        }
+        execute(connection, rollbackPayload.getValue());
     }
 
     private void execute(Connection connection, String sql) {

flamingock-sql-template/src/test/java/io/flamingock/template/sql/SqlTemplateTest.java

@@ -45,7 +45,6 @@
 import static org.mockito.ArgumentMatchers.anyString;
 import static org.mockito.Mockito.*;
 
-@Disabled
 class SqlTemplateTest {
 
     private static final String TEST_TABLE = "test_users";
@@ -164,36 +163,6 @@ void rollbackExecution() throws Exception {
         verify(mockStatement, times(1)).execute(rollbackSql.trim());
     }
 
-    @Test
-    @DisplayName("P0-C: WHEN rollbackPayload is null THEN rollback() does not execute any SQL")
-    void rollbackNullPayloadGuard() throws Exception {
-        SqlTemplate template = new SqlTemplate();
-        // rollbackPayload is null by default — no setRollbackPayload() call
-
-        Connection mockConnection = Mockito.mock(Connection.class);
-        Statement mockStatement = Mockito.mock(Statement.class);
-        when(mockConnection.createStatement()).thenReturn(mockStatement);
-
-        template.rollback(mockConnection);
-
-        verify(mockStatement, never()).execute(anyString());
-    }
-
-    @Test
-    @DisplayName("P0-C: WHEN rollbackPayload value is empty string THEN rollback() does not execute any SQL")
-    void rollbackEmptyPayloadGuard() throws Exception {
-        SqlTemplate template = new SqlTemplate();
-        template.setRollbackPayload(new TemplateString(""));
-
-        Connection mockConnection = Mockito.mock(Connection.class);
-        Statement mockStatement = Mockito.mock(Statement.class);
-        when(mockConnection.createStatement()).thenReturn(mockStatement);
-
-        template.rollback(mockConnection);
-
-        verify(mockStatement, never()).execute(anyString());
-    }
-
     @Test
     @DisplayName("WHEN rollback SQL is executed against a real database THEN the database reflects the rollback")
     void rollbackExecutesAgainstRealDatabase() throws Exception {