Moved Integration test to their own directory

Renamed the Abstract Integration class Refactored existing test currently one test is failing for LogoutHandlerTest Signed-off-by: Joey Smith <[email protected]> Signed-off-by: Joey Smith <[email protected]>
webinertia · Aug 28, 2023 · 3f8795a · 3f8795a
1 parent 3242eb5
commit 3f8795a
Show file tree

Hide file tree

Showing 8 changed files with 101 additions and 53 deletions.
diff --git a/...est/Storage/PageRepositoryFactoryTest.php → ...Integration/PageRepositoryFactoryTest.php b/...est/Storage/PageRepositoryFactoryTest.php → ...Integration/PageRepositoryFactoryTest.php
@@ -2,14 +2,13 @@
 
 declare(strict_types=1);
 
-namespace PageManagerTest\Storage;
+namespace PageManagerTest\Storage\Integration;
 
-use Laminas\Db\Adapter\AdapterInterface;
 use PageManager\Storage\PageRepository;
 use PageManager\Storage\PageRepositoryFactory;
-use Test\AbtractIntegrationTestCase;
+use Test\Integration\AbstractTestCase;
 
-final class PageRepositoryFactoryTest extends AbtractIntegrationTestCase
+final class PageRepositoryFactoryTest extends AbstractTestCase
 {
     protected function setUp(): void
     {

diff --git a/...anagerTest/Storage/PageRepositoryTest.php → ...torage/Integration/PageRepositoryTest.php b/...anagerTest/Storage/PageRepositoryTest.php → ...torage/Integration/PageRepositoryTest.php
@@ -2,13 +2,13 @@
 
 declare(strict_types=1);
 
-namespace PageManagerTest\Storage;
+namespace PageManagerTest\Storage\Integration;
 
 use PageManager\Storage\PageEntity;
 use PageManager\Storage\PageRepository;
-use Test\AbtractIntegrationTestCase;
+use Test\Integration\AbstractTestCase;
 
-final class PageRepositoryTest extends AbtractIntegrationTestCase
+final class PageRepositoryTest extends AbstractTestCase
 {
     protected function setUp(): void
     {

diff --git a/...age/SavePageCommandHandlerFactoryTest.php → ...ion/SavePageCommandHandlerFactoryTest.php b/...age/SavePageCommandHandlerFactoryTest.php → ...ion/SavePageCommandHandlerFactoryTest.php
@@ -2,13 +2,13 @@
 
 declare(strict_types=1);
 
-namespace PageManagerTest\Storage;
+namespace PageManagerTest\Storage\Integration;
 
 use PageManager\Storage\SavePageCommandHandler;
 use PageManager\Storage\SavePageCommandHandlerFactory as Factory;
-use Test\AbtractIntegrationTestCase;
+use Test\Integration\AbstractTestCase;
 
-final class SavePageCommandHandlerFactoryTest extends AbtractIntegrationTestCase
+final class SavePageCommandHandlerFactoryTest extends AbstractTestCase
 {
     protected function setUp(): void
     {

diff --git a/...st/Storage/SavePageCommandHandlerTest.php → ...ntegration/SavePageCommandHandlerTest.php b/...st/Storage/SavePageCommandHandlerTest.php → ...ntegration/SavePageCommandHandlerTest.php
@@ -1,20 +1,15 @@
 <?php
 
-/**
- * Test Type: Integration
- * todo: needs mocked componets as this test is kinda dirty
- */
-
 declare(strict_types=1);
 
-namespace PageManagerTest\Storage;
+namespace PageManagerTest\Storage\Integration;
 
 use League\Tactician\CommandBus;
 use PageManager\Storage\PageEntity;
 use PageManager\Storage\SavePageCommand;
-use Test\AbtractIntegrationTestCase;
+use Test\Integration\AbstractTestCase;
 
-final class SavePageCommandHandlerTest extends AbtractIntegrationTestCase
+final class SavePageCommandHandlerTest extends AbstractTestCase
 {
     /** @var CommandBus */
     protected $commandBus;

diff --git a/test/Test/AbtractIntegrationTestCase.php b/test/Test/AbtractIntegrationTestCase.php
diff --git a/test/Test/Integration/AbstractTestCase.php b/test/Test/Integration/AbstractTestCase.php
@@ -0,0 +1,44 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Test\Integration;
+
+use Mezzio\Application;
+use Mezzio\MiddlewareFactory;
+use PHPUnit\Framework\TestCase;
+use Psr\Container\ContainerInterface;
+
+abstract class AbstractTestCase extends TestCase
+{
+    /** @var Application $app */
+    protected $app;
+    /** @var ContainerInterface */
+    protected $container;
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+        $this->initContainer();
+        // $this->initApp();
+        // $this->initPipeline();
+    }
+
+    protected function initContainer(): void
+    {
+        $this->container = require __DIR__ . '/../../../config/container.php';
+    }
+
+    protected function initApp(): void
+    {
+        /** @var Application */
+        $this->app = $this->container->get(Application::class);
+    }
+
+    protected function initPipeline(): void
+    {
+        /** @var MiddlewareFactory */
+        $factory = $this->container->get(MiddlewareFactory::class);
+        (require __DIR__ . '/../../../config/pipeline.php')($this->app, $factory, $this->container);
+    }
+}
diff --git a/test/UserManagerTest/Handler/LogoutHandlerTest.php b/test/UserManagerTest/Handler/LogoutHandlerTest.php
diff --git a/test/UserManagerTest/Integration/LogoutHandlerTest.php b/test/UserManagerTest/Integration/LogoutHandlerTest.php
@@ -0,0 +1,45 @@
+<?php
+
+declare(strict_types=1);
+
+namespace UserManagerTest\Integration;
+
+use Laminas\Diactoros\Response;
+use Laminas\Diactoros\ServerRequest;
+use Laminas\Diactoros\Uri;
+use Mezzio\Authentication\UserInterface;
+use PHPUnit\Framework\MockObject\MockObject;
+use Test\Integration\AbstractTestCase;
+
+/**
+ *
+ * @runTestsInSeparateProcesses
+ * @preserveGlobalState disabled
+ */
+final class LogoutHandlerTest extends AbstractTestCase
+{
+    protected function setUp(): void
+    {
+        parent::setUp();
+        $this->initApp();
+        $this->initPipeline();
+    }
+
+    public function testUserIsLoggedOut(): void
+    {
+        $sessionData = [
+            'username' => 'jsmith',
+            'roles' => [
+                'Administrator',
+            ],
+        ];
+        $_SESSION[UserInterface::class] = $sessionData;
+        $uri = new Uri('/logout');
+        $request = new ServerRequest([], [], $uri);
+        $request->withAttribute(UserInterface::class, $sessionData);
+        /** @var Response */
+        $response = $this->app->handle($request);
+        $this->assertEquals(302, $response->getStatusCode());
+        $this->assertEquals('/user/login', $response->getHeaderLine('Location'));
+    }
+}