mirror of
https://github.com/FriendsOfTYPO3/tea.git
synced 2024-11-22 21:36:13 +01:00
eeda862e77
The `Product` namespace segment in the domain model namespace `TTN\Tea\Domain\Model` currently serves no purpose and only adds confusion. So let's simplify the extension structure accordingly. (I intended to use this to demonstrate DDD contexts, but never built enough models in the Tea extension for this to actually make sense.) Fixes #1008
439 lines
12 KiB
PHP
439 lines
12 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace TTN\Tea\Tests\Unit\Controller;
|
|
|
|
use PHPUnit\Framework\MockObject\MockObject;
|
|
use TTN\Tea\Controller\FrontEndEditorController;
|
|
use TTN\Tea\Domain\Model\Tea;
|
|
use TTN\Tea\Domain\Repository\TeaRepository;
|
|
use TYPO3\CMS\Core\Context\Context;
|
|
use TYPO3\CMS\Core\Context\UserAspect;
|
|
use TYPO3\CMS\Core\Http\HtmlResponse;
|
|
use TYPO3\CMS\Core\Http\RedirectResponse;
|
|
use TYPO3\CMS\Core\Information\Typo3Version;
|
|
use TYPO3\CMS\Core\Utility\GeneralUtility;
|
|
use TYPO3\CMS\Extbase\Mvc\Controller\ActionController;
|
|
use TYPO3\CMS\Extbase\Mvc\Exception\StopActionException;
|
|
use TYPO3\CMS\Extbase\Persistence\QueryResultInterface;
|
|
use TYPO3\CMS\Fluid\View\TemplateView;
|
|
use TYPO3\TestingFramework\Core\AccessibleObjectInterface;
|
|
use TYPO3\TestingFramework\Core\Unit\UnitTestCase;
|
|
|
|
/**
|
|
* @covers \TTN\Tea\Controller\FrontEndEditorController
|
|
*/
|
|
final class FrontEndEditorControllerTest extends UnitTestCase
|
|
{
|
|
/**
|
|
* @var FrontEndEditorController&MockObject&AccessibleObjectInterface
|
|
*/
|
|
private FrontEndEditorController $subject;
|
|
|
|
/**
|
|
* @var TemplateView&MockObject
|
|
*/
|
|
private TemplateView $viewMock;
|
|
|
|
private Context $context;
|
|
|
|
/**
|
|
* @var TeaRepository&MockObject
|
|
*/
|
|
private TeaRepository $teaRepositoryMock;
|
|
|
|
protected function setUp(): void
|
|
{
|
|
parent::setUp();
|
|
|
|
$this->context = new Context();
|
|
$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'];
|
|
if ((new Typo3Version())->getMajorVersion() < 12) {
|
|
$methodsToMock[] = 'forward';
|
|
}
|
|
$this->subject = $this->getAccessibleMock(
|
|
FrontEndEditorController::class,
|
|
$methodsToMock,
|
|
[$this->context, $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);
|
|
}
|
|
|
|
protected function tearDown(): void
|
|
{
|
|
// empty FIFO queue
|
|
GeneralUtility::makeInstance(Tea::class);
|
|
|
|
parent::tearDown();
|
|
}
|
|
|
|
/**
|
|
* @param int<0, max> $userUid
|
|
*/
|
|
private function setUidOfLoggedInUser(int $userUid): void
|
|
{
|
|
$userAspectMock = $this->createMock(UserAspect::class);
|
|
$userAspectMock->method('get')->with('id')->willReturn($userUid);
|
|
$this->context->setAspect('frontend.user', $userAspectMock);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function isActionController(): void
|
|
{
|
|
self::assertInstanceOf(ActionController::class, $this->subject);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function indexActionForNoLoggedInUserAssignsNothingToView(): void
|
|
{
|
|
$this->setUidOfLoggedInUser(0);
|
|
|
|
$this->viewMock->expects(self::never())->method('assign');
|
|
|
|
$this->subject->indexAction();
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function indexActionForLoggedInUserAssignsTeasOwnedByTheLoggedInUserToView(): void
|
|
{
|
|
$userUid = 5;
|
|
$this->setUidOfLoggedInUser($userUid);
|
|
|
|
$teas = $this->createStub(QueryResultInterface::class);
|
|
$this->teaRepositoryMock->method('findByOwnerUid')->with($userUid)->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 editActionWithOwnTeaAssignsProvidedTeaToView(): void
|
|
{
|
|
$userUid = 5;
|
|
$this->setUidOfLoggedInUser($userUid);
|
|
$tea = new Tea();
|
|
$tea->setOwnerUid($userUid);
|
|
|
|
$this->viewMock->expects(self::once())->method('assign')->with('tea', $tea);
|
|
|
|
$this->subject->editAction($tea);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function editActionWithTeaFromOtherUserThrowsException(): void
|
|
{
|
|
$this->setUidOfLoggedInUser(1);
|
|
$tea = new Tea();
|
|
$tea->setOwnerUid(2);
|
|
|
|
$this->expectException(\RuntimeException::class);
|
|
$this->expectExceptionMessage('You do not have the permissions to edit this tea.');
|
|
$this->expectExceptionCode(1687363749);
|
|
|
|
$this->subject->editAction($tea);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function editActionWithTeaWithoutOwnerThrowsException(): void
|
|
{
|
|
$this->setUidOfLoggedInUser(1);
|
|
$tea = new Tea();
|
|
$tea->setOwnerUid(0);
|
|
|
|
$this->expectException(\RuntimeException::class);
|
|
$this->expectExceptionMessage('You do not have the permissions to edit this tea.');
|
|
$this->expectExceptionCode(1687363749);
|
|
|
|
$this->subject->editAction($tea);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function editActionForOwnTeaReturnsHtmlResponse(): void
|
|
{
|
|
$userUid = 5;
|
|
$this->setUidOfLoggedInUser($userUid);
|
|
$tea = new Tea();
|
|
$tea->setOwnerUid($userUid);
|
|
|
|
$result = $this->subject->editAction($tea);
|
|
|
|
self::assertInstanceOf(HtmlResponse::class, $result);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function updateActionWithOwnTeaPersistsProvidedTea(): void
|
|
{
|
|
$userUid = 5;
|
|
$this->setUidOfLoggedInUser($userUid);
|
|
$tea = new Tea();
|
|
$tea->setOwnerUid($userUid);
|
|
$this->stubRedirect('index');
|
|
|
|
$this->teaRepositoryMock->expects(self::once())->method('update')->with($tea);
|
|
|
|
$this->subject->updateAction($tea);
|
|
}
|
|
|
|
private function mockRedirect(string $actionName): void
|
|
{
|
|
if ((new Typo3Version())->getMajorVersion() < 12) {
|
|
$this->subject->expects(self::once())->method('redirect')
|
|
->with($actionName)
|
|
// @phpstan-ignore-next-line This class does not exist in V12 anymore, but this branch is V11-only.
|
|
->willThrowException(new StopActionException('redirectToUri', 1476045828));
|
|
// @phpstan-ignore-next-line This class does not exist in V12 anymore, but this branch is V11-only.
|
|
$this->expectException(StopActionException::class);
|
|
} else {
|
|
$redirectResponse = $this->createStub(RedirectResponse::class);
|
|
$this->subject->expects(self::once())->method('redirect')->with($actionName)
|
|
->willReturn($redirectResponse);
|
|
}
|
|
}
|
|
|
|
private function stubRedirect(string $actionName): void
|
|
{
|
|
if ((new Typo3Version())->getMajorVersion() < 12) {
|
|
$this->subject->method('redirect')
|
|
// @phpstan-ignore-next-line This class does not exist in V12 anymore, but this branch is V11-only.
|
|
->willThrowException(new StopActionException('redirectToUri', 1476045828));
|
|
// @phpstan-ignore-next-line This class does not exist in V12 anymore, but this branch is V11-only.
|
|
$this->expectException(StopActionException::class);
|
|
} else {
|
|
$redirectResponse = $this->createStub(RedirectResponse::class);
|
|
$this->subject->method('redirect')->willReturn($redirectResponse);
|
|
}
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function updateActionWithOwnTeaRedirectsToIndexAction(): void
|
|
{
|
|
$userUid = 5;
|
|
$this->setUidOfLoggedInUser($userUid);
|
|
$tea = new Tea();
|
|
$tea->setOwnerUid($userUid);
|
|
|
|
$this->mockRedirect('index');
|
|
|
|
$this->subject->updateAction($tea);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function updateActionWithTeaFromOtherUserThrowsException(): void
|
|
{
|
|
$this->setUidOfLoggedInUser(1);
|
|
$tea = new Tea();
|
|
$tea->setOwnerUid(2);
|
|
|
|
$this->expectException(\RuntimeException::class);
|
|
$this->expectExceptionMessage('You do not have the permissions to edit this tea.');
|
|
$this->expectExceptionCode(1687363749);
|
|
|
|
$this->subject->updateAction($tea);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function updateActionWithTeaWithoutOwnerThrowsException(): void
|
|
{
|
|
$this->setUidOfLoggedInUser(1);
|
|
$tea = new Tea();
|
|
$tea->setOwnerUid(0);
|
|
|
|
$this->expectException(\RuntimeException::class);
|
|
$this->expectExceptionMessage('You do not have the permissions to edit this tea.');
|
|
$this->expectExceptionCode(1687363749);
|
|
|
|
$this->subject->updateAction($tea);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function newActionWithTeaAssignsProvidedTeaToView(): void
|
|
{
|
|
$tea = new Tea();
|
|
|
|
$this->viewMock->expects(self::once())->method('assign')->with('tea', $tea);
|
|
|
|
$this->subject->newAction($tea);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function newActionWithNullTeaAssignsProvidedNewTeaToView(): void
|
|
{
|
|
$tea = new Tea();
|
|
GeneralUtility::addInstance(Tea::class, $tea);
|
|
|
|
$this->viewMock->expects(self::once())->method('assign')->with('tea', $tea);
|
|
|
|
$this->subject->newAction(null);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function newActionWithoutTeaAssignsProvidedNewTeaToView(): void
|
|
{
|
|
$tea = new Tea();
|
|
GeneralUtility::addInstance(Tea::class, $tea);
|
|
|
|
$this->viewMock->expects(self::once())->method('assign')->with('tea', $tea);
|
|
|
|
$this->subject->newAction();
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function newActionReturnsHtmlResponse(): void
|
|
{
|
|
$result = $this->subject->newAction();
|
|
|
|
self::assertInstanceOf(HtmlResponse::class, $result);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function createActionSetsLoggedInUserAsOwnerOfProvidedTea(): void
|
|
{
|
|
$userUid = 5;
|
|
$this->setUidOfLoggedInUser($userUid);
|
|
$tea = new Tea();
|
|
$this->stubRedirect('index');
|
|
|
|
$this->subject->createAction($tea);
|
|
|
|
self::assertSame($userUid, $tea->getOwnerUid());
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function createActionPersistsProvidedTea(): void
|
|
{
|
|
$tea = new Tea();
|
|
$this->stubRedirect('index');
|
|
|
|
$this->teaRepositoryMock->expects(self::once())->method('add')->with($tea);
|
|
|
|
$this->subject->createAction($tea);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function createActionRedirectsToIndexAction(): void
|
|
{
|
|
$tea = new Tea();
|
|
|
|
$this->mockRedirect('index');
|
|
|
|
$this->subject->updateAction($tea);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function deleteActionWithOwnTeaRemovesProvidedTea(): void
|
|
{
|
|
$userUid = 5;
|
|
$this->setUidOfLoggedInUser($userUid);
|
|
$tea = new Tea();
|
|
$tea->setOwnerUid($userUid);
|
|
$this->stubRedirect('index');
|
|
|
|
$this->teaRepositoryMock->expects(self::once())->method('remove')->with($tea);
|
|
|
|
$this->subject->deleteAction($tea);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function deleteActionWithOwnTeaRedirectsToIndexAction(): void
|
|
{
|
|
$userUid = 5;
|
|
$this->setUidOfLoggedInUser($userUid);
|
|
$tea = new Tea();
|
|
$tea->setOwnerUid($userUid);
|
|
|
|
$this->mockRedirect('index');
|
|
|
|
$this->subject->deleteAction($tea);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function deleteActionWithTeaFromOtherUserThrowsException(): void
|
|
{
|
|
$this->setUidOfLoggedInUser(1);
|
|
$tea = new Tea();
|
|
$tea->setOwnerUid(2);
|
|
|
|
$this->expectException(\RuntimeException::class);
|
|
$this->expectExceptionMessage('You do not have the permissions to edit this tea.');
|
|
$this->expectExceptionCode(1687363749);
|
|
|
|
$this->subject->deleteAction($tea);
|
|
}
|
|
|
|
/**
|
|
* @test
|
|
*/
|
|
public function deleteActionWithTeaWithoutOwnerThrowsException(): void
|
|
{
|
|
$this->setUidOfLoggedInUser(1);
|
|
$tea = new Tea();
|
|
$tea->setOwnerUid(0);
|
|
|
|
$this->expectException(\RuntimeException::class);
|
|
$this->expectExceptionMessage('You do not have the permissions to edit this tea.');
|
|
$this->expectExceptionCode(1687363749);
|
|
|
|
$this->subject->deleteAction($tea);
|
|
}
|
|
}
|