Mark remaining @Ignored tests with 'TODO SPR-8116'

Each of these tests began failing during the Gradle build porting
process. None seem severe, many are likely due to classpath issues.

In the case of TestNG support, this needs to be added to the Gradle
build in order to execute these tests. See SPR-8116.txt

spring-test/src/test/java/org/springframework/test/annotation/ProfileValueAnnotationAwareTransactionalTests.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2008 the original author or authors.
+ * Copyright 2002-2012 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -76,8 +76,7 @@ public class ProfileValueAnnotationAwareTransactionalTests extends TestCase {
 
 
 	@SuppressWarnings("deprecation")
-	@org.junit.Ignore // causes https://gist.github.com/1165828
-	protected static class DefaultProfileValueSourceTestCase extends AbstractAnnotationAwareTransactionalTests {
+	public static class DefaultProfileValueSourceTestCase extends AbstractAnnotationAwareTransactionalTests {
 
 		int invocationCount = 0;
 
@@ -134,8 +133,7 @@ public class ProfileValueAnnotationAwareTransactionalTests extends TestCase {
 	}
 
 	@ProfileValueSourceConfiguration(HardCodedProfileValueSource.class)
-	@org.junit.Ignore // causes https://gist.github.com/1165832
-	protected static class HardCodedProfileValueSourceTestCase extends DefaultProfileValueSourceTestCase {
+	public static class HardCodedProfileValueSourceTestCase extends DefaultProfileValueSourceTestCase {
 	}
 
 	public static class HardCodedProfileValueSource implements ProfileValueSource {

spring-test/src/test/java/org/springframework/test/context/junit38/FailingBeforeAndAfterMethodsTests.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2007 the original author or authors.
+ * Copyright 2002-2012 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -104,7 +104,7 @@ public class FailingBeforeAndAfterMethodsTests {
 		}
 	}
 
-	@org.junit.Ignore
+	@org.junit.Ignore // TODO SPR-8116
 	@SuppressWarnings("deprecation")
 	@TestExecutionListeners(listeners = AlwaysFailingBeforeTestMethodTestExecutionListener.class, inheritListeners = false)
 	public static class AlwaysFailingBeforeTestMethodTestCase extends AbstractJUnit38SpringContextTests {
@@ -113,7 +113,7 @@ public class FailingBeforeAndAfterMethodsTests {
 		}
 	}
 
-	@org.junit.Ignore
+	@org.junit.Ignore // TODO SPR-8116
 	@SuppressWarnings("deprecation")
 	@TestExecutionListeners(listeners = AlwaysFailingAfterTestMethodTestExecutionListener.class, inheritListeners = false)
 	public static class AlwaysFailingAfterTestMethodTestCase extends AbstractJUnit38SpringContextTests {
@@ -122,7 +122,7 @@ public class FailingBeforeAndAfterMethodsTests {
 		}
 	}
 
-	@org.junit.Ignore
+	@org.junit.Ignore // TODO SPR-8116
 	@SuppressWarnings("deprecation")
 	@ContextConfiguration("FailingBeforeAndAfterMethodsTests-context.xml")
 	public static class FailingBeforeTransactionalTestCase extends AbstractTransactionalJUnit38SpringContextTests {
@@ -136,7 +136,7 @@ public class FailingBeforeAndAfterMethodsTests {
 		}
 	}
 
-	@org.junit.Ignore
+	@org.junit.Ignore // TODO SPR-8116
 	@SuppressWarnings("deprecation")
 	@ContextConfiguration("FailingBeforeAndAfterMethodsTests-context.xml")
 	public static class FailingAfterTransactionalTestCase extends AbstractTransactionalJUnit38SpringContextTests {

spring-test/src/test/java/org/springframework/test/context/junit38/RepeatedJUnit38SpringContextTests.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2007 the original author or authors.
+ * Copyright 2002-2012 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -58,14 +58,12 @@ public class RepeatedJUnit38SpringContextTests extends TestCase {
 	 * explicitly configured with an empty list, thus disabling all default
 	 * listeners.
 	 */
-	@org.junit.Ignore // causes https://gist.github.com/1165825
 	@SuppressWarnings("deprecation")
 	@TestExecutionListeners(listeners = {}, inheritListeners = false)
-	protected static class RepeatedTestCase extends AbstractJUnit38SpringContextTests {
+	public static class RepeatedTestCase extends AbstractJUnit38SpringContextTests {
 
 		int invocationCount = 0;
 
-
 		public RepeatedTestCase(final String name) throws Exception {
 			super(name);
 		}

spring-test/src/test/java/org/springframework/test/context/junit4/ExpectedExceptionSpringRunnerTests.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2011 the original author or authors.
+ * Copyright 2002-2012 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -59,7 +59,7 @@ public class ExpectedExceptionSpringRunnerTests {
 	}
 
 
-	@org.junit.Ignore
+	@org.junit.Ignore // TODO SPR-8116
 	@RunWith(SpringJUnit4ClassRunner.class)
 	@TestExecutionListeners({})
 	public static final class ExpectedExceptionSpringRunnerTestCase {

spring-test/src/test/java/org/springframework/test/context/junit4/FailingBeforeAndAfterMethodsTests.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2009 the original author or authors.
+ * Copyright 2002-2012 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -146,32 +146,32 @@ public class FailingBeforeAndAfterMethodsTests {
 		}
 	}
 
-	@org.junit.Ignore
+	@org.junit.Ignore // TODO SPR-8116
 	@TestExecutionListeners(AlwaysFailingBeforeTestClassTestExecutionListener.class)
 	public static class AlwaysFailingBeforeTestClassTestCase extends BaseTestCase {
 	}
 
-	@org.junit.Ignore
+	@org.junit.Ignore // TODO SPR-8116
 	@TestExecutionListeners(AlwaysFailingAfterTestClassTestExecutionListener.class)
 	public static class AlwaysFailingAfterTestClassTestCase extends BaseTestCase {
 	}
 
-	@org.junit.Ignore
+	@org.junit.Ignore // TODO SPR-8116
 	@TestExecutionListeners(AlwaysFailingPrepareTestInstanceTestExecutionListener.class)
 	public static class AlwaysFailingPrepareTestInstanceTestCase extends BaseTestCase {
 	}
 
-	@org.junit.Ignore
+	@org.junit.Ignore // TODO SPR-8116
 	@TestExecutionListeners(AlwaysFailingBeforeTestMethodTestExecutionListener.class)
 	public static class AlwaysFailingBeforeTestMethodTestCase extends BaseTestCase {
 	}
 
-	@org.junit.Ignore
+	@org.junit.Ignore // TODO SPR-8116
 	@TestExecutionListeners(AlwaysFailingAfterTestMethodTestExecutionListener.class)
 	public static class AlwaysFailingAfterTestMethodTestCase extends BaseTestCase {
 	}
 
-	@org.junit.Ignore
+	@org.junit.Ignore // TODO SPR-8116
 	@ContextConfiguration("FailingBeforeAndAfterMethodsTests-context.xml")
 	public static class FailingBeforeTransactionTestCase extends AbstractTransactionalJUnit4SpringContextTests {
 
@@ -185,7 +185,7 @@ public class FailingBeforeAndAfterMethodsTests {
 		}
 	}
 
-	@org.junit.Ignore
+	@org.junit.Ignore // TODO SPR-8116
 	@ContextConfiguration("FailingBeforeAndAfterMethodsTests-context.xml")
 	public static class FailingAfterTransactionTestCase extends AbstractTransactionalJUnit4SpringContextTests {
 

spring-test/src/test/java/org/springframework/test/context/junit4/RepeatedSpringRunnerTests.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2009 the original author or authors.
+ * Copyright 2002-2012 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -151,7 +151,8 @@ public class RepeatedSpringRunnerTests {
 	 * href="http://jira.springframework.org/browse/SPR-6011"
 	 * target="_blank">SPR-6011</a>.
 	 */
-	@org.junit.Ignore // causing timeouts on cbeams' MBP
+	@org.junit.Ignore // TODO SPR-8116 causing timeouts on cbeams' (otherwise fast) MBP.
+	// Timeouts are 2x-4x their expected range. Something seems wrong indeed.
 	public static final class TimedRepeatedTestCase extends AbstractRepeatedTestCase {
 
 		@Test

spring-test/src/test/java/org/springframework/test/context/junit4/TimedSpringRunnerTests.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2009 the original author or authors.
+ * Copyright 2002-2012 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -56,7 +56,8 @@ public class TimedSpringRunnerTests {
 	}
 
 
-	@org.junit.Ignore // causing timeouts on cbeams' MBP
+	@org.junit.Ignore // TODO SPR-8116 causing timeouts on cbeams' (otherwise fast) MBP.
+	// Timeouts are 2x-5x their expected range. Something seems wrong indeed.
 	@RunWith(SpringJUnit4ClassRunner.class)
 	@TestExecutionListeners( {})
 	public static final class TimedSpringRunnerTestCase {

spring-test/src/test/java/org/springframework/test/context/junit4/orm/HibernateSessionFlushingTests.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2009 the original author or authors.
+ * Copyright 2002-2012 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -39,7 +39,7 @@ import org.springframework.test.context.junit4.orm.service.PersonService;
  * @author Sam Brannen
  * @since 3.0
  */
-@org.junit.Ignore // fix hibernate classpath
+@org.junit.Ignore // TODO SPR-8116 (Hibernate classpath-related)
 @ContextConfiguration
 public class HibernateSessionFlushingTests extends AbstractTransactionalJUnit4SpringContextTests {
 

spring-test/src/test/java/org/springframework/test/context/testng/SPR-8116.txt

@@ -0,0 +1,3 @@
+TODO SPR-8116 re-enable TestNG support for spring-test
+
+These TestNG test classes are currently not run at all.

spring-test/src/test/java/org/springframework/test/util/ReflectionTestUtilsTests.java

@@ -201,7 +201,6 @@ public class ReflectionTestUtilsTests {
 		}.runTest();
 	}
 
-	/* TODO uncomment and fix this compilation issue: https://gist.github.com/1177807
 	@Test
 	public void invokeMethodWithAutoboxingAndUnboxing() {
 		// IntelliJ IDEA 11 won't accept int assignment here
@@ -223,7 +222,6 @@ public class ReflectionTestUtilsTests {
 		Integer sum = invokeMethod(component, "add", new int[] { 1, 2, 3, 4 });
 		assertEquals("add(1,2,3,4)", 10, sum.intValue());
 	}
-	*/
 
 	@Test
 	public void invokeMethodsSimulatingLifecycleEvents() {