- Changed reuse conversion test

Author: dash8x

tests/Feature/Commands/AttachmentsCleanCommandTest.php

@@ -0,0 +1,85 @@
+<?php
+
+namespace Javaabu\Mediapicker\Tests\Feature\Commands;
+
+use Illuminate\Foundation\Testing\RefreshDatabase;
+use Javaabu\Mediapicker\Tests\TestCase;
+use PHPUnit\Framework\Attributes\Test;
+
+class AttachmentsCleanCommandTest extends TestCase
+{
+    use RefreshDatabase;
+
+    #[Test]
+    public function it_can_clean_all_attachments(): void
+    {
+        $attachment = $this->getAttachment();
+
+        $this->assertDatabaseHas('attachments', [
+            'id' => $attachment->id,
+        ]);
+
+        $this->artisan('mediapicker:clear')
+            ->assertSuccessful();
+
+        $this->assertDatabaseEmpty('attachments');
+    }
+
+    /*#[Test]
+    public function it_can_clear_attachments_of_a_specific_model_type(): void
+    {
+        $post = $this->getModel();
+        $attachment = $this->getAttachment($post);
+
+        $other_model = $this->getModelWithConversions();
+        $attachment_2 = $this->getAttachment($other_model);
+
+        $this->assertDatabaseHas('attachments', [
+            'id' => $attachment->id,
+        ]);
+
+        $this->assertDatabaseHas('attachments', [
+            'id' => $attachment_2->id,
+        ]);
+
+        $this->artisan('mediapicker:clear', ['modelType' => 'post'])
+            ->assertSuccessful();
+
+        $this->assertDatabaseMissing('attachments', [
+            'id' => $attachment->id,
+        ]);
+
+        $this->assertDatabaseHas('attachments', [
+            'id' => $attachment_2->id,
+        ]);
+    }
+
+    #[Test]
+    public function it_can_clear_attachments_of_a_specific_collection(): void
+    {
+        $post = $this->getModel();
+        $attachment = $this->getAttachment($post, 'test1');
+
+        $other_model = $this->getModelWithConversions();
+        $attachment_2 = $this->getAttachment($other_model, 'test');
+
+        $this->assertDatabaseHas('attachments', [
+            'id' => $attachment->id,
+        ]);
+
+        $this->assertDatabaseHas('attachments', [
+            'id' => $attachment_2->id,
+        ]);
+
+        $this->artisan('mediapicker:clear', ['collectionName' => 'test1'])
+            ->assertSuccessful();
+
+        $this->assertDatabaseMissing('attachments', [
+            'id' => $attachment->id,
+        ]);
+
+        $this->assertDatabaseHas('attachments', [
+            'id' => $attachment_2->id,
+        ]);
+    }*/
+}

tests/Unit/Concerns/InteractsWithAttachmentsTest.php

@@ -631,10 +631,7 @@ public function it_reuses_conversions_from_other_models(): void
         $this->assertEquals($this->getMediaDirectory($media->getKey() . '/conversions/test-test.jpg'), $path);
         $this->assertFileExists($path);
 
-        $dimensions = $this->getImageDimensions($path);
-        $this->assertEquals(100, $dimensions['width']);
-        $this->assertEquals(100, $dimensions['height']);
-
+        $file_hash = hash_file('sha256', $path);
 
         $attachment_2 = $model_2->addAttachment($media)
             ->toAttachmentCollection();
@@ -644,8 +641,7 @@ public function it_reuses_conversions_from_other_models(): void
         $this->assertEquals($this->getMediaDirectory($media->getKey() . '/conversions/test-test.jpg'), $path_2);
         $this->assertFileExists($path_2);
 
-        $dimensions_2 = $this->getImageDimensions($path_2);
-        $this->assertEquals(100, $dimensions_2['width']);
-        $this->assertEquals(100, $dimensions_2['height']);
+        $file_hash_2 =  hash_file('sha256', $path_2);
+        $this->assertEquals($file_hash, $file_hash_2);
     }
 }