Apply code review suggestions

Author: lsouoliveira

lib/ruby_ui/separator/separator.rb

@@ -4,17 +4,17 @@ module RubyUI
   class Separator < Base
     ORIENTATIONS = %i[horizontal vertical].freeze
 
-    def initialize(as: "div", orientation: :horizontal, decorative: true, **attrs)
+    def initialize(as: :div, orientation: :horizontal, decorative: true, **attrs)
       raise ArgumentError, "Invalid orientation: #{orientation}" unless ORIENTATIONS.include?(orientation.to_sym)
 
-      @as = as
+      @as = as.to_sym
       @orientation = orientation.to_sym
       @decorative = decorative
       super(**attrs)
     end
 
     def view_template(&)
-      public_send(@as, **attrs, &)
+      tag(@as, **attrs, &)
     end
 
     private
@@ -25,10 +25,7 @@ def default_attrs
         class: [
           "shrink-0 bg-border",
           orientation_classes
-        ],
-        data: {
-          orientation: @orientation
-        }
+        ]
       }
     end
 

test/ruby_ui/separator_test.rb

@@ -10,15 +10,15 @@ def test_render
 
     assert_match(/div/, output)
     assert_match(/role="none"/, output)
-    assert_match(/data-orientation="horizontal"/, output)
+    assert_match(/h-\[1px\]/, output)
   end
 
   def test_render_with_vertical_orientation
     output = phlex do
       RubyUI.Separator(orientation: :vertical)
     end
 
-    assert_match(/data-orientation="vertical"/, output)
+    assert_match(/w-\[1px\]/, output)
   end
 
   def test_render_with_decorative_false
@@ -34,6 +34,6 @@ def test_render_with_custom_tag
       RubyUI.Separator(as: "hr")
     end
 
-    assert_match(/hr/, output)
+    assert_match(/<hr/, output)
   end
 end