From c615acd264079dbb2139e3b91f53cb404b6b19dc Mon Sep 17 00:00:00 2001
From: Adrian Hopek <adrian.hopek@blumilk.pl>
Date: Tue, 18 Jan 2022 09:57:10 +0100
Subject: [PATCH] #4 - ecs fix

---
 tests/Feature/UserTest.php | 39 ++++++++++++++++++++++++--------------
 tests/Unit/AvatarTest.php  | 11 +++++++----
 2 files changed, 32 insertions(+), 18 deletions(-)

diff --git a/tests/Feature/UserTest.php b/tests/Feature/UserTest.php
index c8a246e..dad847f 100644
--- a/tests/Feature/UserTest.php
+++ b/tests/Feature/UserTest.php
@@ -16,7 +16,7 @@ class UserTest extends TestCase
 {
     use DatabaseMigrations;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 
@@ -32,26 +32,36 @@ class UserTest extends TestCase
 
         $this->actingAs($admin)
             ->get("/users")
-            ->assertInertia(fn(Assert $page) => $page
-                ->component("Users/Index")
-                ->has("users.data", 11)
+            ->assertInertia(
+                fn(Assert $page) => $page
+                    ->component("Users/Index")
+                    ->has("users.data", 11),
             );
     }
 
     public function testAdminCanSearchUsersList(): void
     {
-        User::factory(["name" => "Test User1"])->create();
-        User::factory(["name" => "Test User2"])->create();
-        User::factory(["name" => "Test User3"])->create();
-        $admin = User::factory(["name" => "John Doe"])->create();
+        User::factory([
+            "name" => "Test User1",
+        ])->create();
+        User::factory([
+            "name" => "Test User2",
+        ])->create();
+        User::factory([
+            "name" => "Test User3",
+        ])->create();
+        $admin = User::factory([
+            "name" => "John Doe",
+        ])->create();
 
         $this->assertDatabaseCount("users", 4);
 
         $this->actingAs($admin)
             ->get("/users?search=test")
-            ->assertInertia(fn(Assert $page) => $page
-                ->component("Users/Index")
-                ->has("users.data", 3)
+            ->assertInertia(
+                fn(Assert $page) => $page
+                    ->component("Users/Index")
+                    ->has("users.data", 3),
             );
     }
 
@@ -64,9 +74,10 @@ class UserTest extends TestCase
 
         $this->actingAs($admin)
             ->get("/users?page=2")
-            ->assertInertia(fn(Assert $page) => $page
-                ->component("Users/Index")
-                ->has("users.data", 1)
+            ->assertInertia(
+                fn(Assert $page) => $page
+                    ->component("Users/Index")
+                    ->has("users.data", 1),
             );
     }
 
diff --git a/tests/Unit/AvatarTest.php b/tests/Unit/AvatarTest.php
index 3cc900b..2da1550 100644
--- a/tests/Unit/AvatarTest.php
+++ b/tests/Unit/AvatarTest.php
@@ -13,7 +13,7 @@ class AvatarTest extends TestCase
 {
     use DatabaseMigrations;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 
@@ -36,7 +36,6 @@ class AvatarTest extends TestCase
         $user->forceDelete();
 
         Storage::assertMissing($user->avatar);
-
     }
 
     public function testAvatarIsReplacedWhenUserChangedTheirName(): void
@@ -46,7 +45,9 @@ class AvatarTest extends TestCase
 
         Storage::assertExists($oldAvatar);
 
-        $user->update(["name" => "John Doe"]);
+        $user->update([
+            "name" => "John Doe",
+        ]);
 
         Storage::assertMissing($oldAvatar);
         Storage::assertExists($user->avatar);
@@ -59,7 +60,9 @@ class AvatarTest extends TestCase
 
         Storage::assertExists($avatar);
 
-        $user->update(["email" => "john.doe@example.com"]);
+        $user->update([
+            "email" => "john.doe@example.com",
+        ]);
 
         Storage::assertExists($avatar);
     }