Polish contribution

See gh-47987

Author: snicoll

spring-boot-project/spring-boot-devtools/src/test/java/org/springframework/boot/devtools/restart/MainMethodTests.java

@@ -68,24 +68,41 @@ void viaJarLauncher() throws Exception {
 	}
 
 	@Test
-	void missingArgsMainMethod() throws Exception {
-		Method actualMain = MissingArgs.class.getMethod("main");
-		MainMethod method = new TestThread(MissingArgs::main).test();
+	void detectPublicMainMethod() throws Exception {
+		Method actualMain = PublicMainMethod.class.getMethod("main", String[].class);
+		MainMethod method = new TestThread(PublicMainMethod::main).test();
 		assertThat(method.getMethod()).isEqualTo(actualMain);
 		assertThat(method.getDeclaringClassName()).isEqualTo(actualMain.getDeclaringClass().getName());
 	}
 
 	@Test
-	void missingArgsPackagePrivateMainMethod() throws Exception {
-		Method actualMain = MissingArgsPackagePrivate.class.getDeclaredMethod("main");
-		MainMethod method = new TestThread(MissingArgsPackagePrivate::main).test();
+	void detectPublicParameterlessMainMethod() throws Exception {
+		Method actualMain = PublicParameterlessMainMethod.class.getMethod("main");
+		MainMethod method = new TestThread(PublicParameterlessMainMethod::main).test();
+		assertThat(method.getMethod()).isEqualTo(actualMain);
+		assertThat(method.getDeclaringClassName()).isEqualTo(actualMain.getDeclaringClass().getName());
+	}
+
+	@Test
+	void detectPackagePrivateMainMethod() throws Exception {
+		Method actualMain = PackagePrivateMainMethod.class.getDeclaredMethod("main", String[].class);
+		MainMethod method = new TestThread(PackagePrivateMainMethod::main).test();
+		assertThat(method.getMethod()).isEqualTo(actualMain);
+		assertThat(method.getDeclaringClassName()).isEqualTo(actualMain.getDeclaringClass().getName());
+	}
+
+	@Test
+	void detectPackagePrivateParameterlessMainMethod() throws Exception {
+		Method actualMain = PackagePrivateParameterlessMainMethod.class.getDeclaredMethod("main");
+		MainMethod method = new TestThread(PackagePrivateParameterlessMainMethod::main).test();
 		assertThat(method.getMethod()).isEqualTo(actualMain);
 		assertThat(method.getDeclaringClassName()).isEqualTo(actualMain.getDeclaringClass().getName());
 	}
 
 	@Test
 	void nonStatic() {
-		assertThatIllegalStateException().isThrownBy(() -> new TestThread(() -> new NonStaticMain().main()).test())
+		assertThatIllegalStateException()
+			.isThrownBy(() -> new TestThread(() -> new NonStaticMainMethod().main()).test())
 			.withMessageContaining("Unable to find main method");
 	}
 
@@ -144,23 +161,39 @@ public static void main(String... args) {
 
 	}
 
-	public static class MissingArgs {
+	public static class PublicMainMethod {
+
+		public static void main(String... args) {
+			mainMethod.set(new MainMethod());
+		}
+
+	}
+
+	public static class PublicParameterlessMainMethod {
 
 		public static void main() {
 			mainMethod.set(new MainMethod());
 		}
 
 	}
 
-	public static class MissingArgsPackagePrivate {
+	public static class PackagePrivateMainMethod {
+
+		static void main(String... args) {
+			mainMethod.set(new MainMethod());
+		}
+
+	}
+
+	public static class PackagePrivateParameterlessMainMethod {
 
 		static void main() {
 			mainMethod.set(new MainMethod());
 		}
 
 	}
 
-	public static class NonStaticMain {
+	public static class NonStaticMainMethod {
 
 		void main(String... args) {
 			mainMethod.set(new MainMethod());