Skip to content
This repository was archived by the owner on Jan 21, 2020. It is now read-only.

Commit af99c8b

Browse files
committed
Don't pass the requested name to the factory
The requested name is not used and it was wrong anyway. It should have been the FQN of the handler and not the container. Also it should be passed as the second argument as the third one is for options.
1 parent dbb3eb6 commit af99c8b

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

test/AppTest/Handler/HomePageHandlerFactoryTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ public function testFactoryWithoutTemplate()
3232

3333
$this->assertInstanceOf(HomePageHandlerFactory::class, $factory);
3434

35-
$homePage = $factory($this->container->reveal(), null, get_class($this->container->reveal()));
35+
$homePage = $factory($this->container->reveal());
3636

3737
$this->assertInstanceOf(HomePageHandler::class, $homePage);
3838
}
@@ -46,7 +46,7 @@ public function testFactoryWithTemplate()
4646

4747
$factory = new HomePageHandlerFactory();
4848

49-
$homePage = $factory($this->container->reveal(), null, get_class($this->container->reveal()));
49+
$homePage = $factory($this->container->reveal());
5050

5151
$this->assertInstanceOf(HomePageHandler::class, $homePage);
5252
}

0 commit comments

Comments
 (0)