Skip to content

Commit 8dd978c

Browse files
author
Mohamed Khaled
committed
Fix PHPCS style violations
Fix code style issues in Registry integration: • Fix multi-line function declaration formatting • Fix MockModel file header issues • Fix newline endings for all interface files • Remove trailing whitespace from all files All PHPCS errors resolved, warnings remain for other test files.
1 parent 840b84d commit 8dd978c

36 files changed

+529
-530
lines changed

src/Providers/AiProviderRegistry.php

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -194,8 +194,11 @@ public function findProviderModelsMetadataForSupport(
194194
* @return ModelInterface The configured model instance.
195195
* @throws InvalidArgumentException If provider or model is not found.
196196
*/
197-
public function getProviderModel(string $idOrClassName, string $modelId, ?ModelConfig $modelConfig = null): ModelInterface
198-
{
197+
public function getProviderModel(
198+
string $idOrClassName,
199+
string $modelId,
200+
?ModelConfig $modelConfig = null
201+
): ModelInterface {
199202
$className = $this->resolveProviderClassName($idOrClassName);
200203

201204
// Use static method from ProviderInterface

src/Providers/Contracts/ModelMetadataDirectoryInterface.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,4 +46,4 @@ public function hasModelMetadata(string $modelId): bool;
4646
* @throws InvalidArgumentException If model metadata not found.
4747
*/
4848
public function getModelMetadata(string $modelId): ModelMetadata;
49-
}
49+
}

src/Providers/Contracts/ProviderAvailabilityInterface.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,4 +22,4 @@ interface ProviderAvailabilityInterface
2222
* @return bool True if the provider is configured and available, false otherwise.
2323
*/
2424
public function isConfigured(): bool;
25-
}
25+
}

src/Providers/Contracts/ProviderInterface.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,4 +57,4 @@ public static function availability(): ProviderAvailabilityInterface;
5757
* @return ModelMetadataDirectoryInterface Model metadata directory.
5858
*/
5959
public static function modelMetadataDirectory(): ModelMetadataDirectoryInterface;
60-
}
60+
}

src/Providers/Models/Contracts/ModelInterface.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,4 +43,4 @@ public function getConfig(): ModelConfig;
4343
* @param ModelConfig $config The model configuration.
4444
*/
4545
public function setConfig(ModelConfig $config): void;
46-
}
46+
}

tests/traits/ArrayTransformationTestTrait.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ protected function assertArrayRoundTrip($original, callable $assertCallback): vo
5353
$array = $original->toArray();
5454
$className = get_class($original);
5555
$restored = $className::fromArray($array);
56-
56+
5757
$this->assertInstanceOf($className, $restored, 'fromArray() should return instance of ' . $className);
5858
$assertCallback($original, $restored);
5959
}
@@ -85,4 +85,4 @@ protected function assertArrayNotHasKeys(array $array, array $unexpectedKeys): v
8585
$this->assertArrayNotHasKey($key, $array, "Array should not contain key: {$key}");
8686
}
8787
}
88-
}
88+
}

tests/unit/Common/AbstractDataValueObjectTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -554,4 +554,4 @@ public static function getJsonSchema(): array
554554
$this->assertIsArray($testObject::getJsonSchema());
555555
$this->assertNotNull($testObject->jsonSerialize());
556556
}
557-
}
557+
}

tests/unit/Files/DTO/FileTest.php

Lines changed: 35 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,9 @@ public function testCreateFromUrl(): void
2323
{
2424
$url = 'https://example.com/image.jpg';
2525
$mimeType = 'image/jpeg';
26-
26+
2727
$file = new File($url, $mimeType);
28-
28+
2929
$this->assertEquals(FileTypeEnum::remote(), $file->getFileType());
3030
$this->assertEquals($url, $file->getUrl());
3131
$this->assertNull($file->getBase64Data());
@@ -42,9 +42,9 @@ public function testCreateFromUrl(): void
4242
public function testCreateFromUrlWithInferredMimeType(): void
4343
{
4444
$url = 'https://example.com/document.pdf';
45-
45+
4646
$file = new File($url);
47-
47+
4848
$this->assertEquals(FileTypeEnum::remote(), $file->getFileType());
4949
$this->assertEquals($url, $file->getUrl());
5050
$this->assertEquals('application/pdf', $file->getMimeType());
@@ -60,9 +60,9 @@ public function testCreateFromDataUri(): void
6060
{
6161
$base64Data = 'SGVsbG8gV29ybGQ=';
6262
$dataUri = 'data:text/plain;base64,' . $base64Data;
63-
63+
6464
$file = new File($dataUri);
65-
65+
6666
$this->assertEquals(FileTypeEnum::inline(), $file->getFileType());
6767
$this->assertNull($file->getUrl());
6868
$this->assertEquals($base64Data, $file->getBase64Data());
@@ -81,9 +81,9 @@ public function testCreateFromDataUriWithMimeTypeOverride(): void
8181
$base64Data = 'SGVsbG8gV29ybGQ=';
8282
$dataUri = 'data:text/plain;base64,' . $base64Data;
8383
$overrideMimeType = 'text/html';
84-
84+
8585
$file = new File($dataUri, $overrideMimeType);
86-
86+
8787
$this->assertEquals(FileTypeEnum::inline(), $file->getFileType());
8888
$this->assertEquals($base64Data, $file->getBase64Data());
8989
$this->assertEquals($overrideMimeType, $file->getMimeType());
@@ -99,9 +99,9 @@ public function testCreateFromPlainBase64(): void
9999
{
100100
$base64Data = 'SGVsbG8gV29ybGQ=';
101101
$mimeType = 'text/plain';
102-
102+
103103
$file = new File($base64Data, $mimeType);
104-
104+
105105
$this->assertEquals(FileTypeEnum::inline(), $file->getFileType());
106106
$this->assertNull($file->getUrl());
107107
$this->assertEquals($base64Data, $file->getBase64Data());
@@ -118,7 +118,7 @@ public function testPlainBase64WithoutMimeTypeThrowsException(): void
118118
{
119119
$this->expectException(InvalidArgumentException::class);
120120
$this->expectExceptionMessage('MIME type is required when providing plain base64 data without data URI format.');
121-
121+
122122
new File('SGVsbG8gV29ybGQ=');
123123
}
124124

@@ -132,10 +132,10 @@ public function testCreateFromLocalFile(): void
132132
// Create a temporary file
133133
$tempFile = tempnam(sys_get_temp_dir(), 'test');
134134
file_put_contents($tempFile, 'Hello World');
135-
135+
136136
try {
137137
$file = new File($tempFile, 'text/plain');
138-
138+
139139
$this->assertEquals(FileTypeEnum::inline(), $file->getFileType());
140140
$this->assertNull($file->getUrl());
141141
$this->assertEquals(base64_encode('Hello World'), $file->getBase64Data());
@@ -154,7 +154,7 @@ public function testInvalidFileFormatThrowsException(): void
154154
{
155155
$this->expectException(InvalidArgumentException::class);
156156
$this->expectExceptionMessage('Invalid file provided. Expected URL, base64 data, or valid local file path.');
157-
157+
158158
new File('not-a-valid-file-or-url', 'text/plain');
159159
}
160160

@@ -167,7 +167,7 @@ public function testNonExistentLocalFileThrowsException(): void
167167
{
168168
$this->expectException(InvalidArgumentException::class);
169169
$this->expectExceptionMessage('Invalid file provided. Expected URL, base64 data, or valid local file path.');
170-
170+
171171
new File('/path/to/non/existent/file.txt', 'text/plain');
172172
}
173173

@@ -181,11 +181,11 @@ public function testDirectoryThrowsException(): void
181181
// Create a directory instead of a file
182182
$tempDir = sys_get_temp_dir() . '/test_dir_' . uniqid();
183183
mkdir($tempDir);
184-
184+
185185
try {
186186
$this->expectException(InvalidArgumentException::class);
187187
$this->expectExceptionMessage('Invalid file provided. Expected URL, base64 data, or valid local file path.');
188-
188+
189189
new File($tempDir, 'text/plain');
190190
} finally {
191191
rmdir($tempDir);
@@ -200,7 +200,7 @@ public function testDirectoryThrowsException(): void
200200
public function testMimeTypeMethods(): void
201201
{
202202
$file = new File('https://example.com/video.mp4');
203-
203+
204204
$this->assertEquals('video/mp4', $file->getMimeType());
205205
$this->assertInstanceOf(\WordPress\AiClient\Files\ValueObjects\MimeType::class, $file->getMimeTypeObject());
206206
$this->assertTrue($file->isVideo());
@@ -217,20 +217,20 @@ public function testMimeTypeMethods(): void
217217
public function testJsonSchema(): void
218218
{
219219
$schema = File::getJsonSchema();
220-
220+
221221
$this->assertIsArray($schema);
222222
$this->assertEquals('object', $schema['type']);
223223
$this->assertArrayHasKey('oneOf', $schema);
224224
$this->assertCount(2, $schema['oneOf']);
225-
225+
226226
// Check remote file schema
227227
$remoteSchema = $schema['oneOf'][0];
228228
$this->assertArrayHasKey('properties', $remoteSchema);
229229
$this->assertArrayHasKey(File::KEY_FILE_TYPE, $remoteSchema['properties']);
230230
$this->assertArrayHasKey(File::KEY_MIME_TYPE, $remoteSchema['properties']);
231231
$this->assertArrayHasKey(File::KEY_URL, $remoteSchema['properties']);
232232
$this->assertEquals([File::KEY_FILE_TYPE, File::KEY_MIME_TYPE, File::KEY_URL], $remoteSchema['required']);
233-
233+
234234
// Check inline file schema
235235
$inlineSchema = $schema['oneOf'][1];
236236
$this->assertArrayHasKey('properties', $inlineSchema);
@@ -249,10 +249,10 @@ public function testDataUriWithoutMimeType(): void
249249
{
250250
$base64Data = 'SGVsbG8gV29ybGQ=';
251251
$dataUri = 'data:;base64,' . $base64Data;
252-
252+
253253
$this->expectException(InvalidArgumentException::class);
254254
$this->expectExceptionMessage('Unable to determine MIME type. Please provide it explicitly.');
255-
255+
256256
new File($dataUri);
257257
}
258258

@@ -265,7 +265,7 @@ public function testUrlWithUnknownExtension(): void
265265
{
266266
$this->expectException(InvalidArgumentException::class);
267267
$this->expectExceptionMessage('Unable to determine MIME type. Please provide it explicitly.');
268-
268+
269269
new File('https://example.com/file.unknown');
270270
}
271271

@@ -278,7 +278,7 @@ public function testToArrayRemoteFile(): void
278278
{
279279
$file = new File('https://example.com/image.jpg', 'image/jpeg');
280280
$json = $file->toArray();
281-
281+
282282
$this->assertIsArray($json);
283283
$this->assertEquals(\WordPress\AiClient\Files\Enums\FileTypeEnum::remote()->value, $json[File::KEY_FILE_TYPE]);
284284
$this->assertEquals('image/jpeg', $json[File::KEY_MIME_TYPE]);
@@ -297,7 +297,7 @@ public function testToArrayInlineFile(): void
297297
$dataUri = 'data:text/plain;base64,' . $base64Data;
298298
$file = new File($dataUri);
299299
$json = $file->toArray();
300-
300+
301301
$this->assertIsArray($json);
302302
$this->assertEquals(\WordPress\AiClient\Files\Enums\FileTypeEnum::inline()->value, $json[File::KEY_FILE_TYPE]);
303303
$this->assertEquals('text/plain', $json[File::KEY_MIME_TYPE]);
@@ -317,9 +317,9 @@ public function testFromArrayRemoteFile(): void
317317
File::KEY_MIME_TYPE => 'image/png',
318318
File::KEY_URL => 'https://example.com/test.png'
319319
];
320-
320+
321321
$file = File::fromArray($json);
322-
322+
323323
$this->assertInstanceOf(File::class, $file);
324324
$this->assertTrue($file->getFileType()->isRemote());
325325
$this->assertEquals('image/png', $file->getMimeType());
@@ -340,9 +340,9 @@ public function testFromArrayInlineFile(): void
340340
File::KEY_MIME_TYPE => 'text/plain',
341341
File::KEY_BASE64_DATA => $base64Data
342342
];
343-
343+
344344
$file = File::fromArray($json);
345-
345+
346346
$this->assertInstanceOf(File::class, $file);
347347
$this->assertTrue($file->getFileType()->isInline());
348348
$this->assertEquals('text/plain', $file->getMimeType());
@@ -361,17 +361,17 @@ public function testArrayRoundTrip(): void
361361
$remoteFile = new File('https://example.com/doc.pdf', 'application/pdf');
362362
$remoteJson = $remoteFile->toArray();
363363
$restoredRemote = File::fromArray($remoteJson);
364-
364+
365365
$this->assertEquals($remoteFile->getFileType()->value, $restoredRemote->getFileType()->value);
366366
$this->assertEquals($remoteFile->getMimeType(), $restoredRemote->getMimeType());
367367
$this->assertEquals($remoteFile->getUrl(), $restoredRemote->getUrl());
368-
368+
369369
// Test inline file
370370
$dataUri = 'data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7';
371371
$inlineFile = new File($dataUri);
372372
$inlineJson = $inlineFile->toArray();
373373
$restoredInline = File::fromArray($inlineJson);
374-
374+
375375
$this->assertEquals($inlineFile->getFileType()->value, $restoredInline->getFileType()->value);
376376
$this->assertEquals($inlineFile->getMimeType(), $restoredInline->getMimeType());
377377
$this->assertEquals($inlineFile->getBase64Data(), $restoredInline->getBase64Data());
@@ -385,11 +385,10 @@ public function testArrayRoundTrip(): void
385385
public function testImplementsWithArrayTransformationInterface(): void
386386
{
387387
$file = new File('https://example.com/test.jpg');
388-
388+
389389
$this->assertInstanceOf(
390390
\WordPress\AiClient\Common\Contracts\WithArrayTransformationInterface::class,
391391
$file
392392
);
393-
394393
}
395-
}
394+
}

tests/unit/Files/Enums/FileTypeEnumTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,4 +53,4 @@ public function testSpecificEnumMethods(): void
5353
$this->assertTrue($remote->isRemote());
5454
$this->assertFalse($remote->isInline());
5555
}
56-
}
56+
}

0 commit comments

Comments
 (0)