From 309fa9219985de8ec25880ee4d4f12cb37da2960 Mon Sep 17 00:00:00 2001
From: Sam Brannen <sam@sambrannen.com>
Date: Mon, 28 Mar 2011 18:26:30 +0000
Subject: [PATCH] [SPR-8089] re-ignoring JMX tests that require the presence of
 jmxremote_optional.jar.

---
 .../jmx/access/MBeanClientInterceptorTests.java           | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/org.springframework.context/src/test/java/org/springframework/jmx/access/MBeanClientInterceptorTests.java b/org.springframework.context/src/test/java/org/springframework/jmx/access/MBeanClientInterceptorTests.java
index 7380c8e379..e457cd6927 100644
--- a/org.springframework.context/src/test/java/org/springframework/jmx/access/MBeanClientInterceptorTests.java
+++ b/org.springframework.context/src/test/java/org/springframework/jmx/access/MBeanClientInterceptorTests.java
@@ -29,6 +29,7 @@ import javax.management.remote.JMXConnectorServer;
 import javax.management.remote.JMXConnectorServerFactory;
 import javax.management.remote.JMXServiceURL;
 
+import org.junit.Ignore;
 import org.springframework.jmx.AbstractMBeanServerTests;
 import org.springframework.jmx.IJmxTestBean;
 import org.springframework.jmx.JmxException;
@@ -185,7 +186,12 @@ public class MBeanClientInterceptorTests extends AbstractMBeanServerTests {
 		}
 	}
 
-	public void testLazyConnectionToRemote() throws Exception {
+	// @Ignore has no effect for JUnit 3.8 tests; however, we leave it
+	// here so that developers can easily search for ignored tests.
+	//
+	// Once fixed, renamed to test* instead of ignore*.
+	@Ignore("Requires jmxremote_optional.jar; see comments in AbstractMBeanServerTests for details.")
+	public void ignoreTestLazyConnectionToRemote() throws Exception {
 		if (!runTests)
 			return;