feat(Platform): remove return types from test methods to satisfy CI rule

Author: qbejs

src/platform/tests/Factory/ProviderConfigFactoryTest.php

@@ -23,7 +23,7 @@
 #[CoversClass(ProviderConfigFactory::class)]
 class ProviderConfigFactoryTest extends TestCase
 {
-    public function testOpenAiDefaults(): void
+    public function testOpenAiDefaults()
     {
         $cfg = ProviderConfigFactory::fromDsn(
             'ai+openai://[email protected]?model=gpt-4o-mini&organization=org_123&headers[x-foo]=bar'
@@ -37,15 +37,15 @@ public function testOpenAiDefaults(): void
         $this->assertSame('bar', $cfg->headers['x-foo'] ?? null);
     }
 
-    public function testOpenAiWithoutHostUsesDefault(): void
+    public function testOpenAiWithoutHostUsesDefault()
     {
         $cfg = ProviderConfigFactory::fromDsn('ai+openai://sk-test@/?model=gpt-4o-mini');
 
         $this->assertSame('https://api.openai.com', $cfg->baseUri);
         $this->assertSame('gpt-4o-mini', $cfg->options['model'] ?? null);
     }
 
-    public function testAzureOpenAiHappyPath(): void
+    public function testAzureOpenAiHappyPath()
     {
         $cfg = ProviderConfigFactory::fromDsn(
             'ai+azure://[email protected]?deployment=gpt-4o&version=2024-08-01-preview&engine=openai'
@@ -59,7 +59,7 @@ public function testAzureOpenAiHappyPath(): void
         $this->assertSame('openai', $cfg->options['engine'] ?? null);
     }
 
-    public function testAzureMetaHappyPath(): void
+    public function testAzureMetaHappyPath()
     {
         $cfg = ProviderConfigFactory::fromDsn(
             'ai+azure://[email protected]?deployment=llama-3.1&version=2024-08-01-preview&engine=meta'
@@ -71,7 +71,7 @@ public function testAzureMetaHappyPath(): void
         $this->assertSame('llama-3.1', $cfg->options['deployment'] ?? null);
     }
 
-    public function testGenericOptionsAndBooleans(): void
+    public function testGenericOptionsAndBooleans()
     {
         $cfg = ProviderConfigFactory::fromDsn(
             'ai+openai://sk@/?model=gpt-4o-mini&timeout=10&verify_peer=true&proxy=http://proxy:8080'
@@ -82,31 +82,31 @@ public function testGenericOptionsAndBooleans(): void
         $this->assertSame('http://proxy:8080', $cfg->options['proxy'] ?? null);
     }
 
-    public function testUnknownProviderThrows(): void
+    public function testUnknownProviderThrows()
     {
         $this->expectException(\InvalidArgumentException::class);
         ProviderConfigFactory::fromDsn('ai+unknown://key@host');
     }
 
-    public function testAzureMissingHostThrows(): void
+    public function testAzureMissingHostThrows()
     {
         $this->expectException(\InvalidArgumentException::class);
         ProviderConfigFactory::fromDsn('ai+azure://AZ_KEY@/?deployment=gpt-4o&version=2024-08-01-preview');
     }
 
-    public function testAzureMissingDeploymentThrows(): void
+    public function testAzureMissingDeploymentThrows()
     {
         $this->expectException(\InvalidArgumentException::class);
         ProviderConfigFactory::fromDsn('ai+azure://[email protected]?version=2024-08-01-preview');
     }
 
-    public function testAzureMissingVersionThrows(): void
+    public function testAzureMissingVersionThrows()
     {
         $this->expectException(\InvalidArgumentException::class);
         ProviderConfigFactory::fromDsn('ai+azure://[email protected]?deployment=gpt-4o');
     }
 
-    public function testAzureUnsupportedEngineThrows(): void
+    public function testAzureUnsupportedEngineThrows()
     {
         $this->expectException(\InvalidArgumentException::class);
         ProviderConfigFactory::fromDsn(

src/platform/tests/Factory/ProviderFactoryTest.php

@@ -30,7 +30,7 @@ protected function tearDown(): void
         AzureMetaBridge::$lastArgs = [];
     }
 
-    public function testBuildsOpenAiWithBearerAuth(): void
+    public function testBuildsOpenAiWithBearerAuth()
     {
         $factory = new ProviderFactory();
 
@@ -49,7 +49,7 @@ public function testBuildsOpenAiWithBearerAuth(): void
         $this->assertArrayNotHasKey('api-key', $headers);
     }
 
-    public function testBuildsAzureOpenAiWithApiKeyHeader(): void
+    public function testBuildsAzureOpenAiWithApiKeyHeader()
     {
         $factory = new ProviderFactory();
 
@@ -73,7 +73,7 @@ public function testBuildsAzureOpenAiWithApiKeyHeader(): void
         $this->assertArrayNotHasKey('Authorization', $headers);
     }
 
-    public function testBuildsAzureMetaWhenEngineMeta(): void
+    public function testBuildsAzureMetaWhenEngineMeta()
     {
         $factory = new ProviderFactory();
 
@@ -95,15 +95,15 @@ public function testBuildsAzureMetaWhenEngineMeta(): void
         $this->assertSame('AZ', $headers['api-key'] ?? null);
     }
 
-    public function testUnsupportedProviderThrows(): void
+    public function testUnsupportedProviderThrows()
     {
         $this->expectException(\InvalidArgumentException::class);
 
         $factory = new ProviderFactory();
         $factory->fromDsn('ai+madeup://[email protected]');
     }
 
-    public function testAzureMissingDeploymentOrVersionBubblesUp(): void
+    public function testAzureMissingDeploymentOrVersionBubblesUp()
     {
         $this->expectException(\InvalidArgumentException::class);