From 1efa2ab3fc54f46a782866d61c275a9ccc216e83 Mon Sep 17 00:00:00 2001
From: Antony Dovgal <tony2001@php.net>
Date: Thu, 12 Apr 2007 12:43:54 +0000
Subject: [PATCH] add new test and SKIPIF sections

---
 ext/reflection/tests/009.phpt      |   2 +
 ext/reflection/tests/010.phpt      |   2 +
 ext/reflection/tests/011.phpt      |   2 +
 ext/reflection/tests/012.phpt      |   2 +
 ext/reflection/tests/013.phpt      |   2 +
 ext/reflection/tests/014.phpt      |   2 +
 ext/reflection/tests/015.phpt      |   2 +
 ext/reflection/tests/016.phpt      |   1 +
 ext/reflection/tests/017.phpt      |   2 +
 ext/reflection/tests/018.phpt      |   2 +
 ext/reflection/tests/019.phpt      |   2 +
 ext/reflection/tests/020.phpt      |   2 +
 ext/reflection/tests/021.phpt      |   2 +
 ext/reflection/tests/022.phpt      |   2 +
 ext/reflection/tests/023.phpt      |   2 +
 ext/reflection/tests/024.phpt      |   2 +
 ext/reflection/tests/025.phpt      | 179 +++++++++++++++++++++++++++++
 ext/reflection/tests/bug37816.phpt |   2 +
 ext/reflection/tests/bug38217.phpt |   4 +-
 ext/reflection/tests/bug38653.phpt |   2 +
 ext/reflection/tests/bug38942.phpt |   2 +
 ext/reflection/tests/bug39001.phpt |   2 +
 ext/reflection/tests/bug39067.phpt |   2 +
 ext/reflection/tests/bug39884.phpt |   2 +
 ext/reflection/tests/bug40431.phpt |   2 +
 25 files changed, 227 insertions(+), 1 deletion(-)
 create mode 100644 ext/reflection/tests/025.phpt

diff --git a/ext/reflection/tests/009.phpt b/ext/reflection/tests/009.phpt
index e88c6d5aba..4e81a025fc 100755
--- a/ext/reflection/tests/009.phpt
+++ b/ext/reflection/tests/009.phpt
@@ -1,5 +1,7 @@
 --TEST--
 ReflectionMethod::__construct() tests
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 
diff --git a/ext/reflection/tests/010.phpt b/ext/reflection/tests/010.phpt
index 361019a08f..8f92feea09 100755
--- a/ext/reflection/tests/010.phpt
+++ b/ext/reflection/tests/010.phpt
@@ -1,5 +1,7 @@
 --TEST--
 ReflectionMethod::__toString() tests (overriden method)
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 class Foo {
diff --git a/ext/reflection/tests/011.phpt b/ext/reflection/tests/011.phpt
index 500a84d311..b39be37c90 100755
--- a/ext/reflection/tests/011.phpt
+++ b/ext/reflection/tests/011.phpt
@@ -1,5 +1,7 @@
 --TEST--
 ReflectionExtension::getClasses()
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 $ext = new ReflectionExtension("reflection");
diff --git a/ext/reflection/tests/012.phpt b/ext/reflection/tests/012.phpt
index 541fdbe141..b8a2694004 100755
--- a/ext/reflection/tests/012.phpt
+++ b/ext/reflection/tests/012.phpt
@@ -1,5 +1,7 @@
 --TEST--
 ReflectionClass::getDefaultProperties()
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 class Foo {
diff --git a/ext/reflection/tests/013.phpt b/ext/reflection/tests/013.phpt
index 87e51c6a89..9ecfa8b66c 100755
--- a/ext/reflection/tests/013.phpt
+++ b/ext/reflection/tests/013.phpt
@@ -1,5 +1,7 @@
 --TEST--
 ReflectionExtension::getFunctions()
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 $ext = new ReflectionExtension("standard");
diff --git a/ext/reflection/tests/014.phpt b/ext/reflection/tests/014.phpt
index 3aaea23468..8b5955f124 100755
--- a/ext/reflection/tests/014.phpt
+++ b/ext/reflection/tests/014.phpt
@@ -1,5 +1,7 @@
 --TEST--
 ReflectionExtension::getConstants()
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 $ext = new ReflectionExtension("standard");
diff --git a/ext/reflection/tests/015.phpt b/ext/reflection/tests/015.phpt
index 2abf510ba6..9035c19b96 100755
--- a/ext/reflection/tests/015.phpt
+++ b/ext/reflection/tests/015.phpt
@@ -1,5 +1,7 @@
 --TEST--
 ReflectionExtension::getINIEntries()
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --INI--
 user_agent=php
 --FILE--
diff --git a/ext/reflection/tests/016.phpt b/ext/reflection/tests/016.phpt
index 1c8ec6be63..e1f71b3697 100755
--- a/ext/reflection/tests/016.phpt
+++ b/ext/reflection/tests/016.phpt
@@ -2,6 +2,7 @@
 ReflectionExtension::getDependencies()
 --SKIPIF--
 <?php
+extension_loaded('reflection') or die('skip'); 
 if (!extension_loaded("xml")) {
   die('skip xml extension not available');
 }
diff --git a/ext/reflection/tests/017.phpt b/ext/reflection/tests/017.phpt
index 58d5a6d19f..56bd508d15 100755
--- a/ext/reflection/tests/017.phpt
+++ b/ext/reflection/tests/017.phpt
@@ -1,5 +1,7 @@
 --TEST--
 ReflectionClass::__toString() (constants)
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 class Foo {
diff --git a/ext/reflection/tests/018.phpt b/ext/reflection/tests/018.phpt
index f25a1aec86..0cebd82ab6 100755
--- a/ext/reflection/tests/018.phpt
+++ b/ext/reflection/tests/018.phpt
@@ -1,5 +1,7 @@
 --TEST--
 Reflection::getModifierNames
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 var_dump(Reflection::getModifierNames(ReflectionMethod::IS_FINAL | ReflectionMethod::IS_PROTECTED));
diff --git a/ext/reflection/tests/019.phpt b/ext/reflection/tests/019.phpt
index 0b1199dbe0..0f7daa21ed 100755
--- a/ext/reflection/tests/019.phpt
+++ b/ext/reflection/tests/019.phpt
@@ -1,5 +1,7 @@
 --TEST--
 ReflectionFunction::getExtensionName
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 $f = new ReflectionFunction("sleep");
diff --git a/ext/reflection/tests/020.phpt b/ext/reflection/tests/020.phpt
index 67dcebe253..8856acb0c7 100755
--- a/ext/reflection/tests/020.phpt
+++ b/ext/reflection/tests/020.phpt
@@ -1,5 +1,7 @@
 --TEST--
 ReflectionObject::hasProperty
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 class Foo {
diff --git a/ext/reflection/tests/021.phpt b/ext/reflection/tests/021.phpt
index 9df3f0aeb4..30dbb5afcb 100755
--- a/ext/reflection/tests/021.phpt
+++ b/ext/reflection/tests/021.phpt
@@ -1,5 +1,7 @@
 --TEST--
 ReflectionClass::hasConstant
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 class Foo {
diff --git a/ext/reflection/tests/022.phpt b/ext/reflection/tests/022.phpt
index 91324613b9..50dbd6ec74 100755
--- a/ext/reflection/tests/022.phpt
+++ b/ext/reflection/tests/022.phpt
@@ -1,5 +1,7 @@
 --TEST--
 ReflectionClass::getConstant
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 class Foo {
diff --git a/ext/reflection/tests/023.phpt b/ext/reflection/tests/023.phpt
index dc7a847f0b..8085f15c15 100755
--- a/ext/reflection/tests/023.phpt
+++ b/ext/reflection/tests/023.phpt
@@ -1,5 +1,7 @@
 --TEST--
 ReflectionClass::getDefaultProperties (filtering parent privates)
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 class C1 {
diff --git a/ext/reflection/tests/024.phpt b/ext/reflection/tests/024.phpt
index b05a4501b3..d1af28dfc9 100755
--- a/ext/reflection/tests/024.phpt
+++ b/ext/reflection/tests/024.phpt
@@ -1,5 +1,7 @@
 --TEST--
 ReflectionObject::__toString (filtering privates/protected dynamic properties)
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 class C1 {
diff --git a/ext/reflection/tests/025.phpt b/ext/reflection/tests/025.phpt
new file mode 100644
index 0000000000..06da4b9a5d
--- /dev/null
+++ b/ext/reflection/tests/025.phpt
@@ -0,0 +1,179 @@
+--TEST--
+ReflectionFunction basic tests
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
+--FILE--
+<?php
+
+/**
+hoho
+*/
+function test ($a, $b = 1, $c = "") {
+	static $var = 1;
+}
+
+$func = new ReflectionFunction("test");
+
+var_dump($func->export("test"));
+echo "--getName--\n";
+var_dump($func->getName());
+echo "--isInternal--\n";
+var_dump($func->isInternal());
+echo "--isUserDefined--\n";
+var_dump($func->isUserDefined());
+echo "--getFilename--\n";
+var_dump($func->getFilename());
+echo "--getStartline--\n";
+var_dump($func->getStartline());
+echo "--getEndline--\n";
+var_dump($func->getEndline());
+echo "--getDocComment--\n";
+var_dump($func->getDocComment());
+echo "--getStaticVariables--\n";
+var_dump($func->getStaticVariables());
+echo "--invoke--\n";
+var_dump($func->invoke(array(1,2,3)));
+echo "--invokeArgs--\n";
+var_dump($func->invokeArgs(array(1,2,3)));
+echo "--returnsReference--\n";
+var_dump($func->returnsReference());
+echo "--getParameters--\n";
+var_dump($func->getParameters());
+echo "--getNumberOfParameters--\n";
+var_dump($func->getNumberOfParameters());
+echo "--getNumberOfRequiredParameters--\n";
+var_dump($func->getNumberOfRequiredParameters());
+
+echo "Done\n";
+
+?>
+--EXPECTF--	
+/**
+hoho
+*/
+Function [ <user> function test ] {
+  @@ %s025.php 6 - 8
+
+  - Parameters [3] {
+    Parameter #0 [ <required> $a ]
+    Parameter #1 [ <optional> $b = 1 ]
+    Parameter #2 [ <optional> $c = '' ]
+  }
+}
+
+NULL
+--getName--
+string(4) "test"
+--isInternal--
+bool(false)
+--isUserDefined--
+bool(true)
+--getFilename--
+string(%d) "%s025.php"
+--getStartline--
+int(6)
+--getEndline--
+int(8)
+--getDocComment--
+string(11) "/**
+hoho
+*/"
+--getStaticVariables--
+array(1) {
+  ["var"]=>
+  int(1)
+}
+--invoke--
+NULL
+--invokeArgs--
+NULL
+--returnsReference--
+bool(false)
+--getParameters--
+array(3) {
+  [0]=>
+  &object(ReflectionParameter)#2 (1) {
+    ["name"]=>
+    string(1) "a"
+  }
+  [1]=>
+  &object(ReflectionParameter)#3 (1) {
+    ["name"]=>
+    string(1) "b"
+  }
+  [2]=>
+  &object(ReflectionParameter)#4 (1) {
+    ["name"]=>
+    string(1) "c"
+  }
+}
+--getNumberOfParameters--
+int(3)
+--getNumberOfRequiredParameters--
+int(1)
+Done
+--UEXPECTF--
+/**
+hoho
+*/
+Function [ <user> function test ] {
+  @@ %s 6 - 8
+
+  - Parameters [3] {
+    Parameter #0 [ <required> $a ]
+    Parameter #1 [ <optional> $b = 1 ]
+    Parameter #2 [ <optional> $c = '' ]
+  }
+}
+
+NULL
+--getName--
+unicode(4) "test"
+--isInternal--
+bool(false)
+--isUserDefined--
+bool(true)
+--getFilename--
+unicode(%d) "%s025.php"
+--getStartline--
+int(6)
+--getEndline--
+int(8)
+--getDocComment--
+unicode(11) "/**
+hoho
+*/"
+--getStaticVariables--
+array(1) {
+  [u"var"]=>
+  int(1)
+}
+--invoke--
+NULL
+--invokeArgs--
+NULL
+--returnsReference--
+bool(false)
+--getParameters--
+array(3) {
+  [0]=>
+  &object(ReflectionParameter)#2 (1) {
+    [u"name"]=>
+    unicode(1) "a"
+  }
+  [1]=>
+  &object(ReflectionParameter)#3 (1) {
+    [u"name"]=>
+    unicode(1) "b"
+  }
+  [2]=>
+  &object(ReflectionParameter)#4 (1) {
+    [u"name"]=>
+    unicode(1) "c"
+  }
+}
+--getNumberOfParameters--
+int(3)
+--getNumberOfRequiredParameters--
+int(1)
+Done
diff --git a/ext/reflection/tests/bug37816.phpt b/ext/reflection/tests/bug37816.phpt
index 18a49046dc..1121d54188 100755
--- a/ext/reflection/tests/bug37816.phpt
+++ b/ext/reflection/tests/bug37816.phpt
@@ -1,5 +1,7 @@
 --TEST--
 Bug #37816 (ReflectionProperty does not throw exception when accessing protected attribute)
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 
diff --git a/ext/reflection/tests/bug38217.phpt b/ext/reflection/tests/bug38217.phpt
index 7a1e22b850..959e16a820 100644
--- a/ext/reflection/tests/bug38217.phpt
+++ b/ext/reflection/tests/bug38217.phpt
@@ -1,5 +1,7 @@
 --TEST--
-#38217 (ReflectionClass::newInstanceArgs() tries to allocate too much memory)
+Bug #38217 (ReflectionClass::newInstanceArgs() tries to allocate too much memory)
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 
diff --git a/ext/reflection/tests/bug38653.phpt b/ext/reflection/tests/bug38653.phpt
index 48f1c6ce41..c9025166d5 100644
--- a/ext/reflection/tests/bug38653.phpt
+++ b/ext/reflection/tests/bug38653.phpt
@@ -1,5 +1,7 @@
 --TEST--
 Bug #38653 (memory leak in ReflectionClass::getConstant())
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 
diff --git a/ext/reflection/tests/bug38942.phpt b/ext/reflection/tests/bug38942.phpt
index 817190c997..a02f596aff 100755
--- a/ext/reflection/tests/bug38942.phpt
+++ b/ext/reflection/tests/bug38942.phpt
@@ -1,5 +1,7 @@
 --TEST--
 Bug #38942 (Double old-style-ctor inheritance)
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 class foo {
diff --git a/ext/reflection/tests/bug39001.phpt b/ext/reflection/tests/bug39001.phpt
index bf8927d661..91fa865188 100644
--- a/ext/reflection/tests/bug39001.phpt
+++ b/ext/reflection/tests/bug39001.phpt
@@ -1,5 +1,7 @@
 --TEST--
 Bug #39001 (ReflectionProperty returns incorrect declaring class for protected properties)
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 
diff --git a/ext/reflection/tests/bug39067.phpt b/ext/reflection/tests/bug39067.phpt
index 5aae083152..a22c53a68a 100644
--- a/ext/reflection/tests/bug39067.phpt
+++ b/ext/reflection/tests/bug39067.phpt
@@ -1,5 +1,7 @@
 --TEST--
 Bug #39067 (getDeclaringClass() and private properties)
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 
diff --git a/ext/reflection/tests/bug39884.phpt b/ext/reflection/tests/bug39884.phpt
index 874fb54078..8882c0cbab 100644
--- a/ext/reflection/tests/bug39884.phpt
+++ b/ext/reflection/tests/bug39884.phpt
@@ -1,5 +1,7 @@
 --TEST--
 Bug #39884 (ReflectionParameter::getClass() throws exception for type hint self)
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 class stubParamTest
diff --git a/ext/reflection/tests/bug40431.phpt b/ext/reflection/tests/bug40431.phpt
index 68c18cf0d4..e881789c79 100644
--- a/ext/reflection/tests/bug40431.phpt
+++ b/ext/reflection/tests/bug40431.phpt
@@ -1,5 +1,7 @@
 --TEST--
 Bug #40431 (dynamic properties may cause crash in ReflectionProperty methods)
+--SKIPIF--
+<?php extension_loaded('reflection') or die('skip'); ?>
 --FILE--
 <?php
 
-- 
2.50.1