Added missing ImportSpecification tests.

Author: Bilge

test/Integration/Porter/Connector/CachingConnectorTest.php

@@ -5,7 +5,7 @@
 use Mockery\MockInterface;
 use ScriptFUSION\Porter\Connector\CachingConnector;
 use ScriptFUSION\Porter\Options\EncapsulatedOptions;
-use ScriptFUSIONTest\Porter\Options\TestOptions;
+use ScriptFUSIONTest\Stubs\TestOptions;
 
 final class CachingConnectorTest extends \PHPUnit_Framework_TestCase
 {

test/Stubs/Invokable.php

@@ -0,0 +1,10 @@
+<?php
+namespace ScriptFUSIONTest\Stubs;
+
+final class Invokable
+{
+    public function __invoke()
+    {
+        // Inteiontionally empty.
+    }
+}

test/Porter/Options/TestOptions.php renamed to test/Stubs/TestOptions.php

@@ -1,5 +1,5 @@
 <?php
-namespace ScriptFUSIONTest\Porter\Options;
+namespace ScriptFUSIONTest\Stubs;
 
 use ScriptFUSION\Porter\Options\EncapsulatedOptions;
 

test/Unit/Porter/ImportSpecificationTest.php

@@ -6,6 +6,7 @@
 use ScriptFUSION\Porter\Specification\DuplicateTransformerException;
 use ScriptFUSION\Porter\Specification\ImportSpecification;
 use ScriptFUSION\Porter\Transform\Transformer;
+use ScriptFUSIONTest\Stubs\Invokable;
 
 final class ImportSpecificationTest extends \PHPUnit_Framework_TestCase
 {
@@ -26,7 +27,10 @@ public function testClone()
     {
         $this->specification
             ->addTransformer(\Mockery::mock(Transformer::class))
-            ->setContext($context = (object)[]);
+            ->setContext($context = (object)[])
+            ->setFetchExceptionHandler($handler = new Invokable)
+        ;
+
         $specification = clone $this->specification;
 
         self::assertNotSame($this->resource, $specification->getResource());
@@ -42,6 +46,7 @@ public function testClone()
         self::assertCount(count($this->specification->getTransformers()), $specification->getTransformers());
 
         self::assertNotSame($context, $specification->getContext());
+        self::assertNotSame($handler, $specification->getFetchExceptionHandler());
     }
 
     public function testProviderData()
@@ -79,10 +84,10 @@ public function testAddTransformers()
 
     public function testAddSameTransformer()
     {
-        $this->specification->addTransformer($transformer1 = \Mockery::mock(Transformer::class));
+        $this->specification->addTransformer($transformer = \Mockery::mock(Transformer::class));
 
         $this->setExpectedException(DuplicateTransformerException::class);
-        $this->specification->addTransformer($transformer1);
+        $this->specification->addTransformer($transformer);
     }
 
     public function testContext()
@@ -97,4 +102,34 @@ public function testCacheAdvice()
             $this->specification->setCacheAdvice($advice)->getCacheAdvice()
         );
     }
+
+    /**
+     * @dataProvider provideFetchAttempts
+     */
+    public function testMaxFetchAttempts($input, $output)
+    {
+        self::assertSame($output, $this->specification->setMaxFetchAttempts($input)->getMaxFetchAttempts());
+    }
+
+    public function provideFetchAttempts()
+    {
+        return [
+            // Valid.
+            [1, 1],
+            [2, 2],
+
+            // Invalid.
+            'Too low, positive' => [0, 1],
+            'Too low, negative' => [-1, 1],
+            'Float in range' => [1.9, 1],
+        ];
+    }
+
+    public function testExceptionHandler()
+    {
+        self::assertSame(
+            $handler = new Invokable,
+            $this->specification->setFetchExceptionHandler($handler)->getFetchExceptionHandler()
+        );
+    }
 }

test/Unit/Porter/Options/EncapsulatedOptionsTest.php

@@ -1,7 +1,7 @@
 <?php
 namespace ScriptFUSIONTest\Unit\Porter\Options;
 
-use ScriptFUSIONTest\Porter\Options\TestOptions;
+use ScriptFUSIONTest\Stubs\TestOptions;
 
 final class EncapsulatedOptionsTest extends \PHPUnit_Framework_TestCase
 {