test fixes

Author: J0nasMayr

src/main/java/it/aboutbits/springboot/emailservice/lib/jpa/EmailRepository.java

@@ -51,7 +51,6 @@ public interface EmailRepository extends JpaRepository<Email, Long> {
                         )
                     )
             order by e.scheduledAt
-            limit :limit
             """)
     List<Long> findCandidateIdsToSend(
             @Param("now") OffsetDateTime now,

src/test/java/it/aboutbits/springboot/emailservice/lib/application/SendScheduledEmailsTest.java

@@ -28,15 +28,13 @@
 import static org.mockito.Mockito.verify;
 
 @SpringBootTest(properties = {
-        "aboutbits.emailservice.scheduling.batch-size=5",
         "aboutbits.emailservice.scheduling.max-attempts=3",
         "aboutbits.emailservice.scheduling.stuck-sending-recovery-threshold=PT5M",
         "aboutbits.emailservice.scheduling.interval=30000"
 })
 @WithPostgres
 @NullMarked
 class SendScheduledEmailsTest {
-    private static final int BATCH_SIZE = 5;
     private static final int MAX_ATTEMPTS = 3;
     private static final int SCHEDULER_INTERVAL_SECONDS = 30;
 
@@ -69,19 +67,6 @@ void givenPendingEmails_sendEmails_shouldMarkAllAsSent() {
         verify(javaMailSender, times(3)).send(any(MimeMessage.class));
     }
 
-    @Test
-    void givenMoreEmailsThanBatchSize_sendEmails_shouldProcessOnlyBatchSizePerPass() {
-        var pending = 3 * BATCH_SIZE;
-        emailRepository.saveAll(IntStream.range(0, pending).mapToObj(_ -> EmailFactory.once().build()).toList());
-
-        sendScheduledEmails.sendEmails();
-
-        assertThat(emailRepository.findAll())
-                .filteredOn(email -> email.getState() == EmailState.SENT)
-                .hasSize(BATCH_SIZE);
-        verify(javaMailSender, times(BATCH_SIZE)).send(any(MimeMessage.class));
-    }
-
     @Test
     void givenRetryableFailure_sendEmails_shouldRescheduleWithBackoffAndIncrementAttempts() {
         doThrow(new MailSendException("smtp blip")).when(javaMailSender).send(any(MimeMessage.class));