Merge pull request #221 from open-runtimes/fix-make-executor-test-non-final

fix: make executor test non final

Author: loks0n

rector.php

@@ -3,6 +3,7 @@
 declare(strict_types=1);
 
 use Rector\Config\RectorConfig;
+use Rector\Privatization\Rector\Class_\FinalizeTestCaseClassRector;
 
 return RectorConfig::configure()
     ->withPaths([
@@ -11,6 +12,9 @@
         __DIR__ . '/tests'
     ])
     ->withSkipPath(__DIR__ . '/tests/resources')
+    ->withSkip([
+        FinalizeTestCaseClassRector::class,
+    ])
     ->withPhpSets(php84: true)
     ->withComposerBased(phpunit: true)
     ->withPreparedSets(

tests/e2e/ExecutorTest.php

@@ -13,7 +13,7 @@
 // TODO: Lengthy log test
 // TODO: Lengthy body test
 
-final class ExecutorTest extends TestCase
+class ExecutorTest extends TestCase
 {
     protected string $endpoint = 'http://executor/v1';
 
@@ -393,8 +393,8 @@ public function testExecute(): void
         $this->assertEquals(201, $response['headers']['status-code']);
 
         $response = $this->client->call(Client::METHOD_POST, '/runtimes/test-exec/executions');
-        $this->assertEquals(200, $response['headers']['status-code']);
-        $this->assertEquals(200, $response['body']['statusCode']);
+        $this->assertEquals(200, $response['headers']['status-code'], 'Failed to execute runtime, response ' . json_encode($response, JSON_PRETTY_PRINT));
+        $this->assertEquals(200, $response['body']['statusCode'], 'Failed execute runtime, response ' . json_encode($response, JSON_PRETTY_PRINT));
         $this->assertEquals('aValue', \json_decode((string) $response['body']['headers'], true)['x-key']);
 
         /** Execute on cold-started runtime */