From 17e963569c479ef12167106f2cdd80fb96bdf7d2 Mon Sep 17 00:00:00 2001
From: Platonides <platonides@users.mediawiki.org>
Date: Sat, 9 Oct 2010 18:15:37 +0000
Subject: [PATCH] Readd after r72566 a phpunit.php so that globals are kept
 global.

---
 maintenance/tests/phpunit/Makefile      |  2 +-
 maintenance/tests/phpunit/bootstrap.php | 34 ++++++++-----------------
 maintenance/tests/phpunit/phpunit.php   | 27 ++++++++++++++++++++
 3 files changed, 39 insertions(+), 24 deletions(-)
 create mode 100755 maintenance/tests/phpunit/phpunit.php

diff --git a/maintenance/tests/phpunit/Makefile b/maintenance/tests/phpunit/Makefile
index d57ee5d3c1..0784c4ac45 100644
--- a/maintenance/tests/phpunit/Makefile
+++ b/maintenance/tests/phpunit/Makefile
@@ -3,7 +3,7 @@
 SHELL = /bin/sh
 CONFIG_FILE = $(shell pwd)/suite.xml
 FLAGS = 
-PU = phpunit --configuration ${CONFIG_FILE}
+PU = php phpunit.php --configuration ${CONFIG_FILE}
 
 all test: warning
 
diff --git a/maintenance/tests/phpunit/bootstrap.php b/maintenance/tests/phpunit/bootstrap.php
index c278af130f..e348b2c13f 100644
--- a/maintenance/tests/phpunit/bootstrap.php
+++ b/maintenance/tests/phpunit/bootstrap.php
@@ -1,31 +1,18 @@
-<?php 
+<?php
 /**
  * Bootstrapping for MediaWiki PHPUnit tests
- * 
+ * This file is included by phpunit and is NOT in the global scope.
+ *
  * @file
  */
 
-/* Configuration */
-
-// This file is not included in the global scope, but rather within a function, so we must global anything we need to
-// have access to in the global scope explicitly
-global $wgCommandLineMode, $IP, $optionsWithArgs, $wgProfiler, $wgAutoloadClasses;
-
-// Evaluate the include path relative to this file
-$IP = dirname( dirname( dirname( dirname( __FILE__ ) ) ) );
-
-// Set a flag which can be used to detect when other scripts have been entered through this entry point or not
-define( 'MW_PHPUNIT_TEST', true );
-
-// Start up MediaWiki in command-line mode
-require_once( "$IP/maintenance/commandLine.inc" );
-
-// Assume UTC for testing purposes
-$wgLocaltimezone = 'UTC';
-
-// To prevent tests from failing with SQLite, we need to turn database caching off
-global $wgCaches;
-$wgCaches[CACHE_DB] = false;
+if ( !defined( 'MW_PHPUNIT_TEST' ) ) {
+	echo <<<EOF
+You are running these tests directly from phpunit. You may not have all globals correctly set.
+Running phpunit.php instead is recommended.
+EOF;
+	require_once ( dirname( __FILE__ ) . "/phpunit.php" );
+}
 
 // Output a notice when running with older versions of PHPUnit
 if ( !version_compare( PHPUnit_Runner_Version::id(), "3.4.1", ">" ) ) {
@@ -74,3 +61,4 @@ abstract class MediaWikiTestSetup extends PHPUnit_Framework_TestCase {
 		}
 	}
 }
+
diff --git a/maintenance/tests/phpunit/phpunit.php b/maintenance/tests/phpunit/phpunit.php
new file mode 100755
index 0000000000..6fde6112ea
--- /dev/null
+++ b/maintenance/tests/phpunit/phpunit.php
@@ -0,0 +1,27 @@
+#!/usr/bin/env php
+<?php
+/**
+ * Bootstrapping for MediaWiki PHPUnit tests
+ * 
+ * @file
+ */
+
+/* Configuration */
+
+// Evaluate the include path relative to this file
+$IP = dirname( dirname( dirname( dirname( __FILE__ ) ) ) );
+
+// Set a flag which can be used to detect when other scripts have been entered through this entry point or not
+define( 'MW_PHPUNIT_TEST', true );
+
+// Start up MediaWiki in command-line mode
+require_once( "$IP/maintenance/commandLine.inc" );
+
+// Assume UTC for testing purposes
+$wgLocaltimezone = 'UTC';
+
+// To prevent tests from failing with SQLite, we need to turn database caching off
+$wgCaches[CACHE_DB] = false;
+
+require_once "/usr/bin/phpunit";
+
-- 
2.20.1