Skip to content

Commit b22192b

Browse files
committed
Update tests
1 parent 1ae5af1 commit b22192b

File tree

3 files changed

+24
-21
lines changed

3 files changed

+24
-21
lines changed

Diff for: src/Scout/ScoutEngine.php

+6-7
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
use MongoDB\BSON\UTCDateTime;
1818
use MongoDB\Collection as MongoDBCollection;
1919
use MongoDB\Database;
20-
use MongoDB\Driver\Cursor;
2120
use MongoDB\Driver\CursorInterface;
2221
use MongoDB\Exception\RuntimeException as MongoDBRuntimeException;
2322
use MongoDB\Laravel\Connection;
@@ -311,9 +310,9 @@ public function mapIds($results): Collection
311310
*
312311
* @see Engine::map()
313312
*
314-
* @param Builder $builder
315-
* @param array|Cursor $results
316-
* @param Model $model
313+
* @param Builder $builder
314+
* @param array $results
315+
* @param Model $model
317316
*
318317
* @return Collection
319318
*/
@@ -328,9 +327,9 @@ public function map(Builder $builder, $results, $model): Collection
328327
*
329328
* @see Engine::lazyMap()
330329
*
331-
* @param Builder $builder
332-
* @param array|Cursor $results
333-
* @param Model $model
330+
* @param Builder $builder
331+
* @param array $results
332+
* @param Model $model
334333
*
335334
* @return LazyCollection
336335
*/

Diff for: tests/Scout/ScoutEngineTest.php

+15-11
Original file line numberDiff line numberDiff line change
@@ -266,7 +266,7 @@ function () {
266266
'$search' => [
267267
'compound' => [
268268
'filter' => [
269-
['equals' => ['path' => '__soft_deleted', 'value' => 0]],
269+
['equals' => ['path' => '__soft_deleted', 'value' => false]],
270270
],
271271
],
272272
],
@@ -286,7 +286,7 @@ function () {
286286
'$search' => [
287287
'compound' => [
288288
'filter' => [
289-
['equals' => ['path' => '__soft_deleted', 'value' => 1]],
289+
['equals' => ['path' => '__soft_deleted', 'value' => true]],
290290
],
291291
],
292292
],
@@ -493,20 +493,24 @@ public function testUpdateWithSoftDelete(): void
493493
->andReturn($collection);
494494
$collection->shouldReceive('bulkWrite')
495495
->once()
496-
->with([
497-
[
498-
'updateOne' => [
499-
['_id' => 'key_1'],
500-
['$setOnInsert' => ['_id' => 'key_1'], '$set' => ['id' => 1]],
501-
['upsert' => true],
496+
->withArgs(function ($pipeline) {
497+
$this->assertSame([
498+
[
499+
'updateOne' => [
500+
['_id' => 'key_1'],
501+
['$setOnInsert' => ['_id' => 'key_1'], '$set' => ['id' => 1, '__soft_deleted' => false]],
502+
['upsert' => true],
503+
],
502504
],
503-
],
504-
]);
505+
], $pipeline);
506+
507+
return true;
508+
});
505509

506510
$model = new SearchableModel(['id' => 1]);
507511
$model->delete();
508512

509-
$engine = new ScoutEngine($database, softDelete: false);
513+
$engine = new ScoutEngine($database, softDelete: true);
510514
$engine->update(EloquentCollection::make([$model]));
511515
}
512516

Diff for: tests/Scout/ScoutIntegrationTest.php

+3-3
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,8 @@ public function setUp(): void
4242
{
4343
parent::setUp();
4444

45+
$this->skipIfSearchIndexManagementIsNotSupported();
46+
4547
// Init the SQL database with some objects that will be indexed
4648
// Test data copied from Laravel Scout tests
4749
// https://github.com/laravel/scout/blob/10.x/tests/Integration/SearchableTests.php
@@ -88,8 +90,6 @@ public function setUp(): void
8890
/** This test create the search index for tests performing search */
8991
public function testItCanCreateTheCollection()
9092
{
91-
$this->skipIfSearchIndexManagementIsNotSupported();
92-
9393
$collection = DB::connection('mongodb')->getCollection('prefix_scout_users');
9494
$collection->drop();
9595

@@ -111,7 +111,7 @@ public function testItCanCreateTheCollection()
111111
['$search' => ['index' => 'scout', 'exists' => ['path' => 'name']]],
112112
])->toArray();
113113

114-
if (count($indexedDocuments) > 0) {
114+
if (count($indexedDocuments) >= 44) {
115115
break;
116116
}
117117

0 commit comments

Comments
 (0)