Fix transaction

edeandrea · Jan 2, 2025 · 339b11d · 339b11d
1 parent 54898da
commit 339b11d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/org/ericoleg/ndnp/ai/NotificationService.java b/src/main/java/org/ericoleg/ndnp/ai/NotificationService.java
@@ -67,7 +67,7 @@ private String updateStatus(long claimId, String status) {
 	}
 
 	@Transactional
-	Optional<Claim> updateStatusIfFound(long claimId, String status) {
+	public Optional<Claim> updateStatusIfFound(long claimId, String status) {
 		return Claim.<Claim>findByIdOptional(claimId)
 			.map(claim -> {
 				// Capitalize the first letter

diff --git a/src/test/java/org/ericoleg/ndnp/ai/NotificationServiceTests.java b/src/test/java/org/ericoleg/ndnp/ai/NotificationServiceTests.java
@@ -25,7 +25,6 @@
 import io.quarkiverse.mailpit.test.model.Message;
 
 @QuarkusTest
-@TestTransaction
 @WithMailbox
 class NotificationServiceTests {
 	@InjectMailbox
@@ -40,11 +39,12 @@ void afterEach() {
 	}
 
 	@Test
+	@TestTransaction
 	void emailSendsWhenUserExists() {
 		var status = "Denied";
 		var claimId = 1L;
 		var claim = Claim.<Claim>findByIdOptional(claimId)
-			.orElseThrow(() -> new IllegalArgumentException("Marty McFly's claim should be found!"));
+		                 .orElseThrow(() -> new IllegalArgumentException("Marty McFly's claim should be found!"));
 
 		assertThat(this.emailService.updateClaimStatus(claimId, status))
 			.isNotNull()