decidim-admin 0.27.5 → 0.28.0.rc4

data/lib/decidim/admin/test/filters_participatory_space_user_roles_examples.rb

@@ -95,48 +95,28 @@ end
 shared_examples "filterable participatory space user roles" do
   context "when filtering by invite Accepted" do
     context "when filtering by null" do
-      it "returns participatory space users" do
-        apply_filter("Invite accepted", "Yes")
-
-        within ".stack tbody" do
-          expect(page).to have_content(invited_user2.name)
-          expect(page).to have_css("tr", count: 1)
-        end
+      include_examples "admin is filtering participatory space users", label: "Invite accepted", value: "Yes" do
+        let(:compare_with) { invited_user2.name }
       end
     end
 
     context "when filtering by not null" do
-      it "returns participatory space users" do
-        apply_filter("Invite accepted", "No")
-
-        within ".stack tbody" do
-          expect(page).to have_content(invited_user1.name)
-          expect(page).to have_css("tr", count: 1)
-        end
+      include_examples "admin is filtering participatory space users", label: "Invite accepted", value: "No" do
+        let(:compare_with) { invited_user1.name }
       end
     end
   end
 
   context "when filtering by logged in" do
     context "when filtering by null" do
-      it "returns participatory space users" do
-        apply_filter("Ever logged in", "Yes")
-
-        within ".stack tbody" do
-          expect(page).to have_content(invited_user2.name)
-          expect(page).to have_css("tr", count: 1)
-        end
+      include_examples "admin is filtering participatory space users", label: "Ever logged in", value: "Yes" do
+        let(:compare_with) { invited_user2.name }
       end
     end
 
     context "when filtering by not null" do
-      it "returns participatory space users" do
-        apply_filter("Ever logged in", "No")
-
-        within ".stack tbody" do
-          expect(page).to have_content(invited_user1.name)
-          expect(page).to have_css("tr", count: 1)
-        end
+      include_examples "admin is filtering participatory space users", label: "Ever logged in", value: "No" do
+        let(:compare_with) { invited_user1.name }
       end
     end
   end
@@ -144,22 +124,11 @@ end
 
 shared_examples "searchable participatory space user roles" do
   context "when searching by name or nickname or email" do
-    it "can be searched by name" do
-      search_by_text(name)
-
-      within ".stack tbody" do
-        expect(page).to have_content(name)
-        expect(page).to have_css("tr", count: 1)
-      end
+    include_examples "admin is searching participatory space users" do
+      let(:value) { name }
     end
-
-    it "can be searched by email" do
-      search_by_text(email)
-
-      within ".stack tbody" do
-        expect(page).to have_content(email)
-        expect(page).to have_css("tr", count: 1)
-      end
+    include_examples "admin is searching participatory space users" do
+      let(:value) { email }
     end
   end
 end

data/lib/decidim/admin/test/filters_participatory_space_users_examples.rb

@@ -1,50 +1,52 @@
 # frozen_string_literal: true
 
+shared_examples "admin is filtering participatory space users" do |label:, value:|
+  it "returns participatory space users" do
+    apply_filter(label, value)
+
+    within ".table-list tbody" do
+      expect(page).to have_content(compare_with)
+      expect(page).to have_css("tr", count: 1)
+    end
+  end
+end
+
+shared_examples "admin is searching participatory space users" do
+  it "returns participatory space users" do
+    search_by_text(value)
+
+    within ".table-list tbody" do
+      expect(page).to have_content(value)
+      expect(page).to have_css("tr", count: 1)
+    end
+  end
+end
+
 shared_examples "filterable participatory space users" do
   context "when filtering by invitation sent at" do
     context "when filtering by null" do
-      it "returns participatory space users" do
-        apply_filter("Invitation sent", "Not sent")
-
-        within ".stack tbody" do
-          expect(page).to have_content(invited_user2.name)
-          expect(page).to have_css("tr", count: 1)
-        end
+      include_examples "admin is filtering participatory space users", label: "Invitation sent", value: "Not sent" do
+        let(:compare_with) { invited_user2.name }
       end
     end
 
     context "when filtering by not null" do
-      it "returns participatory space users" do
-        apply_filter("Invitation sent", "Sent")
-
-        within ".stack tbody" do
-          expect(page).to have_content(invited_user1.name)
-          expect(page).to have_css("tr", count: 1)
-        end
+      include_examples "admin is filtering participatory space users", label: "Invitation sent", value: "Sent" do
+        let(:compare_with) { invited_user1.name }
       end
     end
   end
 
   context "when filtering by invitation accepted at" do
     context "when filtering by null" do
-      it "returns participatory space users" do
-        apply_filter("Invitation accepted", "Not accepted")
-
-        within ".stack tbody" do
-          expect(page).to have_content(invited_user2.name)
-          expect(page).to have_css("tr", count: 1)
-        end
+      include_examples "admin is filtering participatory space users", label: "Invitation accepted", value: "Not accepted" do
+        let(:compare_with) { invited_user2.name }
       end
     end
 
     context "when filtering by not null" do
-      it "returns participatory space users" do
-        apply_filter("Invitation accepted", "Accepted")
-
-        within ".stack tbody" do
-          expect(page).to have_content(invited_user1.name)
-          expect(page).to have_css("tr", count: 1)
-        end
+      include_examples "admin is filtering participatory space users", label: "Invitation accepted", value: "Accepted" do
+        let(:compare_with) { invited_user1.name }
       end
     end
   end
@@ -52,22 +54,11 @@ end
 
 shared_examples "searchable participatory space users" do
   context "when searching by name or nickname or email" do
-    it "can be searched by name" do
-      search_by_text(name)
-
-      within ".stack tbody" do
-        expect(page).to have_content(name)
-        expect(page).to have_css("tr", count: 1)
-      end
+    include_examples "admin is searching participatory space users" do
+      let(:value) { name }
     end
-
-    it "can be searched by email" do
-      search_by_text(email)
-
-      within ".stack tbody" do
-        expect(page).to have_content(email)
-        expect(page).to have_css("tr", count: 1)
-      end
+    include_examples "admin is searching participatory space users" do
+      let(:value) { email }
     end
   end
 end

data/lib/decidim/admin/test/forms/attachment_collection_form_examples.rb

@@ -9,7 +9,7 @@ module Decidim
         described_class.from_params(
           attributes
         ).with_context(
-          collection_for: collection_for,
+          collection_for:,
           current_organization: organization
         )
       end

data/lib/decidim/admin/test/forms/attachment_form_examples.rb

@@ -9,7 +9,7 @@ module Decidim
         described_class.from_params(
           attributes
         ).with_context(
-          attached_to: attached_to,
+          attached_to:,
           current_organization: organization
         )
       end

data/lib/decidim/admin/test/forms/category_form_examples.rb

@@ -50,7 +50,7 @@ module Decidim
       end
 
       context "when the parent_id is set" do
-        let!(:category) { create :category, participatory_space: participatory_space }
+        let!(:category) { create :category, participatory_space: }
 
         context "and it is set to a first-class category" do
           let(:parent_id) { category.id }

data/lib/decidim/admin/test/manage_attachment_collections_examples.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 shared_examples "manage attachment collections examples" do
-  let!(:attachment_collection) { create(:attachment_collection, collection_for: collection_for) }
+  let!(:attachment_collection) { create(:attachment_collection, collection_for:) }
 
   before do
     visit current_path
@@ -15,7 +15,7 @@ shared_examples "manage attachment collections examples" do
 
   it "can view an attachment collection details" do
     within "#attachment_collections table" do
-      click_link translated(attachment_collection.name, locale: :en)
+      click_link "Edit"
     end
 
     expect(page).to have_selector("input#attachment_collection_name_en[value='#{translated(attachment_collection.name, locale: :en)}']")
@@ -24,7 +24,7 @@ shared_examples "manage attachment collections examples" do
   end
 
   it "can add attachment collections to a process" do
-    find(".card-title a.new").click
+    click_link "New attachment folder"
 
     within ".new_attachment_collection" do
       fill_in_i18n(
@@ -49,7 +49,7 @@ shared_examples "manage attachment collections examples" do
     expect(page).to have_admin_callout("successfully")
 
     within "#attachment_collections table" do
-      expect(page).to have_link("Application forms")
+      expect(page).to have_text("Application forms")
     end
   end
 
@@ -75,12 +75,12 @@ shared_examples "manage attachment collections examples" do
     expect(page).to have_admin_callout("successfully")
 
     within "#attachment_collections table" do
-      expect(page).to have_link("Latest application forms")
+      expect(page).to have_text("Latest application forms")
     end
   end
 
   context "when deleting a attachment collection" do
-    let!(:attachment_collection2) { create(:attachment_collection, collection_for: collection_for) }
+    let!(:attachment_collection2) { create(:attachment_collection, collection_for:) }
 
     context "when the attachment collection has no associated content", :slow do
       before do
@@ -95,13 +95,13 @@ shared_examples "manage attachment collections examples" do
         expect(page).to have_admin_callout("successfully")
 
         within "#attachment_collections table" do
-          expect(page).to have_no_content(translated(attachment_collection2.name))
+          expect(page).not_to have_content(translated(attachment_collection2.name))
         end
       end
     end
 
     context "when the attachment collection has associated content" do
-      let!(:attachment) { create(:attachment, attached_to: collection_for, attachment_collection: attachment_collection) }
+      let!(:attachment) { create(:attachment, attached_to: collection_for, attachment_collection:) }
 
       before do
         visit current_path
@@ -109,7 +109,7 @@ shared_examples "manage attachment collections examples" do
 
       it "cannot delete it" do
         within find("tr", text: translated(attachment_collection.name)) do
-          expect(page).to have_no_selector("a.action-icon--remove")
+          expect(page).not_to have_selector("a.action-icon--remove")
         end
       end
     end

data/lib/decidim/admin/test/manage_attachments_examples.rb

@@ -2,7 +2,7 @@
 
 shared_examples "manage attachments examples" do
   context "when processing attachments" do
-    let!(:attachment) { create(:attachment, attached_to: attached_to, attachment_collection: attachment_collection) }
+    let!(:attachment) { create(:attachment, attached_to:, attachment_collection:) }
 
     before do
       visit current_path
@@ -19,7 +19,7 @@ shared_examples "manage attachments examples" do
 
     it "can view an attachment details" do
       within "#attachments table" do
-        click_link translated(attachment.title, locale: :en)
+        click_link "Edit"
       end
 
       expect(page).to have_selector("input#attachment_title_en[value='#{translated(attachment.title, locale: :en)}']")
@@ -30,7 +30,7 @@ shared_examples "manage attachments examples" do
     end
 
     it "can add attachments without a collection to a process" do
-      find(".card-title a.new").click
+      click_link "New attachment"
 
       within ".new_attachment" do
         fill_in_i18n(
@@ -59,12 +59,12 @@ shared_examples "manage attachments examples" do
       expect(page).to have_admin_callout("successfully")
 
       within "#attachments table" do
-        expect(page).to have_link("Very Important Document")
+        expect(page).to have_text("Very Important Document")
       end
     end
 
     it "can add attachments within a collection to a process" do
-      find(".card-title a.new").click
+      click_link "New attachment"
 
       within ".new_attachment" do
         fill_in_i18n(
@@ -95,7 +95,7 @@ shared_examples "manage attachments examples" do
       expect(page).to have_admin_callout("successfully")
 
       within "#attachments table" do
-        expect(page).to have_link("Document inside a collection")
+        expect(page).to have_text("Document inside a collection")
         expect(page).to have_text(translated(attachment_collection.name, locale: :en))
       end
     end
@@ -116,7 +116,7 @@ shared_examples "manage attachments examples" do
 
       within "#attachments" do
         within find("tr", text: translated(attachment.title)) do
-          expect(page).to have_no_text(translated(attachment_collection.name, locale: :en))
+          expect(page).not_to have_text(translated(attachment_collection.name, locale: :en))
         end
       end
     end
@@ -128,7 +128,7 @@ shared_examples "manage attachments examples" do
 
       expect(page).to have_admin_callout("successfully")
 
-      expect(page).to have_no_content(translated(attachment.title, locale: :en))
+      expect(page).not_to have_content(translated(attachment.title, locale: :en))
     end
 
     it "can update an attachment" do
@@ -153,7 +153,7 @@ shared_examples "manage attachments examples" do
       expect(page).to have_admin_callout("successfully")
 
       within "#attachments table" do
-        expect(page).to have_link("This is a nice photo")
+        expect(page).to have_text("This is a nice photo")
       end
     end
   end

data/lib/decidim/admin/test/manage_categories_examples.rb

@@ -15,11 +15,11 @@ shared_examples "manage categories examples" do
     expect(page).to have_selector("input#category_name_en[value='#{translated(category.name, locale: :en)}']")
     expect(page).to have_selector("input#category_weight[value='#{category.weight}']")
 
-    expect(page).to have_selector("select#category_parent_id")
+    expect(page).to have_select(id: "category_parent_id")
   end
 
   it "creates a new category" do
-    find(".card-title a.new").click
+    click_link "New category"
 
     within ".new_category" do
       fill_in_i18n(
@@ -67,7 +67,7 @@ shared_examples "manage categories examples" do
   end
 
   context "when deleting a category" do
-    let!(:category2) { create(:category, participatory_space: participatory_space) }
+    let!(:category2) { create(:category, participatory_space:) }
 
     context "when the category has no associated content" do
       context "when the category has no subcategories" do
@@ -83,7 +83,7 @@ shared_examples "manage categories examples" do
           expect(page).to have_admin_callout("successfully")
 
           within "#categories table" do
-            expect(page).to have_no_content(translated(category2.name))
+            expect(page).not_to have_content(translated(category2.name))
           end
         end
       end
@@ -110,14 +110,14 @@ shared_examples "manage categories examples" do
     end
 
     context "when the category has associated content" do
-      let!(:component) { create(:component, participatory_space: participatory_space) }
-      let!(:dummy_resource) { create(:dummy_resource, component: component, category: category) }
+      let!(:component) { create(:component, participatory_space:) }
+      let!(:dummy_resource) { create(:dummy_resource, component:, category:) }
 
       it "cannot delete it" do
         visit current_path
 
         within find("tr", text: translated(category.name)) do
-          expect(page).to have_no_selector("a.action-icon--remove")
+          expect(page).not_to have_selector("a.action-icon--remove")
         end
       end
     end

data/lib/decidim/admin/test/manage_component_permissions_examples.rb

@@ -11,7 +11,7 @@ shared_examples "Managing component permissions" do
   end
 
   let!(:component) do
-    create(:component, participatory_space: participatory_space)
+    create(:component, participatory_space:)
   end
 
   before do
@@ -28,7 +28,7 @@ shared_examples "Managing component permissions" do
     end
 
     it "saves permission settings in the component" do
-      within "form.new_component_permissions" do
+      within "#components form" do
         within ".foo-permission" do
           check "Example authorization (Direct)"
           fill_in "Allowed postal codes", with: "08002"
@@ -42,7 +42,7 @@ shared_examples "Managing component permissions" do
         include(
           "authorization_handlers" => {
             "dummy_authorization_handler" => {
-              "options" => { "allowed_postal_codes" => "08002" }
+              "options" => { "allowed_postal_codes" => "08002", "allowed_scope_id" => "" }
             }
           }
         )
@@ -56,7 +56,7 @@ shared_examples "Managing component permissions" do
       within ".component-#{component.id}" do
         click_link "Permissions"
       end
-      within "form.new_component_permissions" do
+      within "#components form" do
         within ".foo-permission" do
           check "Example authorization (Direct)"
           fill_in "Allowed postal codes", with: "08002"
@@ -90,7 +90,7 @@ shared_examples "Managing component permissions" do
     end
 
     it "removes the action from the permissions hash" do
-      within "form.new_component_permissions" do
+      within "#components form" do
         within ".foo-permission" do
           uncheck "Example authorization (Direct)"
           uncheck "Another example authorization (Direct)"
@@ -125,7 +125,7 @@ shared_examples "Managing component permissions" do
     end
 
     it "changes the configured action in the permissions hash" do
-      within "form.new_component_permissions" do
+      within "#components form" do
         within ".foo-permission" do
           uncheck "Example authorization (Direct)"
           check "Another example authorization (Direct)"
@@ -149,7 +149,7 @@ shared_examples "Managing component permissions" do
     end
 
     it "adds an authorization to the configured action in the component permissions hash" do
-      within "form.new_component_permissions" do
+      within "#components form" do
         within ".foo-permission" do
           check "Another example authorization (Direct)"
           fill_in "Passport number", with: "AXXXXXXXX"
@@ -164,7 +164,7 @@ shared_examples "Managing component permissions" do
         include(
           "authorization_handlers" => {
             "dummy_authorization_handler" => {
-              "options" => { "allowed_postal_codes" => "08002" }
+              "options" => { "allowed_postal_codes" => "08002", "allowed_scope_id" => "" }
             },
             "another_dummy_authorization_handler" => {
               "options" => { "passport_number" => "AXXXXXXXX" }
@@ -177,13 +177,13 @@ shared_examples "Managing component permissions" do
 
   context "when managing resource permissions" do
     let!(:resource) do
-      create(:dummy_resource, component: component)
+      create(:dummy_resource, component:)
     end
 
     let(:edit_resource_permissions_path) do
-      ::Decidim::EngineRouter.admin_proxy(participatory_space).edit_component_permissions_path(component.id,
-                                                                                               resource_name: resource.resource_manifest.name,
-                                                                                               resource_id: resource.id)
+      Decidim::EngineRouter.admin_proxy(participatory_space).edit_component_permissions_path(component.id,
+                                                                                             resource_name: resource.resource_manifest.name,
+                                                                                             resource_id: resource.id)
     end
 
     let(:component_settings) { nil }
@@ -199,20 +199,20 @@ shared_examples "Managing component permissions" do
     end
 
     it "shows the resource permissions settings" do
-      expect(page).to have_content(translated(resource.title))
+      expect(page).to have_content("Edit permissions")
     end
 
     context "when resources permissions are disabled" do
       let(:component_settings) { { resources_permissions_enabled: false } }
 
-      it "doesn't show the resource permissions settings" do
+      it "does not show the resource permissions settings" do
         expect(page).not_to have_content(resource.title)
       end
     end
 
     context "when setting permissions" do
       it "saves permission settings for the resource" do
-        within "form.new_component_permissions" do
+        within "#components form" do
           within ".foo-permission" do
             check "Example authorization (Direct)"
             fill_in "Allowed postal codes", with: "08002"
@@ -226,7 +226,7 @@ shared_examples "Managing component permissions" do
           include(
             "authorization_handlers" => {
               "dummy_authorization_handler" => {
-                "options" => { "allowed_postal_codes" => "08002" }
+                "options" => { "allowed_postal_codes" => "08002", "allowed_scope_id" => "" }
               }
             }
           )
@@ -253,7 +253,7 @@ shared_examples "Managing component permissions" do
       end
 
       it "removes the action from the permissions hash" do
-        within "form.new_component_permissions" do
+        within "#components form" do
           within ".foo-permission" do
             uncheck "Example authorization (Direct)"
             uncheck "Another example authorization (Direct)"
@@ -286,7 +286,7 @@ shared_examples "Managing component permissions" do
       end
 
       it "changes the configured action in the resource permissions hash" do
-        within "form.new_component_permissions" do
+        within "#components form" do
           within ".foo-permission" do
             uncheck "Example authorization (Direct)"
             check "Another example authorization (Direct)"
@@ -310,7 +310,7 @@ shared_examples "Managing component permissions" do
       end
 
       it "adds an authorization to the configured action in the resource permissions hash" do
-        within "form.new_component_permissions" do
+        within "#components form" do
           within ".foo-permission" do
             check "Another example authorization (Direct)"
             fill_in "Passport number", with: "AXXXXXXXX"
@@ -325,7 +325,7 @@ shared_examples "Managing component permissions" do
           include(
             "authorization_handlers" => {
               "dummy_authorization_handler" => {
-                "options" => { "allowed_postal_codes" => "08002" }
+                "options" => { "allowed_postal_codes" => "08002", "allowed_scope_id" => "" }
               },
               "another_dummy_authorization_handler" => {
                 "options" => { "passport_number" => "AXXXXXXXX" }
@@ -354,7 +354,7 @@ shared_examples "Managing component permissions" do
       end
 
       it "changes the configured action in the resource permissions hash" do
-        within "form.new_component_permissions" do
+        within "#components form" do
           within ".foo-permission" do
             uncheck "Example authorization (Direct)"
             check "Another example authorization (Direct)"
@@ -378,7 +378,7 @@ shared_examples "Managing component permissions" do
       end
 
       it "adds an authorization to the configured action in the resource permissions hash" do
-        within "form.new_component_permissions" do
+        within "#components form" do
           within ".foo-permission" do
             check "Another example authorization (Direct)"
             fill_in "Passport number", with: "AXXXXXXXX"
@@ -393,7 +393,7 @@ shared_examples "Managing component permissions" do
           include(
             "authorization_handlers" => {
               "dummy_authorization_handler" => {
-                "options" => { "allowed_postal_codes" => "08002" }
+                "options" => { "allowed_postal_codes" => "08002", "allowed_scope_id" => "" }
               },
               "another_dummy_authorization_handler" => {
                 "options" => { "passport_number" => "AXXXXXXXX" }
@@ -421,7 +421,7 @@ shared_examples "Managing component permissions" do
       end
 
       it "saves the action from the permissions hash as an empty hash" do
-        within "form.new_component_permissions" do
+        within "#components form" do
           within ".foo-permission" do
             uncheck "Example authorization (Direct)"
             uncheck "Another example authorization (Direct)"