[TASK] Remove no longer needed Controller unit test (#1644)

Using unit tests for controller is not considered best practice.
We already have functional tests in place and can therefore remove the
unit test.

Related: #1565

Tests/Unit/Controller/TeaControllerTest.php

@@ -1,107 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace TTN\Tea\Tests\Unit\Controller;
-
-use PHPUnit\Framework\MockObject\MockObject;
-use TTN\Tea\Controller\TeaController;
-use TTN\Tea\Domain\Model\Tea;
-use TTN\Tea\Domain\Repository\TeaRepository;
-use TYPO3\CMS\Core\Http\HtmlResponse;
-use TYPO3\CMS\Extbase\Mvc\Controller\ActionController;
-use TYPO3\CMS\Extbase\Persistence\QueryResultInterface;
-use TYPO3\CMS\Fluid\View\TemplateView;
-use TYPO3\TestingFramework\Core\AccessibleObjectInterface;
-use TYPO3\TestingFramework\Core\Unit\UnitTestCase;
-
-/**
- * Note: Unit tests for controllers are not considered best practice anymore. Instead, functional tests should be used.
- * We're currently in the process of migrating all controller tests to functional tests.
- *
- * @covers \TTN\Tea\Controller\TeaController
- */
-final class TeaControllerTest extends UnitTestCase
-{
-    /**
-     * @var TeaController&MockObject&AccessibleObjectInterface
-     */
-    private TeaController $subject;
-
-    /**
-     * @var TemplateView&MockObject
-     */
-    private TemplateView $viewMock;
-
-    /**
-     * @var TeaRepository&MockObject
-     */
-    private TeaRepository $teaRepositoryMock;
-
-    protected function setUp(): void
-    {
-        parent::setUp();
-
-        $this->teaRepositoryMock = $this->createMock(TeaRepository::class);
-        // We need to create an accessible mock in order to be able to set the protected `view`.
-        $methodsToMock = ['htmlResponse', 'redirect', 'redirectToUri'];
-        $this->subject = $this->getAccessibleMock(TeaController::class, $methodsToMock, [$this->teaRepositoryMock]);
-
-        $this->viewMock = $this->createMock(TemplateView::class);
-        $this->subject->_set('view', $this->viewMock);
-
-        $responseStub = $this->createStub(HtmlResponse::class);
-        $this->subject->method('htmlResponse')->willReturn($responseStub);
-    }
-
-    /**
-     * @test
-     */
-    public function isActionController(): void
-    {
-        self::assertInstanceOf(ActionController::class, $this->subject);
-    }
-
-    /**
-     * @test
-     */
-    public function indexActionAssignsAllTeaAsTeasToView(): void
-    {
-        $teas = $this->createStub(QueryResultInterface::class);
-        $this->teaRepositoryMock->method('findAll')->willReturn($teas);
-        $this->viewMock->expects(self::once())->method('assign')->with('teas', $teas);
-
-        $this->subject->indexAction();
-    }
-
-    /**
-     * @test
-     */
-    public function indexActionReturnsHtmlResponse(): void
-    {
-        $result = $this->subject->indexAction();
-
-        self::assertInstanceOf(HtmlResponse::class, $result);
-    }
-
-    /**
-     * @test
-     */
-    public function showActionAssignsPassedTeaAsTeaToView(): void
-    {
-        $tea = new Tea();
-        $this->viewMock->expects(self::once())->method('assign')->with('tea', $tea);
-
-        $this->subject->showAction($tea);
-    }
-
-    /**
-     * @test
-     */
-    public function showActionAssignsReturnsHtmlResponse(): void
-    {
-        $result = $this->subject->showAction(new Tea());
-
-        self::assertInstanceOf(HtmlResponse::class, $result);
-    }
-}