From: Ilia Alshanetsky <iliaa@php.net>
Date: Thu, 24 Jul 2003 17:45:15 +0000 (+0000)
Subject: MFH:
X-Git-Tag: php-4.3.3RC2~37
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=0ea41bf2b156deb512aa9d48163b17e89e330775;p=php

MFH:
  When encountering a test that users GET/POST make run-tests.php
  use CGI sapi if it is avaliable.
  If CLI sapi is not avaliable print a clear error when make test
  is executed.
---

diff --git a/Makefile.global b/Makefile.global
index 52b177b9dd..e2c591ad41 100644
--- a/Makefile.global
+++ b/Makefile.global
@@ -50,11 +50,15 @@ install-tester:
 
 install-su: install-pear install-tester
 
-test: $(SAPI_CLI_PATH)
-	@TEST_PHP_EXECUTABLE=$(top_builddir)/$(SAPI_CLI_PATH) \
-	 TEST_PHP_SRCDIR=$(top_srcdir) \
-	 CC="$(CC)" \
-			$(top_builddir)/$(SAPI_CLI_PATH) -d 'open_basedir=' -d 'safe_mode=0' -d 'output_buffering=0' $(top_srcdir)/run-tests.php $(TESTS)
+test: 
+	-@if test -x $(SAPI_CLI_PATH) && test ! -z $(SAPI_CLI_PATH); then \
+		TEST_PHP_EXECUTABLE=$(top_builddir)/$(SAPI_CLI_PATH) \
+		TEST_PHP_SRCDIR=$(top_srcdir) \
+		CC="$(CC)" \
+			$(top_builddir)/$(SAPI_CLI_PATH) -d 'open_basedir=' -d 'safe_mode=0' -d 'output_buffering=0' $(top_srcdir)/run-tests.php $(TESTS); \
+	else \
+		echo "ERROR: Cannot run tests without CLI sapi."; \
+	fi
 
 clean:
 	find . -name \*.lo | xargs rm -f
diff --git a/run-tests.php b/run-tests.php
index 7264567a84..da91aeb67f 100755
--- a/run-tests.php
+++ b/run-tests.php
@@ -99,7 +99,7 @@ if (getenv('TEST_PHP_EXECUTABLE')) {
 		putenv("TEST_PHP_EXECUTABLE=$php");
 	}
 }
-if (!file_exists($php)) {
+if (empty($php) || !file_exists($php)) {
 	error("environment variable TEST_PHP_EXECUTABLE must be set to specify PHP executable!");
 }
 
@@ -567,6 +567,14 @@ TEST $file
 	}
 	fclose($fp);
 
+	/* For GET/POST tests, check if cgi sapi is avaliable and if it is, use it. */
+	if ((!empty($section_text['GET']) || !empty($section_text['POST']))) {
+		if (file_exists("./sapi/cgi/php")) {
+			$old_php = $php;
+			$php = realpath("./sapi/cgi/php") . ' -C ';
+		}
+	}
+
 	$shortname = str_replace($GLOBALS['cwd'].'/', '', $file);
 	$tested = trim($section_text['TEST'])." [$shortname]";
 
@@ -610,6 +618,9 @@ TEST $file
 				} else {
 					echo "\n";
 				}
+				if (isset($old_php)) {
+					$php = $old_php;
+				}
 				return 'SKIPPED';
 			}
 			if (eregi("^info", trim($output))) {
@@ -696,6 +707,11 @@ COMMAND $cmd
 	$output = trim($out);
 	$output = preg_replace('/\r\n/',"\n",$output);
 
+	/* when using CGI, strip the headers from the output */
+	if (isset($old_php) && ($pos = strpos($output, "\n\n")) !== FALSE) {
+		$output = substr($output, ($pos + 2));
+	}
+
 	if (isset($section_text['EXPECTF']) || isset($section_text['EXPECTREGEX'])) {
 		if (isset($section_text['EXPECTF'])) {
 			$wanted = trim($section_text['EXPECTF']);
@@ -722,6 +738,9 @@ COMMAND $cmd
 		if (preg_match("/^$wanted_re\$/s", $output)) {
 			@unlink($tmp_file);
 			echo "PASS $tested\n";
+			if (isset($old_php)) {
+				$php = $old_php;
+			}
 			return 'PASSED';
 		}
 
@@ -733,6 +752,9 @@ COMMAND $cmd
 		if ($ok) {
 			@unlink($tmp_file);
 			echo "PASS $tested\n";
+			if (isset($old_php)) {
+				$php = $old_php;
+			}
 			return 'PASSED';
 		}
 	}
@@ -791,6 +813,10 @@ $output
 		error_report($file,$logname,$tested);
 	}
 
+	if (isset($old_php)) {
+		$php = $old_php;
+	}
+
 	return $warn ? 'WARNED' : 'FAILED';
 }