Update tests

daniel-de-wit · Jan 27, 2025 · e2e9364 · e2e9364
1 parent ba2dcaa
commit e2e9364
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 1 deletion.
diff --git a/tests/Integration/LaravelIdeHelperHookPaperclipServiceProviderTest.php b/tests/Integration/LaravelIdeHelperHookPaperclipServiceProviderTest.php
@@ -27,7 +27,7 @@ protected function getPackageProviders($app): array
     /**
      * @test
      */
-    public function it_auto_registration_off_model_hook(): void
+    public function it_auto_registers_model_hook(): void
     {
         /** @var Application $app */
         $app = $this->app;
@@ -43,4 +43,24 @@ public function it_auto_registration_off_model_hook(): void
             (array) $config->get('ide-helper.model_hooks', []),
         );
     }
+
+    /**
+     * @test
+     */
+    public function it_auto_registers_model_hook_with_wrong_service_provider_order(): void
+    {
+        /** @var Application $app */
+        $app = $this->app;
+
+        $app->loadDeferredProvider(LaravelIdeHelperHookPaperclipServiceProvider::class);
+        $app->loadDeferredProvider(IdeHelperServiceProvider::class);
+
+        /** @var Repository $config */
+        $config = $app->get('config');
+
+        $this->assertContains(
+            PaperclipHook::class,
+            (array) $config->get('ide-helper.model_hooks', []),
+        );
+    }
 }
diff --git a/tests/Unit/Hooks/PaperclipHookTest.php b/tests/Unit/Hooks/PaperclipHookTest.php
@@ -26,6 +26,7 @@ public function it_writes_paperclip_properties(): void
         /** @var ModelsCommand|MockInterface $command */
         $command = Mockery::mock(ModelsCommand::class)
             ->shouldReceive('setProperty')
+            ->once()
             ->with(
                 'foobar',
                 '\\Czim\\Paperclip\\Contracts\\AttachmentInterface|\\SplFileInfo|\\Czim\\FileHandling\\Storage\\File\\SplFileInfoStorableFile|\\Czim\\FileHandling\\Contracts\\Support\\RawContentInterface|string',
@@ -37,6 +38,7 @@ public function it_writes_paperclip_properties(): void
         /** @var ModelWithAttachment|MockInterface $model */
         $model = Mockery::mock(ModelWithAttachment::class)
             ->shouldReceive('getAttachedFiles')
+            ->once()
             ->andReturn([
                 'foobar' => Mockery::mock(AttachmentInterface::class),
             ])