From d7f42ddd4dfbd9929767fc7d00937cad2bfe6239 Mon Sep 17 00:00:00 2001
From: Hafez Divandari <hafezdivandari@gmail.com>
Date: Sat, 23 Nov 2024 16:27:05 +0330
Subject: [PATCH] fix tests with notices

---
 .../AuthorizedAccessTokenControllerTest.php   | 13 ++--
 tests/Unit/ClientControllerTest.php           | 60 ++++++++-----------
 .../PersonalAccessTokenControllerTest.php     | 13 ++--
 3 files changed, 34 insertions(+), 52 deletions(-)

diff --git a/tests/Unit/AuthorizedAccessTokenControllerTest.php b/tests/Unit/AuthorizedAccessTokenControllerTest.php
index 042c316a..f53449aa 100644
--- a/tests/Unit/AuthorizedAccessTokenControllerTest.php
+++ b/tests/Unit/AuthorizedAccessTokenControllerTest.php
@@ -99,16 +99,13 @@ public function test_tokens_can_be_deleted()
 
     public function test_not_found_response_is_returned_if_user_doesnt_have_token()
     {
-        $request = Request::create('/', 'GET');
-
-        $this->tokenRepository->shouldReceive('findForUser')->with(3, 1)->andReturnNull();
+        $user = m::mock(Authenticatable::class);
+        $user->shouldReceive('getAuthIdentifier')->andReturn(1);
 
-        $request->setUserResolver(function () {
-            $user = m::mock(Authenticatable::class);
-            $user->shouldReceive('getAuthIdentifier')->andReturn(1);
+        $request = Request::create('/', 'GET');
+        $request->setUserResolver(fn () => $user);
 
-            return $user;
-        });
+        $this->tokenRepository->shouldReceive('findForUser')->with(3, $user)->andReturnNull();
 
         $this->assertSame(404, $this->controller->destroy($request, 3)->status());
     }
diff --git a/tests/Unit/ClientControllerTest.php b/tests/Unit/ClientControllerTest.php
index 55a5f5eb..3e0a8042 100644
--- a/tests/Unit/ClientControllerTest.php
+++ b/tests/Unit/ClientControllerTest.php
@@ -20,13 +20,13 @@ class ClientControllerTest extends TestCase
 
     public function test_all_the_clients_for_the_current_user_can_be_retrieved()
     {
-        $clientRepository = m::mock(ClientRepository::class);
-        $clientRepository->shouldReceive('forUser')->once()->with(1)
-            ->andReturn($clients = (new Client)->newCollection());
-
         $user = m::mock(Authenticatable::class);
         $user->shouldReceive('getAuthIdentifier')->andReturn(1);
 
+        $clientRepository = m::mock(ClientRepository::class);
+        $clientRepository->shouldReceive('forUser')->once()->with($user)
+            ->andReturn($clients = (new Client)->newCollection());
+
         $request = Request::create('/', 'GET');
         $request->setUserResolver(fn () => $user);
 
@@ -114,18 +114,15 @@ public function test_public_clients_can_be_stored()
 
     public function test_clients_can_be_updated()
     {
+        $user = m::mock(Authenticatable::class);
+        $user->shouldReceive('getAuthIdentifier')->andReturn(1);
+
         $clients = m::mock(ClientRepository::class);
         $client = m::mock(Client::class);
-        $clients->shouldReceive('findForUser')->with(1, 1)->andReturn($client);
+        $clients->shouldReceive('findForUser')->with(1, $user)->andReturn($client);
 
         $request = Request::create('/', 'GET', ['name' => 'client name', 'redirect' => 'http://localhost']);
-
-        $request->setUserResolver(function () {
-            $user = m::mock(Authenticatable::class);
-            $user->shouldReceive('getAuthIdentifier')->andReturn(1);
-
-            return $user;
-        });
+        $request->setUserResolver(fn () => $user);
 
         $clients->shouldReceive('update')->once()->with(
             $client, 'client name', ['http://localhost']
@@ -152,17 +149,14 @@ public function test_clients_can_be_updated()
 
     public function test_404_response_if_client_doesnt_belong_to_user()
     {
+        $user = m::mock(Authenticatable::class);
+        $user->shouldReceive('getAuthIdentifier')->andReturn(1);
+
         $clients = m::mock(ClientRepository::class);
-        $clients->shouldReceive('findForUser')->with(1, 1)->andReturnNull();
+        $clients->shouldReceive('findForUser')->with(1, $user)->andReturnNull();
 
         $request = Request::create('/', 'GET', ['name' => 'client name', 'redirect' => 'http://localhost']);
-
-        $request->setUserResolver(function () {
-            $user = m::mock(Authenticatable::class);
-            $user->shouldReceive('getAuthIdentifier')->andReturn(1);
-
-            return $user;
-        });
+        $request->setUserResolver(fn () => $user);
 
         $clients->shouldReceive('update')->never();
 
@@ -177,18 +171,15 @@ public function test_404_response_if_client_doesnt_belong_to_user()
 
     public function test_clients_can_be_deleted()
     {
+        $user = m::mock(Authenticatable::class);
+        $user->shouldReceive('getAuthIdentifier')->andReturn(1);
+
         $clients = m::mock(ClientRepository::class);
         $client = m::mock(Client::class);
-        $clients->shouldReceive('findForUser')->with(1, 1)->andReturn($client);
+        $clients->shouldReceive('findForUser')->with(1, $user)->andReturn($client);
 
         $request = Request::create('/', 'GET', ['name' => 'client name', 'redirect' => 'http://localhost']);
-
-        $request->setUserResolver(function () {
-            $user = m::mock(Authenticatable::class);
-            $user->shouldReceive('getAuthIdentifier')->andReturn(1);
-
-            return $user;
-        });
+        $request->setUserResolver(fn () => $user);
 
         $clients->shouldReceive('delete')->once()->with(
             m::type(Client::class)
@@ -207,17 +198,14 @@ public function test_clients_can_be_deleted()
 
     public function test_404_response_if_client_doesnt_belong_to_user_on_delete()
     {
+        $user = m::mock(Authenticatable::class);
+        $user->shouldReceive('getAuthIdentifier')->andReturn(1);
+
         $clients = m::mock(ClientRepository::class);
-        $clients->shouldReceive('findForUser')->with(1, 1)->andReturnNull();
+        $clients->shouldReceive('findForUser')->with(1, $user)->andReturnNull();
 
         $request = Request::create('/', 'GET', ['name' => 'client name', 'redirect' => 'http://localhost']);
-
-        $request->setUserResolver(function () {
-            $user = m::mock(Authenticatable::class);
-            $user->shouldReceive('getAuthIdentifier')->andReturn(1);
-
-            return $user;
-        });
+        $request->setUserResolver(fn () => $user);
 
         $clients->shouldReceive('delete')->never();
 
diff --git a/tests/Unit/PersonalAccessTokenControllerTest.php b/tests/Unit/PersonalAccessTokenControllerTest.php
index 8f57c8a5..e89c0967 100644
--- a/tests/Unit/PersonalAccessTokenControllerTest.php
+++ b/tests/Unit/PersonalAccessTokenControllerTest.php
@@ -115,17 +115,14 @@ public function test_tokens_can_be_deleted()
 
     public function test_not_found_response_is_returned_if_user_doesnt_have_token()
     {
-        $request = Request::create('/', 'GET');
+        $user = m::mock(Authenticatable::class);
+        $user->shouldReceive('getAuthIdentifier')->andReturn(1);
 
         $tokenRepository = m::mock(TokenRepository::class);
-        $tokenRepository->shouldReceive('findForUser')->with(3, 1)->andReturnNull();
-
-        $request->setUserResolver(function () {
-            $user = m::mock(Authenticatable::class);
-            $user->shouldReceive('getAuthIdentifier')->andReturn(1);
+        $tokenRepository->shouldReceive('findForUser')->with(3, $user)->andReturnNull();
 
-            return $user;
-        });
+        $request = Request::create('/', 'GET');
+        $request->setUserResolver(fn () => $user);
 
         $validator = m::mock(Factory::class);
         $controller = new PersonalAccessTokenController($tokenRepository, $validator);