inline_forms 8.1.21 → 8.1.40

data/test/dummy/app/views/_inline_forms_tabs.html.erb

@@ -0,0 +1,40 @@
+<div class="contain-to-grid fixed" id="inline_forms_model_top_bar_container">
+  <nav id="inline_forms_model_top_bar" class="top-bar">
+    <div class="top-bar-left">
+      <ul class="menu">
+        <li class="menu-text" id="inline_forms_menu">
+          <h1 style="margin: 0; font-size: inherit;">
+            <a>
+              <%= t(controller_name) %>
+            </a>
+          </h1>
+        </li>
+      </ul>
+    </div>
+    <div class="top-bar-right">
+      <ul class="menu">
+        <li>
+          <%= form_tag request.path, :method => 'get' do %>
+            <div class="row collapse">
+              <div class="large-1 small-1 columns">
+                <%= link_to "<i class='fi-x'></i>".html_safe, request.path, :title => 'reset', :class => "button expanded inline_forms_model_top_bar_buttons" %>
+              </div>
+              <div class="large-6 small-6 columns">
+                <%= text_field_tag :search,  params[:search], :placeholder => 'zoek op naam...', :id => 'input_search' %>
+              </div>
+              <div class="large-3 small-3 columns">
+                <%= submit_tag 'zoek', :class => "button expanded inline_forms_model_top_bar_buttons" %>
+              </div>
+              <div class="large-2 small-2 columns">
+                &nbsp;
+              </div>
+            </div>
+          <% end %>
+        </li>
+        <li>
+          <%= link_to_new_record(@Klass, "new_#{@Klass.to_s.singularize.underscore}_path", @Klass.to_s.pluralize.downcase + '_list') %>
+        </li>
+      </ul>
+    </div>
+  </nav>
+</div>

data/test/dummy/config/application.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+# Minimal host app for the engine's integration tests (test/integration/).
+# Boots the inline_forms engine against an in-memory SQLite database with the
+# host-side gems the engine expects (will_paginate, paper_trail, turbo-rails)
+# but without Devise, CanCanCan, Sprockets or ActionText: authorization is
+# optional in the engine (cancan_enabled? rescues NameError), tests drive the
+# UI through Turbo-Frame requests (the frame layout never renders the
+# current_user header), and the asset-precompile initializer skips hosts
+# without a pipeline.
+require "rails"
+require "active_record/railtie"
+require "action_controller/railtie"
+require "action_view/railtie"
+
+Bundler.require(*Rails.groups)
+
+require "inline_forms"
+
+module Dummy
+  class Application < Rails::Application
+    config.load_defaults 8.1
+    config.root = File.expand_path("..", __dir__)
+    config.eager_load = false
+    config.secret_key_base = "inline_forms_dummy_test_secret"
+    config.logger = ActiveSupport::Logger.new(File::NULL)
+    config.active_support.deprecation = :stderr
+    config.action_controller.allow_forgery_protection = false
+    # Schema is defined by test/integration_test_helper.rb on the in-memory DB.
+    config.active_record.maintain_test_schema = false
+    # Raise in-place so integration failures carry the real backtrace.
+    config.action_dispatch.show_exceptions = :none
+  end
+end

data/test/dummy/config/database.yml

@@ -0,0 +1,3 @@
+test:
+  adapter: sqlite3
+  database: ":memory:"

data/test/dummy/config/environment.rb

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+require_relative "application"
+
+Rails.application.initialize!

data/test/dummy/config/initializers/inline_forms.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+# Mirrors the initializer the installer seeds in host apps; the header
+# (`inline_forms/_header`) iterates MODEL_TABS for admin users. Kept empty so
+# the dummy needs no CanCan (`can?` is only called per entry).
+MODEL_TABS = %w[].freeze

data/test/dummy/config/initializers/paper_trail_yaml_safe_load.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+# Mirrors the initializer the installer writes into generated apps
+# (installer_core.rb): PaperTrail serializes object/object_changes as YAML,
+# and Psych safe-loading needs these classes permitted or version.changeset /
+# version.reify silently degrade (empty changeset) or raise.
+permitted = [
+  Symbol,
+  Date,
+  Time,
+  BigDecimal,
+  ActiveSupport::TimeWithZone,
+  ActiveSupport::TimeZone,
+  ActiveSupport::HashWithIndifferentAccess
+]
+
+%w[
+  ActiveRecord::Type::Time::Value
+  ActiveRecord::Type::Date::Value
+  ActiveRecord::Type::DateTime::Value
+].each do |const_name|
+  klass = const_name.safe_constantize
+  permitted << klass if klass
+end
+
+Rails.application.config.active_record.yaml_column_permitted_classes ||= []
+Rails.application.config.active_record.yaml_column_permitted_classes |= permitted
+ActiveRecord.yaml_column_permitted_classes |= Rails.application.config.active_record.yaml_column_permitted_classes

data/test/dummy/config/routes.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+# Mirrors the routes `rails g inline_forms` injects for each resource.
+Rails.application.routes.draw do
+  # Devise stand-in: the inline_forms header links destroy_user_session_path.
+  delete "logout", to: proc { [ 204, {}, [] ] }, as: :destroy_user_session
+
+  resources :widgets do
+    post "revert", on: :member
+    get "list_versions", on: :member
+  end
+
+  resources :machines do
+    post "revert", on: :member
+    get "list_versions", on: :member
+  end
+
+  resources :parts do
+    post "revert", on: :member
+    get "list_versions", on: :member
+  end
+end

data/test/integration/inline_forms_crud_test.rb

@@ -0,0 +1,141 @@
+# frozen_string_literal: true
+
+require_relative "../integration_test_helper"
+
+# End-to-end (controller + views + Turbo frames) CRUD through the engine's
+# InlineFormsController, running against the test/dummy host app.
+class InlineFormsCrudTest < InlineFormsIntegrationTestCase
+  setup do
+    @widget = Widget.create!(name: "Alpha", priority: 2)
+  end
+
+  test "index lists rows inside the list turbo-frame" do
+    get widgets_path(update: "widgets_list"), headers: frame_headers("widgets_list")
+
+    assert_response :success
+    assert_includes response.body, %(<turbo-frame id="widgets_list" class="list_container">)
+    assert_includes response.body, %(<turbo-frame id="widget_#{@widget.id}">)
+    assert_includes response.body, "Alpha"
+  end
+
+  test "index applies inline_forms_search when ?search= is passed" do
+    Widget.create!(name: "Beta")
+
+    get widgets_path(update: "widgets_list", search: "Alp"),
+        headers: frame_headers("widgets_list")
+
+    assert_response :success
+    assert_includes response.body, "Alpha"
+    refute_includes response.body, "Beta"
+  end
+
+  test "row show renders attribute frames" do
+    frame = "widget_#{@widget.id}"
+    get widget_path(@widget, update: frame), headers: frame_headers(frame)
+
+    assert_response :success
+    assert_includes response.body, %(<turbo-frame id="#{frame}">)
+    assert_includes response.body, %(<turbo-frame id="widget_#{@widget.id}_name">)
+  end
+
+  test "attribute edit renders the input in its frame" do
+    frame = "widget_#{@widget.id}_name"
+    get edit_widget_path(@widget, attribute: "name", form_element: "text_field",
+                         update: frame),
+        headers: frame_headers(frame)
+
+    assert_response :success
+    assert_includes response.body, %(<turbo-frame id="#{frame}">)
+    assert_includes response.body, %(name="name")
+    assert_includes response.body, %(value="Alpha")
+  end
+
+  test "attribute update persists and renders the show state" do
+    frame = "widget_#{@widget.id}_name"
+    put widget_path(@widget, attribute: "name", form_element: "text_field",
+                    update: frame),
+        params: { name: "Renamed" }, headers: frame_headers(frame)
+
+    assert_response :success
+    assert_equal "Renamed", @widget.reload.name
+    assert_includes response.body, "Renamed"
+  end
+
+  test "failed save re-renders the edit state, not a fake show (8.1.19)" do
+    frame = "widget_#{@widget.id}_name"
+    put widget_path(@widget, attribute: "name", form_element: "text_field",
+                    update: frame),
+        params: { name: "" }, headers: frame_headers(frame)
+
+    assert_response :success
+    assert_equal "Alpha", @widget.reload.name, "blank name must not persist"
+    assert_includes response.body, %(name="name"),
+      "expected the edit input to be re-rendered after a failed save"
+  end
+
+  test "destroy responds with the destroyed-row state and an undo pointing at the destroy version" do
+    frame = "widget_#{@widget.id}"
+    assert_difference -> { Widget.count }, -1 do
+      delete widget_path(@widget, update: frame), headers: frame_headers(frame)
+    end
+    assert_response :success
+
+    destroy_version = PaperTrail::Version.where(item_type: "Widget",
+                                                item_id: @widget.id,
+                                                event: "destroy").last
+    assert destroy_version, "expected a PaperTrail destroy version"
+    assert_includes response.body, "/widgets/#{destroy_version.id}/revert",
+      "undo link must target the destroy version (8.1.13)"
+  end
+
+  # revert responds with turbo_stream (replaces row + versions panel at once,
+  # see render_revert_response); the undo/Restore links request that format.
+  def turbo_stream_headers(frame_id)
+    frame_headers(frame_id).merge("Accept" => "text/vnd.turbo-stream.html")
+  end
+
+  test "revert of a destroy version restores the row (undo)" do
+    frame = "widget_#{@widget.id}"
+    delete widget_path(@widget, update: frame), headers: frame_headers(frame)
+    destroy_version = PaperTrail::Version.where(item_type: "Widget",
+                                                item_id: @widget.id,
+                                                event: "destroy").last
+
+    assert_difference -> { Widget.count }, +1 do
+      post revert_widget_path(destroy_version.id, update: "widgets_list"),
+           headers: turbo_stream_headers("widgets_list")
+    end
+    assert_response :success
+    assert Widget.exists?(@widget.id), "undo must restore the destroyed row"
+    assert_equal "Alpha", Widget.find(@widget.id).name
+  end
+
+  test "revert of a destroy version is idempotent when the row already exists (8.1.17)" do
+    frame = "widget_#{@widget.id}"
+    delete widget_path(@widget, update: frame), headers: frame_headers(frame)
+    destroy_version = PaperTrail::Version.where(item_type: "Widget",
+                                                item_id: @widget.id,
+                                                event: "destroy").last
+
+    post revert_widget_path(destroy_version.id, update: "widgets_list"),
+         headers: turbo_stream_headers("widgets_list")
+    assert Widget.exists?(@widget.id)
+
+    # Replaying the same restore must not raise RecordNotUnique.
+    post revert_widget_path(destroy_version.id, update: "widgets_list"),
+         headers: turbo_stream_headers("widgets_list")
+    assert_response :success
+    assert_equal 1, Widget.where(id: @widget.id).count
+  end
+
+  test "versions panel lists an update event" do
+    @widget.update!(name: "Renamed once")
+    frame = "widget_#{@widget.id}"
+
+    get list_versions_widget_path(@widget, update: frame),
+        headers: frame_headers(frame)
+
+    assert_response :success
+    assert_includes response.body, "Renamed once"
+  end
+end

data/test/integration/legacy_elements_test.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+require_relative "../integration_test_helper"
+
+# First-ever render coverage for two legacy display elements that no example
+# app exercises (see stuff/improvement-plan.md §4): simple_file_field (raw
+# filename column + host-provided download route) and pdf_link (host-provided
+# /:attribute/:id report route).
+class LegacyElementsTest < InlineFormsIntegrationTestCase
+  setup do
+    @widget = Widget.create!(name: "Legacy")
+  end
+
+  test "simple_file_field blank state renders the inline-edit plus link" do
+    frame = "widget_#{@widget.id}"
+    get widget_path(@widget, update: frame), headers: frame_headers(frame)
+
+    assert_response :success
+    assert_includes response.body, %(<turbo-frame id="widget_#{@widget.id}_manual">)
+    assert_includes response.body, "fi-plus"
+  end
+
+  test "simple_file_field edit renders a file input" do
+    frame = "widget_#{@widget.id}_manual"
+    get edit_widget_path(@widget, attribute: "manual",
+                         form_element: "simple_file_field", update: frame),
+        headers: frame_headers(frame)
+
+    assert_response :success
+    assert_includes response.body, %(type="file")
+    assert_includes response.body, %(name="manual")
+  end
+
+  test "pdf_link show renders preview and pdf links for the attribute route" do
+    frame = "widget_#{@widget.id}"
+    get widget_path(@widget, update: frame), headers: frame_headers(frame)
+
+    assert_response :success
+    assert_includes response.body, %(href="/report/#{@widget.id}")
+    assert_includes response.body, %(href="/report/#{@widget.id}.pdf")
+  end
+end

data/test/integration/native_inputs_test.rb

@@ -0,0 +1,138 @@
+# frozen_string_literal: true
+
+require_relative "../integration_test_helper"
+
+# Engine-level render coverage for the native-input form elements introduced
+# in 8.1.25/8.1.26 (previously only covered by the example-app gate).
+class NativeInputsTest < InlineFormsIntegrationTestCase
+  setup do
+    @widget = Widget.create!(
+      name: "Native",
+      released_on: Date.new(2026, 5, 17),
+      meeting_at: Time.utc(2000, 1, 1, 9, 15),
+      start_month: Date.new(2026, 5, 1),
+      priority: 2,
+      rating: 3
+    )
+  end
+
+  def edit_attribute(attribute, form_element)
+    frame = "widget_#{@widget.id}_#{attribute}"
+    get edit_widget_path(@widget, attribute: attribute, form_element: form_element,
+                         update: frame),
+        headers: frame_headers(frame)
+    assert_response :success
+    response.body
+  end
+
+  test "date_select renders a native date input with ISO value" do
+    body = edit_attribute("released_on", "date_select")
+    assert_includes body, %(type="date")
+    assert_includes body, %(value="2026-05-17")
+    refute_includes body, "<script"
+  end
+
+  test "time_select renders a native time input with HH:MM value" do
+    body = edit_attribute("meeting_at", "time_select")
+    assert_includes body, %(type="time")
+    assert_includes body, %(value="09:15")
+  end
+
+  test "month_year_picker renders a native month input with YYYY-MM value" do
+    body = edit_attribute("start_month", "month_year_picker")
+    assert_includes body, %(type="month")
+    assert_includes body, %(value="2026-05")
+  end
+
+  test "month_year_picker update accepts native and legacy formats" do
+    frame = "widget_#{@widget.id}_start_month"
+
+    put widget_path(@widget, attribute: "start_month",
+                    form_element: "month_year_picker", update: frame),
+        params: { start_month: "2026-09" }, headers: frame_headers(frame)
+    assert_equal Date.new(2026, 9, 1), @widget.reload.start_month
+
+    put widget_path(@widget, attribute: "start_month",
+                    form_element: "month_year_picker", update: frame),
+        params: { start_month: "November 2027" }, headers: frame_headers(frame)
+    assert_equal Date.new(2027, 11, 1), @widget.reload.start_month
+  end
+
+  test "dropdown_with_other renders a datalist-backed combobox without inline scripts" do
+    Kind.create!(name: "Gadget")
+    Kind.create!(name: "Gizmo")
+
+    body = edit_attribute("kind", "dropdown_with_other")
+    assert_includes body, "<datalist"
+    assert_includes body, %(<option value="Gadget">)
+    assert_includes body, %(<option value="Gizmo">)
+    assert_includes body, %(name="_widget[kind_other]")
+    refute_includes body, "<script"
+  end
+
+  test "dropdown_with_other update matches an existing record by name" do
+    kind = Kind.create!(name: "Gadget")
+    frame = "widget_#{@widget.id}_kind"
+
+    put widget_path(@widget, attribute: "kind",
+                    form_element: "dropdown_with_other", update: frame),
+        params: { _widget: { kind_other: "Gadget" } }, headers: frame_headers(frame)
+
+    assert_response :success
+    @widget.reload
+    assert_equal kind.id, @widget.kind_id
+    assert_nil @widget.kind_other
+  end
+
+  test "dropdown_with_other update stores unmatched text as the other value" do
+    Kind.create!(name: "Gadget")
+    frame = "widget_#{@widget.id}_kind"
+
+    put widget_path(@widget, attribute: "kind",
+                    form_element: "dropdown_with_other", update: frame),
+        params: { _widget: { kind_other: "Completely new thing" } },
+        headers: frame_headers(frame)
+
+    assert_response :success
+    @widget.reload
+    assert_equal 0, @widget.kind_id
+    assert_equal "Completely new thing", @widget.kind_other
+  end
+
+  test "slider_with_values renders a native range input with label data" do
+    body = edit_attribute("rating", "slider_with_values")
+    assert_includes body, %(type="range")
+    assert_includes body, "slider_with_values"
+    assert_includes body, "data-slider-values"
+    assert_includes body, %(<output)
+    refute_includes body, "<script"
+  end
+
+  test "slider_with_values show renders a progress bar (clickable, non-interactive)" do
+    frame = "widget_#{@widget.id}"
+    get widget_path(@widget, update: frame), headers: frame_headers(frame)
+
+    assert_response :success
+    assert_includes response.body, "<progress"
+  end
+
+  test "color_field renders a native color input and round-trips a hex value" do
+    @widget.update!(accent: "#a3381e")
+    body = edit_attribute("accent", "color_field")
+    assert_includes body, %(type="color")
+    assert_includes body, %(value="#a3381e")
+
+    frame = "widget_#{@widget.id}_accent"
+    put widget_path(@widget, attribute: "accent", form_element: "color_field",
+                    update: frame),
+        params: { accent: "#2563EB" }, headers: frame_headers(frame)
+    assert_response :success
+    assert_equal "#2563eb", @widget.reload.accent, "hex is normalized to lowercase"
+
+    # Junk input is rejected server-side (stored as nil), not persisted raw.
+    put widget_path(@widget, attribute: "accent", form_element: "color_field",
+                    update: frame),
+        params: { accent: "red; background:url(x)" }, headers: frame_headers(frame)
+    assert_nil @widget.reload.accent
+  end
+end

data/test/integration/open_after_create_test.rb

@@ -0,0 +1,85 @@
+# frozen_string_literal: true
+
+require_relative "../integration_test_helper"
+
+# "Open after create" (stuff/2026-07-10-nested-creation-options.md, Option A):
+# a successful top-level create of a class with an :associated (has_many)
+# panel responds with Turbo Streams that refresh the list and open the new
+# record's row, so the nested panel is immediately usable. Everything else
+# (classes without :associated, nested creates, failed saves, non-stream
+# clients) keeps the plain frame responses.
+class OpenAfterCreateTest < InlineFormsIntegrationTestCase
+  # Turbo form submissions advertise turbo-stream before text/html.
+  def stream_headers(frame_id)
+    { "Turbo-Frame" => frame_id,
+      "Accept" => "text/vnd.turbo-stream.html, text/html" }
+  end
+
+  test "top-level create with an associated panel streams the list and the opened row" do
+    assert_difference -> { Machine.count }, +1 do
+      post machines_path(update: "machines_list"),
+           params: { name: "Lathe" },
+           headers: stream_headers("machines_list")
+    end
+    assert_response :success
+    assert_equal "text/vnd.turbo-stream.html", response.media_type
+
+    machine = Machine.find_by!(name: "Lathe")
+    assert_includes response.body,
+      %(<turbo-stream action="replace" target="machines_list">)
+    assert_includes response.body,
+      %(<turbo-stream action="replace" target="machine_#{machine.id}">)
+
+    # The opened row exposes the parts panel with its "new part" link,
+    # carrying the freshly created parent's id.
+    assert_includes response.body, %(id="machine_#{machine.id}_parts_list_auto_header")
+    assert_match(
+      %r{/parts/new\?[^"]*parent_id=#{machine.id}},
+      response.body,
+      "expected the associated panel's new-record link for the created parent"
+    )
+  end
+
+  test "top-level create without turbo-stream accept keeps the list frame response" do
+    post machines_path(update: "machines_list"),
+         params: { name: "Mill" },
+         headers: frame_headers("machines_list")
+    assert_response :success
+    assert_equal "text/html", response.media_type
+    assert_includes response.body, %(<turbo-frame id="machines_list">)
+  end
+
+  test "create of a class without an associated panel is unchanged" do
+    post parts_path(update: "parts_list"),
+         params: { name: "Plain" },
+         headers: stream_headers("parts_list")
+    assert_response :success
+    assert_equal "text/html", response.media_type
+    assert_includes response.body, %(<turbo-frame id="parts_list">)
+  end
+
+  test "nested create under a parent keeps the frame response" do
+    machine = Machine.create!(name: "Host")
+    frame = "machine_#{machine.id}_parts"
+
+    assert_difference -> { Part.count }, +1 do
+      post parts_path(update: frame, parent_class: "Machine", parent_id: machine.id),
+           params: { name: "Bolt" },
+           headers: stream_headers(frame)
+    end
+    assert_response :success
+    assert_equal "text/html", response.media_type
+    assert_equal machine.id, Part.find_by!(name: "Bolt").machine_id
+  end
+
+  test "failed top-level create re-renders the new form, not streams" do
+    assert_no_difference -> { Machine.count } do
+      post machines_path(update: "machines_list"),
+           params: { name: "" },
+           headers: stream_headers("machines_list")
+    end
+    assert_response :success
+    assert_equal "text/html", response.media_type
+    assert_includes response.body, 'class="edit_form"'
+  end
+end

data/test/integration_test_helper.rb

@@ -0,0 +1,80 @@
+# frozen_string_literal: true
+
+# Boots the test/dummy host app for the engine's integration tests. Fast path
+# (seconds) next to the full release gate (build gems -> inline_forms create
+# MyApp --example -> rails test); catches controller/view/Turbo regressions
+# without an install cycle. See stuff/prompt/test-the-example-app.md for the
+# release gate, which remains authoritative.
+ENV["RAILS_ENV"] = "test"
+
+require_relative "dummy/config/environment"
+require "rails/test_help"
+require "minitest/autorun"
+
+# In-memory SQLite: define the schema on every boot (maintain_test_schema is
+# off; there are no migration files to run).
+ActiveRecord::Schema.verbose = false
+ActiveRecord::Schema.define do
+  create_table :widgets, force: true do |t|
+    t.string  :name
+    t.text    :notes
+    t.date    :released_on
+    t.time    :meeting_at
+    t.date    :start_month
+    t.integer :priority
+    t.boolean :active
+    t.integer :kind_id
+    t.string  :kind_other
+    t.integer :rating
+    t.string  :manual
+    t.string  :accent
+    t.timestamps
+  end
+
+  create_table :kinds, force: true do |t|
+    t.string :name
+    t.timestamps
+  end
+
+  # Machine has_many :parts with an :associated panel (open-after-create).
+  create_table :machines, force: true do |t|
+    t.string :name
+    t.timestamps
+  end
+
+  create_table :parts, force: true do |t|
+    t.string  :name
+    t.integer :machine_id
+    t.timestamps
+  end
+  add_index :parts, :machine_id
+
+  # PaperTrail (paper_trail 17.x shape).
+  create_table :versions, force: true do |t|
+    t.string   :item_type, null: false
+    t.bigint   :item_id, null: false
+    t.string   :event, null: false
+    t.string   :whodunnit
+    t.text     :object, limit: 1_073_741_823
+    t.text     :object_changes, limit: 1_073_741_823
+    t.datetime :created_at
+  end
+  add_index :versions, [ :item_type, :item_id ]
+end
+
+class InlineFormsIntegrationTestCase < ActionDispatch::IntegrationTest
+  # The inline UI drives every request through a <turbo-frame>; the frame
+  # layout (turbo_rails/frame) also spares the dummy the full chrome (header
+  # needs current_user, assets, ...).
+  def frame_headers(frame_id)
+    { "Turbo-Frame" => frame_id, "Accept" => "text/html" }
+  end
+
+  setup do
+    Widget.delete_all
+    Kind.delete_all
+    Part.delete_all
+    Machine.delete_all
+    PaperTrail::Version.delete_all
+  end
+end

data/test/plain_text_configuration_test.rb

@@ -20,7 +20,7 @@ class PlainTextConfigurationTest < Minitest::Test
 
     def inline_forms_attribute_list
       [
-        [:description, :plain_text]
+        [ :description, :plain_text ]
       ]
     end
   end
@@ -40,7 +40,7 @@ class PlainTextConfigurationTest < Minitest::Test
 
     def inline_forms_attribute_list
       [
-        [:description, :plain_text]
+        [ :description, :plain_text ]
       ]
     end
   end
@@ -56,7 +56,7 @@ class PlainTextConfigurationTest < Minitest::Test
 
     def inline_forms_attribute_list
       [
-        [:description, :rich_text]
+        [ :description, :rich_text ]
       ]
     end
   end