Verify RestartEndpoint is enabled without causing a restart (#1617)

ryanjbaxter · web-flow · commit 598baeef0b11 · 2025-12-03T20:09:33.000-05:00
Fixes #1362
diff --git a/spring-cloud-context/src/test/java/org/springframework/cloud/autoconfigure/LifecycleMvcAutoConfigurationTests.java b/spring-cloud-context/src/test/java/org/springframework/cloud/autoconfigure/LifecycleMvcAutoConfigurationTests.java
@@ -58,7 +58,6 @@ public void environmentWebEndpointExtensionGloballyDisabled() {
 	}
 
 	@Test
-	@Disabled("TODO: https://github.com/spring-cloud/spring-cloud-commons/issues/1362")
 	public void environmentWebEndpointExtensionEnabled() {
 		beanCreated("writableEnvironmentEndpointWebExtension", "management.endpoint.env.enabled=true",
 				"management.endpoint.env.post.enabled=true", "management.endpoints.web.exposure.include=env");
@@ -77,7 +76,7 @@ public void restartEndpointGloballyDisabled() {
 
 	@Test
 	public void restartEndpointEnabled() {
-		beanCreatedAndEndpointEnabled("restartEndpoint", RestartEndpoint.class, RestartEndpoint::restart,
+		beanCreatedAndEndpointEnabled("restartEndpoint", RestartEndpoint.class, RestartEndpoint::getTimeout,
 				"management.endpoint.restart.enabled=true", "management.endpoints.web.exposure.include=restart");
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,6 @@ public void environmentWebEndpointExtensionGloballyDisabled() {`
`58`	`58`	`}`
`59`	`59`
`60`	`60`	`@Test`
`61`		`- @Disabled("TODO: https://github.com/spring-cloud/spring-cloud-commons/issues/1362")`
`62`	`61`	`public void environmentWebEndpointExtensionEnabled() {`
`63`	`62`	`beanCreated("writableEnvironmentEndpointWebExtension", "management.endpoint.env.enabled=true",`
`64`	`63`	`"management.endpoint.env.post.enabled=true", "management.endpoints.web.exposure.include=env");`
`@@ -77,7 +76,7 @@ public void restartEndpointGloballyDisabled() {`
`77`	`76`
`78`	`77`	`@Test`
`79`	`78`	`public void restartEndpointEnabled() {`
`80`		`- beanCreatedAndEndpointEnabled("restartEndpoint", RestartEndpoint.class, RestartEndpoint::restart,`
	`79`	`+ beanCreatedAndEndpointEnabled("restartEndpoint", RestartEndpoint.class, RestartEndpoint::getTimeout,`
`81`	`80`	`"management.endpoint.restart.enabled=true", "management.endpoints.web.exposure.include=restart");`
`82`	`81`	`}`
`83`	`82`