Merge pull request #92 from captn3m0/fix_mock_strategy

Fix mock client strategy

Author: dbu

CHANGELOG.md

@@ -3,6 +3,8 @@
 
 ## Unreleased
 
+- MockClientStategy class fixed
+
 ## 1.2.0 - 2017-02-12
 
 ### Added

spec/Strategy/MockClientStrategySpec.php

@@ -0,0 +1,25 @@
+<?php
+
+namespace spec\Http\Discovery\Strategy;
+
+use Http\Client\HttpClient;
+use Http\Discovery\ClassDiscovery;
+use Http\Discovery\Strategy\DiscoveryStrategy;
+use Http\Discovery\Strategy;
+use PhpSpec\ObjectBehavior;
+use spec\Http\Discovery\Helper\DiscoveryHelper;
+
+class MockClientStrategySpec extends ObjectBehavior
+{
+    function let()
+    {
+        ClassDiscovery::setStrategies([Strategy\MockClientStrategy::class]);
+    }
+
+    function it_should_return_the_mock_client(DiscoveryStrategy $strategy)
+    {
+        $candidates = $this->getCandidates(HttpClient::class);
+        $candidates->shouldBeArray();
+        $candidates->shouldHaveCount(1);
+    }
+}

src/Strategy/MockClientStrategy.php

@@ -18,7 +18,7 @@ final class MockClientStrategy implements DiscoveryStrategy
     public static function getCandidates($type)
     {
         return ($type === HttpClient::class)
-            ? ['class' => Mock::class, 'condition' => Mock::class]
+            ? [['class' => Mock::class, 'condition' => Mock::class]]
             : [];
     }
 }