Add test and refactor

Author: jaumarar

src/DotEnv.php

@@ -38,7 +38,7 @@ private function setProcessors(array $processors = null) : DotEnv
         /**
          * Fill with default processors
          */
-        if ($processors === null){
+        if ($processors === null) {
             $this->processors = [
                 BooleanProcessor::class,
                 QuotedProcessor::class

src/Processor/QuotedProcessor.php

@@ -24,7 +24,7 @@ public function execute()
         return substr($this->value, 1, -1);
     }
 
-    public function isWrappedByChar(string $value, string $char) : bool
+    private function isWrappedByChar(string $value, string $char) : bool
     {
         return $value[0] === $char && $value[-1] === $char;
     }

tests/DotenvTest.php

@@ -6,6 +6,7 @@
 use DevCoder\Option;
 use DevCoder\Processor\BooleanProcessor;
 use DevCoder\Processor\QuotedProcessor;
+use PHPUnit\Framework\Assert;
 use PHPUnit\Framework\TestCase;
 
 class DotenvTest extends TestCase
@@ -53,6 +54,52 @@ public function testFileNotExist() {
         (new DotEnv($this->env('.env.not-exists')))->load();
     }
 
+    public function testUncompatibleProcessors() {
+        $this->assertProcessors(
+            [],
+            []
+        );
+
+        $this->assertProcessors(
+            null,
+            [BooleanProcessor::class, QuotedProcessor::class]
+        );
+
+        $this->assertProcessors(
+            [null],
+            []
+        );
+
+        $this->assertProcessors(
+            [new \stdClass()],
+            []
+        );
+
+        $this->assertProcessors(
+            [QuotedProcessor::class, null],
+            [QuotedProcessor::class]
+        );
+    }
+
+    private function assertProcessors(array $processorsToUse = null, array $expectedProcessors = [])
+    {
+        $dotEnv = new DotEnv($this->env('.env.default'), $processorsToUse);
+        $dotEnv->load();
+
+        $this->assertEquals(
+            $expectedProcessors,
+            $this->getProtectedProperty($dotEnv, 'processors')
+        );
+    }
+
+    private function getProtectedProperty(object $object, string $property) {
+        $reflection = new \ReflectionClass($object);
+        $reflectionProperty = $reflection->getProperty($property);
+        $reflectionProperty->setAccessible(true);
+
+        return $reflectionProperty->getValue($object);
+    }
+
     /**
      * @runInSeparateProcess
      */