Refactor CLI detection into a separate function to be able to mock it

Author: fabacino

src/Debug.php

@@ -154,7 +154,7 @@ public function printValue($var, int $flags = null)
         }
 
         $output = static::debugValue($var, $flags);
-        if (PHP_SAPI !== 'cli') {
+        if (!$this->isCli()) {
             $output = "<pre>{$output}</pre>";
         }
         echo $output;
@@ -204,4 +204,14 @@ public function logValue($var, int $flags = null)
     {
         $this->Logger->addDebug(static::debugValue($var, $flags));
     }
+
+    /**
+     * Check whether we are in a CLI environment.
+     *
+     * @return bool
+     */
+    protected function isCli(): bool
+    {
+        return substr(PHP_SAPI, 0, 3) === 'cli';
+    }
 }

tests/DbgTest.php

@@ -45,6 +45,27 @@ public function testPrintArray()
     [2] => third
 )
 
+EOT;
+        $this->assertEquals($expected, $this->captureOutput($var));
+    }
+
+    /**
+     * Test printing output in non-CLI environment.
+     *
+     * @return void
+     */
+    public function testPrintArrayNonCli()
+    {
+        TestDebug::init();
+        $var = ['first', 'second', 'third'];
+        $expected = <<<'EOT'
+<pre>Array
+(
+    [0] => first
+    [1] => second
+    [2] => third
+)
+</pre>
 EOT;
         $this->assertEquals($expected, $this->captureOutput($var));
     }

tests/TestDebug.php

@@ -0,0 +1,28 @@
+<?php
+
+/**
+ * Debug class for testing.
+ *
+ * @copyright Fabian Wiget <fabacino@gmail.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Fabacino\Debug\Test;
+
+class TestDebug extends \Fabacino\Debug\Debug
+{
+    /**
+     * Check whether we are in a CLI environment.
+     *
+     * @return bool
+     */
+    protected function isCli(): bool
+    {
+        // PHPUnit is always executed through CLI. In order to be able to check
+        // functionality which is working only in a non-CLI environment, we just
+        // pretend that PHP is running in a non-CLI environment.
+        return false;
+    }
+}