decidim-decidim_awesome 0.6.7 → 0.8.1

data/lib/decidim/decidim_awesome/custom_fields.rb

@@ -0,0 +1,94 @@
+# frozen_string_literal: true
+
+module Decidim
+  module DecidimAwesome
+    class CustomFields
+      def initialize(fields)
+        @fields = if fields.respond_to? :map
+                    fields.map { |f| JSON.parse(f) }.flatten
+                  else
+                    JSON.parse(fields)
+                  end
+      end
+
+      attr_reader :fields, :xml, :errors, :data
+
+      def apply_xml(xml)
+        parse_xml(xml)
+        map_fields!
+      rescue StandardError => e
+        @errors = e.message
+      end
+
+      def to_json(*_args)
+        @fields
+      end
+
+      def translate!
+        translate_values!
+      end
+
+      private
+
+      def parse_xml(xml)
+        @xml = xml
+        @data = Nokogiri.XML(xml).xpath("//dl/dd")
+        return if @data.present?
+
+        apply_to_first_textarea
+      end
+
+      def map_fields!
+        return unless data
+
+        @fields.map! do |field|
+          if field["name"] # ignore headers/paragraphs
+            value = data.search("##{field["name"]} div")
+            field["userData"] = value.map { |v| v.attribute("alt")&.value || v.inner_html(encoding: "UTF-8") } if value.present?
+          end
+
+          field
+        end
+      end
+
+      # Finds the first textarea and applies non-xml compatible content
+      # when textarea has not wysiwyg assigned, strips html
+      def apply_to_first_textarea
+        # quill editor might leave html traces without any user content
+        # so we won't process it if there is no text (html free) result
+        text = Nokogiri.HTML(xml).html? ? Nokogiri.HTML(xml).text.strip : text.strip
+        return if text.blank?
+
+        textarea = @fields.find { |field| field["type"] == "textarea" }
+        @errors = I18n.t(".invalid_xml", scope: "decidim.decidim_awesome.custom_fields.errors")
+        return unless textarea
+
+        textarea["userData"] = [textarea["subtype"] == "textarea" ? text : xml]
+        @errors = I18n.t(".invalid_fields", scope: "decidim.decidim_awesome.custom_fields.errors", field: textarea["label"] || textarea["name"])
+      end
+
+      def translate_values!
+        deep_transform_values!(@fields) do |value|
+          next value unless value.is_a? String
+          next value unless (match = value.match(/^(.*\..*)$/))
+
+          I18n.t(match[1], raise: true)
+        rescue I18n::MissingTranslationData
+          value
+        end
+        @fields
+      end
+
+      def deep_transform_values!(object, &block)
+        case object
+        when Hash
+          object.transform_values! { |value| deep_transform_values!(value, &block) }
+        when Array
+          object.map! { |e| deep_transform_values!(e, &block) }
+        else
+          yield(object)
+        end
+      end
+    end
+  end
+end

data/lib/decidim/decidim_awesome/engine.rb

@@ -8,30 +8,58 @@ module Decidim
   module DecidimAwesome
     # This is the engine that runs on the public interface of decidim_awesome.
     class Engine < ::Rails::Engine
+      include AwesomeHelpers
+
       isolate_namespace Decidim::DecidimAwesome
 
       routes do
         post :editor_images, to: "editor_images#create"
       end
 
-      initializer "decidim_awesome.view_helpers" do
-        ActionView::Base.send :include, AwesomeHelpers
-      end
-
-      initializer "decidim_decidim_awesome.assets" do |app|
-        app.config.assets.precompile += %w(decidim_decidim_awesome_manifest.js decidim_decidim_awesome_manifest.css)
-        # add to precompile any present theme asset
-        Dir.glob(Rails.root.join("app", "assets", "themes", "*.*")).each do |path|
-          app.config.assets.precompile << path
-        end
-      end
-
       # Prepare a zone to create overrides
       # https://edgeguides.rubyonrails.org/engines.html#overriding-models-and-controllers
+      # overrides
       config.to_prepare do
-        Dir.glob("#{Engine.root}/app/awesome_overrides/**/*_override.rb").each do |override|
-          require_dependency override
+        # activate Decidim LayoutHelper for the overriden views
+        ActiveSupport.on_load :action_controller do
+          helper Decidim::LayoutHelper if respond_to?(:helper)
+        end
+        # Include additional helpers globally
+        ActionView::Base.include(Decidim::DecidimAwesome::AwesomeHelpers)
+
+        # override user's admin property
+        Decidim::User.include(Decidim::DecidimAwesome::UserOverride) if DecidimAwesome.enabled?(:scoped_admins)
+
+        # redirect unauthorized scoped admins to allowed places or custom redirects if configured
+        Decidim::ErrorsController.include(Decidim::DecidimAwesome::NotFoundRedirect) if DecidimAwesome.enabled?([:scoped_admins, :custom_redirects])
+
+        # Custom fields need to deal with several places
+        if DecidimAwesome.enabled?(:proposal_custom_fields)
+          Decidim::Proposals::ApplicationHelper.include(Decidim::DecidimAwesome::Proposals::ApplicationHelperOverride)
+          Decidim::Proposals::ProposalWizardCreateStepForm.include(Decidim::DecidimAwesome::Proposals::ProposalWizardCreateStepFormOverride)
+          Decidim::AmendmentsHelper.include(Decidim::DecidimAwesome::AmendmentsHelperOverride)
         end
+
+        Decidim::MenuPresenter.include(Decidim::DecidimAwesome::MenuPresenterOverride)
+        Decidim::MenuItemPresenter.include(Decidim::DecidimAwesome::MenuItemPresenterOverride)
+
+        # Late registering of components to take into account initializer values
+        DecidimAwesome.registered_components.each do |manifest, block|
+          Decidim.register_component(manifest, &block) unless DecidimAwesome.disabled_components.include?(manifest)
+        end
+      end
+
+      initializer "decidim.middleware" do |app|
+        app.config.middleware.insert_after Decidim::Middleware::CurrentOrganization, Decidim::DecidimAwesome::CurrentConfig
+      end
+
+      initializer "decidim_decidim_awesome.webpacker.assets_path" do
+        Decidim.register_assets_path File.expand_path("app/packs", root)
+      end
+
+      initializer "decidim_decidim_awesome.add_cells_view_paths" do
+        Cell::ViewModel.view_paths << File.expand_path("#{Decidim::DecidimAwesome::Engine.root}/app/cells")
+        Cell::ViewModel.view_paths << File.expand_path("#{Decidim::DecidimAwesome::Engine.root}/app/views")
       end
     end
   end

data/lib/decidim/decidim_awesome/iframe_component/admin_engine.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+require "rails"
+require "decidim/core"
+
+module Decidim
+  module DecidimAwesome
+    module IframeComponent
+      # This is the engine is used to create the component Map.
+      class AdminEngine < ::Rails::Engine
+        isolate_namespace Decidim::DecidimAwesome::IframeComponent
+
+        routes do
+          root to: "iframe#settings"
+        end
+
+        def load_seed
+          nil
+        end
+      end
+    end
+  end
+end

data/lib/decidim/decidim_awesome/iframe_component/component.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-require_dependency "decidim/components/namer"
+require "decidim/components/namer"
 
-Decidim.register_component(:awesome_iframe) do |component|
+Decidim::DecidimAwesome.register_component(:awesome_iframe) do |component|
   component.engine = Decidim::DecidimAwesome::IframeComponent::Engine
-  # component.admin_engine = Decidim::DecidimAwesome::AdminIframeEngine
+  component.admin_engine = Decidim::DecidimAwesome::IframeComponent::AdminEngine
   component.icon = "decidim/meetings/icon.svg" # TODO: create a Icon
   component.permissions_class_name = "Decidim::DecidimAwesome::Permissions"
 
@@ -15,7 +15,7 @@ Decidim.register_component(:awesome_iframe) do |component|
     # Add your global settings
     # Available types: :integer, :boolean
     settings.attribute :announcement, type: :text, translated: true, editor: true
-    settings.attribute :iframe, type: :text
+    settings.attribute :iframe, type: :text, default: ""
     settings.attribute :viewport_width, type: :boolean, default: false
     settings.attribute :no_margins, type: :boolean, default: false
   end
@@ -23,7 +23,7 @@ Decidim.register_component(:awesome_iframe) do |component|
   component.settings(:step) do |settings|
     # Add your settings per step
     settings.attribute :announcement, type: :text, translated: true, editor: true
-    settings.attribute :iframe, type: :text
+    settings.attribute :iframe, type: :text, default: ""
   end
 
   # component.register_stat :some_stat do |context, start_at, end_at|
@@ -43,7 +43,7 @@ Decidim.register_component(:awesome_iframe) do |component|
       published_at: Time.current,
       participatory_space: participatory_space,
       settings: {
-        announcement: { en: Faker::Lorem.paragraphs(2).join("\n") }
+        announcement: { en: Faker::Lorem.paragraphs(number: 2).join("\n") }
       }
     }
 

data/lib/decidim/decidim_awesome/map_component/admin_engine.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+require "rails"
+require "decidim/core"
+
+module Decidim
+  module DecidimAwesome
+    module MapComponent
+      # This is the engine is used to create the component Map.
+      class AdminEngine < ::Rails::Engine
+        isolate_namespace Decidim::DecidimAwesome::MapComponent
+
+        routes do
+          root to: "map#settings"
+        end
+
+        def load_seed
+          nil
+        end
+      end
+    end
+  end
+end

data/lib/decidim/decidim_awesome/map_component/component.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-require_dependency "decidim/components/namer"
+require "decidim/components/namer"
 
-Decidim.register_component(:awesome_map) do |component|
+Decidim::DecidimAwesome.register_component(:awesome_map) do |component|
   component.engine = Decidim::DecidimAwesome::MapComponent::Engine
-  # component.admin_engine = Decidim::DecidimAwesome::AdminMapEngine
+  component.admin_engine = Decidim::DecidimAwesome::MapComponent::AdminEngine
   component.icon = "decidim/meetings/icon.svg" # TODO: create a Icon
   component.permissions_class_name = "Decidim::DecidimAwesome::Permissions"
 
@@ -22,7 +22,9 @@ Decidim.register_component(:awesome_map) do |component|
     settings.attribute :collapse, type: :boolean, default: false
     settings.attribute :menu_amendments, type: :boolean, default: true
     settings.attribute :menu_meetings, type: :boolean, default: true
+    settings.attribute :menu_categories, type: :boolean, default: true
     settings.attribute :menu_hashtags, type: :boolean, default: true
+    settings.attribute :menu_merge_components, type: :boolean, default: false
   end
 
   component.settings(:step) do |settings|
@@ -31,7 +33,6 @@ Decidim.register_component(:awesome_map) do |component|
     settings.attribute :show_not_answered, type: :boolean, default: true
     settings.attribute :show_evaluating, type: :boolean, default: true
     settings.attribute :show_accepted, type: :boolean, default: true
-    # Not possible yet (needs graphql update):
     settings.attribute :show_rejected, type: :boolean, default: false
     settings.attribute :show_withdrawn, type: :boolean, default: false
   end
@@ -53,7 +54,7 @@ Decidim.register_component(:awesome_map) do |component|
       published_at: Time.current,
       participatory_space: participatory_space,
       settings: {
-        announcement: { en: Faker::Lorem.paragraphs(2).join("\n") }
+        announcement: { en: Faker::Lorem.paragraphs(number: 2).join("\n") }
       }
     }
 

data/lib/decidim/decidim_awesome/map_component/engine.rb

@@ -14,6 +14,40 @@ module Decidim
           root to: "map#show"
         end
 
+        initializer "decidim_decidim_awesome.content_blocks" do |_app|
+          # do not register this if awesome_map is disabled
+          next if DecidimAwesome.disabled_components.include?(:awesome_map)
+
+          # === Home Map block ===
+          Decidim.content_blocks.register(:homepage, :awesome_map) do |content_block|
+            content_block.cell = "decidim/decidim_awesome/content_blocks/map"
+            content_block.settings_form_cell = "decidim/decidim_awesome/content_blocks/map_form"
+            content_block.public_name_key = "decidim.decidim_awesome.content_blocks.map.name"
+
+            content_block.settings do |settings|
+              settings.attribute :title, type: :text, translated: true
+
+              settings.attribute :map_height, type: :integer, default: 500
+              settings.attribute :map_center, type: :string, default: ""
+              settings.attribute :map_zoom, type: :integer, default: 8
+              settings.attribute :truncate, type: :integer, default: 255
+              settings.attribute :collapse, type: :boolean, default: false
+              settings.attribute :menu_amendments, type: :boolean, default: true
+              settings.attribute :menu_meetings, type: :boolean, default: true
+              settings.attribute :menu_hashtags, type: :boolean, default: true
+              settings.attribute :menu_categories, type: :boolean, default: true
+              settings.attribute :menu_merge_components, type: :boolean, default: true
+
+              settings.attribute :show_not_answered, type: :boolean, default: true
+              settings.attribute :show_accepted, type: :boolean, default: true
+              settings.attribute :show_withdrawn, type: :boolean, default: false
+              settings.attribute :show_evaluating, type: :boolean, default: true
+              settings.attribute :show_rejected, type: :boolean, default: false
+            end
+          end
+          # === TODO: processes groups map block ===
+        end
+
         def load_seed
           nil
         end

data/lib/decidim/decidim_awesome/menu_hacker.rb

@@ -4,6 +4,7 @@ module Decidim
   module DecidimAwesome
     class MenuHacker
       include Decidim::TranslatableAttributes
+
       def initialize(name, view)
         @name = name
         @organization = view.try(:current_organization)
@@ -12,7 +13,7 @@ module Decidim
       end
 
       # returns a combined array of the Decidim defined menu and the hacked stored as config vars
-      def items(include_invisible = false)
+      def items(include_invisible: false)
         return @items if @items
 
         @items = default_items

data/lib/decidim/decidim_awesome/system_checker.rb

@@ -14,7 +14,7 @@ module Decidim
           @overrides = checksums.map do |package, files|
             props = {
               spec: ::Gem::Specification.find_by_name(package),
-              files: files.map { |file, signatures| [file, signatures.values] }.to_h
+              files: files.transform_values(&:values)
             }
             [package, to_struct(props)]
           end

data/lib/decidim/decidim_awesome/test/factories.rb

@@ -15,8 +15,8 @@ FactoryBot.define do
     awesome_config { create :awesome_config }
   end
 
-  factory :editor_image, class: "Decidim::DecidimAwesome::EditorImage" do
-    image { Decidim::Dev.test_file("city.jpeg", "image/jpeg") }
+  factory :awesome_editor_image, class: "Decidim::DecidimAwesome::EditorImage" do
+    file { Decidim::Dev.test_file("city.jpeg", "image/jpeg") }
     path { Faker::Internet.url(host: "", scheme: "") }
     author { create :user }
     organization { create :organization }

data/lib/decidim/decidim_awesome/test/initializer.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+Decidim::DecidimAwesome.configure do |config|
+  if ENV["FEATURES"] == "disabled"
+    [
+      :allow_images_in_full_editor,
+      :allow_images_in_small_editor,
+      :allow_images_in_proposals,
+      :use_markdown_editor,
+      :allow_images_in_markdown_editor,
+      :auto_save_forms,
+      :intergram_for_admins,
+      :intergram_for_public,
+      :scoped_styles,
+      :proposal_custom_fields,
+      :menu,
+      :scoped_admins,
+      :custom_redirects
+    ].each do |conf|
+      config.send("#{conf}=", :disabled)
+    end
+
+    config.disabled_components = [:awesome_map, :awesome_iframe]
+  end
+end

data/lib/decidim/decidim_awesome/test/layouts/decidim/_head.html.erb

@@ -1,2 +1,30 @@
-<%= stylesheet_link_tag    "application", media: "all" %>
-<%= javascript_include_tag "application" %>
+<%= csrf_meta_tags %>
+
+<meta name="viewport" content="width=device-width, initial-scale=1.0">
+<% available_locales.each do |locale| %>
+  <link rel="alternate" href="<%= url_for(request.parameters.merge(locale: locale)) %>" hreflang="<%= locale %>">
+<% end %>
+
+<meta name="twitter:card" content="summary_large_image">
+<meta name="twitter:site" content="@<%= decidim_meta_twitter_handler %>">
+<meta name="twitter:title" content="<%= decidim_page_title %>">
+<meta name="twitter:description" content="<%= decidim_meta_description %>">
+<meta name="twitter:image" content="<%= decidim_meta_image_url %>">
+
+<meta property="og:url" content="<%= decidim_meta_url %>">
+<meta property="og:type" content="article">
+<meta property="og:title" content="<%= decidim_page_title %>">
+<meta property="og:description" content="<%= decidim_meta_description %>">
+<meta property="og:image" content="<%= decidim_meta_image_url %>">
+
+<%= favicon %>
+<%= stylesheet_pack_tag "decidim_core", media: "all" %>
+<%= invisible_captcha_styles %>
+
+<%= organization_colors %>
+<%= javascript_pack_tag "decidim_core", defer: false %>
+
+<%= render partial: "layouts/decidim/head_extra" %>
+<%== current_organization.header_snippets if Decidim.enable_html_header_snippets %>
+<%= snippets.display(:head) %>
+<%= content_for :header_snippets %>

data/lib/decidim/decidim_awesome/test/layouts/decidim/admin/_header.html.erb

@@ -1,3 +1,8 @@
-<%= stylesheet_link_tag    "decidim/admin/application", media: "all" %>
-<%= javascript_include_tag "decidim/admin/application" %>
+<meta name="viewport" content="width=device-width, initial-scale=1">
+<%= csrf_meta_tags %>
+<%= organization_colors %>
 
+<%= stylesheet_pack_tag "decidim_admin", media: "all" %>
+<%= javascript_pack_tag "decidim_admin", defer: false %>
+
+<%= snippets.display(:head) %>

data/lib/decidim/decidim_awesome/test/shared_examples/box_label_editor.rb

@@ -0,0 +1,116 @@
+# frozen_string_literal: true
+
+shared_examples "edits box label inline" do |test_case, key|
+  let(:data) { '[{"type":"text","label":"Short Name","subtype":"text","className":"form-control","name":"text-1476748004559"}]' }
+
+  it "updates the label when no changes" do
+    link = find("[data-key=#{key}] a.awesome-auto-edit", match: :first)
+    expect(page).not_to have_css("input.awesome-auto-edit")
+    link.click
+    expect(page).to have_css("input.awesome-auto-edit")
+    find("body").click
+    expect(page).not_to have_css("input.awesome-auto-edit")
+    link.click
+    input = find("[data-key=#{key}] input.awesome-auto-edit")
+    input.fill_in with: "A new làbel\n"
+    sleep 1
+    expect(page).not_to have_css("input.awesome-auto-edit")
+    expect(page).to have_css("span.awesome-auto-edit[data-key=a_new_label]")
+
+    find("*[type=submit]").click
+    case test_case
+    when :css
+      expect(page).to have_content("body {background: red;}")
+    when :fields
+      expect(page).to have_content("Occupation")
+      expect(page).to have_content("Street Sweeper")
+      expect(page).not_to have_content("Short Bio")
+    when :admins
+      expect(page).not_to have_content(user.name.to_s)
+      expect(page).to have_content(user2.name.to_s)
+      expect(page).to have_content(user3.name.to_s)
+    end
+    expect(page).to have_css("span.awesome-auto-edit[data-key=a_new_label]")
+    expect(page).not_to have_css("span.awesome-auto-edit[data-key=#{key}]")
+  end
+
+  it "updates the label with previous changes" do
+    case test_case
+    when :css
+      sleep 1
+      page.execute_script("document.querySelector(\"[data-key=#{key}] .CodeMirror\").CodeMirror.setValue(\"div {background: brown;}\");")
+    when :fields
+      sleep 2
+      page.execute_script("$('.proposal_custom_fields_container[data-key=#{key}] .proposal_custom_fields_editor')[0].FormBuilder.actions.setData(#{data})")
+    when :admins
+      sleep 1
+      page.execute_script("$('.multiusers-select:first').append(new Option('#{user.name}', #{user.id}, true, true)).trigger('change');")
+    end
+
+    link = find("[data-key=#{key}] a.awesome-auto-edit", match: :first)
+    expect(page).not_to have_css("input.awesome-auto-edit")
+    link.click
+    expect(page).to have_css("input.awesome-auto-edit")
+    find("body").click
+    expect(page).not_to have_css("input.awesome-auto-edit")
+    link.click
+    input = find("[data-key=#{key}] input.awesome-auto-edit")
+    input.fill_in with: "A new làbel\n"
+    sleep 1
+    expect(page).not_to have_css("input.awesome-auto-edit")
+    expect(page).to have_css("span.awesome-auto-edit[data-key=a_new_label]")
+
+    find("*[type=submit]").click
+    case test_case
+    when :css
+      expect(page).to have_admin_callout("updated successfully")
+      expect(page).to have_content("div {background: brown;}")
+    when :fields
+      expect(page).to have_content("Short Name")
+    when :admins
+      expect(page).to have_content(user.name.to_s)
+      expect(page).to have_content(user2.name.to_s)
+      expect(page).to have_content(user3.name.to_s)
+    end
+    expect(page).to have_css("span.awesome-auto-edit[data-key=a_new_label]")
+    expect(page).not_to have_css("span.awesome-auto-edit[data-key=#{key}]")
+  end
+
+  it "updates the label with post changes" do
+    link = find("[data-key=#{key}] a.awesome-auto-edit", match: :first)
+    expect(page).not_to have_css("input.awesome-auto-edit")
+    link.click
+    expect(page).to have_css("input.awesome-auto-edit")
+    find("body").click
+    expect(page).not_to have_css("input.awesome-auto-edit")
+    link.click
+    input = find("[data-key=#{key}] input.awesome-auto-edit")
+    input.fill_in with: "A new làbel\n"
+    sleep 1
+    expect(page).not_to have_css("input.awesome-auto-edit")
+    expect(page).to have_css("span.awesome-auto-edit[data-key=a_new_label]")
+
+    case test_case
+    when :css
+      sleep 1
+      page.execute_script('document.querySelector("[data-key=a_new_label] .CodeMirror").CodeMirror.setValue("body {background: lilac;}");')
+      find("*[type=submit]").click
+      expect(page).to have_admin_callout("updated successfully")
+      expect(page).to have_content("body {background: lilac;}")
+    when :fields
+      expect(page).to have_content("Full Name")
+      sleep 2
+      page.execute_script("$('.proposal_custom_fields_container[data-key=a_new_label] .proposal_custom_fields_editor')[0].FormBuilder.actions.setData(#{data})")
+      find("*[type=submit]").click
+      expect(page).to have_content("Short Name")
+    when :admins
+      page.execute_script("$('.multiusers-select:first').append(new Option('#{user.name}', #{user.id}, true, true)).trigger('change');")
+      find("*[type=submit]").click
+      expect(page).to have_content(user.name.to_s)
+      expect(page).to have_content(user2.name.to_s)
+      expect(page).to have_content(user3.name.to_s)
+    end
+    expect(page).to have_css("span.awesome-auto-edit[data-key=a_new_label]")
+    expect(page).not_to have_css("span.awesome-auto-edit[data-key=#{key}]")
+  end
+end

data/lib/decidim/decidim_awesome/test/shared_examples/config_examples.rb

@@ -3,6 +3,7 @@
 shared_examples "javascript config vars" do
   it "has DecidimAwesome object" do
     expect(page.body).to have_content("window.DecidimAwesome")
+    expect(page.body).to have_content("window.DecidimAwesome.version")
     expect(page.body).to have_content("window.DecidimAwesome.editor_uploader_path")
     expect(page.body).to have_content("window.DecidimAwesome.texts")
   end
@@ -25,3 +26,20 @@ shared_examples "do not have menu link" do |item|
     end
   end
 end
+
+shared_examples "forbids disabled feature" do
+  let(:feature) { :menu }
+  let(:features) { [feature] }
+  before do
+    features.each do |feat|
+      allow(Decidim::DecidimAwesome.config).to receive(feat).and_return(:disabled)
+    end
+  end
+
+  it "redirects with error" do
+    action
+
+    expect(flash[:alert]).not_to be_empty
+    expect(response).to redirect_to("/admin/")
+  end
+end

data/lib/decidim/decidim_awesome/test/shared_examples/controller_examples.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+shared_examples "a blank component" do |engine|
+  routes { engine.routes }
+
+  describe "GET settings" do
+    it "redirects to settings" do
+      get :settings, params: { component_id: component }
+      expect(response).to have_http_status(:redirect)
+      expect(response.location).to include("#{component.id}/edit")
+    end
+  end
+end