Fixes #1496: delete_access_collection() invalidates the access collection cache before attempting to delete. Removed a misplaced unset() in a unit test that was causing test objects not to be properly deleted.

git-svn-id: http://code.elgg.org/elgg/trunk@3922 36083f99-b078-4883-b0ff-0f9b5a30f544

1 files changed, 2 insertions, 9 deletions

diff --git a/engine/tests/api/entity_getter_functions.php b/engine/tests/api/entity_getter_functions.php
index 579b3cea0..d09f1066d 100644
--- a/engine/tests/api/entity_getter_functions.php
+++ b/engine/tests/api/entity_getter_functions.php
@@ -25,19 +25,14 @@ class ElggCoreEntityGetterFunctionsTest extends ElggCoreUnitTest {
 			'object' => array(),
 			'user' => array(),
 			'group' => array(),
-		//'site'	=> array()
+			//'site'	=> array()
 		);
 
 		// sites are a bit wonky.  Don't use them just now.
 		$this->types = array('object', 'user', 'group');
 
 		// create some fun objects to play with.
-		// one with no subtype
-//		$e = new ElggObject();
-//		$e->subtype = $subtype;
-//		$e->save();
-
-		// and 5 with random subtypes
+		// 5 with random subtypes
 		for ($i=0; $i<5; $i++) {
 			$subtype = "test_object_subtype_" . rand();
 			$e = new ElggObject();
@@ -74,10 +69,8 @@ class ElggCoreEntityGetterFunctionsTest extends ElggCoreUnitTest {
 	 */
 	public function tearDown() {
 		//$this->swallowErrors();
-
 		foreach ($this->entities as $e) {
 			$e->delete();
-			unset($this->entities);
 		}
 	}