decidim-dev 0.30.6 → 0.31.0.rc1

data/lib/decidim/dev/assets/participatory_processes.json

@@ -12,7 +12,6 @@
       "es": "In laudantium."
     },
     "slug": "distinctio-cupiditate",
-    "hashtag": "harum",
     "short_description": {
       "ca": "<p>Harum magnam id.</p>",
       "en": "<p>Aut quam doloribus.</p>",

data/lib/decidim/dev/assets/participatory_processes_with_null.json

@@ -12,7 +12,6 @@
       "es": "In laudantium."
     },
     "slug": "distinctio-cupiditate",
-    "hashtag": "harum",
     "short_description": {
       "ca": "<p>Harum magnam id.</p>",
       "en": "<p>Aut quam doloribus.</p>",

data/lib/decidim/dev/assets/valid_election_census.csv

@@ -0,0 +1,3 @@
+email;token
+user1@example.org;token123
+user2@example.org;token456

data/lib/decidim/dev/assets/valid_emails.csv

@@ -0,0 +1,27 @@
+first_test_email@example.org
+second_test_email@example.org
+third_test_email@example.org
+this_is_a_test_email@test.org
+test.123@test.org
+example.123@test.org
+this_is_another_test_email@test.com
+test_user_001@example.com
+test.account@example.com
+test.example@example.com
+example.account@example.com
+new_email_test@example.com
+one_more_example_email@test.org
+another_example_email@test.org
+first_test@test.org
+one_example_email@test.org
+second_test@test.com
+four.test@test.org
+the_test_email@example.org
+these_test_email@test.org
+more_example_email@test.org
+the_new_email_test@example.test
+another_example_email@test.example
+this_is_the_new_email_test@example.test
+one_more_example_email@example.test
+another_example_email@example.test
+first_test@example.test

data/lib/decidim/dev/component.rb

@@ -33,8 +33,8 @@ Decidim.register_component(:dummy) do |component|
     settings.attribute :amendment_reaction_enabled, type: :boolean, default: true
     settings.attribute :amendment_promotion_enabled, type: :boolean, default: true
     settings.attribute :amendments_visibility, type: :string, default: "all"
-    settings.attribute :endorsements_enabled, type: :boolean, default: false
-    settings.attribute :endorsements_blocked, type: :boolean, default: false
+    settings.attribute :likes_enabled, type: :boolean, default: false
+    settings.attribute :likes_blocked, type: :boolean, default: false
   end
 
   component.register_resource(:dummy_resource) do |resource|

data/lib/decidim/dev/engine.rb

@@ -25,7 +25,7 @@ module Decidim
         ActiveSupport.on_load(:action_controller) { include Decidim::Dev::NeedsDevelopmentTools } if Rails.env.development? || ENV.fetch("DECIDIM_DEV_ENGINE", nil)
       end
 
-      initializer "decidim_dev.webpacker.assets_path" do
+      initializer "decidim_dev.shakapacker.assets_path" do
         Decidim.register_assets_path File.expand_path("app/packs", root)
       end
 
@@ -45,12 +45,6 @@ module Decidim
           end
         end
       end
-
-      initializer "decidim_dev.graphql_api" do
-        next unless Decidim.module_installed?(:api)
-
-        Decidim::Api.add_orphan_type Decidim::Dev::DummyResourceType
-      end
     end
   end
 end

data/lib/decidim/dev/test/base_spec_helper.rb

@@ -4,6 +4,15 @@ require "decidim/dev"
 
 ENV["RAILS_ENV"] ||= "test"
 ENV["RAILS_SYSTEM_TESTING_SCREENSHOT_HTML"] ||= "1"
+ENV["DECIDIM_API_JWT_SECRET"] ||= "dummy-api-jwt-secret"
+ENV["DECIDIM_AVAILABLE_LOCALES"] ||= "en,ca,es"
+ENV["DECIDIM_ENABLE_MACHINE_TRANSLATION"] ||= "true"
+ENV["DECIDIM_SPAM_DETECTION_BACKEND_RESOURCE"] ||= "memory"
+ENV["DECIDIM_SPAM_DETECTION_BACKEND_USER"] ||= "memory"
+ENV["DECIDIM_SMS_GATEWAY_SERVICE"] ||= "Decidim::Verifications::Sms::ExampleGateway"
+ENV["DECIDIM_TIMESTAMP_SERVICE"] ||= "Decidim::Initiatives::DummyTimestamp"
+ENV["DECIDIM_PDF_SIGNATURE_SERVICE"] ||= "Decidim::PdfSignatureExample"
+ENV["DECIDIM_MACHINE_TRANSLATION_SERVICE"] ||= "Decidim::Dev::DummyTranslator"
 
 engine_spec_dir = File.join(Dir.pwd, "spec")
 

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

@@ -15,8 +15,6 @@ FactoryBot.define do
     transient do
       skip_injection { false }
       users { nil }
-      # user_groups correspondence to users is by sorting order
-      user_groups { [] }
     end
     title { generate_localized_title(:dummy_resource_title, skip_injection:) }
     component { create(:dummy_component, skip_injection:) }
@@ -27,11 +25,11 @@ FactoryBot.define do
       published_at { Time.current }
     end
 
-    trait :with_endorsements do
+    trait :with_likes do
       after :create do |resource, evaluator|
         5.times.collect do
-          create(:endorsement, resource:, skip_injection: evaluator.skip_injection,
-                               author: build(:user, organization: resource.component.organization, skip_injection: evaluator.skip_injection))
+          create(:like, resource:, skip_injection: evaluator.skip_injection,
+                        author: build(:user, organization: resource.component.organization, skip_injection: evaluator.skip_injection))
         end
       end
     end
@@ -61,13 +59,8 @@ FactoryBot.define do
 
     after :build do |resource, evaluator|
       evaluator.authors_list.each do |coauthor|
-        resource.coauthorships << if coauthor.is_a?(Decidim::UserGroup)
-                                    build(:coauthorship, author: coauthor.users.first, user_group: coauthor, coauthorable: resource,
-                                                         organization: evaluator.component.organization, skip_injection: evaluator.skip_injection)
-                                  else
-                                    build(:coauthorship, author: coauthor, coauthorable: resource, organization: evaluator.component.organization,
-                                                         skip_injection: evaluator.skip_injection)
-                                  end
+        resource.coauthorships << build(:coauthorship, author: coauthor, coauthorable: resource, organization: evaluator.component.organization,
+                                                       skip_injection: evaluator.skip_injection)
       end
     end
   end

data/lib/decidim/dev/test/rspec_support/activestorage_matchers.rb

@@ -28,7 +28,7 @@ module ActiveStorageMatchers
         when :redirect, :representation
           ActiveStorage::Blob.find_signed(key_match)
         when :disk
-          decoded = ActiveStorage.verifier.verified(key_match, purpose: :blob_key)
+          decoded = ActiveStorage.verifier.verified(key_match, purpose: :blob_key).with_indifferent_access
           ActiveStorage::Blob.find_by(key: decoded[:key]) if decoded
         end
     end

data/lib/decidim/dev/test/rspec_support/capybara.rb

@@ -30,28 +30,21 @@ module Decidim
   end
 end
 
-1.step do
-  port = rand(5000..6999)
+# Expected values: "", "2", "3", etc. (see parallel_tests documentation)
+parallel_run_idx = ENV.fetch("TEST_ENV_NUMBER", "").to_i
+parallel_run_idx -= 1 if parallel_run_idx.positive?
+Capybara.server_port = 1.step do |num|
+  port = 4999 + num + (100 * parallel_run_idx)
+  next if port == 5432 # Reserved for PostgreSQL
   next if port == 6379 # Reserved for Redis
 
+  # Make sure the port is not reserved by any other application.
   begin
-    redis = Redis.new
-    reserved_ports = (redis.get("decidim_test_capybara_reserved_ports") || "").split(",").map(&:to_i)
-    unless reserved_ports.include?(port)
-      reserved_ports << port
-      if ParallelTests.last_process?
-        redis.del("decidim_test_capybara_reserved_ports")
-      else
-        redis.set("decidim_test_capybara_reserved_ports", reserved_ports.sort.join(","))
-      end
-      break
-    end
-  rescue Redis::CannotConnectError
-    # Redis is not available
-    break
+    Socket.tcp("127.0.0.1", port, connect_timeout: 5).close
+    warn "Port #{port} is already in use, trying another one."
+  rescue Errno::ECONNREFUSED
+    break port
   end
-ensure
-  Capybara.server_port = port
 end
 
 Capybara.register_driver :headless_chrome do |app|

data/lib/decidim/dev/test/rspec_support/comments.rb

@@ -3,7 +3,7 @@
 module CommentsHelpers
   def have_comment_from(user, text, opts = {})
     within "#comments" do
-      have_content(decidim_sanitize_translated(user.name).gsub("\n", " "), **opts) && have_content(text, **opts)
+      have_content(user.name, **opts).and have_content(text, **opts)
     end
   end
 

data/lib/decidim/dev/test/rspec_support/component.rb

@@ -18,7 +18,7 @@ RSpec.configure do |config|
           t.datetime :published_at
           t.datetime :deleted_at
           t.integer :coauthorships_count, null: false, default: 0
-          t.integer :endorsements_count, null: false, default: 0
+          t.integer :likes_count, null: false, default: 0
           t.integer :comments_count, null: false, default: 0
           t.integer :follows_count, null: false, default: 0
 
@@ -53,7 +53,7 @@ RSpec.configure do |config|
           t.datetime :published_at
           t.datetime :deleted_at
           t.integer :coauthorships_count, null: false, default: 0
-          t.integer :endorsements_count, null: false, default: 0
+          t.integer :likes_count, null: false, default: 0
           t.integer :comments_count, null: false, default: 0
 
           t.references :decidim_component, index: false

data/lib/decidim/dev/test/rspec_support/component_context.rb

@@ -114,111 +114,3 @@ shared_context "when managing a component as a process admin" do
            participatory_process:)
   end
 end
-
-shared_context "when publishing and unpublishing the component" do
-  let(:title) { translated(current_component.name) }
-  # When resources are being created or modified, the following jobs are enqueued among the ones that we need to wait for.
-  # When running in CI, we have situations when job processing takes longer, causing flaky tests.
-  # Adding a list of exceptions here, helps us to avoid those situations.
-  let(:job_exceptions) do
-    [
-      Decidim::MachineTranslationResourceJob,
-      ActiveStorage::AnalyzeJob
-    ]
-  end
-
-  context "when component is unpublished" do
-    before do
-      current_component.unpublish!
-      current_component.participatory_space.try_add_to_index_as_search_resource
-
-      visit decidim_admin_participatory_processes.components_path(current_component.participatory_space)
-    end
-
-    it "reindexes on publication" do
-      Decidim::SearchableResource.where(resource:).delete_all
-      expect(Decidim::SearchableResource.where(resource:).count).to be_zero
-
-      within "tr", text: title do
-        click_on "Publish"
-      end
-
-      expect(page).to have_admin_callout("The component has been successfully published")
-
-      perform_enqueued_jobs(except: job_exceptions)
-
-      expect(Decidim::SearchableResource.where(resource:).count).to be_positive
-      expect(component.reload).to be_published
-    end
-  end
-
-  context "when component is published" do
-    before do
-      current_component.publish!
-      current_component.participatory_space.try_add_to_index_as_search_resource
-
-      visit decidim_admin_participatory_processes.components_path(current_component.participatory_space)
-    end
-
-    it "removes records from index" do
-      perform_enqueued_jobs(except: job_exceptions)
-
-      expect(Decidim::SearchableResource.where(resource:).count).to be_positive
-
-      within ".sidebar-menu" do
-        click_on "Components"
-      end
-
-      within "tr", text: title do
-        click_on "Hide"
-      end
-
-      perform_enqueued_jobs(except: job_exceptions)
-
-      expect(Decidim::SearchableResource.where(resource:).count).to be_positive
-
-      within "tr", text: title do
-        click_on "Unpublish"
-      end
-
-      expect(page).to have_admin_callout("The component has been successfully unpublished")
-
-      perform_enqueued_jobs(except: job_exceptions)
-
-      expect(Decidim::SearchableResource.where(resource:).count).to be_zero
-      expect(current_component.reload).not_to be_published
-    end
-  end
-end
-
-shared_context "when cycling through publication states" do
-  include_context "when managing a component as an admin" do
-    let(:title) { translated(current_component.name) }
-
-    it "cycles through unpublished and published states successfully" do
-      visit decidim_admin_participatory_processes.components_path(current_component.participatory_space)
-
-      within ".sidebar-menu" do
-        click_on "Components"
-      end
-
-      within "tr", text: title do
-        click_on "Hide"
-      end
-
-      expect(page).to have_admin_callout("The component has been successfully hidden from the menu.")
-
-      within "tr", text: title do
-        click_on "Unpublish"
-      end
-
-      expect(page).to have_admin_callout("The component has been successfully unpublished")
-
-      within "tr", text: title do
-        click_on "Publish"
-      end
-
-      expect(page).to have_admin_callout("The component has been successfully published")
-    end
-  end
-end

data/lib/decidim/dev/test/rspec_support/editor_context.rb

@@ -1,32 +1,20 @@
 # frozen_string_literal: true
 
-shared_context "with editor content containing hashtags and mentions" do
-  let(:hashtag) { create(:hashtag, organization:) }
-  let(:hashtag2) { create(:hashtag, organization:) }
+shared_context "with editor content containing mentions" do
   let(:user) { create(:user, :confirmed, organization:) }
   let(:user2) { create(:user, :confirmed, organization:) }
-  let(:user_group) { create(:user_group, :confirmed, organization:) }
-  let(:user_group2) { create(:user_group, :confirmed, organization:) }
 
   let(:html) do
     <<~HTML
-      <p>Paragraph with a hashtag #{hashtag.to_global_id} and another hashtag #{hashtag2.to_global_id}</p>
       <p>Paragraph with a user mention #{user.to_global_id} and another user mention #{user2.to_global_id}</p>
-      <p>Paragraph with a user group mention #{user_group.to_global_id} and another user group mention #{user_group2.to_global_id}</p>
     HTML
   end
   let(:editor_html) do
     <<~HTML
-      <p>Paragraph with a hashtag #{html_hashtag(hashtag)} and another hashtag #{html_hashtag(hashtag2)}</p>
       <p>Paragraph with a user mention #{html_mention(user)} and another user mention #{html_mention(user2)}</p>
-      <p>Paragraph with a user group mention #{html_mention(user_group)} and another user group mention #{html_mention(user_group2)}</p>
     HTML
   end
 
-  def html_hashtag(hashtag)
-    %(<span data-type="hashtag" data-label="##{hashtag.name}">##{hashtag.name}</span>)
-  end
-
   def html_mention(mentionable)
     mention = "@#{mentionable.nickname}"
     label = "#{mention} (#{CGI.escapeHTML(mentionable.name)})"

data/lib/decidim/dev/test/rspec_support/gamification.rb

@@ -4,7 +4,7 @@ RSpec.configure do |config|
   config.before(:suite) do
     Decidim::Gamification.register_badge(:test) do |badge|
       badge.levels = [1, 5, 10]
-      badge.valid_for = [:user, :user_group]
+      badge.valid_for = [:user]
       badge.reset = ->(_user) { 100 }
     end
   end

data/lib/decidim/dev/test/rspec_support/helpers.rb

@@ -134,6 +134,22 @@ module Decidim
         JS
       )
     end
+
+    def select_text(selector)
+      page.execute_script(
+        <<~JS
+          var selection = document.getSelection();
+          var range = document.createRange();
+          var element = document.querySelector("#{selector}");
+
+          range.selectNodeContents(element);
+          selection.removeAllRanges();
+          selection.addRange(range);
+          document.dispatchEvent(new MouseEvent("selectstart"));
+          document.dispatchEvent(new MouseEvent("mouseup"));
+        JS
+      )
+    end
   end
 
   module FrontEndFileTestHelpers

data/lib/decidim/dev/test/rspec_support/rake_tasks.rb

@@ -40,22 +40,6 @@ module RakeTaskOutputHelpers
       $stdout = original_stdout
     end
   end
-
-  def check_no_errors_have_been_printed
-    expect($stdout.string).not_to include("ERROR:")
-  end
-
-  def check_some_errors_have_been_printed
-    expect($stdout.string).to include("ERROR:")
-  end
-
-  def check_error_printed(type)
-    expect($stdout.string).to include("ERROR: [#{type}]")
-  end
-
-  def check_message_printed(message)
-    expect($stdout.string).to include(message)
-  end
 end
 
 RSpec.configure do |config|

data/lib/decidim/dev/test/rspec_support/screenshot_helper.rb

@@ -20,9 +20,9 @@ module ActionDispatch
         end
 
         # Add the file URI scheme so terminal emulators can open one click
-        def display_image
+        def display_image(html:, **)
           message = "[Screenshot Image]: file://#{image_path}\n"
-          message << "[Screenshot HTML]: file://#{html_path}\n" if save_html?
+          message << "[Screenshot HTML]: file://#{html_path}\n" if html
 
           case output_type
           when "artifact"

data/lib/decidim/dev/test/rspec_support/{webpacker.rb → shakapacker.rb}

@@ -6,12 +6,12 @@ RSpec.configure do |config|
     raise "package.json file does not exist" unless Rails.root.join("package.json").exist?
     raise "Node modules directory does not exist" unless Rails.root.join("node_modules").exist?
 
-    Dir.chdir(Rails.root) { Webpacker.compile }
+    Dir.chdir(Rails.root) { Shakapacker.compile }
   rescue Errno::ENOENT
     node_modules_contents = `ls #{Rails.root.join("node_modules")}`
 
     message = <<~ERROR
-      There was an error during the Webpacker compilation
+      There was an error during the Shakapacker compilation
       #{"=" * 80}
       Node version: #{`node -v`}
       #{"=" * 80}

data/lib/decidim/dev/test/rspec_support/z_download_helper.rb

@@ -43,6 +43,5 @@ RSpec.configure do |config|
     FileUtils.mkdir_p DownloadHelper::PATH.to_s
     page.driver.browser.download_path = DownloadHelper::PATH.to_s
     clear_downloads
-    driven_by :headless_chrome
   end
 end

data/lib/decidim/dev/test/spec_helper.rb

@@ -24,7 +24,7 @@ RSpec.configure do |config|
   config.raise_errors_for_deprecations!
   config.example_status_persistence_file_path = ".rspec-failures"
   config.filter_run_when_matching :focus
-  config.profile_examples = 10
+  config.profile_examples = 10 unless ENV.fetch("CI", false)
   config.default_formatter = "doc" if config.files_to_run.one?
 
   # If you are not using ActiveRecord, or you'd prefer not to run each of your

data/lib/decidim/dev/version.rb

@@ -4,7 +4,7 @@ module Decidim
   # This holds the decidim-dev version.
   module Dev
     def self.version
-      "0.30.6"
+      "0.31.0.rc1"
     end
   end
 end

data/lib/decidim/dev.rb

@@ -15,8 +15,6 @@ require "decidim/dev/auth_engine"
 # Also, to avoid further headaches :)
 # require "decidim/dev/component"
 
-require "decidim/dev/api"
-
 module Decidim
   # Decidim::Dev holds all the convenience logic and libraries to be able to
   # create external libraries that create test apps and test themselves against

data/rubocop-decidim.yml

@@ -5,16 +5,24 @@
 # For further details, see:
 # https://github.com/rubocop/rubocop/issues/4154#issuecomment-316004878
 
-require:
+plugins:
   - rubocop-rubycw
-  - rubocop-capybara
-  - rubocop-factory_bot
 
 inherit_from:
-  - config/rubocop/performance.yml
-  - config/rubocop/ruby.yml
-  - config/rubocop/rails.yml
-  - config/rubocop/rspec.yml
-  - config/rubocop/faker.yml
-  - config/rubocop/graphql.yml
+  - config/rubocop/capybara/configuration.yml
+  - config/rubocop/capybara/disabled.yml
+  - config/rubocop/performance/configuration.yml
+  - config/rubocop/performance/disabled.yml
+  - config/rubocop/ruby/configuration.yml
+  - config/rubocop/ruby/disabled.yml
+  - config/rubocop/rails/configuration.yml
+  - config/rubocop/rails/disabled.yml
+  - config/rubocop/rspec/configuration.yml
+  - config/rubocop/rspec/disabled.yml
+  - config/rubocop/faker/configuration.yml
+  - config/rubocop/faker/disabled.yml
+  - config/rubocop/graphql/configuration.yml
+  - config/rubocop/graphql/disabled.yml
+  - config/rubocop/factory_bot/configuration.yml
+  - config/rubocop/factory_bot/disabled.yml
   - config/rubocop/disabled.yml