Merge pull request #189 from elguitarraverde/patch-1

fix: scripts attributes not working

Author: yajra

src/Html/Builder.php

@@ -100,11 +100,11 @@ public static function useWebpack(): void
      * @param  array  $attributes
      * @return \Illuminate\Support\HtmlString
      */
-    public function scripts(string $script = null, array $attributes = ['type' => 'text/javascript']): HtmlString
+    public function scripts(string $script = null, array $attributes = []): HtmlString
     {
         $script = $script ?: $this->generateScripts();
         $attributes = $this->html->attributes(
-            array_merge($attributes, ['type' => static::$jsType])
+            array_merge($attributes, ['type' => $attributes['type'] ?? static::$jsType])
         );
 
         return new HtmlString("<script{$attributes}>$script</script>");

tests/BuilderTest.php

@@ -11,6 +11,31 @@
 
 class BuilderTest extends TestCase
 {
+     /** @test */
+     public function it_can_get_script_default_type_attribute()
+     {
+         $html = $this->getHtmlBuilder()->scripts()->toHtml();
+ 
+         $this->assertStringContainsString('type="text/javascript"', $html);
+     }
+
+    /** @test */
+     public function it_can_set_script_type_attribute()
+     {
+        $html = $this->getHtmlBuilder()->scripts(attributes: ['type' => 'module'])->toHtml();
+ 
+        $this->assertStringContainsString('type="module"', $html);
+     }
+
+     /** @test */
+    public function it_can_set_multiple_script_attributes()
+    {
+        $html = $this->getHtmlBuilder()->scripts(attributes: ['prop1' => 'val1', 'prop2' => 'val2'])->toHtml();
+
+        $this->assertStringContainsString('prop1="val1"', $html);
+        $this->assertStringContainsString('prop2="val2"', $html);
+    }
+
     /** @test */
     public function it_can_use_vitejs_module_script()
     {