+ <%= description %> +
+Content
", + ca: "Contingut
", + es: "Contenido
" + } + end + + let(:attributes) do + { + static_page: { + slug: "help", + title: { en: "Help", ca: "Ajuda", es: "Ayuda" }, + content:, + attachment: attachment_params + } + } + end + + context "when everything is OK" do + it { is_expected.to be_valid } + end + + describe "attachment attributes added by decorator" do + it "responds to documents" do + expect(subject).to respond_to(:documents) + end + + it "responds to add_documents" do + expect(subject).to respond_to(:add_documents) + end + + it "responds to photos" do + expect(subject).to respond_to(:photos) + end + + it "responds to add_photos" do + expect(subject).to respond_to(:add_photos) + end + + it "responds to attachment" do + expect(subject).to respond_to(:attachment) + end + end +end diff --git a/spec/decorators/decidim/admin/update_static_page_decorator_spec.rb b/spec/decorators/decidim/admin/update_static_page_decorator_spec.rb new file mode 100644 index 00000000..bc61ec6c --- /dev/null +++ b/spec/decorators/decidim/admin/update_static_page_decorator_spec.rb @@ -0,0 +1,112 @@ +# frozen_string_literal: true + +require "rails_helper" +require "spec_helper" + +describe Decidim::Admin::UpdateStaticPage do + let(:form_klass) { Decidim::Admin::StaticPageForm } + + let(:organization) { create(:organization) } + let(:user) { create(:user, :admin, :confirmed, organization:) } + let(:attachment_params) { nil } + let(:uploaded_photos) { [] } + + let(:form) do + form_klass.from_params( + form_params + ).with_context( + current_organization: organization, + current_user: user + ) + end + + let!(:static_page) { create(:static_page, organization:) } + + describe "call" do + let(:form_params) do + { + static_page: { + id: static_page.id, + slug: static_page.slug, + title: { en: "Updated title" }, + content: { en: "Updated content
" }, + weight: 5, + allow_public_access: true, + add_documents: nil, + add_photos: uploaded_photos + } + } + end + + let(:command) do + described_class.new(form, static_page) + end + + describe "when the form is not valid" do + before do + allow(form).to receive(:invalid?).and_return(true) + end + + it "broadcast invalid" do + expect { command.call }.to broadcast(:invalid) + end + + it "does not update the static page" do + expect do + command.call + end.not_to(change { static_page.reload.content }) + end + end + + describe "when the form is valid" do + it "broadcast ok" do + expect { command.call }.to broadcast(:ok) + end + + it "updates the static page content" do + expect do + command.call + end.to(change { static_page.reload.content["en"] }) + end + + it "updates the static page title" do + expect do + command.call + end.to(change { static_page.reload.title["en"] }.to("Updated title")) + end + + it "updates the static page weight" do + expect do + command.call + end.to(change { static_page.reload.weight }.to(5)) + end + + it "updates the static page allow_public_access" do + expect do + command.call + end.to(change { static_page.reload.allow_public_access }.to(true)) + end + + context "when photos are uploaded" do + let(:attachment_params) do + blob = ActiveStorage::Blob.create_and_upload!( + io: Rack::Test::UploadedFile.new(Decidim::Core::Engine.root.join("db", "seeds", "city.jpeg"), "image/jpeg"), + filename: "city.jpeg", + content_type: "image/jpeg" + ) + { + title: "My attachment", + file: blob.signed_id + } + end + let(:uploaded_photos) { [attachment_params] } + + it "creates an attachment for the static page" do + expect { command.call }.to change(Decidim::Attachment, :count).by(1) + last_attachment = Decidim::Attachment.last + expect(last_attachment.attached_to).to eq(static_page) + end + end + end + end +end diff --git a/spec/system/admin/static_pages/admin_static_page_form_spec.rb b/spec/system/admin/static_pages/admin_static_page_form_spec.rb new file mode 100644 index 00000000..ca211552 --- /dev/null +++ b/spec/system/admin/static_pages/admin_static_page_form_spec.rb @@ -0,0 +1,32 @@ +# frozen_string_literal: true + +require "rails_helper" + +describe "Admin static page form", type: :system do + let(:organization) { create(:organization) } + let!(:user) { create(:user, :admin, :confirmed, organization:) } + let!(:tos_page) { organization.static_pages.find_by!(slug: "terms-of-service") } + let!(:other_page) { create(:static_page, organization:) } + + before do + switch_to_host(organization.host) + login_as user, scope: :user + end + + describe "editing a terms-of-service static page" do + it "does not show the attachment fields" do + visit decidim_admin.edit_static_page_path(tos_page) + + expect(page).to have_no_css("label", text: /documents/i) + expect(page).to have_no_css("label", text: /imatge/i) + end + end + + describe "editing a regular static page" do + it "shows the attachment fields" do + visit decidim_admin.edit_static_page_path(other_page) + + expect(page).to have_css("legend", text: I18n.t("decidim.admin.static_pages.form.attachment_legend")) + end + end +end