fix: use GET instead of HEAD to speed up connection test

Interestingly, a HEAD to a JobData table (like done in the connection test) last approx > 15 sec whereas a GET ~ 1 sec. Therefore, we use GET again.
jobrouter · Sep 3, 2024 · f82fd72 · f82fd72
1 parent 66043ca
commit f82fd72
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Classes/Controller/TableTestController.php b/Classes/Controller/TableTestController.php
@@ -59,7 +59,7 @@ public function __invoke(ServerRequestInterface $request): ResponseInterface
             }
 
             $this->restClientFactory->create($connection)->request(
-                'HEAD',
+                'GET',
                 \sprintf('application/jobdata/tables/%s/datasets', $table->tableGuid),
             );
             return $this->buildResponse();

diff --git a/Tests/Unit/Controller/TableTestControllerTest.php b/Tests/Unit/Controller/TableTestControllerTest.php
@@ -172,7 +172,7 @@ public function invokeReturnsResponseWithErrorWhenExceptionIsThrown(): void
 
         $this->clientStub
             ->method('request')
-            ->with('HEAD', 'application/jobdata/tables/sometableguid/datasets')
+            ->with('GET', 'application/jobdata/tables/sometableguid/datasets')
             ->willThrowException(new \Exception('some exception message'));
         $this->restClientFactoryMock
             ->method('create')