[Fix] SubstituteBindings to use findOneBy() (#346)

* fix substituteImplicitBindings using findOneBy()

The function findBy() will bring back an array of results, when, for the implicit bind to work, only one entity is expected.

* fix substituteImplicitBindings using findOneBy()

Tests fixes

Author: rafaelucena

src/Middleware/SubstituteBindings.php

@@ -71,7 +71,7 @@ protected function substituteImplicitBindings(Route $route)
                 if ($parameter->getClass()->implementsInterface(UrlRoutable::class)) {
                     $name = call_user_func([$class, 'getRouteKeyName']);
 
-                    $entity = $repository->findBy([
+                    $entity = $repository->findOneBy([
                         $name => $id
                     ]);
                 } else {

tests/Middleware/SubstituteBindingsTest.php

@@ -161,7 +161,7 @@ public function test_not_id_binding()
         $entity       = new BindableEntityWithInterface();
         $entity->id   = 1;
         $entity->name = 'NAMEVALUE';
-        $this->repository->shouldReceive('findBy')->once()->with(['name' => 'NAMEVALUE'])->andReturn($entity);
+        $this->repository->shouldReceive('findOneBy')->with(['name' => 'NAMEVALUE'])->andReturn($entity);
 
         $this->assertEquals(1, $router->dispatch(Request::create('foo/NAMEVALUE', 'GET'))->getContent());
     }