fix tests

Author: iamgergo

tests/Casts/MetaValueTest.php

@@ -18,7 +18,7 @@ protected function setUp(): void
     {
         parent::setUp();
 
-        $this->cast = new MetaValue();
+        $this->cast = new MetaValue;
 
         $this->model = new class extends Model
         {

tests/Jobs/MoveFileTest.php

@@ -28,41 +28,41 @@ public function test_move_file_job_can_be_dispatched(): void
 
     public function test_move_file_job_moves_file_to_storage(): void
     {
-        Storage::fake('public');
-
         $file = UploadedFile::fake()->image('test.jpg');
         $tempPath = $file->getRealPath();
 
         $medium = Medium::factory()->make([
             'disk' => 'public',
-            'path' => 'media/test.jpg',
+            'file_name' => 'test.jpg',
         ]);
 
         $job = new MoveFile($medium, $tempPath, true);
         $job->handle();
 
-        Storage::disk('public')->assertExists('media/test.jpg');
+        Storage::disk('public')->assertExists('test.jpg');
+
+        Storage::disk('public')->delete('test.jpg');
     }
 
     public function test_move_file_job_deletes_original_when_not_preserved(): void
     {
-        Storage::fake('public');
-
         $tempDir = Storage::disk('local')->path('root-tmp');
         $tempPath = $tempDir.'/test-file.jpg';
         File::ensureDirectoryExists($tempDir);
         File::put($tempPath, 'test content');
 
         $medium = Medium::factory()->make([
             'disk' => 'public',
-            'path' => 'media/test.jpg',
+            'file_name' => 'test.jpg',
         ]);
 
         $job = new MoveFile($medium, $tempPath, false);
         $job->handle();
 
-        Storage::disk('public')->assertExists('media/test.jpg');
+        Storage::disk('public')->assertExists('test.jpg');
         $this->assertFalse(File::exists($tempPath));
+
+        Storage::disk('public')->delete('test.jpg');
     }
 
     public function test_move_file_job_preserves_original_when_preserve_is_true(): void
@@ -76,13 +76,15 @@ public function test_move_file_job_preserves_original_when_preserve_is_true(): v
 
         $medium = Medium::factory()->make([
             'disk' => 'public',
-            'path' => 'media/test.jpg',
+            'file_name' => 'test.jpg',
         ]);
 
         $job = new MoveFile($medium, $tempPath, true);
         $job->handle();
 
-        Storage::disk('public')->assertExists('media/test.jpg');
+        Storage::disk('public')->assertExists('test.jpg');
         $this->assertTrue(File::exists($tempPath));
+
+        Storage::disk('public')->delete('test.jpg');
     }
 }

tests/Listeners/FormatRootStubsTest.php

@@ -8,7 +8,6 @@
 use Cone\Root\Tests\TestCase;
 use Illuminate\Foundation\Events\VendorTagPublished;
 use Illuminate\Support\Facades\App;
-use Illuminate\Support\Facades\File;
 
 final class FormatRootStubsTest extends TestCase
 {
@@ -19,7 +18,7 @@ public function test_listener_formats_root_stubs(): void
 
         $event = new VendorTagPublished('root-stubs', [$tempFile]);
 
-        $listener = new FormatRootStubs();
+        $listener = new FormatRootStubs;
         $listener->handle($event);
 
         $contents = file_get_contents($tempFile);
@@ -37,7 +36,7 @@ public function test_listener_ignores_non_root_stubs_tag(): void
 
         $event = new VendorTagPublished('other-tag', [$tempFile]);
 
-        $listener = new FormatRootStubs();
+        $listener = new FormatRootStubs;
         $listener->handle($event);
 
         $contents = file_get_contents($tempFile);
@@ -57,7 +56,7 @@ public function test_listener_handles_multiple_files(): void
 
         $event = new VendorTagPublished('root-stubs', [$tempFile1, $tempFile2]);
 
-        $listener = new FormatRootStubs();
+        $listener = new FormatRootStubs;
         $listener->handle($event);
 
         $contents1 = file_get_contents($tempFile1);

tests/Notifications/RootChannelTest.php

@@ -19,9 +19,9 @@ public function test_root_channel_sends_notification(): void
 
         $user = User::factory()->create();
 
-        $notification = new TestRootNotification();
+        $notification = new TestRootNotification;
 
-        (new RootChannel())->send($user, $notification);
+        (new RootChannel)->send($user, $notification);
 
         $this->assertDatabaseHas('root_notifications', [
             'notifiable_type' => User::class,
@@ -35,9 +35,9 @@ public function test_root_channel_includes_notification_type(): void
     {
         $user = User::factory()->create();
 
-        $notification = new TestRootNotification();
+        $notification = new TestRootNotification;
 
-        (new RootChannel())->send($user, $notification);
+        (new RootChannel)->send($user, $notification);
 
         $this->assertDatabaseHas('root_notifications', [
             'type' => TestRootNotification::class,
@@ -49,7 +49,7 @@ class TestRootNotification extends RootNotification
 {
     public function toRoot(object $notifiable): RootMessage
     {
-        return (new RootMessage())
+        return (new RootMessage)
             ->subject('Test Subject')
             ->message('Test Message');
     }

tests/Notifications/RootMessageTest.php

@@ -18,23 +18,23 @@ public function test_root_message_can_be_created(): void
 
     public function test_root_message_can_set_subject(): void
     {
-        $message = new RootMessage();
+        $message = new RootMessage;
         $message->subject('Test Subject');
 
         $this->assertSame('Test Subject', $message->toArray()['subject']);
     }
 
     public function test_root_message_can_set_message(): void
     {
-        $message = new RootMessage();
+        $message = new RootMessage;
         $message->message('Test Message');
 
         $this->assertSame('Test Message', $message->toArray()['message']);
     }
 
     public function test_root_message_can_set_data(): void
     {
-        $message = new RootMessage();
+        $message = new RootMessage;
         $message->data(['key' => 'value']);
 
         $this->assertSame(['key' => 'value'], $message->toArray()['data']);
@@ -54,7 +54,7 @@ public function test_root_message_can_be_converted_to_array(): void
 
     public function test_root_message_methods_are_fluent(): void
     {
-        $message = new RootMessage();
+        $message = new RootMessage;
 
         $result = $message->subject('Test')
             ->message('Message')

tests/Policies/MediumPolicyTest.php

@@ -21,7 +21,7 @@ protected function setUp(): void
     {
         parent::setUp();
 
-        $this->policy = new MediumPolicy();
+        $this->policy = new MediumPolicy;
         $this->user = User::factory()->create();
         $this->medium = Medium::factory()->create();
     }

tests/Support/ClassListTest.php

@@ -18,7 +18,7 @@ public function test_a_class_list_can_be_created_with_classes(): void
 
     public function test_a_class_list_can_add_classes(): void
     {
-        $classList = new ClassList();
+        $classList = new ClassList;
 
         $classList->add('foo');
         $this->assertSame(['foo'], $classList->toArray());

tests/Traits/MakeableTest.php

@@ -33,15 +33,12 @@ class TestMakeable
     public function __construct(
         public string $param1,
         public string $param2
-    ) {
-    }
+    ) {}
 }
 
 class TestMakeableNoParams
 {
     use Makeable;
 
-    public function __construct()
-    {
-    }
+    public function __construct() {}
 }

tests/View/Components/AlertComponentTest.php

@@ -11,14 +11,14 @@ final class AlertComponentTest extends TestCase
 {
     public function test_alert_component_can_be_instantiated(): void
     {
-        $component = new AlertComponent();
+        $component = new AlertComponent;
 
         $this->assertInstanceOf(AlertComponent::class, $component);
     }
 
     public function test_alert_component_renders(): void
     {
-        $component = new AlertComponent();
+        $component = new AlertComponent;
 
         $view = $component->render();
 

tests/View/Components/ChartTest.php

@@ -11,7 +11,7 @@ final class ChartTest extends TestCase
 {
     public function test_chart_component_renders(): void
     {
-        $component = new Chart();
+        $component = new Chart;
 
         $view = $component->render();