Test fixes

app/components/dialog_component.rb

@@ -4,14 +4,14 @@ class DialogComponent < ViewComponent::Base
       title_div = content_tag(:div, class: "flex items-center justify-between gap-2") do
         title = content_tag(:h2, title, class: class_names("font-medium text-primary", drawer? ? "text-lg" : "")) if title
         close_icon = render ButtonComponent.new(variant: "icon", class: "ml-auto", icon: "x", tabindex: "-1", data: { action: "dialog#close" }) unless hide_close_icon
-        safe_join([title, close_icon].compact)
+        safe_join([ title, close_icon ].compact)
       end
 
       subtitle = content_tag(:p, subtitle, class: "text-sm text-secondary") if subtitle
 
       block_content = capture(&block) if block
 
-      safe_join([title_div, subtitle, block_content].compact)
+      safe_join([ title_div, subtitle, block_content ].compact)
     end
   end
 
@@ -79,10 +79,10 @@ class DialogComponent < ViewComponent::Base
     merged_opts = opts.dup
     data = merged_opts.delete(:data) || {}
 
-    data[:controller] = ["dialog", "hotkey", data[:controller]].compact.join(" ") 
+    data[:controller] = [ "dialog", "hotkey", data[:controller] ].compact.join(" ")
     data[:dialog_auto_open_value] = auto_open
     data[:dialog_reload_on_close_value] = reload_on_close
-    data[:action] = ["mousedown->dialog#clickOutside", data[:action]].compact.join(" ")
+    data[:action] = [ "mousedown->dialog#clickOutside", data[:action] ].compact.join(" ")
     data[:hotkey] = "esc:dialog#close"
     merged_opts[:data] = data
 

app/views/family_merchants/new.html.erb

@@ -1,6 +1,6 @@
 <%= render DialogComponent.new do |dialog| %>
   <% dialog.with_header(title: t(".title")) %>
   <% dialog.with_body do %>
-    <%= render "form", family_merchant: @family_merchant %>
+    <%= render "form", family_merchant: @merchant %>
   <% end %>
 <% end %>

app/views/transactions/_form.html.erb

@@ -1,4 +1,6 @@
-<%= styled_form_with model: @entry, url: transactions_path, class: "space-y-4 text-subdued", data: { controller: "transaction-form" } do |f| %>
+<%# locals: (entry:, income_categories:, expense_categories:) %>
+
+<%= styled_form_with model: entry, url: transactions_path, class: "space-y-4 text-subdued", data: { controller: "transaction-form" } do |f| %>
   <% if entry.errors.any? %>
     <%= render "shared/form_errors", model: entry %>
   <% end %>
@@ -21,7 +23,7 @@
 
     <%= f.money_field :amount, label: t(".amount"), required: true %>
     <%= f.fields_for :entryable do |ef| %>
-      <% categories = params[:nature] == "inflow" ? @income_categories : @expense_categories %>
+      <% categories = params[:nature] == "inflow" ? income_categories : expense_categories %>
       <%= ef.collection_select :category_id, categories, :id, :name, { prompt: t(".category_prompt"), label: t(".category") } %>
     <% end %>
     <%= f.date_field :date, label: t(".date"), required: true, min: Entry.min_supported_date, max: Date.current, value: Date.current %>

app/views/transactions/new.html.erb

@@ -1,6 +1,6 @@
 <%= render DialogComponent.new do |dialog| %>
   <% dialog.with_header(title: "New transaction") %>
   <% dialog.with_body do %>
-    <%= render "form", transaction: @transaction %>
+    <%= render "form", entry: @entry, income_categories: @income_categories, expense_categories: @expense_categories %>
   <% end %>
 <% end %>