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

diff --git a/engine/tests/suite.php b/engine/tests/suite.php
index 7299585272bcb381d7babaeb5a32ccafc281bcd7..dbe88065ae25190521f780504c3f9446fa20cc03 100755 (executable)
--- 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);