Tests the response macros

hotwired-laravel · Dec 31, 2020 · 2466cc2 · 2466cc2
1 parent 765b446
commit 2466cc2
Show file tree

Hide file tree

Showing 4 changed files with 103 additions and 3 deletions.
diff --git a/src/NamesResolver.php b/src/NamesResolver.php
@@ -27,6 +27,11 @@ public static function resourceNameSingular(Model $model): string
         return static::resourceNameSingularFor(class_basename($model));
     }
 
+    public static function resourceVariableName(Model $model): string
+    {
+        return Str::camel(static::resourceNameSingular($model));
+    }
+
     public static function partialNameFor(Model $model): string
     {
         $root = static::resourceName($model);

diff --git a/src/TurboStreamResponseMacro.php b/src/TurboStreamResponseMacro.php
@@ -52,7 +52,7 @@ private function renderModelAddedStream(Model $model, string $action = null)
                 : NamesResolver::partialNameFor($model),
             'data' => method_exists($model, 'hotwirePartialData')
                 ? $model->hotwirePartialData()
-                : [NamesResolver::resourceNameSingular($model) => $model],
+                : [NamesResolver::resourceVariableName($model) => $model],
         ]));
     }
 
@@ -62,13 +62,13 @@ private function renderModelUpdatedStream(Model $model)
             'target' => method_exists($model, 'hotwireTargetDomId')
                 ? $model->hotwireTargetDomId()
                 : NamesResolver::resourceId($model::class, $model->id),
-            'action' => 'update',
+            'action' => 'replace',
             'resourcePartialName' => method_exists($model, 'hotwirePartialName')
                 ? $model->hotwirePartialName()
                 : NamesResolver::partialNameFor($model),
             'data' => method_exists($model, 'hotwirePartialData')
                 ? $model->hotwirePartialData()
-                : [NamesResolver::resourceNameSingular($model) => $model],
+                : [NamesResolver::resourceVariableName($model) => $model],
         ]));
     }
 }
diff --git a/tests/Http/ResponseMacrosTest.php b/tests/Http/ResponseMacrosTest.php
@@ -0,0 +1,94 @@
+<?php
+
+namespace Tonysm\TurboLaravel\Tests\Http;
+
+use Illuminate\Support\Facades\View;
+use Tonysm\TurboLaravel\Tests\TestCase;
+
+class ResponseMacrosTest extends TestCase
+{
+    public function setUp(): void
+    {
+        parent::setUp();
+
+        View::addNamespace('test-stubs', __DIR__ . '/stubs/');
+    }
+
+    /** @test */
+    public function streams_model_on_create()
+    {
+        $testModel = TestModel::create(['name' => 'test']);
+
+        $expected = <<<html
+<turbo-stream target="test_models" action="append">
+    <template>
+        <div id="test_model_{$testModel->id}">hello</div>
+    </template>
+</turbo-stream>
+html;
+
+        $resp = response()->turboStream($testModel);
+
+        $this->assertEquals($expected, trim($resp->getContent()));
+        $this->assertEquals('text/html; turbo-stream', $resp->headers->get('Content-Type'));
+    }
+
+    /** @test */
+    public function streams_model_on_update()
+    {
+        $testModel = TestModel::create(['name' => 'test'])->fresh();
+
+        $expected = <<<html
+<turbo-stream target="test_model_{$testModel->id}" action="replace">
+    <template>
+        <div id="test_model_{$testModel->id}">hello</div>
+    </template>
+</turbo-stream>
+html;
+
+        $resp = response()->turboStream($testModel);
+
+        $this->assertEquals($expected, trim($resp->getContent()));
+        $this->assertEquals('text/html; turbo-stream', $resp->headers->get('Content-Type'));
+    }
+
+    /** @test */
+    public function streams_model_on_delete()
+    {
+        $testModel = tap(TestModel::create(['name' => 'test']))->delete();
+
+        $expected = <<<html
+<turbo-stream target="test_model_{$testModel->id}" action="remove"></turbo-stream>
+html;
+
+        $resp = response()->turboStream($testModel);
+
+        $this->assertEquals($expected, trim($resp->getContent()));
+        $this->assertEquals('text/html; turbo-stream', $resp->headers->get('Content-Type'));
+    }
+
+    /** @test */
+    public function streams_custom_view()
+    {
+        $testModel = TestModel::create(['name' => 'test']);
+
+        $expected = <<<html
+<div id="test_model_{$testModel->id}">hello</div>
+html;
+
+        $resp = response()->turboStreamView(View::file(__DIR__ . '/stubs/_test_model.blade.php', [
+            'testModel' => $testModel,
+        ]));
+
+        $this->assertEquals($expected, trim($resp->getContent()));
+        $this->assertEquals('text/html; turbo-stream', $resp->headers->get('Content-Type'));
+    }
+}
+
+class TestModel extends \Tonysm\TurboLaravel\Tests\TestModel
+{
+    public function hotwirePartialName()
+    {
+        return "test-stubs::_test_model";
+    }
+}
diff --git a/tests/Http/stubs/_test_model.blade.php b/tests/Http/stubs/_test_model.blade.php
@@ -0,0 +1 @@
+<div id="@domid($testModel)">hello</div>