BAEL-8140: code review and puml

Author: etrandafir93

persistence-modules/spring-boot-persistence-5/puml/nested-transactions.puml

@@ -0,0 +1,44 @@
+@startuml Nested Transactions in publishArticle_v2
+
+participant "Client" as C
+participant "Blog" as B
+participant "ArticleRepo" as AR
+participant "AuditService" as AS
+database "Database" as DB
+
+C -> B: publishArticle_v2()
+activate B
+note right of B: @Transactional
+B -> DB: Begin TX1
+activate DB #LightBlue
+
+    B -> AR: save()
+    activate AR
+    AR -> DB: INSERT article
+    AR --> B:
+    deactivate AR
+
+    B -> AS: saveAudit()
+    activate AS
+    note right of AS: @Transactional\n(REQUIRES_NEW)
+    AS -> DB: Begin TX2
+    activate DB #LightGreen
+
+    AS -> DB: INSERT audit
+
+    AS -> DB: Commit TX2
+    deactivate DB
+    AS --> B:
+    deactivate AS
+
+B -> DB: Commit / Rollback TX1
+deactivate DB
+B --> C:
+deactivate B
+
+note over DB
+  TX1: Article save
+  TX2: Audit save (independent, always commits)
+end note
+
+@enduml

persistence-modules/spring-boot-persistence-5/puml/sequential-transactions.puml

@@ -0,0 +1,50 @@
+@startuml Sequential Transactions in publishArticle_v3
+
+participant "Client" as C
+participant "Blog" as B
+participant "TransactionTemplate" as TT
+participant "ArticleRepo" as AR
+participant "AuditRepo" as AuR
+database "Database" as DB
+
+C -> B: publishArticle_v3()
+activate B
+
+B -> TT: execute()
+activate TT
+TT -> DB: Begin TX1
+activate DB #LightBlue
+
+TT -> AR: save()
+activate AR
+AR -> DB: INSERT article
+AR --> TT
+deactivate AR
+
+TT -> DB: Commit / Rollback TX1
+deactivate DB
+TT --> B
+deactivate TT
+
+
+B -> DB: Begin TX2
+activate DB #LightGreen
+
+B -> AuR: save()
+activate AuR
+AuR -> DB: INSERT audit (FAILURE)
+AuR --> B:
+deactivate AuR
+
+B -> DB: Commit TX2
+deactivate DB
+
+B --> C:
+deactivate B
+
+note over DB
+  TX1 completes (commit or rollback) BEFORE TX2 begins
+  TX2: Independent transaction for FAILURE audit
+end note
+
+@enduml

persistence-modules/spring-boot-persistence-5/src/test/java/com/baeldung/rollbackonly/BlogIntegrationTest.java

@@ -51,7 +51,7 @@ void whenPublishingAnInvalidArticle_thenThrowsUnexpectedRollbackException() {
         assertThatThrownBy(() -> articleService.publishArticle(
                 new Article("Test Article", null)))
             .isInstanceOf(UnexpectedRollbackException.class)
-            .hasMessageContaining("Transaction silently rolled back because it has been marked as rollback-only");
+            .hasMessageContaining("marked as rollback-only");
 
         assertThat(auditRepo.findAll())
             .isEmpty();