decidim-department_admin 0.7.2

data/spec/system/admin_invite_department_admin_spec.rb

@@ -0,0 +1,118 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+#
+# To "downgrade" an admin to department_admin, it must first be removed from admins
+# create a new admin as department_admin.
+#
+describe "Admin invite user", type: :system do
+  let!(:area) { create(:area) }
+
+  let(:organization) { create(:organization) }
+
+  let!(:admin) { create(:user, :admin, :confirmed, organization: organization) }
+  let(:user_manager) { create(:user, :user_manager, :confirmed, organization: organization) }
+
+  let(:department_admin) do
+    user = create(:user, :confirmed, organization: organization)
+    user.roles << "department_admin"
+    user.areas << area
+    user.save!
+    user
+  end
+
+  before do
+    switch_to_host(organization.host)
+    login_as admin, scope: :user
+    visit decidim_admin.new_user_path
+  end
+
+  it "admin is able to create department admins" do
+    fill_the_form_for_department_admin("Cabello Loco", "my@email.net")
+    submit_form
+    check_succeess
+    user = check_is_department_admin("my@email.net")
+    check_assigned_area(user, area)
+  end
+
+  it "admin is able to add department_admin role to existing user" do
+    fill_the_form_for_department_admin(user_manager.name, user_manager.email)
+    submit_form
+    check_succeess
+    check_is_department_admin(user_manager.email)
+    check_assigned_area(user_manager, area)
+  end
+
+  context "when departments are reorganized" do
+    let!(:new_area) { create(:area) }
+
+    before do
+      department_admin
+      visit decidim_admin.new_user_path
+    end
+
+    it "admin is able to change department_admin's area/department" do
+      fill_the_form_for_department_admin(department_admin.name, department_admin.email, new_area)
+      submit_form
+      check_succeess
+      check_is_department_admin(department_admin.email)
+      check_assigned_area(department_admin, new_area)
+    end
+  end
+
+  context "when a department_admin is promoted to Admin" do
+    it "no longer has the `department_admin` role" do
+      department_admin
+      fill_the_form_for_admin(department_admin.name, department_admin.email)
+      submit_form
+      check_succeess
+      check_is_admin(department_admin.email)
+    end
+  end
+
+  def fill_the_form_for_department_admin(name, email, selected_area = area)
+    within "form.new_user" do
+      fill_in :user_name, with: name
+      fill_in :user_email, with: email
+      find("#user_role").find("option[value='department_admin']").select_option
+      expect(page).to have_css("#user_area_id")
+      find("#user_area_id").find("option[value='#{selected_area.id}']").select_option
+    end
+  end
+
+  def fill_the_form_for_admin(name, email, _selected_area = area)
+    within "form.new_user" do
+      fill_in :user_name, with: name
+      fill_in :user_email, with: email
+      find("#user_role").find("option[value='admin']").select_option
+    end
+  end
+
+  def submit_form
+    find("*[name=commit][type=submit]").click
+  end
+
+  def check_succeess
+    expect(page).to have_content("Participant successfully invited.")
+    expect(page).to have_current_path "/admin/users"
+  end
+
+  def check_is_admin(email)
+    user = Decidim::User.find_by(email: email).reload
+    expect(user).to be_admin
+    expect(user.roles).not_to include("department_admin")
+    expect(user.areas).to be_empty
+  end
+
+  def check_is_department_admin(email)
+    user = Decidim::User.find_by(email: email)
+    expect(user.roles).to include("department_admin")
+    user
+  end
+
+  def check_assigned_area(user, area)
+    expect(user.areas.last).to eq(area)
+    expect(user.areas.size).to eq(1)
+  end
+end

data/spec/system/department_admin_manages_newsletters_spec.rb

@@ -0,0 +1,237 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+describe "Admin manages newsletters", type: :system do
+  let(:organization) { create(:organization) }
+  let(:area) { create(:area, organization: organization) }
+  let(:department_admin) { create(:department_admin, :confirmed, name: "Sarah Kerrigan", organization: organization, area: area) }
+  let!(:deliverable_users) { create_list(:user, 5, :confirmed, newsletter_notifications_at: Time.current, organization: organization) }
+
+  before do
+    switch_to_host(organization.host)
+    login_as department_admin, scope: :user
+  end
+
+  def wait_redirect
+    sleep(0.5)
+  end
+
+  def fill_newsletter_form
+    fill_in_i18n(
+      :newsletter_subject,
+      "#newsletter-subject-tabs",
+      en: "A fancy newsletter for %{name}",
+      es: "Un correo electrónico muy chulo para %{name}",
+      ca: "Un correu electrònic flipant per a %{name}"
+    )
+
+    fill_in_i18n_editor(
+      :newsletter_settings_body,
+      "#newsletter-settings--body-tabs",
+      en: "Hello %{name}! Relevant content.",
+      es: "Hola, %{name}! Contenido relevante.",
+      ca: "Hola, %{name}! Contingut rellevant."
+    )
+
+    find("*[type=submit]").click
+  end
+
+  context "when creating and previewing a newsletter" do
+    it "allows a newsletter to be created" do
+      visit decidim_admin.newsletters_path
+
+      within ".secondary-nav" do
+        find(".button.new").click
+      end
+
+      within "#basic_only_text .card-footer" do
+        click_link("Use this template")
+      end
+
+      within "#new_newsletter_" do
+        fill_newsletter_form
+      end
+
+      expect(page).to have_content("Preview")
+      expect(page).to have_content("A fancy newsletter for #{department_admin.name}")
+    end
+  end
+
+  context "with existing newsletter" do
+    let!(:newsletter) do
+      create(:newsletter,
+             organization: organization,
+             subject: {
+               en: "A fancy newsletter for %{name}",
+               es: "Un correo electrónico muy chulo para %{name}",
+               ca: "Un correu electrònic flipant per a %{name}",
+             },
+             body: {
+               en: "Hello %{name}! Relevant content.",
+               es: "Hola, %{name}! Contenido relevante.",
+               ca: "Hola, %{name}! Contingut rellevant.",
+             },
+             author: department_admin)
+    end
+
+    it "previews a newsletter" do
+      visit decidim_admin.newsletter_path(newsletter)
+
+      expect(page).to have_content("A fancy newsletter for Sarah Kerrigan")
+      expect(page).to have_css("iframe.email-preview[src=\"#{decidim_admin.preview_newsletter_path(newsletter)}\"]")
+
+      visit decidim_admin.preview_newsletter_path(newsletter)
+      expect(page).to have_content("Hello Sarah Kerrigan! Relevant content.")
+    end
+  end
+
+  context "when updating the newsletter" do
+    let!(:newsletter) { create(:newsletter, organization: organization, author: department_admin) }
+
+    it "allows a newsletter to be updated" do
+      visit decidim_admin.newsletters_path
+      within("tr[data-newsletter-id=\"#{newsletter.id}\"]") do
+        click_link "Edit"
+      end
+
+      within ".edit_newsletter" do
+        fill_newsletter_form
+      end
+
+      expect(page).to have_content("Preview")
+      expect(page).to have_content("A fancy newsletter")
+    end
+  end
+
+  context "when selecting newsletter recipients" do
+    let!(:newsletter) { create(:newsletter, organization: organization, author: department_admin) }
+
+    context "when followers are selected" do
+      let!(:participatory_processes) { create_list(:participatory_process, 2, organization: organization, area: area) }
+      let!(:followers) do
+        deliverable_users.each do |follower|
+          create(:follow, followable: participatory_processes.first, user: follower)
+        end
+      end
+
+      it "sends to followers" do
+        visit decidim_admin.select_recipients_to_deliver_newsletter_path(newsletter)
+        perform_enqueued_jobs do
+          within(".newsletter_deliver") do
+            check("Send to followers")
+            uncheck("Send to participants")
+            within ".participatory_processes-block" do
+              select translated(participatory_processes.first.title), from: :newsletter_participatory_space_types_participatory_processes__ids
+            end
+          end
+
+          within ".button--double" do
+            accept_confirm { find("*", text: "Deliver").click }
+          end
+
+          wait_redirect
+          expect(page).to have_content("Newsletters")
+          expect(page).to have_admin_callout("successfully")
+        end
+
+        within "tbody" do
+          expect(page).to have_content("5 / 5")
+        end
+      end
+    end
+
+    context "when participants are selected" do
+      let!(:participatory_process) { create(:participatory_process, organization: organization, area: area) }
+      let!(:component) { create(:dummy_component, organization: newsletter.organization, participatory_space: participatory_process) }
+
+      before do
+        deliverable_users.each do |participant|
+          create(:dummy_resource, component: component, author: participant, published_at: Time.current)
+        end
+      end
+
+      it "sends to participants" do
+        visit decidim_admin.select_recipients_to_deliver_newsletter_path(newsletter)
+        perform_enqueued_jobs do
+          within(".newsletter_deliver") do
+            uncheck("Send to followers")
+            check("Send to participants")
+            within ".participatory_processes-block" do
+              select translated(component.participatory_space.title), from: :newsletter_participatory_space_types_participatory_processes__ids
+            end
+          end
+
+          within ".button--double" do
+            accept_confirm { find("*", text: "Deliver").click }
+          end
+
+          wait_redirect
+          expect(page).to have_content("Newsletters")
+          expect(page).to have_admin_callout("successfully")
+          expect(page).to have_content("Has been sent to")
+          within "tbody" do
+            expect(page).to have_content("5 / 5")
+          end
+        end
+      end
+    end
+
+    context "when selecting both followers and participants" do
+      let!(:participatory_process) { create(:participatory_process, organization: organization, area: area) }
+      let!(:component) { create(:dummy_component, organization: newsletter.organization, participatory_space: participatory_process) }
+
+      let!(:followers) do
+        deliverable_users.each do |follower|
+          create(:follow, followable: component.participatory_space, user: follower)
+        end
+      end
+
+      before do
+        deliverable_users.each do |participant|
+          create(:dummy_resource, component: component, author: participant, published_at: Time.current)
+        end
+      end
+
+      it "sends to participants" do
+        visit decidim_admin.select_recipients_to_deliver_newsletter_path(newsletter)
+        perform_enqueued_jobs do
+          within(".newsletter_deliver") do
+            check("Send to followers")
+            check("Send to participants")
+            within ".participatory_processes-block" do
+              select translated(component.participatory_space.title), from: :newsletter_participatory_space_types_participatory_processes__ids
+            end
+          end
+
+          within ".button--double" do
+            accept_confirm { find("*", text: "Deliver").click }
+          end
+
+          wait_redirect
+          expect(page).to have_content("Newsletters")
+          expect(page).to have_admin_callout("successfully")
+        end
+
+        within "tbody" do
+          expect(page).to have_content("5 / 5")
+        end
+      end
+    end
+  end
+
+  context "when deleting a newsletter" do
+    let!(:newsletter) { create(:newsletter, organization: organization, author: department_admin) }
+
+    it "deletes a newsletter" do
+      visit decidim_admin.newsletters_path
+
+      within("tr[data-newsletter-id=\"#{newsletter.id}\"]") do
+        accept_confirm { click_link "Delete" }
+      end
+
+      expect(page).to have_content("successfully")
+      expect(page).to have_no_css("tr[data-newsletter-id=\"#{newsletter.id}\"]")
+    end
+  end
+end

data/spec/system/department_admin_should_be_able_to_access_admin_dashboard_spec.rb

@@ -0,0 +1,75 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+describe "Department admin should be able to access Admin Dashboard", type: :system do
+  let(:organization) { create(:organization) }
+  let(:area) { create(:area) }
+  let(:department_admin) { create(:department_admin, :confirmed, organization: organization, area: area, accepted_tos_version: Time.current) }
+  let!(:process) { create(:participatory_process, :published, organization: organization, area: area) }
+
+  before do
+    switch_to_host(organization.host)
+    login_as department_admin, scope: :user
+    visit decidim_admin.root_path
+  end
+
+  it "is able to access the admin Dashboard" do
+    expect(page).to have_content("Dashboard")
+    expect(page).to have_content("Welcome to the Admin Panel.")
+  end
+
+  context "when accessing the dashboard some left menu elements should be accessible" do
+    it "is able to access 'Processes'" do
+      expect(page).to have_content("Processes")
+      click_link "Processes"
+      expect(page).to have_current_path "/admin/participatory_processes"
+      expect(page).to have_content("Participatory processes")
+    end
+
+    it "is able to access 'Assemblies'" do
+      expect(page).to have_content("Assemblies")
+      click_link "Assemblies"
+      expect(page).to have_current_path "/admin/assemblies"
+      expect(page).to have_content("New assembly")
+    end
+
+    it "is able to access 'Participants'" do
+      expect(page).to have_content("Participants")
+      click_link "Participants"
+      expect(page).to have_current_path "/admin/users"
+      expect(page).to have_content("New admin")
+    end
+
+    it "is able to access 'Newsletter'" do
+      expect(page).to have_content("Newsletter")
+      click_link "Newsletter"
+      expect(page).to have_current_path "/admin/newsletters"
+      expect(page).to have_content("New newsletter")
+    end
+
+    it "is able to access 'Conferences'" do
+      expect(page).to have_content("Conferences")
+      click_link "Conferences"
+      expect(page).to have_current_path "/admin/conferences"
+      expect(page).to have_content("New Conference")
+    end
+    # TODO: not supported at the moment
+    # it "should be able to access 'Initiatives'"
+    # it "should be able to access 'Consultations'"
+  end
+
+  context "when accessing the dashboard some left menu elements should NOT be accessible" do
+    it "is not able to access 'Pages'" do
+      expect(page).not_to have_content("Pages")
+    end
+
+    it "is not able to access 'Settings'" do
+      expect(page).not_to have_content("Settings")
+    end
+
+    it "is not able to access 'Admin Activity Log'" do
+      expect(page).not_to have_content("Admin activity log")
+    end
+  end
+end

data/spec/system/department_admin_should_be_able_to_manage_assemblies_spec.rb

@@ -0,0 +1,99 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+describe "Admin manages assemblies", versioning: true, type: :system do
+  let(:organization) { create(:organization) }
+  let(:area) { create(:area, organization: organization) }
+  let(:department_admin) { create(:department_admin, :confirmed, organization: organization, area: area) }
+
+  before do
+    switch_to_host(organization.host)
+    login_as department_admin, scope: :user
+  end
+
+  shared_examples "creating an assembly" do
+    let(:image1_filename) { "city.jpeg" }
+    let(:image1_path) { Decidim::Dev.asset(image1_filename) }
+
+    let(:image2_filename) { "city2.jpeg" }
+    let(:image2_path) { Decidim::Dev.asset(image2_filename) }
+    let(:attributes) { attributes_for(:assembly, organization: organization) }
+
+    before do
+      visit decidim_admin_assemblies.assemblies_path
+      click_link "New assembly"
+    end
+
+    it "creates a new assembly" do
+      within ".new_assembly" do
+        fill_in_i18n(:assembly_title, "#assembly-title-tabs", **attributes[:title].except("machine_translations"))
+        fill_in_i18n(:assembly_subtitle, "#assembly-subtitle-tabs", **attributes[:subtitle].except("machine_translations"))
+        fill_in_i18n_editor(:assembly_short_description, "#assembly-short_description-tabs", **attributes[:short_description].except("machine_translations"))
+        fill_in_i18n_editor(:assembly_description, "#assembly-description-tabs", **attributes[:description].except("machine_translations"))
+        fill_in_i18n_editor(:assembly_purpose_of_action, "#assembly-purpose_of_action-tabs", **attributes[:purpose_of_action].except("machine_translations"))
+        fill_in_i18n_editor(:assembly_composition, "#assembly-composition-tabs", **attributes[:composition].except("machine_translations"))
+        fill_in_i18n_editor(:assembly_internal_organisation, "#assembly-internal_organisation-tabs", **attributes[:internal_organisation].except("machine_translations"))
+        fill_in_i18n_editor(:assembly_announcement, "#assembly-announcement-tabs", **attributes[:announcement].except("machine_translations"))
+        fill_in_i18n_editor(:assembly_closing_date_reason, "#assembly-closing_date_reason-tabs", **attributes[:closing_date_reason].except("machine_translations"))
+
+        fill_in_i18n(:assembly_participatory_scope, "#assembly-participatory_scope-tabs", **attributes[:participatory_scope].except("machine_translations"))
+        fill_in_i18n(:assembly_participatory_structure, "#assembly-participatory_structure-tabs", **attributes[:participatory_structure].except("machine_translations"))
+        fill_in_i18n(:assembly_meta_scope, "#assembly-meta_scope-tabs", **attributes[:meta_scope].except("machine_translations"))
+        fill_in_i18n(:assembly_local_area, "#assembly-local_area-tabs", **attributes[:local_area].except("machine_translations"))
+        fill_in_i18n(:assembly_target, "#assembly-target-tabs", **attributes[:target].except("machine_translations"))
+
+        fill_in :assembly_slug, with: "slug"
+        fill_in :assembly_hashtag, with: "#hashtag"
+        fill_in :assembly_weight, with: 1
+      end
+
+      dynamically_attach_file(:assembly_hero_image, image1_path)
+      dynamically_attach_file(:assembly_banner_image, image2_path)
+
+      within ".new_assembly" do
+        find("*[type=submit]").click
+      end
+
+      expect(page).to have_admin_callout("successfully")
+      expect(Decidim::Assembly.last.area).to eq(area)
+
+      within ".container" do
+        # expect(page).to have_current_path decidim_admin_assemblies.assemblies_path(q: { parent_id_eq: parent_assembly&.id })
+        expect(page).to have_content(translated(attributes[:title]))
+      end
+    end
+  end
+
+  context "when managing parent assemblies" do
+    let(:parent_assembly) { nil }
+    let!(:assembly) { create :assembly, organization: organization }
+
+    before do
+      switch_to_host(organization.host)
+      login_as department_admin, scope: :user
+      visit decidim_admin_assemblies.assemblies_path
+    end
+
+    # it_behaves_like "manage assemblies"
+    it_behaves_like "creating an assembly"
+  end
+
+  context "when managing child assemblies" do
+    let!(:parent_assembly) { create :assembly, organization: organization, area: area }
+    let!(:child_assembly) { create :assembly, organization: organization, parent: parent_assembly, area: area }
+    let(:assembly) { child_assembly }
+
+    before do
+      switch_to_host(organization.host)
+      login_as department_admin, scope: :user
+      visit decidim_admin_assemblies.assemblies_path
+      within find("tr", text: translated(parent_assembly.title)) do
+        click_link "Assemblies"
+      end
+    end
+
+    # it_behaves_like "manage assemblies"
+    it_behaves_like "creating an assembly"
+  end
+end

data/spec/system/department_admin_should_be_able_to_manage_conferences_spec.rb

@@ -0,0 +1,74 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+describe "Admin manages conferences", versioning: true, type: :system do
+  let(:organization) { create(:organization) }
+  let(:area) { create(:area, organization: organization) }
+  let(:department_admin) { create(:department_admin, :confirmed, organization: organization, area: area) }
+
+  before do
+    switch_to_host(organization.host)
+    login_as department_admin, scope: :user
+  end
+
+  shared_examples "creating an conference" do
+    let(:image1_filename) { "city.jpeg" }
+    let(:image1_path) { Decidim::Dev.asset(image1_filename) }
+
+    let(:image2_filename) { "city2.jpeg" }
+    let(:image2_path) { Decidim::Dev.asset(image2_filename) }
+
+    before do
+      visit decidim_admin_conferences.assemblies_path
+      click_link "New conference"
+    end
+
+    it "creates a new conference" do
+      within ".new_conference" do
+        fill_in_i18n(
+          :conference_title,
+          "#conference-title-tabs",
+          en: "My conference",
+          es: "Mi conferencia",
+          ca: "Mi jornada"
+        )
+        fill_in_i18n(
+          :conference_subtitle,
+          "#conference-subtitle-tabs",
+          en: "Subtitle",
+          es: "Subtítulo",
+          ca: "Subtítol"
+        )
+        fill_in_i18n_editor(
+          :conference_short_description,
+          "#conference-short_description-tabs",
+          en: "Short description",
+          es: "Descripción corta",
+          ca: "Descripció curta"
+        )
+        fill_in_i18n_editor(
+          :conference_description,
+          "#conference-description-tabs",
+          en: "A longer description",
+          es: "Descripción más larga",
+          ca: "Descripció més llarga"
+        )
+
+        fill_in :conference_slug, with: "slug"
+        fill_in :conference_hashtag, with: "#hashtag"
+        attach_file :conference_hero_image, image1_path
+        attach_file :conference_banner_image, image2_path
+
+        find("*[type=submit]").click
+      end
+
+      expect(page).to have_admin_callout("successfully")
+      expect(Decidim::Conference.last.area).to eq(area)
+
+      within ".container" do
+        expect(page).to have_content("My conference")
+      end
+    end
+  end
+end