test(graphql): @stream tests fix for "laravel/scout:>=10.11.8".

LastDragon-ru · Nov 28, 2024 · 6bd5c78 · 6bd5c78
1 parent 3d81688
commit 6bd5c78
Show file tree

Hide file tree

Showing 2 changed files with 55 additions and 19 deletions.
diff --git a/packages/graphql/src/Stream/Streams/ScoutTest.php b/packages/graphql/src/Stream/Streams/ScoutTest.php
@@ -75,22 +75,40 @@ public function testGetItems(): void {
 
         self::assertEquals($expected, [...$items]);
         self::assertQueryLogEquals(
-            [
-                [
-                    'query'    => 'select count(*) as aggregate from "test_objects"',
-                    'bindings' => [],
+            match ((new RequiresLaravelScout('>=10.11.8'))->isSatisfied()) {
+                true  => [
+                    [
+                        'query'    => 'select count(*) as aggregate from "test_objects"',
+                        'bindings' => [],
+                    ],
+                    [
+                        'query'    => <<<SQL
+                        select *
+                        from "test_objects"
+                        order by "id" asc, "test_objects"."id" desc
+                        limit {$limit} offset {$offset}
+                    SQL
+                        ,
+                        'bindings' => [],
+                    ],
                 ],
-                [
-                    'query'    => <<<SQL
+                false => [
+                    [
+                        'query'    => 'select count(*) as aggregate from "test_objects"',
+                        'bindings' => [],
+                    ],
+                    [
+                        'query'    => <<<SQL
                         select *
                         from "test_objects"
                         order by "id" asc, "id" desc
                         limit {$limit} offset {$offset}
                     SQL
-                    ,
-                    'bindings' => [],
+                        ,
+                        'bindings' => [],
+                    ],
                 ],
-            ],
+            },
             $query,
         );
     }
@@ -111,22 +129,40 @@ public function testGetLength(): void {
 
         self::assertEquals($count, $length);
         self::assertQueryLogEquals(
-            [
-                [
-                    'query'    => 'select count(*) as aggregate from "test_objects"',
-                    'bindings' => [],
+            match ((new RequiresLaravelScout('>=10.11.8'))->isSatisfied()) {
+                true  => [
+                    [
+                        'query'    => 'select count(*) as aggregate from "test_objects"',
+                        'bindings' => [],
+                    ],
+                    [
+                        'query'    => <<<'SQL'
+                        select *
+                        from "test_objects"
+                        order by "id" asc, "test_objects"."id" desc
+                        limit 1 offset 0
+                    SQL
+                        ,
+                        'bindings' => [],
+                    ],
                 ],
-                [
-                    'query'    => <<<'SQL'
+                false => [
+                    [
+                        'query'    => 'select count(*) as aggregate from "test_objects"',
+                        'bindings' => [],
+                    ],
+                    [
+                        'query'    => <<<'SQL'
                         select *
                         from "test_objects"
                         order by "id" asc, "id" desc
                         limit 1 offset 0
                     SQL
-                    ,
-                    'bindings' => [],
+                        ,
+                        'bindings' => [],
+                    ],
                 ],
-            ],
+            },
             $query,
         );
     }

diff --git a/phpstan-larastan.neon b/phpstan-larastan.neon
@@ -267,7 +267,7 @@ services:
             - phpstan.broker.dynamicFunctionReturnTypeExtension
 
     -
-        class: Larastan\Larastan\ReturnTypes\CollectionFilterDynamicReturnTypeExtension
+        class: Larastan\Larastan\ReturnTypes\CollectionFilterRejectDynamicReturnTypeExtension
         tags:
             - phpstan.broker.dynamicMethodReturnTypeExtension