Add method to safely read environment variable

Author: sebastianfeldmann

src/Runner/Util.php

@@ -47,4 +47,17 @@ public static function getExecType(string $action): string
     {
         return substr($action, 0, 1) === '\\' ? 'php' : 'cli';
     }
+
+    /**
+     * Try to read an environment variable
+     *
+     * @param  string $name
+     * @param  string $default
+     * @return string
+     */
+    public static function getEnv(string $name, string $default = ''): string
+    {
+        $var = getenv($name);
+        return $var ?: $_ENV[$name] ?? $_SERVER[$name] ?? $default;
+    }
 }

tests/unit/Runner/UtilTest.php

@@ -46,4 +46,32 @@ public function testIsTypeValid(): void
         $this->assertTrue(Util::isTypeValid('cli'));
         $this->assertFalse(Util::isTypeValid('foo'));
     }
+
+    /**
+     * Tests Util::getEnv
+     */
+    public function testCanReadEnvVar(): void
+    {
+        $_ENV['foo'] = 'bar';
+        $this->assertEquals('bar', Util::getEnv('foo'));
+        unset($_ENV['foo']);
+    }
+
+    /**
+     * Tests Util::getEnv
+     */
+    public function testUsesServerSuperglobalAsFallback(): void
+    {
+        $_SERVER['foo'] = 'bar';
+        $this->assertEquals('bar', Util::getEnv('foo'));
+        unset($_SERVER['foo']);
+    }
+
+    /**
+     * Tests Util::getEnv
+     */
+    public function testReturnsDefaultIdEnvNotSet(): void
+    {
+        $this->assertEquals('baz', Util::getEnv('fiz', 'baz'));
+    }
 }