decidim-core 0.31.5 → 0.31.7

data/db/migrate/20250217192438_convert_user_groups_into_users.rb

@@ -33,8 +33,8 @@ class ConvertUserGroupsIntoUsers < ActiveRecord::Migration[7.0]
   def up
     User.old_group.find_each do |group|
       if group.email.blank? || another_user_with_same_email_in_organization?(group)
-        group.update_attribute(:email, "user_group_#{group.id}@#{group.organization.host}.invalid")
         group.update_attribute(:extended_data, (group.extended_data || {}).merge("patched" => true, "previous_email" => group.email))
+        group.update_attribute(:email, "user_group_#{group.id}@#{group.organization.host}.invalid")
 
         group.reload
       end
@@ -49,7 +49,8 @@ class ConvertUserGroupsIntoUsers < ActiveRecord::Migration[7.0]
     User.new_group.find_each do |group|
       group.update_attribute(:officialized_at, nil)
       group.update_attribute(:type, "Decidim::UserGroup")
-      group.update_attribute(:extended_data, (group.extended_data || {}).except("group"))
+      group.update_attribute(:email, group.extended_data["previous_email"]) if group.extended_data["previous_email"].present?
+      group.update_attribute(:extended_data, (group.extended_data || {}).except("group", "patched", "previous_email"))
     end
   end
   # rubocop:enable Rails/SkipsModelValidations

data/decidim-core.gemspec

@@ -74,6 +74,9 @@ Gem::Specification.new do |s|
   s.add_dependency "premailer-rails", "~> 1.10"
   s.add_dependency "rack", "~> 2.2", ">= 2.2.8.1"
   s.add_dependency "rack-attack", "~> 6.0"
+  # Workaround until shakapacker releases a new version compatible with rack-proxy v1.0
+  # @see https://github.com/shakacode/shakapacker/issues/1220
+  s.add_dependency "rack-proxy", "~> 0.8.3", "< 1.0"
   s.add_dependency "rails", "~> 7.2.0", ">= 7.2.2.2"
   s.add_dependency "rails-i18n", "~> 7.0"
   s.add_dependency "ransack", "~> 4.2.0"

data/lib/decidim/api/functions/user_entity_list.rb

@@ -11,6 +11,7 @@ module Decidim
     #
     class UserEntityList
       include NeedsApiFilterAndOrder
+      include NeedsApiDefaultOrder
 
       def initialize
         @model_class = Decidim::UserBaseEntity
@@ -24,6 +25,7 @@ module Decidim
                  .includes(avatar_attachment: :blob)
         add_filter_keys(args[:filter])
         add_order_keys(args[:order].to_h)
+        add_default_order
         @query
       end
     end

data/lib/decidim/attachment_attributes.rb

@@ -15,24 +15,39 @@ module Decidim
       #
       # name - The attribute's name
       #
-      #
       # Example:
       #
-      #   attachment_attribute :photos
+      #   attachments_attribute :photos
       #   # This will create two attributes of the following types:
       #   #   attribute :photos, Array[Integer]
       #   #   attribute :add_photos, Array
-      #   # In addition, it will generate the getter method for the attribute
-      #   # returning an array of the Decidim::Attachment records.
+      #   # In addition, it will generate:
+      #   #   - A setter that handles String (JSON/CSV), Integer, or Array inputs.
+      #   #   - A getter that falls back to add_photos when the attribute is blank.
+      #   #   - Private helpers for parsing and extracting IDs.
       #
       # Returns nothing.
-      def attachments_attribute(name)
+      def attachments_attribute(name) # rubocop:disable Metrics/CyclomaticComplexity,Metrics/PerceivedComplexity
         attribute name, Array[Integer]
         attribute :"add_#{name}", Array
 
-        # Define the getter method that fetches the attachment records based on
-        # their types. For Strings and Integers, assumes they are IDs and will
-        # fetch the attachment record matching that ID.
+        # Setter: coerces String (JSON or comma-separated IDs) and bare Integer
+        # values into the expected Array[Integer] format before delegating to super.
+        define_method :"#{name}=" do |value|
+          case value
+          when String
+            parsed = send(:"parse_string_#{name}", value)
+            parsed.any? ? super(parsed) : super(value)
+          when Integer
+            super([value])
+          else
+            super(value)
+          end
+        end
+
+        # Getter: resolves stored integer IDs to Decidim::Attachment records,
+        # caching the result on the instance. Falls back to extracting IDs from
+        # add_<name> when the attribute itself is blank, then resolves those too.
         variable_name = "@#{name}_records"
         define_method name do
           return instance_variable_get(variable_name) if instance_variable_defined?(variable_name)
@@ -40,9 +55,15 @@ module Decidim
           original = @attributes[name.to_s].value_before_type_cast
           return original if original && !original.is_a?(Array)
 
+          ids = if super().blank? && send(:"add_#{name}").present?
+                  send(:"extract_ids_from_add_#{name}")
+                else
+                  super()
+                end
+
           instance_variable_set(
             variable_name,
-            super().map do |attachment|
+            ids.map do |attachment|
               if attachment.is_a?(Integer)
                 Decidim::Attachment.find_by(id: attachment)
               else
@@ -51,6 +72,34 @@ module Decidim
             end.compact
           )
         end
+
+        # Private helpers -------------------------------------------------------
+
+        define_method :"extract_ids_from_add_#{name}" do
+          send(:"add_#{name}")
+            .select { |item| item.is_a?(Hash) && (item[:id].present? || item["id"].present?) }
+            .map { |item| (item[:id] || item["id"]).to_i }
+        end
+
+        define_method :"parse_string_#{name}" do |value|
+          return [] if value.blank?
+
+          send(:"parse_#{name}_ids", value)
+        end
+
+        define_method :"parse_#{name}_ids" do |value|
+          ids = begin
+            Array(JSON.parse(value))
+          rescue JSON::ParserError
+            value.split(",").map(&:strip)
+          end
+
+          ids.map(&:to_i).reject(&:zero?)
+        end
+
+        private :"extract_ids_from_add_#{name}"
+        private :"parse_string_#{name}"
+        private :"parse_#{name}_ids"
       end
     end
   end

data/lib/decidim/command.rb

@@ -53,7 +53,7 @@ module Decidim
     protected
 
     def event_arguments
-      raise "#{self.class} must implement #event_arguments"
+      raise "#{self.class} must implement #event_arguments (passed to #{event_namespace})"
     end
 
     def event_namespace

data/lib/decidim/content_renderers/base_renderer.rb

@@ -12,6 +12,16 @@ module Decidim
     class BaseRenderer
       include Decidim::ContentProcessor::Common
 
+      ReplacementContext = Struct.new(:placement, :node_name, :attribute_name, :ancestor_names, keyword_init: true) do
+        def text?
+          placement == :text
+        end
+
+        def attribute?
+          placement == :attribute
+        end
+      end
+
       # @return [String] the content to be formatted
       attr_reader :content
 
@@ -34,6 +44,108 @@ module Decidim
       def render(_options = nil)
         content
       end
+
+      protected
+
+      def replace_pattern_by_context(text, pattern, skip_ancestor_tags: %w(code pre script style), on_missing: "")
+        return text unless text.respond_to?(:gsub)
+
+        skip_ancestor_tags = Array(skip_ancestor_tags).map(&:to_s)
+
+        has_match = pattern.is_a?(String) ? text.include?(pattern) : pattern.match?(text)
+        return text unless has_match
+
+        fragment = html_fragment(text)
+        attr_modified = replace_pattern_in_attributes(fragment, pattern, skip_ancestor_tags:, on_missing:) do |match, context|
+          yield(match, context)
+        end
+        text_modified = replace_pattern_in_text_nodes(fragment, pattern, skip_ancestor_tags:, on_missing:) do |match, context|
+          yield(match, context)
+        end
+
+        return text unless attr_modified || text_modified
+
+        fragment.to_s
+      end
+
+      private
+
+      def replace_pattern_in_attributes(fragment, pattern, skip_ancestor_tags:, on_missing:)
+        modified = false
+        fragment.xpath(".//*").each do |node|
+          next if skip_replacement_for_node?(node, skip_ancestor_tags)
+
+          node.attribute_nodes.each do |attribute|
+            replaced_value = attribute.value.gsub(pattern) do |match|
+              replace_match(match, replacement_context(node, placement: :attribute, attribute_name: attribute.name), on_missing:) do |resolved_match, context|
+                yield(resolved_match, context)
+              end
+            end
+            unless replaced_value == attribute.value
+              attribute.value = replaced_value
+              modified = true
+            end
+          end
+        end
+        modified
+      end
+
+      def replace_pattern_in_text_nodes(fragment, pattern, skip_ancestor_tags:, on_missing:)
+        modified = false
+        fragment.xpath(".//text()").each do |node|
+          parent = node.parent
+          next if skip_replacement_for_node?(parent, skip_ancestor_tags)
+
+          original_text = node.text
+          has_node_match = pattern.is_a?(String) ? original_text.include?(pattern) : pattern.match?(original_text)
+          next unless has_node_match
+
+          doc = node.document
+          context = replacement_context(parent, placement: :text)
+          last_pos = 0
+
+          original_text.scan(pattern) do
+            m = Regexp.last_match
+            node.add_previous_sibling(Nokogiri::XML::Text.new(original_text[last_pos...m.begin(0)], doc)) if m.begin(0) > last_pos
+
+            replacement = replace_match(m[0], context, on_missing:) do |resolved_match, ctx|
+              yield(resolved_match, ctx)
+            end
+
+            Loofah.fragment(replacement.to_s).children.to_a.each do |child|
+              node.add_previous_sibling(child)
+            end
+
+            last_pos = m.end(0)
+          end
+
+          node.add_previous_sibling(Nokogiri::XML::Text.new(original_text[last_pos..], doc)) if last_pos < original_text.length
+          node.remove
+          modified = true
+        end
+        modified
+      end
+
+      def replace_match(match, context, on_missing:)
+        yield(match, context)
+      rescue ActiveRecord::RecordNotFound
+        on_missing.respond_to?(:call) ? on_missing.call(match, context) : on_missing
+      end
+
+      def replacement_context(node, placement:, attribute_name: nil)
+        ReplacementContext.new(
+          placement:,
+          node_name: node&.name,
+          attribute_name:,
+          ancestor_names: node ? node.ancestors.map(&:name) : []
+        )
+      end
+
+      def skip_replacement_for_node?(node, skip_ancestor_tags)
+        return false unless node
+
+        ([node.name] + node.ancestors.map(&:name)).any? { |name| skip_ancestor_tags.include?(name) }
+      end
     end
   end
 end

data/lib/decidim/content_renderers/blob_renderer.rb

@@ -33,11 +33,10 @@ module Decidim
       protected
 
       def replace_pattern(text, pattern)
-        return text unless text.respond_to?(:gsub)
-
-        text.gsub(pattern) do
-          blob_gid = Regexp.last_match(1)
-          variation_key = Regexp.last_match(3)
+        replace_pattern_by_context(text, pattern) do |match, _context|
+          match_data = match.match(GLOBAL_ID_REGEX)
+          blob_gid = match_data[1]
+          variation_key = match_data[3]
 
           blob = GlobalID::Locator.locate(blob_gid)
           if variation_key
@@ -50,8 +49,6 @@ module Decidim
           else
             blob_url(blob)
           end
-        rescue ActiveRecord::RecordNotFound => _e
-          ""
         end
       end
 

data/lib/decidim/content_renderers/mention_resource_renderer.rb

@@ -27,17 +27,15 @@ module Decidim
       protected
 
       def replace_pattern(text, pattern, editor:)
-        return text unless text.respond_to?(:gsub)
-
-        text.gsub(pattern) do |resource_gid|
+        replace_pattern_by_context(text, pattern) do |resource_gid, context|
           resource = GlobalID::Locator.locate(resource_gid)
-          if editor
+          if context.attribute?
+            render_resource_url(resource, editor:)
+          elsif editor
             render_editor(resource_gid, resource)
           else
             render_resource_link(resource)
           end
-        rescue ActiveRecord::RecordNotFound => _e
-          ""
         end
       end
 
@@ -50,6 +48,12 @@ module Decidim
         link_to mention_title(resource), resource_path(resource)
       end
 
+      def render_resource_url(resource, editor:)
+        return resource_path(resource) if editor
+
+        Decidim::ResourceLocatorPresenter.new(resource).url
+      end
+
       def mention_title(resource)
         title = presenter_for(resource).title
 

data/lib/decidim/content_renderers/resource_renderer.rb

@@ -11,17 +11,26 @@ module Decidim
       #
       # @return [String] the content ready to display (contains HTML)
       def render(_options = nil)
-        return content unless content.respond_to?(:gsub)
-
-        content.gsub(regex) do |resource_gid|
+        replace_pattern_by_context(content, regex, on_missing: proc { |match, _| "~#{match.split("/").last}" }) do |resource_gid, context|
           resource = GlobalID::Locator.locate(resource_gid)
-          resource.presenter.display_mention
-        rescue ActiveRecord::RecordNotFound
-          resource_id = resource_gid.split("/").last
-          "~#{resource_id}"
+
+          if context.attribute?
+            resource_attribute_value(resource)
+          else
+            resource.presenter.display_mention
+          end
         end
       end
 
+      protected
+
+      def resource_attribute_value(resource)
+        presenter = resource.presenter
+        return presenter.profile_path if presenter.respond_to?(:profile_path)
+
+        Decidim::ResourceLocatorPresenter.new(resource).path
+      end
+
       def regex
         raise "Not implemented"
       end

data/lib/decidim/content_renderers/user_renderer.rb

@@ -24,17 +24,15 @@ module Decidim
       protected
 
       def replace_pattern(text, pattern, editor:)
-        return text unless text.respond_to?(:gsub)
-
-        text.gsub(pattern) do |mentionable_gid|
-          mentionable = GlobalID::Locator.locate(mentionable_gid)
-          if editor
-            render_editor(mentionable)
+        replace_pattern_by_context(text, pattern) do |user_gid, context|
+          user = GlobalID::Locator.locate(user_gid)
+          if context.attribute?
+            render_profile_path(user)
+          elsif editor
+            render_editor(user)
           else
-            render_text(mentionable)
+            render_text(user)
           end
-        rescue ActiveRecord::RecordNotFound => _e
-          ""
         end
       end
 
@@ -55,6 +53,10 @@ module Decidim
           presenter_for(user).display_mention
         end
       end
+
+      def render_profile_path(user)
+        presenter_for(user).profile_path
+      end
     end
   end
 end

data/lib/decidim/core/content_blocks/registry_manager.rb

@@ -51,11 +51,11 @@ module Decidim
             ]
 
             content_block.settings do |settings|
-              settings.attribute :title, type: :text, translated: true
+              settings.attribute :title, type: :text, translated: true, required: true
               settings.attribute :short_description, type: :text, translated: true
-              settings.attribute :action_button_title, type: :text, translated: true
-              settings.attribute :action_button_subtitle, type: :text, translated: true
-              settings.attribute :action_button_url, type: :text
+              settings.attribute :action_button_title, type: :text, translated: true, required: true
+              settings.attribute :action_button_subtitle, type: :text, translated: true, required: true
+              settings.attribute :action_button_url, type: :text, required: true
             end
 
             content_block.default!

data/lib/decidim/core/engine.rb

@@ -495,6 +495,14 @@ module Decidim
         end
       end
 
+      initializer "decidim_core.delete_account" do
+        config.to_prepare do
+          ActiveSupport::Notifications.subscribe("decidim.destroy_account:after") do |_event_name, data|
+            Decidim::DeleteUserMailer.delete(user_email: data[:user_email], user_name: data[:user_name], locale: data[:locale], organization: data[:organization]).deliver_later
+          end
+        end
+      end
+
       initializer "decidim_core.add_cells_view_paths" do
         Cell::ViewModel.view_paths << Rails.root.join("app/views") # for partials
         Cell::ViewModel.view_paths << File.expand_path("#{Decidim::Core::Engine.root}/app/cells")

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

@@ -85,6 +85,8 @@ FactoryBot.define do
     "#{Faker::Lorem.sentence(word_count: 1, supplemental: true, random_words_to_add: 3)} #{n}"
   end
 
+  sequence(:component_position)
+
   factory :category, class: "Decidim::Category" do
     transient do
       skip_injection { false }
@@ -463,6 +465,7 @@ FactoryBot.define do
     manifest_name { "dummy" }
     published_at { Time.current }
     deleted_at { nil }
+    weight { generate(:component_position) }
     settings do
       {
         dummy_global_translatable_text: generate_localized_title(:dummy_global_translatable_text, skip_injection:),

data/lib/decidim/core/test/shared_examples/admin_resource_gallery_examples.rb

@@ -87,7 +87,7 @@ end
 
 shared_examples "admin creates resource gallery" do
   context "when uploading images" do
-    let(:uploaded_photos) do
+    let(:uploaded_attachments) do
       [
         {
           title: "Picture of the city",
@@ -99,10 +99,10 @@ shared_examples "admin creates resource gallery" do
         }
       ]
     end
-    let(:photos) { [] }
+    let(:current_attachments) { [] }
 
     it "creates a gallery for the resource" do
-      expect { command.call }.to change(Decidim::Attachment, :count).by(uploaded_photos.count)
+      expect { command.call }.to change(Decidim::Attachment, :count).by(uploaded_attachments.count)
 
       resource = resource_class.last
       expect(resource.photos.count).to eq(2)
@@ -111,7 +111,7 @@ shared_examples "admin creates resource gallery" do
     end
 
     context "when gallery is left blank" do
-      let(:uploaded_photos) { [] }
+      let(:uploaded_attachments) { [] }
 
       it "broadcasts ok" do
         expect { command.call }.to broadcast(:ok)
@@ -122,7 +122,7 @@ end
 
 shared_examples "admin manages resource gallery" do
   context "when managing images" do
-    let(:uploaded_photos) do
+    let(:uploaded_attachments) do
       [
         {
           title: "Picture of the city",
@@ -134,10 +134,10 @@ shared_examples "admin manages resource gallery" do
         }
       ]
     end
-    let(:photos) { [] }
+    let(:current_attachments) { [] }
 
     it "creates a gallery for the resource" do
-      expect { command.call }.to change(Decidim::Attachment, :count).by(uploaded_photos.count)
+      expect { command.call }.to change(Decidim::Attachment, :count).by(uploaded_attachments.count)
       resource = resource_class.last
       expect(resource.photos.count).to eq(2)
       last_attachment = Decidim::Attachment.last
@@ -145,7 +145,7 @@ shared_examples "admin manages resource gallery" do
     end
 
     context "when gallery is left blank" do
-      let(:uploaded_photos) { [] }
+      let(:uploaded_attachments) { [] }
 
       it "broadcasts ok" do
         expect { command.call }.to broadcast(:ok)
@@ -155,8 +155,8 @@ shared_examples "admin manages resource gallery" do
     context "when images are removed" do
       let!(:image1) { create(:attachment, :with_image, attached_to: resource) }
       let!(:image2) { create(:attachment, :with_image, attached_to: resource) }
-      let(:uploaded_photos) { [] }
-      let(:current_photos) { [image1] }
+      let(:uploaded_attachments) { [] }
+      let(:current_attachments) { [image1.id] }
 
       it "to decrease the number of photos in the gallery" do
         expect(resource.attachments.count).to eq(2)

data/lib/decidim/core/test/shared_examples/amendable/amendment_event_examples.rb

@@ -8,7 +8,7 @@ shared_examples "amendment event" do
   it_behaves_like "a simple event email"
   it_behaves_like "a simple event notification"
 
-  let(:emendation_author_nickname) { "@#{emendation.creator_author.nickname}" }
+  let(:emendation_author_name) { emendation.creator_author.name }
   let(:emendation_path) { Decidim::ResourceLocatorPresenter.new(emendation).path }
   let(:emendation_author_path) { Decidim::UserPresenter.new(emendation.creator_author).profile_path }
   let(:amendable_path) { Decidim::ResourceLocatorPresenter.new(amendable).path }
@@ -16,5 +16,5 @@ shared_examples "amendment event" do
   let(:email_intro) { "An amendment has been #{amendment_type} for #{amendable_title}. You can see it from this page:" }
   let(:email_outro) { "You have received this notification because you are following #{amendable_title}. You can stop receiving notifications following the previous link." }
 
-  let(:notification_title) { "The <a href=\"#{emendation_path}\">amendment</a> created by <a href=\"#{emendation_author_path}\">#{emendation_author_nickname}</a> has been #{amendment_type} for <a href=\"#{amendable_path}\">#{amendable_title}</a>." } # rubocop:disable Layout/LineLength
+  let(:notification_title) { "The <a href=\"#{emendation_path}\">amendment</a> created by <a href=\"#{emendation_author_path}\">#{emendation_author_name}</a> has been #{amendment_type} for <a href=\"#{amendable_path}\">#{amendable_title}</a>." } # rubocop:disable Layout/LineLength
 end

data/lib/decidim/core/test/shared_examples/amendable/amendment_promoted_event_examples.rb

@@ -10,15 +10,15 @@ shared_examples "amendment promoted event" do
   it_behaves_like "a simple event email"
   it_behaves_like "a simple event notification"
 
-  let(:emendation_author_nickname) { "@#{emendation.creator_author.nickname}" }
+  let(:emendation_author_name) { emendation.creator_author.name }
   let(:emendation_path) { Decidim::ResourceLocatorPresenter.new(emendation).path }
   let(:emendation_author_path) { Decidim::UserPresenter.new(emendation.creator_author).profile_path }
   let(:amendable_path) { Decidim::ResourceLocatorPresenter.new(amendable).path }
 
-  let(:email_subject) { "An amendment from #{emendation_author_nickname} has been published as a new proposal" }
+  let(:email_subject) { "An amendment from #{emendation_author_name} has been published as a new proposal" }
   let(:email_intro) { "A rejected amendment for #{amendable_title} has been published as a new #{amendable_type}. You can see it from this page:" }
   let(:email_outro) { "You have received this notification because you are following #{amendable_title}. You can stop receiving notifications following the previous link." }
   # rubocop:disable Layout/LineLength
-  let(:notification_title) { "A <a href=\"#{emendation_path}\">rejected amendment</a> for <a href=\"#{amendable_path}\">#{amendable_title}</a> has been published as a new #{amendable_type} by <a href=\"#{emendation_author_path}\">#{emendation_author_nickname}</a>." }
+  let(:notification_title) { "A <a href=\"#{emendation_path}\">rejected amendment</a> for <a href=\"#{amendable_path}\">#{amendable_title}</a> has been published as a new #{amendable_type} by <a href=\"#{emendation_author_path}\">#{emendation_author_name}</a>." }
   # rubocop:enable Layout/LineLength
 end

data/lib/decidim/core/test/shared_examples/comments_examples.rb

@@ -147,6 +147,16 @@ shared_examples "comments" do
       expect(page).to have_css(".comment-thread")
     end
 
+    it "opens the login modal when clicking the comment button" do
+      visit resource_path
+
+      expect(page).to have_no_css("#loginModal-content", visible: :visible)
+
+      find(".add-comment-mobile").click
+
+      expect(page).to have_css("#loginModal-content", visible: :visible)
+    end
+
     context "when user visit a mobile browser" do
       before do
         driven_by(:iphone)
@@ -176,6 +186,22 @@ shared_examples "comments" do
       expect(page).to have_css(".add-comment form")
     end
 
+    it "does not show login modal when clicking the comment button" do
+      visit resource_path
+
+      find("a.add-comment-mobile").click
+
+      expect(page).to have_no_css("#loginModal-content", visible: :visible)
+    end
+
+    it "guides the user to the comment form when clicking the comment button" do
+      visit resource_path
+
+      find("a.add-comment-mobile").click
+
+      expect(page).to have_css("#add-comment-anchor")
+    end
+
     context "when user visit a computer browser" do
       before do
         switch_to_host(organization.host)
@@ -1000,16 +1026,16 @@ shared_examples "comments" do
         let(:content) { "A valid user mention: @#{mentioned_user.nickname}" }
 
         context "when text finish with a mention" do
-          it "shows the tribute container" do
-            expect(page).to have_css(".tribute-container", text: mentioned_user.name, wait: 10)
+          it "shows the suggestions menu" do
+            expect(page).to have_css(".editor-suggestions-item", text: "@#{mentioned_user.nickname} (#{mentioned_user.name})", wait: 10)
           end
         end
 
         context "when text contains a mention" do
           let(:content) { "A valid user mention: @#{mentioned_user.nickname}." }
 
-          it "shows the tribute container" do
-            expect(page).to have_no_css(".tribute-container", text: mentioned_user.name)
+          it "hides the suggestions menu" do
+            expect(page).to have_no_css(".editor-suggestions-item", text: "@#{mentioned_user.nickname} (#{mentioned_user.name})")
           end
         end
       end
@@ -1018,8 +1044,8 @@ shared_examples "comments" do
         let!(:mentioned_user) { create(:user, organization:) }
         let(:content) { "A unconfirmed user mention: @#{mentioned_user.nickname}" }
 
-        it "do not show the tribute container" do
-          expect(page).to have_no_css(".tribute-container", text: mentioned_user.name)
+        it "does not show the suggestions menu" do
+          expect(page).to have_no_css(".editor-suggestions-item", text: "@#{mentioned_user.nickname} (#{mentioned_user.name})")
         end
       end
     end

data/lib/decidim/core/test/shared_examples/has_space_in_mcell_examples.rb

@@ -8,7 +8,8 @@ shared_examples_for "has space in m-cell" do
     let(:show_space) { true }
 
     it "renders the space where the model belongs to" do
-      expect(cell_html).to have_content(decidim_escape_translated(model.component.participatory_space.title))
+      expect(cell_html).to have_content(translated_attribute(model.component.participatory_space.title))
+      expect(cell_html.to_s).to include(decidim_escape_translated(model.component.participatory_space.title).gsub(""", "\""))
     end
   end
 end