123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593 |
- <?php
- /**
- * SPDX-FileCopyrightText: 2016-2024 Nextcloud GmbH and Nextcloud contributors
- * SPDX-FileCopyrightText: 2016 ownCloud, Inc.
- * SPDX-License-Identifier: AGPL-3.0-or-later
- */
- namespace Test\AppFramework\Http;
- use OC\AppFramework\Http\Dispatcher;
- use OC\AppFramework\Http\Request;
- use OC\AppFramework\Middleware\MiddlewareDispatcher;
- use OC\AppFramework\Utility\ControllerMethodReflector;
- use OCP\AppFramework\Controller;
- use OCP\AppFramework\Http;
- use OCP\AppFramework\Http\DataResponse;
- use OCP\AppFramework\Http\JSONResponse;
- use OCP\AppFramework\Http\ParameterOutOfRangeException;
- use OCP\AppFramework\Http\Response;
- use OCP\Diagnostics\IEventLogger;
- use OCP\IConfig;
- use OCP\IRequest;
- use OCP\IRequestId;
- use PHPUnit\Framework\MockObject\MockObject;
- use Psr\Container\ContainerInterface;
- use Psr\Log\LoggerInterface;
- class TestController extends Controller {
- /**
- * @param string $appName
- * @param \OCP\IRequest $request
- */
- public function __construct($appName, $request) {
- parent::__construct($appName, $request);
- }
- /**
- * @param int $int
- * @param bool $bool
- * @param double $foo
- * @param int $test
- * @param integer $test2
- * @return array
- */
- public function exec($int, $bool, $foo, $test = 4, $test2 = 1) {
- $this->registerResponder('text', function ($in) {
- return new JSONResponse(['text' => $in]);
- });
- return [$int, $bool, $test, $test2];
- }
- /**
- * @param int $int
- * @param bool $bool
- * @param int $test
- * @param int $test2
- * @return DataResponse
- */
- public function execDataResponse($int, $bool, $test = 4, $test2 = 1) {
- return new DataResponse([
- 'text' => [$int, $bool, $test, $test2]
- ]);
- }
- }
- /**
- * Class DispatcherTest
- *
- * @package Test\AppFramework\Http
- * @group DB
- */
- class DispatcherTest extends \Test\TestCase {
- /** @var MiddlewareDispatcher */
- private $middlewareDispatcher;
- /** @var Dispatcher */
- private $dispatcher;
- private $controllerMethod;
- /** @var Controller|MockObject */
- private $controller;
- private $response;
- /** @var IRequest|MockObject */
- private $request;
- private $lastModified;
- private $etag;
- /** @var Http|MockObject */
- private $http;
- private $reflector;
- /** @var IConfig|MockObject */
- private $config;
- /** @var LoggerInterface|MockObject */
- private $logger;
- /** @var IEventLogger|MockObject */
- private $eventLogger;
- /** @var ContainerInterface|MockObject */
- private $container;
- protected function setUp(): void {
- parent::setUp();
- $this->controllerMethod = 'test';
- $this->config = $this->createMock(IConfig::class);
- $this->logger = $this->createMock(LoggerInterface::class);
- $this->eventLogger = $this->createMock(IEventLogger::class);
- $this->container = $this->createMock(ContainerInterface::class);
- $app = $this->getMockBuilder(
- 'OC\AppFramework\DependencyInjection\DIContainer')
- ->disableOriginalConstructor()
- ->getMock();
- $request = $this->getMockBuilder(
- '\OC\AppFramework\Http\Request')
- ->disableOriginalConstructor()
- ->getMock();
- $this->http = $this->getMockBuilder(
- \OC\AppFramework\Http::class)
- ->disableOriginalConstructor()
- ->getMock();
- $this->middlewareDispatcher = $this->getMockBuilder(
- '\OC\AppFramework\Middleware\MiddlewareDispatcher')
- ->disableOriginalConstructor()
- ->getMock();
- $this->controller = $this->getMockBuilder(
- '\OCP\AppFramework\Controller')
- ->setMethods([$this->controllerMethod])
- ->setConstructorArgs([$app, $request])
- ->getMock();
- $this->request = $this->getMockBuilder(
- '\OC\AppFramework\Http\Request')
- ->disableOriginalConstructor()
- ->getMock();
- $this->reflector = new ControllerMethodReflector();
- $this->dispatcher = new Dispatcher(
- $this->http,
- $this->middlewareDispatcher,
- $this->reflector,
- $this->request,
- $this->config,
- \OC::$server->getDatabaseConnection(),
- $this->logger,
- $this->eventLogger,
- $this->container,
- );
- $this->response = $this->createMock(Response::class);
- $this->lastModified = new \DateTime('now', new \DateTimeZone('GMT'));
- $this->etag = 'hi';
- }
- /**
- * @param string $out
- * @param string $httpHeaders
- */
- private function setMiddlewareExpectations($out = null,
- $httpHeaders = null, $responseHeaders = [],
- $ex = false, $catchEx = true) {
- if ($ex) {
- $exception = new \Exception();
- $this->middlewareDispatcher->expects($this->once())
- ->method('beforeController')
- ->with($this->equalTo($this->controller),
- $this->equalTo($this->controllerMethod))
- ->will($this->throwException($exception));
- if ($catchEx) {
- $this->middlewareDispatcher->expects($this->once())
- ->method('afterException')
- ->with($this->equalTo($this->controller),
- $this->equalTo($this->controllerMethod),
- $this->equalTo($exception))
- ->willReturn($this->response);
- } else {
- $this->middlewareDispatcher->expects($this->once())
- ->method('afterException')
- ->with($this->equalTo($this->controller),
- $this->equalTo($this->controllerMethod),
- $this->equalTo($exception))
- ->willThrowException($exception);
- return;
- }
- } else {
- $this->middlewareDispatcher->expects($this->once())
- ->method('beforeController')
- ->with($this->equalTo($this->controller),
- $this->equalTo($this->controllerMethod));
- $this->controller->expects($this->once())
- ->method($this->controllerMethod)
- ->willReturn($this->response);
- }
- $this->response->expects($this->once())
- ->method('render')
- ->willReturn($out);
- $this->response->expects($this->once())
- ->method('getStatus')
- ->willReturn(Http::STATUS_OK);
- $this->response->expects($this->once())
- ->method('getHeaders')
- ->willReturn($responseHeaders);
- $this->http->expects($this->once())
- ->method('getStatusHeader')
- ->with($this->equalTo(Http::STATUS_OK))
- ->willReturn($httpHeaders);
- $this->middlewareDispatcher->expects($this->once())
- ->method('afterController')
- ->with($this->equalTo($this->controller),
- $this->equalTo($this->controllerMethod),
- $this->equalTo($this->response))
- ->willReturn($this->response);
- $this->middlewareDispatcher->expects($this->once())
- ->method('afterController')
- ->with($this->equalTo($this->controller),
- $this->equalTo($this->controllerMethod),
- $this->equalTo($this->response))
- ->willReturn($this->response);
- $this->middlewareDispatcher->expects($this->once())
- ->method('beforeOutput')
- ->with($this->equalTo($this->controller),
- $this->equalTo($this->controllerMethod),
- $this->equalTo($out))
- ->willReturn($out);
- }
- public function testDispatcherReturnsArrayWith2Entries(): void {
- $this->setMiddlewareExpectations('');
- $response = $this->dispatcher->dispatch($this->controller, $this->controllerMethod);
- $this->assertNull($response[0]);
- $this->assertEquals([], $response[1]);
- $this->assertNull($response[2]);
- }
- public function testHeadersAndOutputAreReturned(): void {
- $out = 'yo';
- $httpHeaders = 'Http';
- $responseHeaders = ['hell' => 'yeah'];
- $this->setMiddlewareExpectations($out, $httpHeaders, $responseHeaders);
- $response = $this->dispatcher->dispatch($this->controller,
- $this->controllerMethod);
- $this->assertEquals($httpHeaders, $response[0]);
- $this->assertEquals($responseHeaders, $response[1]);
- $this->assertEquals($out, $response[3]);
- }
- public function testExceptionCallsAfterException(): void {
- $out = 'yo';
- $httpHeaders = 'Http';
- $responseHeaders = ['hell' => 'yeah'];
- $this->setMiddlewareExpectations($out, $httpHeaders, $responseHeaders, true);
- $response = $this->dispatcher->dispatch($this->controller,
- $this->controllerMethod);
- $this->assertEquals($httpHeaders, $response[0]);
- $this->assertEquals($responseHeaders, $response[1]);
- $this->assertEquals($out, $response[3]);
- }
- public function testExceptionThrowsIfCanNotBeHandledByAfterException(): void {
- $out = 'yo';
- $httpHeaders = 'Http';
- $responseHeaders = ['hell' => 'yeah'];
- $this->setMiddlewareExpectations($out, $httpHeaders, $responseHeaders, true, false);
- $this->expectException(\Exception::class);
- $this->dispatcher->dispatch(
- $this->controller,
- $this->controllerMethod
- );
- }
- private function dispatcherPassthrough() {
- $this->middlewareDispatcher->expects($this->once())
- ->method('beforeController');
- $this->middlewareDispatcher->expects($this->once())
- ->method('afterController')
- ->willReturnCallback(function ($a, $b, $in) {
- return $in;
- });
- $this->middlewareDispatcher->expects($this->once())
- ->method('beforeOutput')
- ->willReturnCallback(function ($a, $b, $in) {
- return $in;
- });
- }
- public function testControllerParametersInjected(): void {
- $this->request = new Request(
- [
- 'post' => [
- 'int' => '3',
- 'bool' => 'false',
- 'double' => 1.2,
- ],
- 'method' => 'POST'
- ],
- $this->createMock(IRequestId::class),
- $this->createMock(IConfig::class)
- );
- $this->dispatcher = new Dispatcher(
- $this->http, $this->middlewareDispatcher, $this->reflector,
- $this->request,
- $this->config,
- \OC::$server->getDatabaseConnection(),
- $this->logger,
- $this->eventLogger,
- $this->container
- );
- $controller = new TestController('app', $this->request);
- // reflector is supposed to be called once
- $this->dispatcherPassthrough();
- $response = $this->dispatcher->dispatch($controller, 'exec');
- $this->assertEquals('[3,false,4,1]', $response[3]);
- }
- public function testControllerParametersInjectedDefaultOverwritten(): void {
- $this->request = new Request(
- [
- 'post' => [
- 'int' => '3',
- 'bool' => 'false',
- 'double' => 1.2,
- 'test2' => 7
- ],
- 'method' => 'POST',
- ],
- $this->createMock(IRequestId::class),
- $this->createMock(IConfig::class)
- );
- $this->dispatcher = new Dispatcher(
- $this->http, $this->middlewareDispatcher, $this->reflector,
- $this->request,
- $this->config,
- \OC::$server->getDatabaseConnection(),
- $this->logger,
- $this->eventLogger,
- $this->container
- );
- $controller = new TestController('app', $this->request);
- // reflector is supposed to be called once
- $this->dispatcherPassthrough();
- $response = $this->dispatcher->dispatch($controller, 'exec');
- $this->assertEquals('[3,false,4,7]', $response[3]);
- }
- public function testResponseTransformedByUrlFormat(): void {
- $this->request = new Request(
- [
- 'post' => [
- 'int' => '3',
- 'bool' => 'false',
- 'double' => 1.2,
- ],
- 'urlParams' => [
- 'format' => 'text'
- ],
- 'method' => 'GET'
- ],
- $this->createMock(IRequestId::class),
- $this->createMock(IConfig::class)
- );
- $this->dispatcher = new Dispatcher(
- $this->http, $this->middlewareDispatcher, $this->reflector,
- $this->request,
- $this->config,
- \OC::$server->getDatabaseConnection(),
- $this->logger,
- $this->eventLogger,
- $this->container
- );
- $controller = new TestController('app', $this->request);
- // reflector is supposed to be called once
- $this->dispatcherPassthrough();
- $response = $this->dispatcher->dispatch($controller, 'exec');
- $this->assertEquals('{"text":[3,false,4,1]}', $response[3]);
- }
- public function testResponseTransformsDataResponse(): void {
- $this->request = new Request(
- [
- 'post' => [
- 'int' => '3',
- 'bool' => 'false',
- 'double' => 1.2,
- ],
- 'urlParams' => [
- 'format' => 'json'
- ],
- 'method' => 'GET'
- ],
- $this->createMock(IRequestId::class),
- $this->createMock(IConfig::class)
- );
- $this->dispatcher = new Dispatcher(
- $this->http, $this->middlewareDispatcher, $this->reflector,
- $this->request,
- $this->config,
- \OC::$server->getDatabaseConnection(),
- $this->logger,
- $this->eventLogger,
- $this->container
- );
- $controller = new TestController('app', $this->request);
- // reflector is supposed to be called once
- $this->dispatcherPassthrough();
- $response = $this->dispatcher->dispatch($controller, 'execDataResponse');
- $this->assertEquals('{"text":[3,false,4,1]}', $response[3]);
- }
- public function testResponseTransformedByAcceptHeader(): void {
- $this->request = new Request(
- [
- 'post' => [
- 'int' => '3',
- 'bool' => 'false',
- 'double' => 1.2,
- ],
- 'server' => [
- 'HTTP_ACCEPT' => 'application/text, test',
- 'HTTP_CONTENT_TYPE' => 'application/x-www-form-urlencoded'
- ],
- 'method' => 'PUT'
- ],
- $this->createMock(IRequestId::class),
- $this->createMock(IConfig::class)
- );
- $this->dispatcher = new Dispatcher(
- $this->http, $this->middlewareDispatcher, $this->reflector,
- $this->request,
- $this->config,
- \OC::$server->getDatabaseConnection(),
- $this->logger,
- $this->eventLogger,
- $this->container
- );
- $controller = new TestController('app', $this->request);
- // reflector is supposed to be called once
- $this->dispatcherPassthrough();
- $response = $this->dispatcher->dispatch($controller, 'exec');
- $this->assertEquals('{"text":[3,false,4,1]}', $response[3]);
- }
- public function testResponseTransformedBySendingMultipartFormData(): void {
- $this->request = new Request(
- [
- 'post' => [
- 'int' => '3',
- 'bool' => 'false',
- 'double' => 1.2,
- ],
- 'server' => [
- 'HTTP_ACCEPT' => 'application/text, test',
- 'HTTP_CONTENT_TYPE' => 'multipart/form-data'
- ],
- 'method' => 'POST'
- ],
- $this->createMock(IRequestId::class),
- $this->createMock(IConfig::class)
- );
- $this->dispatcher = new Dispatcher(
- $this->http, $this->middlewareDispatcher, $this->reflector,
- $this->request,
- $this->config,
- \OC::$server->getDatabaseConnection(),
- $this->logger,
- $this->eventLogger,
- $this->container
- );
- $controller = new TestController('app', $this->request);
- // reflector is supposed to be called once
- $this->dispatcherPassthrough();
- $response = $this->dispatcher->dispatch($controller, 'exec');
- $this->assertEquals('{"text":[3,false,4,1]}', $response[3]);
- }
- public function testResponsePrimarilyTransformedByParameterFormat(): void {
- $this->request = new Request(
- [
- 'post' => [
- 'int' => '3',
- 'bool' => 'false',
- 'double' => 1.2,
- ],
- 'get' => [
- 'format' => 'text'
- ],
- 'server' => [
- 'HTTP_ACCEPT' => 'application/json, test'
- ],
- 'method' => 'POST'
- ],
- $this->createMock(IRequestId::class),
- $this->createMock(IConfig::class)
- );
- $this->dispatcher = new Dispatcher(
- $this->http, $this->middlewareDispatcher, $this->reflector,
- $this->request,
- $this->config,
- \OC::$server->getDatabaseConnection(),
- $this->logger,
- $this->eventLogger,
- $this->container
- );
- $controller = new TestController('app', $this->request);
- // reflector is supposed to be called once
- $this->dispatcherPassthrough();
- $response = $this->dispatcher->dispatch($controller, 'exec');
- $this->assertEquals('{"text":[3,false,4,1]}', $response[3]);
- }
- public function rangeDataProvider(): array {
- return [
- [PHP_INT_MIN, PHP_INT_MAX, 42, false],
- [0, 12, -5, true],
- [-12, 0, 5, true],
- [7, 14, 5, true],
- [7, 14, 10, false],
- [-14, -7, -10, false],
- ];
- }
- /**
- * @dataProvider rangeDataProvider
- */
- public function testEnsureParameterValueSatisfiesRange(int $min, int $max, int $input, bool $throw): void {
- $this->reflector = $this->createMock(ControllerMethodReflector::class);
- $this->reflector->expects($this->any())
- ->method('getRange')
- ->willReturn([
- 'min' => $min,
- 'max' => $max,
- ]);
- $this->dispatcher = new Dispatcher(
- $this->http,
- $this->middlewareDispatcher,
- $this->reflector,
- $this->request,
- $this->config,
- \OC::$server->getDatabaseConnection(),
- $this->logger,
- $this->eventLogger,
- $this->container,
- );
- if ($throw) {
- $this->expectException(ParameterOutOfRangeException::class);
- }
- $this->invokePrivate($this->dispatcher, 'ensureParameterValueSatisfiesRange', ['myArgument', $input]);
- if (!$throw) {
- // do not mark this test risky
- $this->assertTrue(true);
- }
- }
- }
|