Add spec tests

Author: GromNaN

src/GridFS/Bucket.php

@@ -252,7 +252,11 @@ public function delete(mixed $id)
      */
     public function deleteByName(string $filename): void
     {
-        $this->collectionWrapper->deleteFileAndChunksByFilename($filename);
+        $count = $this->collectionWrapper->deleteFileAndChunksByFilename($filename);
+
+        if ($count === 0) {
+            throw FileNotFoundException::byFilename($filename);
+        }
     }
 
     /**
@@ -672,9 +676,9 @@ public function rename(mixed $id, string $newFilename)
      */
     public function renameByName(string $filename, string $newFilename): void
     {
-        $matchedCount = $this->collectionWrapper->updateFilenameForFilename($filename, $newFilename);
+        $count = $this->collectionWrapper->updateFilenameForFilename($filename, $newFilename);
 
-        if (! $matchedCount) {
+        if ($count === 0) {
             throw FileNotFoundException::byFilename($filename);
         }
     }

src/GridFS/CollectionWrapper.php

@@ -74,7 +74,7 @@ public function deleteChunksByFilesId(mixed $id): void
     /**
      * Delete all GridFS files and chunks for a given filename.
      */
-    public function deleteFileAndChunksByFilename(string $filename): ?int
+    public function deleteFileAndChunksByFilename(string $filename): int
     {
         /** @var iterable<array{_id: mixed}> $files */
         $files = $this->findFiles(['filename' => $filename], [

tests/GridFS/BucketFunctionalTest.php

@@ -182,17 +182,10 @@ public function testDeleteByName(): void
         $this->assertCollectionCount($this->chunksCollection, 0);
     }
 
-    public function testDeleteByNameShouldIgnoreNonexistentFiles(): void
+    public function testDeleteByNameShouldRequireFileToExist(): void
     {
-        $this->bucket->uploadFromStream('filename', self::createStream('foobar'));
-
-        $this->assertCollectionCount($this->filesCollection, 1);
-        $this->assertCollectionCount($this->chunksCollection, 1);
-
-        $this->bucket->deleteByName('nonexistent-filename');
-
-        $this->assertCollectionCount($this->filesCollection, 1);
-        $this->assertCollectionCount($this->chunksCollection, 1);
+        $this->expectException(FileNotFoundException::class);
+        $this->bucket->deleteByName('nonexistent-name');
     }
 
     public function testDownloadingFileWithMissingChunk(): void

tests/UnifiedSpecTests/Operation.php

@@ -789,6 +789,12 @@ private function executeForBucket(Bucket $bucket)
 
                 return $bucket->delete($args['id']);
 
+            case 'deleteByName':
+                assertArrayHasKey('filename', $args);
+                assertIsString($args['filename']);
+
+                return $bucket->deleteByName($args['filename']);
+
             case 'downloadByName':
                 assertArrayHasKey('filename', $args);
                 assertIsString($args['filename']);
@@ -803,6 +809,21 @@ private function executeForBucket(Bucket $bucket)
 
                 return stream_get_contents($bucket->openDownloadStream($args['id']));
 
+            case 'rename':
+                assertArrayHasKey('id', $args);
+                assertArrayHasKey('newFilename', $args);
+                assertIsString($args['newFilename']);
+
+                return $bucket->rename($args['id'], $args['newFilename']);
+
+            case 'renameByName':
+                assertArrayHasKey('filename', $args);
+                assertArrayHasKey('newFilename', $args);
+                assertIsString($args['filename']);
+                assertIsString($args['newFilename']);
+
+                return $bucket->renameByName($args['filename'], $args['newFilename']);
+
             case 'uploadWithId':
                 assertArrayHasKey('id', $args);
                 $args['_id'] = $args['id'];

tests/UnifiedSpecTests/Util.php

@@ -132,8 +132,11 @@ final class Util
         ],
         Bucket::class => [
             'delete' => ['id'],
+            'deleteByName' => ['filename'],
             'downloadByName' => ['filename', 'revision'],
             'download' => ['id'],
+            'rename' => ['id', 'newFilename'],
+            'renameByName' => ['filename', 'newFilename'],
             'uploadWithId' => ['id', 'filename', 'source', 'chunkSizeBytes', 'disableMD5', 'contentType', 'metadata'],
             'upload' => ['filename', 'source', 'chunkSizeBytes', 'disableMD5', 'contentType', 'metadata'],
         ],