fix and polish

batch-boot-jpa-sample/docker/docker-compose.yml

@@ -9,6 +9,11 @@ services:
       - POSTGRES_USER=appuser
       - POSTGRES_PASSWORD=secret
       - POSTGRES_DB=appdb
+    healthcheck:
+      test: ["CMD-SHELL", "pg_isready -U appuser -d appdb"]
+      interval: 10s
+      timeout: 5s
+      retries: 5
     ports:
       - "5432:5432"
     networks:

batch-boot-jpa-sample/pom.xml

@@ -243,7 +243,7 @@
                 <configuration>
                     <java>
                         <googleJavaFormat>
-                            <version>1.24.0</version>
+                            <version>1.25.0</version>
                             <style>AOSP</style>
                         </googleJavaFormat>
                         <importOrder />

batch-boot-jpa-sample/src/main/java/com/example/bootbatchjpa/Application.java → batch-boot-jpa-sample/src/main/java/com/example/bootbatchjpa/BatchApplication.java

@@ -7,9 +7,9 @@
 
 @SpringBootApplication
 @EnableConfigurationProperties({ApplicationProperties.class})
-public class Application {
+public class BatchApplication {
 
     public static void main(String[] args) {
-        SpringApplication.run(Application.class, args);
+        SpringApplication.run(BatchApplication.class, args);
     }
 }

batch-boot-jpa-sample/src/main/java/com/example/bootbatchjpa/config/BatchConfig.java

@@ -9,7 +9,12 @@
 import java.util.Map;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-import org.springframework.batch.core.*;
+import org.springframework.batch.core.BatchStatus;
+import org.springframework.batch.core.Job;
+import org.springframework.batch.core.JobExecution;
+import org.springframework.batch.core.JobExecutionListener;
+import org.springframework.batch.core.JobParameters;
+import org.springframework.batch.core.Step;
 import org.springframework.batch.core.configuration.annotation.EnableBatchProcessing;
 import org.springframework.batch.core.configuration.annotation.StepScope;
 import org.springframework.batch.core.job.builder.JobBuilder;
@@ -60,7 +65,6 @@ Job allCustomersJob(
                 .start(step)
                 .incrementer(new RunIdIncrementer())
                 .listener(this)
-                .start(step)
                 .build();
     }
 

batch-boot-jpa-sample/src/main/java/com/example/bootbatchjpa/config/Initializer.java

@@ -27,11 +27,12 @@ public void run(String... args) {
         List<Customer> customerList =
                 Instancio.ofList(Customer.class)
                         .size(1000)
+                        .ignore(field(Customer.class, "id"))
                         .generate(
                                 field(Customer.class, "gender"), gen -> gen.oneOf("male", "female"))
                         .create();
         log.info("Saving Customers of size :{}", customerList.size());
-        customerList = customerRepository.saveAllAndFlush(customerList);
+        customerList = customerRepository.saveAll(customerList);
         log.info("Inserted customers of size :{}", customerList.size());
     }
 }

batch-boot-jpa-sample/src/test/java/com/example/bootbatchjpa/TestBatchApplication.java

@@ -0,0 +1,11 @@
+package com.example.bootbatchjpa;
+
+import com.example.bootbatchjpa.common.ContainersConfig;
+import org.springframework.boot.SpringApplication;
+
+public class TestBatchApplication {
+
+    public static void main(String[] args) {
+        SpringApplication.from(BatchApplication::main).with(ContainersConfig.class).run(args);
+    }
+}

batch-boot-jpa-sample/src/test/java/com/example/bootbatchjpa/common/AbstractIntegrationTest.java

@@ -3,7 +3,6 @@
 import static com.example.bootbatchjpa.utils.AppConstants.PROFILE_TEST;
 import static org.springframework.boot.test.context.SpringBootTest.WebEnvironment.RANDOM_PORT;
 
-import com.example.bootbatchjpa.TestApplication;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.autoconfigure.web.servlet.AutoConfigureMockMvc;
@@ -14,7 +13,7 @@
 @ActiveProfiles({PROFILE_TEST})
 @SpringBootTest(
         webEnvironment = RANDOM_PORT,
-        classes = {TestApplication.class})
+        classes = {ContainersConfig.class})
 @AutoConfigureMockMvc
 public abstract class AbstractIntegrationTest {
 

batch-boot-jpa-sample/src/test/java/com/example/bootbatchjpa/common/ContainersConfig.java

@@ -0,0 +1,17 @@
+package com.example.bootbatchjpa.common;
+
+import org.springframework.boot.test.context.TestConfiguration;
+import org.springframework.boot.testcontainers.service.connection.ServiceConnection;
+import org.springframework.context.annotation.Bean;
+import org.testcontainers.containers.PostgreSQLContainer;
+import org.testcontainers.utility.DockerImageName;
+
+@TestConfiguration(proxyBeanMethods = false)
+public class ContainersConfig {
+
+    @Bean
+    @ServiceConnection
+    PostgreSQLContainer<?> postgreSQLContainer() {
+        return new PostgreSQLContainer<>(DockerImageName.parse("postgres").withTag("17.2-alpine"));
+    }
+}

batch-boot-jpa-sample/src/test/java/com/example/bootbatchjpa/repository/SchemaValidationTest.java → batch-boot-jpa-sample/src/test/java/com/example/bootbatchjpa/repository/InnerSchemaValidationTest.java

@@ -2,17 +2,17 @@
 
 import static org.assertj.core.api.Assertions.assertThat;
 
-import com.example.bootbatchjpa.common.TestContainersConfig;
+import com.example.bootbatchjpa.common.ContainersConfig;
 import com.zaxxer.hikari.HikariDataSource;
 import javax.sql.DataSource;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.autoconfigure.jdbc.AutoConfigureTestDatabase;
 import org.springframework.boot.test.autoconfigure.orm.jpa.DataJpaTest;
-import org.springframework.boot.testcontainers.context.ImportTestcontainers;
+import org.springframework.context.annotation.Import;
 
 @DataJpaTest(properties = {"spring.jpa.hibernate.ddl-auto=validate"})
-@ImportTestcontainers(TestContainersConfig.class)
+@Import(ContainersConfig.class)
 @AutoConfigureTestDatabase
 class SchemaValidationTest {
 

batch-boot-jpa-sample/src/test/java/com/example/bootbatchjpa/web/controllers/CustomerControllerIT.java

@@ -39,7 +39,7 @@ void setUp() {
                                 field(Customer.class, "gender"), gen -> gen.oneOf("male", "female"))
                         .create();
 
-        customerList = customerRepository.saveAllAndFlush(customerList);
+        customerList = customerRepository.saveAll(customerList);
     }
 
     @Test

httpClients/boot-http-proxy/src/test/java/com/example/rest/proxy/TestApplication.java

@@ -6,6 +6,6 @@
 class TestApplication {
 
     public static void main(String[] args) {
-        SpringApplication.from(Application::main).with(ContainersConfig.class).run(args);
+        SpringApplication.from(BatchApplication::main).with(ContainersConfig.class).run(args);
     }
 }

httpClients/boot-rest-template/src/test/java/com/example/rest/template/TestApplication.java

@@ -6,6 +6,6 @@
 class TestApplication {
 
     public static void main(String[] args) {
-        SpringApplication.from(Application::main).with(ContainersConfig.class).run(args);
+        SpringApplication.from(BatchApplication::main).with(ContainersConfig.class).run(args);
     }
 }