Cleaned up all tests with tearDown nulling refs

git-svn-id: https://svn.codehaus.org/grails/trunk@3981 1cfb16fd-6d17-0410-8ff1-b7e8e1e2867d

Author: marcpalmer

test/commons/org/codehaus/groovy/grails/commons/DynamicMethodsMetaClassTests.java

@@ -50,6 +50,10 @@ public void set(Object object, Object newValue) {
         this.groovyObject = (GroovyObject)groovyClass.newInstance();
     }
 
+    protected void tearDown() {
+        groovyObject = null;
+    }
+
     public void testInvokeExistingJavaMethod() {
         assertNotNull(groovyObject.invokeMethod("toString", new Object[0]));
     }

test/commons/org/codehaus/groovy/grails/commons/test/AbstractGrailsIntegrationTests.java

@@ -163,7 +163,6 @@ protected HttpServletRequest createMockRequest(String uri) {
 	protected abstract void onSetUp();
 
     protected final void tearDown() throws Exception {
-        super.tearDown();
 		if(TransactionSynchronizationManager.hasResource(this.sessionFactory)) {
 		    SessionHolder holder = (SessionHolder) TransactionSynchronizationManager.getResource(this.sessionFactory);
 		    org.hibernate.Session s = holder.getSession();
@@ -172,7 +171,18 @@ protected final void tearDown() throws Exception {
 		    SessionFactoryUtils.releaseSession(s, this.sessionFactory);
 		}
         onTearDown();
-    }	
+
+        gcl = null;
+        ga = null;
+        messageSource = null;
+        parentContext = null;
+        applicationContext = null;
+        servletContext = null;
+        sessionFactory = null;
+        session = null;
+
+        super.tearDown();
+    }
 
     /**
      * Called directly before destruction of the TestCase in the junit.framework.TestCase#tearDown() method

test/commons/org/codehaus/groovy/grails/commons/test/AbstractGrailsMockTests.java

@@ -65,6 +65,7 @@ protected final void tearDown() throws Exception {
 
         ga = null;
         ctx = null;
+        gcl = null;
 
         super.tearDown();
     }

test/commons/org/codehaus/groovy/grails/commons/test/AbstractGrailsResourceTests.java

@@ -175,8 +175,20 @@ protected final void tearDown() throws Exception {
 		    SessionFactoryUtils.releaseSession(s, this.sessionFactory);
 		}
         onTearDown();
-    }	
 
+        gcl = null;
+        ga = null;
+        messageSource = null;
+        parentContext = null;
+        applicationContext = null;
+        servletContext = null;
+        sessionFactory = null;
+        session = null;
+        resourcePattern = null;
+        resolver = null;
+        resources = null;
+    }
+    
     /**
      * Called directly before destruction of the TestCase in the junit.framework.TestCase#tearDown() method
      */

test/commons/org/codehaus/groovy/grails/injection/GrailsInjectionOperationTests.java

@@ -34,7 +34,15 @@ public void setOperation(GrailsInjectionOperation injectionOperation) {
 	}
 
 
-	protected void onSetUp() throws Exception {
+    protected void onTearDown() throws Exception
+    {
+        resources = null;
+        injectionOperation = null;
+        ga = null;
+	    super.onTearDown();
+    }
+
+    protected void onSetUp() throws Exception {
 		super.onSetUp();
 
 		resources = new PathMatchingResourcePatternResolver().getResources("classpath:org/codehaus/groovy/grails/injection/grails-app/domain/*.groovy");

test/commons/org/codehaus/groovy/grails/plugins/DefaultGrailsPluginTests.java

@@ -36,6 +36,17 @@ public class DefaultGrailsPluginTests extends AbstractGrailsMockTests {
     private Class disabled;
     private Class observed;
 
+    protected void onTearDown()
+    {
+        versioned = null;
+        notVersion = null;
+        notPluginClass = null;
+        disabled = null;
+        observed = null;
+        super.onTearDown();
+        //To change body of overridden methods use File | Settings | File Templates.
+    }
+
     protected void onSetUp() {
 		versioned = gcl.parseClass("class MyGrailsPlugin {\n" +
 						"def version = 1.1;" +

test/groovy/org/codehaus/groovy/grails/cli/AbstractCliTests.groovy

@@ -16,6 +16,7 @@ abstract class AbstractCliTests extends GroovyTestCase {
 
 	void tearDown() {
 		ant.delete(dir:appBase, failonerror:false)
+		ant = null
 	}
 
 }

test/groovy/org/codehaus/groovy/grails/commons/DefaultGrailsControllerClassTests.groovy

@@ -11,6 +11,10 @@ class DefaultGrailsControllerClassTests extends GroovyTestCase {
 		gcl = new GroovyClassLoader()
 	}
 
+    void tearDown() {
+        gcl = null
+    }
+
     void testInterceptorInheritance() {
         gcl.parseClass("""
 abstract class ParentController {

test/groovy/org/codehaus/groovy/grails/commons/DefaultGrailsDomainClassTests.groovy

@@ -10,7 +10,11 @@ class DefaultGrailsDomainClassTests extends GroovyTestCase {
 	void setUp() {
 		gcl = new GroovyClassLoader()
 	}
-	 
+
+    void tearDown() {
+        gcl = null
+    }
+
 	void testFetchMode() {
 		gcl.parseClass(
 				"""

test/groovy/org/codehaus/groovy/grails/commons/metaclass/ExpandoMetaClassCreationHandleTests.groovy

@@ -35,7 +35,10 @@ class ExpandoMetaClassCreationHandleTests extends GroovyTestCase {
 
 	void tearDown() {
 		registry.metaClassCreationHandle = original
+		original = null
+		registry = null
 	}
+
 	void testExpandoCreationHandle() {
 		def metaClass = registry.getMetaClass(URL.class)
 		if(!(metaClass instanceof ExpandoMetaClass)) {