ScoutEngine: fix delete() TypeError message to reference EloquentCollection (#3457)

Author: zigzagdev

src/Scout/ScoutEngine.php

@@ -151,7 +151,7 @@ public function update($models)
     #[Override]
     public function delete($models): void
     {
-        assert($models instanceof EloquentCollection, new TypeError(sprintf('Argument #1 ($models) must be of type %s, %s given', Collection::class, get_debug_type($models))));
+        assert($models instanceof EloquentCollection, new TypeError(sprintf('Argument #1 ($models) must be of type %s, %s given', EloquentCollection::class, get_debug_type($models))));
 
         if ($models->isEmpty()) {
             return;

tests/Scout/ScoutEngineTest.php

@@ -21,6 +21,7 @@
 use MongoDB\Laravel\Tests\Scout\Models\SearchableModel;
 use MongoDB\Laravel\Tests\TestCase;
 use PHPUnit\Framework\Attributes\DataProvider;
+use TypeError;
 
 use function array_replace_recursive;
 use function count;
@@ -670,4 +671,17 @@ public function testDeleteWithRemoveableScoutCollection(): void
         $engine = new ScoutEngine($database, softDelete: false);
         $engine->delete($job->models);
     }
+
+    public function testDeleteRejectsNonEloquentCollection(): void
+    {
+        $database = $this->createMock(Database::class);
+        $engine = new ScoutEngine($database, softDelete: false);
+
+        $this->expectException(TypeError::class);
+        $this->expectExceptionMessage(
+            'Argument #1 ($models) must be of type Illuminate\Database\Eloquent\Collection',
+        );
+
+        $engine->delete(LaravelCollection::make([1, 2, 3]));
+    }
 }