Merge pull request #25 from assertwell/fix/check-for-runkit-on-functions

Mark tests that use defineFunction() or deleteFunction() as skipped if Runkit is unavailable
assertwell · Apr 15, 2021 · e397aa7 · e397aa7
2 parents 510f0ab + 054618f
commit e397aa7
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/src/Functions.php b/src/Functions.php
@@ -56,6 +56,10 @@ protected function restoreFunctions()
      */
     protected function defineFunction($name, \Closure $closure)
     {
+        if (! Runkit::isAvailable()) {
+            $this->markTestSkipped('defineFunction() requires Runkit be available, skipping.');
+        }
+
         if (function_exists($name)) {
             throw new FunctionExistsException(sprintf(
                 'Function %1$s() already exists. You may redefine it using %2$s::redefineFunction() instead.',
@@ -130,6 +134,10 @@ protected function deleteFunction($name)
             return $this;
         }
 
+        if (! Runkit::isAvailable()) {
+            $this->markTestSkipped('deleteFunction() requires Runkit be available, skipping.');
+        }
+
         $namespaced = Runkit::makeNamespaced($name);
 
         if (! Runkit::function_rename($name, $namespaced)) {