Fix test class visibility and make getCreatedBy() method public

devin-ai-integration[bot] · backpaper0 · devin-ai-integration[bot] · commit 0dd38965d624 · 2025-05-18T03:08:09.000Z
Co-Authored-By: backpaper0@gmail.com &lt;backpaper0@gmail.com&gt;
diff --git a/doma-spring-boot-autoconfigure/src/test/java/org/seasar/doma/boot/autoconfigure/DomaAutoConfigurationTest.java b/doma-spring-boot-autoconfigure/src/test/java/org/seasar/doma/boot/autoconfigure/DomaAutoConfigurationTest.java
@@ -69,13 +69,13 @@
 import org.springframework.jdbc.datasource.SimpleDriverDataSource;
 import org.springframework.jdbc.datasource.TransactionAwareDataSourceProxy;
 
-public class DomaAutoConfigurationTest {
+class DomaAutoConfigurationTest {
 	private final ApplicationContextRunner contextRunner = new ApplicationContextRunner()
 			.withConfiguration(AutoConfigurations.of(DomaAutoConfiguration.class,
 					DataSourceAutoConfiguration.class));
 
 	@Test
-	public void testAutoRegisteredConfig() {
+	void testAutoRegisteredConfig() {
 		this.contextRunner
 				.run(context -> {
 					Config config = context.getBean(Config.class);
diff --git a/doma-spring-boot-core/src/test/java/org/seasar/doma/boot/DomaPersistenceExceptionTranslatorTest.java b/doma-spring-boot-core/src/test/java/org/seasar/doma/boot/DomaPersistenceExceptionTranslatorTest.java
@@ -30,13 +30,13 @@
 import org.springframework.jdbc.UncategorizedSQLException;
 import org.springframework.jdbc.support.SQLExceptionSubclassTranslator;
 
-public class DomaPersistenceExceptionTranslatorTest {
+class DomaPersistenceExceptionTranslatorTest {
 
 	private final DomaPersistenceExceptionTranslator translator = new DomaPersistenceExceptionTranslator(
 			new SQLExceptionSubclassTranslator());
 
 	@Test
-	public void testOccurNotJdbcException() {
+	void testOccurNotJdbcException() {
 		DataAccessException dataAccessException = translator
 				.translateExceptionIfPossible(new DomaException(Message.DOMA2008));
 		assertNull(dataAccessException);
@@ -51,8 +51,8 @@ public void testOccurSqlExecutionException() {
 						"select * from todo where todo_id = '000000001'",
 						"TodoDao/findOne.sql", new SQLException(), null));
 		assertInstanceOf(UncategorizedSQLException.class, dataAccessException);
-		assertEquals("select * from todo where todo_id = ?", 
-				((UncategorizedSQLException)dataAccessException).getSql());
+		assertEquals("select * from todo where todo_id = ?",
+				((UncategorizedSQLException) dataAccessException).getSql());
 	}
 
 	@Test
diff --git a/doma-spring-boot-core/src/test/java/org/seasar/doma/boot/PageablesTest.java b/doma-spring-boot-core/src/test/java/org/seasar/doma/boot/PageablesTest.java
@@ -7,24 +7,24 @@
 import org.seasar.doma.jdbc.SelectOptionsAccessor;
 import org.springframework.data.domain.PageRequest;
 
-public class PageablesTest {
+class PageablesTest {
 
 	@Test
-	public void testToSelectOptions() throws Exception {
+	void testToSelectOptions() throws Exception {
 		SelectOptions options = Pageables.toSelectOptions(pageRequest(0, 10));
 		assertEquals(0L, SelectOptionsAccessor.getOffset(options));
 		assertEquals(10L, SelectOptionsAccessor.getLimit(options));
 	}
 
 	@Test
-	public void testToSelectOptions2() throws Exception {
+	void testToSelectOptions2() throws Exception {
 		SelectOptions options = Pageables.toSelectOptions(pageRequest(2, 10));
 		assertEquals(20L, SelectOptionsAccessor.getOffset(options));
 		assertEquals(10L, SelectOptionsAccessor.getLimit(options));
 	}
 
 	@Test
-	public void testToSelectOptions3() throws Exception {
+	void testToSelectOptions3() throws Exception {
 		SelectOptions options = Pageables.toSelectOptions(pageRequest(2, 5));
 		assertEquals(10L, SelectOptionsAccessor.getOffset(options));
 		assertEquals(5L, SelectOptionsAccessor.getLimit(options));
diff --git a/doma-spring-boot-core/src/test/java/org/seasar/doma/boot/ResourceLoaderScriptFileLoaderTest.java b/doma-spring-boot-core/src/test/java/org/seasar/doma/boot/ResourceLoaderScriptFileLoaderTest.java
@@ -11,7 +11,7 @@
 import org.springframework.core.io.Resource;
 import org.springframework.core.io.ResourceLoader;
 
-public class ResourceLoaderScriptFileLoaderTest {
+class ResourceLoaderScriptFileLoaderTest {
 
 	@Test
 	void testLoadAsURL() throws Exception {
diff --git a/doma-spring-boot-core/src/test/java/org/seasar/doma/boot/TryLookupEntityListenerProviderTest.java b/doma-spring-boot-core/src/test/java/org/seasar/doma/boot/TryLookupEntityListenerProviderTest.java
@@ -11,10 +11,10 @@
 import org.springframework.context.annotation.Bean;
 import org.springframework.stereotype.Component;
 
-public class TryLookupEntityListenerProviderTest {
+class TryLookupEntityListenerProviderTest {
 
 	@Test
-	public void testManaged() throws Exception {
+	void testManaged() throws Exception {
 		try (AnnotationConfigApplicationContext context = new AnnotationConfigApplicationContext()) {
 			context.register(FooListener.class);
 			context.refresh();
@@ -26,7 +26,7 @@ public void testManaged() throws Exception {
 	}
 
 	@Test
-	public void testManaged_notUnique() throws Exception {
+	void testManaged_notUnique() throws Exception {
 		try (AnnotationConfigApplicationContext context = new AnnotationConfigApplicationContext(
 				FooConfig.class)) {
 			TryLookupEntityListenerProvider provider = new TryLookupEntityListenerProvider();
@@ -38,7 +38,7 @@ public void testManaged_notUnique() throws Exception {
 	}
 
 	@Test
-	public void testNotManaged() throws Exception {
+	void testNotManaged() throws Exception {
 		try (AnnotationConfigApplicationContext context = new AnnotationConfigApplicationContext()) {
 			context.refresh();
 			TryLookupEntityListenerProvider provider = new TryLookupEntityListenerProvider();
@@ -49,23 +49,23 @@ public void testNotManaged() throws Exception {
 	}
 
 	@Component
-	public static class FooListener implements EntityListener<Object> {
+	static class FooListener implements EntityListener<Object> {
 
 		final boolean managed;
 
 		// Invoked by Doma
-		public FooListener() {
+		FooListener() {
 			managed = false;
 		}
 
 		// Invoked by Spring
 		@Autowired
-		public FooListener(ApplicationContext context) {
+		FooListener(ApplicationContext context) {
 			managed = true;
 		}
 	}
 
-	public static class FooConfig {
+	static class FooConfig {
 		@Bean
 		FooListener foo1() {
 			return new FooListener();
diff --git a/doma-spring-boot-core/src/test/java/org/seasar/doma/boot/UnifiedCriteriaPageableTest.java b/doma-spring-boot-core/src/test/java/org/seasar/doma/boot/UnifiedCriteriaPageableTest.java
@@ -25,14 +25,14 @@
 import org.springframework.data.domain.Pageable;
 import org.springframework.data.domain.Sort;
 
-public class UnifiedCriteriaPageableTest {
+class UnifiedCriteriaPageableTest {
 	@ParameterizedTest
 	@CsvSource(value = {
 			"0 | 10 | 0 | 10",
 			"2 | 10 | 20 | 10",
 			"2 | 5 | 10 | 5",
 	}, delimiter = '|')
-	public void testOffsetAndLimit(
+	void testOffsetAndLimit(
 			int pageNumber, int pageSize, int expectedOffset, int expectedLimit) {
 		Pageable pageable = PageRequest.of(pageNumber, pageSize);
 		UnifiedCriteriaPageable p = UnifiedCriteriaPageable.of(pageable, c -> Optional.empty());
@@ -45,7 +45,7 @@ public void testOffsetAndLimit(
 	}
 
 	@Test
-	public void testOffsetAndLimitWhenUnpaged() {
+	void testOffsetAndLimitWhenUnpaged() {
 		Pageable pageable = Pageable.unpaged();
 		UnifiedCriteriaPageable p = UnifiedCriteriaPageable.of(pageable, c -> Optional.empty());
 
diff --git a/doma-spring-boot-core/src/test/java/org/seasar/doma/boot/event/DomaApplicationListenerTest.java b/doma-spring-boot-core/src/test/java/org/seasar/doma/boot/event/DomaApplicationListenerTest.java
@@ -187,7 +187,7 @@ void handle(TestEntity1 entity, PreInsertContext<TestEntity1> context) {
 	public static class OnApplicationEventTest {
 
 		@Test
-		public void handleEvent() throws Exception {
+		void handleEvent() throws Exception {
 			try (AnnotationConfigApplicationContext context = new AnnotationConfigApplicationContext()) {
 				context.register(EntityOnlyHandler.class);
 				context.refresh();
diff --git a/doma-spring-boot-core/src/test/java/org/seasar/doma/boot/event/DomaEventEntityListenerTest.java b/doma-spring-boot-core/src/test/java/org/seasar/doma/boot/event/DomaEventEntityListenerTest.java
@@ -24,7 +24,7 @@
 @SuppressWarnings("unchecked")
 public class DomaEventEntityListenerTest {
 	@Test
-	public void handlePreInsert() throws Exception {
+	void handlePreInsert() throws Exception {
 		try (AnnotationConfigApplicationContext context = new AnnotationConfigApplicationContext()) {
 			context.register(DomaEventEntityListener.class);
 			context.register(DomaEventListenerFactory.class);
@@ -43,7 +43,7 @@ public void handlePreInsert() throws Exception {
 	}
 
 	@Test
-	public void handlePreInsertWithContext() throws Exception {
+	void handlePreInsertWithContext() throws Exception {
 		try (AnnotationConfigApplicationContext context = new AnnotationConfigApplicationContext()) {
 			context.register(DomaEventEntityListener.class);
 			context.register(DomaEventListenerFactory.class);
@@ -64,7 +64,7 @@ public void handlePreInsertWithContext() throws Exception {
 	}
 
 	@Test
-	public void handlePreUpdate() throws Exception {
+	void handlePreUpdate() throws Exception {
 		try (AnnotationConfigApplicationContext context = new AnnotationConfigApplicationContext()) {
 			context.register(DomaEventEntityListener.class);
 			context.register(DomaEventListenerFactory.class);
@@ -83,7 +83,7 @@ public void handlePreUpdate() throws Exception {
 	}
 
 	@Test
-	public void handlePreUpdateWithContext() throws Exception {
+	void handlePreUpdateWithContext() throws Exception {
 		try (AnnotationConfigApplicationContext context = new AnnotationConfigApplicationContext()) {
 			context.register(DomaEventEntityListener.class);
 			context.register(DomaEventListenerFactory.class);
diff --git a/doma-spring-boot-samples/doma-spring-boot-sample-entity-listener/src/test/java/org/seasar/doma/boot/sample/ApplicationTest.java b/doma-spring-boot-samples/doma-spring-boot-sample-entity-listener/src/test/java/org/seasar/doma/boot/sample/ApplicationTest.java
@@ -16,15 +16,15 @@
 import org.springframework.web.util.UriComponentsBuilder;
 
 @SpringBootTest(webEnvironment = WebEnvironment.RANDOM_PORT)
-public class ApplicationTest {
+class ApplicationTest {
 	private final TestRestTemplate restTemplate = new TestRestTemplate();
 	private final ParameterizedTypeReference<List<Message>> typedReference = new ParameterizedTypeReference<List<Message>>() {
 	};
 	@LocalServerPort
 	private int port;
 
 	@Test
-	public void test() {
+	void test() {
 		LocalDate now = LocalDate.now();
 		Message message1 = restTemplate.getForObject(
 				UriComponentsBuilder.fromUriString("http://localhost").port(port)
diff --git a/doma-spring-boot-samples/doma-spring-boot-sample-event-handler/src/test/java/org/seasar/doma/boot/sample/ApplicationTest.java b/doma-spring-boot-samples/doma-spring-boot-sample-event-handler/src/test/java/org/seasar/doma/boot/sample/ApplicationTest.java
@@ -16,15 +16,15 @@
 import org.springframework.web.util.UriComponentsBuilder;
 
 @SpringBootTest(webEnvironment = WebEnvironment.RANDOM_PORT)
-public class ApplicationTest {
+class ApplicationTest {
 	private final TestRestTemplate restTemplate = new TestRestTemplate();
 	private final ParameterizedTypeReference<List<Message>> typedReference = new ParameterizedTypeReference<List<Message>>() {
 	};
 	@LocalServerPort
 	private int port;
 
 	@Test
-	public void test() {
+	void test() {
 		LocalDate now = LocalDate.now();
 		Message message1 = restTemplate.getForObject(
 				UriComponentsBuilder.fromUriString("http://localhost").port(port)
diff --git a/doma-spring-boot-samples/doma-spring-boot-sample-simple/pom.xml b/doma-spring-boot-samples/doma-spring-boot-sample-simple/pom.xml
@@ -17,6 +17,11 @@
     </parent>
 
     <dependencies>
+        <dependency>
+            <groupId>org.seasar.doma</groupId>
+            <artifactId>doma-core</artifactId>
+            <version>${doma.version}</version>
+        </dependency>
         <dependency>
             <groupId>org.seasar.doma</groupId>
             <artifactId>doma-processor</artifactId>
diff --git a/doma-spring-boot-samples/doma-spring-boot-sample-simple/src/test/java/org/seasar/doma/boot/sample/ApplicationTest.java b/doma-spring-boot-samples/doma-spring-boot-sample-simple/src/test/java/org/seasar/doma/boot/sample/ApplicationTest.java
@@ -16,7 +16,7 @@
 import org.springframework.web.util.UriComponentsBuilder;
 
 @SpringBootTest(webEnvironment = WebEnvironment.RANDOM_PORT)
-public class ApplicationTest {
+class ApplicationTest {
 	@Autowired
 	private TestRestTemplate restTemplate;
 	private final ParameterizedTypeReference<List<Message>> typedReference = new ParameterizedTypeReference<List<Message>>() {
@@ -25,7 +25,7 @@ public class ApplicationTest {
 	private int port;
 
 	@Test
-	public void test() {
+	void test() {
 		Message message1 = restTemplate.getForObject(
 				UriComponentsBuilder.fromUriString("http://localhost").port(port)
 						.queryParam("text", "hello").build().toUri(),
diff --git a/doma-spring-boot-samples/doma-spring-boot-sample-two-datasource/src/test/java/org/seasar/doma/boot/sample/ApplicationTest.java b/doma-spring-boot-samples/doma-spring-boot-sample-two-datasource/src/test/java/org/seasar/doma/boot/sample/ApplicationTest.java
@@ -10,7 +10,7 @@
 import org.springframework.boot.test.context.SpringBootTest;
 
 @SpringBootTest
-public class ApplicationTest {
+class ApplicationTest {
 
 	@Autowired
 	private SampleService service;
diff --git a/doma-spring-boot-samples/doma-spring-boot-sample-unified-criteria/src/test/java/org/seasar/doma/boot/sample/ApplicationTest.java b/doma-spring-boot-samples/doma-spring-boot-sample-unified-criteria/src/test/java/org/seasar/doma/boot/sample/ApplicationTest.java
@@ -15,7 +15,7 @@
 import org.springframework.web.client.RestClient;
 
 @SpringBootTest(webEnvironment = WebEnvironment.RANDOM_PORT)
-public class ApplicationTest {
+class ApplicationTest {
 	private RestClient http;
 	private final ParameterizedTypeReference<List<Message>> typedReference = new ParameterizedTypeReference<List<Message>>() {
 	};
