diff options
author | Roeland Jago Douma <rullzer@users.noreply.github.com> | 2020-10-04 11:07:49 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-10-04 11:07:49 +0300 |
commit | f4707d178e4d0a85ca63b2e13bf8f897de9a941e (patch) | |
tree | d9e4756b16d5fef6460aab4815dc028fc6479c87 /tests | |
parent | eba83d22bbcf45caf400704b8794acce180c5ba9 (diff) | |
parent | 95a301ea570bfc724877530975297aad1e7536c8 (diff) |
Merge pull request #23047 from nextcloud/techdebt/noid/warn-on-database-abuse
Log the number of queries built and executed
Diffstat (limited to 'tests')
-rw-r--r-- | tests/lib/AppFramework/Http/DispatcherTest.php | 55 |
1 files changed, 46 insertions, 9 deletions
diff --git a/tests/lib/AppFramework/Http/DispatcherTest.php b/tests/lib/AppFramework/Http/DispatcherTest.php index c4c973aec90..e308e5f2b3b 100644 --- a/tests/lib/AppFramework/Http/DispatcherTest.php +++ b/tests/lib/AppFramework/Http/DispatcherTest.php @@ -33,6 +33,9 @@ use OCP\AppFramework\Http\DataResponse; use OCP\AppFramework\Http\JSONResponse; use OCP\AppFramework\Http\Response; use OCP\IConfig; +use OCP\IRequest; +use PHPUnit\Framework\MockObject\MockObject; +use Psr\Log\LoggerInterface; class TestController extends Controller { /** @@ -72,7 +75,12 @@ class TestController extends Controller { } } - +/** + * Class DispatcherTest + * + * @package Test\AppFramework\Http + * @group DB + */ class DispatcherTest extends \Test\TestCase { /** @var MiddlewareDispatcher */ private $middlewareDispatcher; @@ -80,16 +88,24 @@ class DispatcherTest extends \Test\TestCase { private $dispatcher; private $controllerMethod; 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; protected function setUp(): void { parent::setUp(); $this->controllerMethod = 'test'; + $this->config = $this->createMock(IConfig::class); + $this->logger = $this->createMock(LoggerInterface::class); $app = $this->getMockBuilder( 'OC\AppFramework\DependencyInjection\DIContainer') ->disableOriginalConstructor() @@ -99,7 +115,7 @@ class DispatcherTest extends \Test\TestCase { ->disableOriginalConstructor() ->getMock(); $this->http = $this->getMockBuilder( - '\OC\AppFramework\Http') + \OC\AppFramework\Http::class) ->disableOriginalConstructor() ->getMock(); @@ -124,7 +140,10 @@ class DispatcherTest extends \Test\TestCase { $this->http, $this->middlewareDispatcher, $this->reflector, - $this->request + $this->request, + $this->config, + \OC::$server->getDatabaseConnection(), + $this->logger ); $this->response = $this->createMock(Response::class); @@ -299,7 +318,10 @@ class DispatcherTest extends \Test\TestCase { ); $this->dispatcher = new Dispatcher( $this->http, $this->middlewareDispatcher, $this->reflector, - $this->request + $this->request, + $this->config, + \OC::$server->getDatabaseConnection(), + $this->logger ); $controller = new TestController('app', $this->request); @@ -330,7 +352,10 @@ class DispatcherTest extends \Test\TestCase { ); $this->dispatcher = new Dispatcher( $this->http, $this->middlewareDispatcher, $this->reflector, - $this->request + $this->request, + $this->config, + \OC::$server->getDatabaseConnection(), + $this->logger ); $controller = new TestController('app', $this->request); @@ -364,7 +389,10 @@ class DispatcherTest extends \Test\TestCase { ); $this->dispatcher = new Dispatcher( $this->http, $this->middlewareDispatcher, $this->reflector, - $this->request + $this->request, + $this->config, + \OC::$server->getDatabaseConnection(), + $this->logger ); $controller = new TestController('app', $this->request); @@ -397,7 +425,10 @@ class DispatcherTest extends \Test\TestCase { ); $this->dispatcher = new Dispatcher( $this->http, $this->middlewareDispatcher, $this->reflector, - $this->request + $this->request, + $this->config, + \OC::$server->getDatabaseConnection(), + $this->logger ); $controller = new TestController('app', $this->request); @@ -431,7 +462,10 @@ class DispatcherTest extends \Test\TestCase { ); $this->dispatcher = new Dispatcher( $this->http, $this->middlewareDispatcher, $this->reflector, - $this->request + $this->request, + $this->config, + \OC::$server->getDatabaseConnection(), + $this->logger ); $controller = new TestController('app', $this->request); @@ -467,7 +501,10 @@ class DispatcherTest extends \Test\TestCase { ); $this->dispatcher = new Dispatcher( $this->http, $this->middlewareDispatcher, $this->reflector, - $this->request + $this->request, + $this->config, + \OC::$server->getDatabaseConnection(), + $this->logger ); $controller = new TestController('app', $this->request); |