File tree Expand file tree Collapse file tree 6 files changed +8
-10
lines changed
tests/Unit/Subscriptions/Broadcasters Expand file tree Collapse file tree 6 files changed +8
-10
lines changed Original file line number Diff line number Diff line change @@ -25,9 +25,9 @@ final class ASTUnserializationBench
2525 }
2626 GRAPHQL;
2727
28- protected string $ documentNode ;
28+ private string $ documentNode ;
2929
30- protected string $ documentAST ;
30+ private string $ documentAST ;
3131
3232 public function prepareSchema (): void
3333 {
Original file line number Diff line number Diff line change @@ -54,7 +54,7 @@ public function unserialize(string $context): GraphQLContext
5454 $ rawRequest ['server ' ],
5555 $ rawRequest ['content ' ],
5656 );
57- $ request ->setUserResolver (fn () => $ this ->getRestoredPropertyValue ($ rawUser ));
57+ $ request ->setUserResolver (fn (): mixed => $ this ->getRestoredPropertyValue ($ rawUser ));
5858 } else {
5959 $ request = null ;
6060 }
Original file line number Diff line number Diff line change @@ -112,7 +112,7 @@ protected static function applyArgBuilderDirectives(
112112 $ value = $ argument ->toPlain ();
113113
114114 foreach ($ argument ->directives as $ directive ) {
115- if (! ( $ directive instanceof ArgBuilderDirective) ) {
115+ if (! $ directive instanceof ArgBuilderDirective) {
116116 continue ;
117117 }
118118
Original file line number Diff line number Diff line change @@ -43,7 +43,7 @@ public function transform($argumentValue): ?string
4343 return null ;
4444 }
4545
46- if (! ( $ argumentValue instanceof UploadedFile) ) {
46+ if (! $ argumentValue instanceof UploadedFile) {
4747 $ uploadedFileClass = UploadedFile::class;
4848 throw new \InvalidArgumentException ("Expected argument ` {$ this ->nodeName ()}` to be instanceof {$ uploadedFileClass }. " );
4949 }
Original file line number Diff line number Diff line change @@ -73,7 +73,7 @@ public function enhanceBuilder(?callable $directiveFilter = null): ScoutBuilder
7373
7474 foreach ($ this ->argumentsWithScoutBuilderDirectives as $ argument ) {
7575 foreach ($ argument ->directives as $ directive ) {
76- if (! ( $ directive instanceof ScoutBuilderDirective) ) {
76+ if (! $ directive instanceof ScoutBuilderDirective) {
7777 continue ;
7878 }
7979
Original file line number Diff line number Diff line change @@ -50,8 +50,7 @@ public function testBroadcastChannelNameIsNotModified(): void
5050
5151 public function testAuthorized (): void
5252 {
53- $ broadcastManager = $ this ->createMock (BroadcastManager::class);
54- $ redisBroadcaster = new EchoBroadcaster ($ broadcastManager );
53+ $ redisBroadcaster = new EchoBroadcaster ($ this ->createMock (BroadcastManager::class));
5554
5655 $ request = new Request ();
5756 $ request ['channel_name ' ] = 'abc ' ;
@@ -65,8 +64,7 @@ public function testAuthorized(): void
6564
6665 public function testUnauthorized (): void
6766 {
68- $ broadcastManager = $ this ->createMock (BroadcastManager::class);
69- $ redisBroadcaster = new EchoBroadcaster ($ broadcastManager );
67+ $ redisBroadcaster = new EchoBroadcaster ($ this ->createMock (BroadcastManager::class));
7068
7169 $ response = $ redisBroadcaster ->unauthorized (new Request ());
7270 $ this ->assertSame (403 , $ response ->getStatusCode ());
You can’t perform that action at this time.
0 commit comments