ConfigurationException - previous must be (child of) \Exception, not Exception.

Author: SpacePossum

src/Exception/ConfigurationException.php

@@ -13,18 +13,18 @@
 final class ConfigurationException extends InvalidArgumentException
 {
     /**
-     * @param string         $option
-     * @param string         $expected
-     * @param mixed          $value
-     * @param int            $code
-     * @param Exception|null $previous
+     * @param string          $option
+     * @param string          $expected
+     * @param mixed           $value
+     * @param int             $code
+     * @param \Exception|null $previous
      */
     public function __construct(
         string $option,
         string $expected,
         $value,
         int $code = 0,
-        Exception $previous = null
+        \Exception $previous = null
     ) {
         parent::__construct(
             \sprintf(

tests/Exception/ConfigurationExceptionTest.php

@@ -17,12 +17,25 @@
  */
 final class ConfigurationExceptionTest extends TestCase
 {
-    public function testConstruct()
+    public function testConstructWithDefaults()
     {
         $e = new ConfigurationException('test', 'A', 'B');
 
         $this->assertSame(0, $e->getCode());
         $this->assertNull($e->getPrevious());
         $this->assertSame('Option "test" must be A, got "string#B".', $e->getMessage());
     }
+
+    public function testConstruct()
+    {
+        $e = new ConfigurationException(
+            'test',
+            'integer',
+            new \SplFileInfo(__FILE__),
+            789,
+            new \BadMethodCallException(__METHOD__)
+        );
+
+        $this->assertSame('Option "test" must be integer, got "SplFileInfo".', $e->getMessage());
+    }
 }