From 3ed9bad2b1dbf5364febbb2c84ae4b6a631dc4b6 Mon Sep 17 00:00:00 2001
From: brettp <brettp@36083f99-b078-4883-b0ff-0f9b5a30f544>
Date: Tue, 13 Oct 2009 21:54:05 +0000
Subject: Test suite now requires admin for web-based tests. Uses access
 overrides for cli-based tests.

git-svn-id: http://code.elgg.org/elgg/trunk@3531 36083f99-b078-4883-b0ff-0f9b5a30f544
---
 engine/tests/suite.php | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

(limited to 'engine/tests/suite.php')

diff --git a/engine/tests/suite.php b/engine/tests/suite.php
index 729958527..dbe88065a 100755
--- a/engine/tests/suite.php
+++ b/engine/tests/suite.php
@@ -11,9 +11,6 @@
 
 require_once(dirname( __FILE__ ) . '/../start.php');
 
-// Ensure that only logged-in users can see this page
-gatekeeper();
-
 $vendor_path = "$CONFIG->path/vendors/simpletest";
 $test_path = "$CONFIG->path/engine/tests";
 
@@ -35,9 +32,14 @@ if ($CONFIG->debug > 0) {
 	if (TextReporter::inCli()) {
 		// In CLI error codes are returned.
 		// 0 is success.
+		elgg_set_ignore_access(TRUE);
 		exit ($suite->Run(new TextReporter()) ? 0 : 1 );
 	}
+	// Ensure that only logged-in users can see this page
+	admin_gatekeeper();
+	$old = elgg_set_ignore_access(TRUE);
 	$suite->Run(new HtmlReporter());
+	elgg_set_ignore_access($old);
 } else {
 	// @todo display an error?
 	exit (1);
-- 
cgit v1.2.3