<%= t(".archive_help") %>
+| <%= t("models.debate.fields.title", scope: "decidim.debates") %> | -<%= t("models.debate.fields.start_time", scope: "decidim.debates") %> | -<%= t("models.debate.fields.end_time", scope: "decidim.debates") %> | - <%= th_resource_scope_label %> -<%= t("actions.title", scope: "decidim.debates") %> | -||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
|
- <%= link_to present(debate).title, resource_locator(debate).path, target: :blank %> - |
- - <% if debate.start_time %> - <%= l debate.start_time, format: :long %> - <% end %> - | -- <% if debate.end_time %> - <%= l debate.end_time, format: :long %> - <% end %> - | - <%= td_resource_scope_for(debate.scope) %> -
- <% if allowed_to? :update, :debate, debate: debate %>
- <%= icon_link_to "pencil", edit_debate_path(debate), t("actions.edit", scope: "decidim.debates"), class: "action-icon--edit" %>
- <% end %>
+ <% if debates.any? %>
+
+
+ <% if allowed_to? :archive, :debate, debate: debate %>
+ <%= icon_link_to "folder", archive_debate_path(id: debate.id, archive: !debate.archived?), t("actions.#{ debate.archived? ? "unarchive" : "archive" }", scope: "decidim.debates"), class: "action-icon--archive #{"action-icon--highlighted" if debate.archived?}", method: :post %>
+ <% else %>
+
+ <% end %>
+
+ <% if allowed_to? :delete, :debate, debate: debate %>
+ <%= icon_link_to "circle-x", debate_path(debate), t("actions.destroy", scope: "decidim.debates"), method: :delete, class: "action-icon--remove", data: { confirm: t("actions.confirm_destroy", scope: "decidim.debates") } %>
+ <% else %>
+
+ <% end %>
+
|
+
", "
") do - Decidim::Faker::Localized.paragraph(3) + Decidim::Faker::Localized.paragraph(sentence_count: 3) end, instructions: Decidim::Faker::Localized.wrapped("", "
") do - Decidim::Faker::Localized.paragraph(3) + Decidim::Faker::Localized.paragraph(sentence_count: 3) end, start_time: 3.weeks.from_now, end_time: 3.weeks.from_now + 4.hours, @@ -109,22 +109,27 @@ Decidim::Comments::Seed.comments_for(debate) end - closed_debate = Decidim::Debates::Debate.last - closed_debate.conclusions = Decidim::Faker::Localized.wrapped("", "
") do - Decidim::Faker::Localized.paragraph(3) + Decidim::Debates::Debate.last(2).each do |debate| + debate.conclusions = Decidim::Faker::Localized.wrapped("", "
") do + Decidim::Faker::Localized.paragraph(sentence_count: 3) + end + debate.closed_at = Time.current + debate.save! end - closed_debate.closed_at = Time.current - closed_debate.save! + + archived_debate = Decidim::Debates::Debate.last + archived_debate.archived_at = Time.current + archived_debate.save! params = { component: component, category: participatory_space.categories.sample, - title: Decidim::Faker::Localized.sentence(2), + title: Decidim::Faker::Localized.sentence(word_count: 2), description: Decidim::Faker::Localized.wrapped("", "
") do - Decidim::Faker::Localized.paragraph(3) + Decidim::Faker::Localized.paragraph(sentence_count: 3) end, instructions: Decidim::Faker::Localized.wrapped("", "
") do - Decidim::Faker::Localized.paragraph(3) + Decidim::Faker::Localized.paragraph(sentence_count: 3) end, author: user } diff --git a/decidim-debates/spec/commands/decidim/debates/admin/archive_debate_spec.rb b/decidim-debates/spec/commands/decidim/debates/admin/archive_debate_spec.rb new file mode 100644 index 000000000000..fc55ee80db4b --- /dev/null +++ b/decidim-debates/spec/commands/decidim/debates/admin/archive_debate_spec.rb @@ -0,0 +1,48 @@ +# frozen_string_literal: true + +require "spec_helper" + +describe Decidim::Debates::Admin::ArchiveDebate do + subject { described_class.new(archive, debate, user) } + + let(:organization) { create :organization, available_locales: [:en, :ca, :es], default_locale: :en } + let(:participatory_process) { create :participatory_process, organization: organization } + let(:current_component) { create :component, participatory_space: participatory_process, manifest_name: "debates" } + let(:category) { create :category, participatory_space: participatory_process } + let(:user) { create :user, :admin, :confirmed, organization: organization } + let(:debate) { create :debate, component: current_component, archived_at: archived_at } + let(:archive) { true } + let(:archived_at) { nil } + + context "when the debate object is not valid" do + before do + debate.title = nil + end + + it "is not valid" do + expect { subject.call }.to broadcast(:invalid) + end + end + + context "when everything is ok" do + context "when the archive parameter is true" do + let(:archived_at) { nil } + let(:archive) { true } + + it "archives the debate" do + subject.call + expect(debate.archived_at).to be_present + end + end + + context "when the archive parameter is false" do + let(:archived_at) { 1.day.ago } + let(:archive) { false } + + it "unarchives the debate" do + subject.call + expect(debate.archived_at).not_to be_present + end + end + end +end diff --git a/decidim-debates/spec/models/decidim/debates/debate_spec.rb b/decidim-debates/spec/models/decidim/debates/debate_spec.rb index 1e1ee727a443..dee941e077b7 100644 --- a/decidim-debates/spec/models/decidim/debates/debate_spec.rb +++ b/decidim-debates/spec/models/decidim/debates/debate_spec.rb @@ -125,4 +125,20 @@ it { is_expected.to be_falsey } end end + + describe "archived?" do + subject { debate.archived? } + + context "when the debate has an archived_at time" do + let(:debate) { build :debate, archived_at: 2.days.ago } + + it { is_expected.to be_truthy } + end + + context "when the debate does not have an archived_at time" do + let(:debate) { build :debate, archived_at: nil } + + it { is_expected.to be_falsey } + end + end end diff --git a/decidim-debates/spec/permissions/decidim/debates/admin/permissions_spec.rb b/decidim-debates/spec/permissions/decidim/debates/admin/permissions_spec.rb index e96cb68a8a5a..781efc0b1f94 100644 --- a/decidim-debates/spec/permissions/decidim/debates/admin/permissions_spec.rb +++ b/decidim-debates/spec/permissions/decidim/debates/admin/permissions_spec.rb @@ -79,4 +79,34 @@ it { is_expected.to eq false } end end + + describe "debate archive" do + let(:action) do + { scope: :admin, action: :archive, subject: :debate } + end + + context "when the debate is closed" do + let(:debate) { create :debate, :closed, component: debates_component } + + it { is_expected.to eq true } + + context "and it is not official" do + let(:debate) { create :debate, :closed, author: user, component: debates_component } + + it { is_expected.to eq true } + end + end + + context "when debate is open" do + let(:debate) { create :debate, component: debates_component } + + it { is_expected.to eq true } + + context "and it is not official" do + let(:debate) { create :debate, author: user, component: debates_component } + + it { is_expected.to eq true } + end + end + end end diff --git a/decidim-debates/spec/shared/manage_debates_examples.rb b/decidim-debates/spec/shared/manage_debates_examples.rb index 2c0fe4f5b72d..ae538fc12138 100644 --- a/decidim-debates/spec/shared/manage_debates_examples.rb +++ b/decidim-debates/spec/shared/manage_debates_examples.rb @@ -63,7 +63,7 @@ it "creates a new debate" do within ".card-title" do - page.find(".button.button--title").click + click_link "New Debate" end within ".new_debate" do @@ -191,4 +191,46 @@ end end end + + describe "archiving a debate" do + let!(:debate) { create(:debate, :closed, component: current_component) } + + it "archives a debate" do + within find("tr", text: translated(debate.title)) do + page.find(".action-icon--archive").click + end + + within ".callout-wrapper" do + expect(page).to have_content("successfully") + end + + expect(page).to have_no_content(translated(debate.title)) + + click_link "Archived debates" + expect(page).to have_content(translated(debate.title)) + end + end + + describe "unarchiving a debate" do + let!(:debate) { create(:debate, :closed, component: current_component, archived_at: 1.day.ago) } + + before do + click_link "Archived debates" + end + + it "unarchives a debate" do + within find("tr", text: translated(debate.title)) do + page.find(".action-icon--archive").click + end + + within ".callout-wrapper" do + expect(page).to have_content("successfully") + end + + expect(page).to have_no_content(translated(debate.title)) + + click_link "Active debates" + expect(page).to have_content(translated(debate.title)) + end + end end