decidim-dev 0.26.10 → 0.27.0.rc1

data/config/locales/sv.yml

@@ -1,3 +1,4 @@
+---
 sv:
   activemodel:
     attributes:
@@ -14,8 +15,6 @@ sv:
       dummy:
         settings:
           global:
-            guided: Guidad ingång
-            guided_help: Hjälptext
             readonly_attribute: Skrivskyddat attribut
             test: Ett test
             test_choices:

data/config/locales/sw-KE.yml

@@ -1 +1,2 @@
+---
 sw:

data/config/locales/ti-ER.yml

@@ -1 +1,2 @@
+---
 ti:

data/config/locales/tr-TR.yml

@@ -1,3 +1,4 @@
+---
 tr:
   activemodel:
     attributes:

data/config/locales/uk.yml

@@ -1,3 +1,4 @@
+---
 uk:
   activemodel:
     attributes:

data/config/locales/val-ES.yml

@@ -1 +1,2 @@
+---
 val:

data/config/locales/vi.yml

@@ -1 +1,2 @@
+---
 vi:

data/config/locales/zh-CN.yml

@@ -1,3 +1,4 @@
+---
 zh-CN:
   activemodel:
     attributes:

data/config/locales/zh-TW.yml

@@ -1,64 +1,2 @@
+---
 zh-TW:
-  activemodel:
-    attributes:
-      dummy_resource:
-        created_at: 建立於
-        decidim_scope_id: 範圍
-        field: 我的欄位
-        start_date: 開始日期
-        title: 標題
-        translatable_text: 可翻譯的文字
-        updated_at: 更新於
-  decidim:
-    components:
-      dummy:
-        settings:
-          global:
-            guided: 導引輸入
-            guided_help: 說明文字
-            guided_readonly: 禁用輸入
-            guided_rich: 導引豐富輸入
-            guided_rich_help_html: HTML <strong>說明</strong> 文字
-            guided_rich_readonly_html: 禁用輸入 HTML <strong>說明</strong> 文字
-            readonly_attribute: 唯讀屬性
-            test: A 測試
-            test_choices:
-              a: A選項
-              b: B選項
-              c: C選項
-            test_options:
-              bar: Bar
-              baz: Baz
-              foo: Foo
-          step:
-            endorsements_blocked: 封鎖支持
-            endorsements_enabled: 啟用支持
-            readonly_step_attribute: 唯讀步驟屬性
-            test_options:
-              bar: Bar
-              baz: Baz
-              foo: Foo
-    dummy:
-      admin:
-        exports:
-          dummies: 測試帳戶
-    gamification:
-      badges:
-        test:
-          conditions:
-            - 在Decidim中使用測試環境。
-          description: 參與者通過建立測試可獲得此徽章。
-          description_another: 此參與者已建立了 %{score} 個測試。
-          description_own: 您已建立了 %{score} 個測試。
-          name: 測試
-          next_level_in: 再建立 %{score} 個測試即可達到下一個等級！
-          unearned_another: 這個參與者尚未創建任何測試。
-          unearned_own: 您尚未建立任何測試。
-    resource_links:
-      test_link:
-        dummy_resource_dummy: 相關的測試帳戶
-    statistics:
-      bar: Bar
-      dummies_count_high: 高級測試帳戶
-      dummies_count_medium: 中級測試帳戶
-      foo: Foo

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

@@ -512,17 +512,7 @@
         },
         "weight": 0,
         "permissions": null,
-        "published_at": "2020-01-22 07:55:04 UTC",
-        "specific_data": {
-          "body": {
-            "en": "English content",
-            "ca": "Catalan content",
-            "es": "Spanish content",
-            "machine_translations": {
-              "es": "Machine translation spanish content"
-            }
-          }
-        }
+        "published_at": "2020-01-22 07:55:04 UTC"
       },
       {
         "manifest_name": "proposals",

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

@@ -1,72 +1,51 @@
 {
-    "Response": {
-      "MetaInfo": {
-        "Timestamp": "2013-02-08T16:26:39.382+0000"
+  "items": [
+    {
+      "title": "5 Rue Daunou, 75002 Paris, France",
+      "id": "here:af:streetsection:bI4Le6cyA.1mlQyLunYpjC:CggIBCCi-9SPARABGgE1KGQ",
+      "resultType": "houseNumber",
+      "houseNumberType": "PA",
+      "address": {
+        "label": "5 Rue Daunou, 75002 Paris, France",
+        "countryCode": "FRA",
+        "countryName": "France",
+        "stateCode": "IDF",
+        "state": "Île-de-France",
+        "county": "Paris",
+        "city": "Paris",
+        "district": "2e Arrondissement",
+        "street": "Rue Daunou",
+        "postalCode": "75002",
+        "houseNumber": "5"
       },
-      "View": [
+      "position": {
+        "lat": 48.86926,
+        "lng": 2.3321
+      },
+      "access": [
         {
-          "_type": "SearchResultsViewType",
-          "ViewId": 0,
-          "Result": [
-            {
-              "Relevance": 1.0,
-              "MatchLevel": "houseNumber",
-              "MatchQuality": {
-                "State": 1.0,
-                "City": 1.0,
-                "Street": [
-                  1.0
-                ],
-                "HouseNumber": 1.0
-              },
-              "MatchType": "pointAddress",
-              "Location": {
-                "LocationId": "NT_ArsGdYbpo6dqjPQel9gTID_4",
-                "LocationType": "point",
-                "DisplayPosition": {
-                  "Latitude": 40.7504692,
-                  "Longitude": -73.9933777
-                },
-                "NavigationPosition": [
-                  {
-                    "Latitude": 40.7500305,
-                    "Longitude": -73.9942398
-                  }
-                ],
-                "MapView": {
-                  "TopLeft": {
-                    "Latitude": 40.7515934,
-                    "Longitude": -73.9948616
-                  },
-                  "BottomRight": {
-                    "Latitude": 40.7493451,
-                    "Longitude": -73.9918938
-                  }
-                },
-                "Address": {
-                  "Label": "4 Penn Plz, New York, NY 10001, United States",
-                  "Country": "USA",
-                  "State": "NY",
-                  "County": "New York",
-                  "City": "New York",
-                  "Street": "Penn Plz",
-                  "HouseNumber": "4",
-                  "PostalCode": "10001",
-                  "AdditionalData": [
-                    {
-                      "value": "United States",
-                      "key": "CountryName"
-                    },
-                    {
-                      "value": "New York",
-                      "key": "StateName"
-                    }
-                  ]
-                }
-              }
-            }
-          ]
+          "lat": 48.86931,
+          "lng": 2.33215
+        }
+      ],
+      "mapView": {
+        "west": 2.33073,
+        "south": 48.86836,
+        "east": 2.33347,
+        "north": 48.87016
+      },
+      "scoring": {
+        "queryScore": 0.97,
+        "fieldScore": {
+          "country": 1,
+          "city": 1,
+          "streets": [
+            1
+          ],
+          "houseNumber": 1,
+          "postalCode": 0.82
         }
-      ]
+      }
     }
-  }
+  ]
+}

data/lib/decidim/dev/common_rake.rb

@@ -3,6 +3,7 @@
 require "bundler/gem_tasks"
 require "rspec/core/rake_task"
 require "decidim/dev"
+require "parallel_tests/tasks"
 
 RSpec::Core::RakeTask.new(:spec) do |t|
   t.rspec_opts = "--format progress --format RspecJunitFormatter -o ~/rspec/rspec.xml" if ENV["CI"]

data/lib/decidim/dev/railtie.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "rails"
+require "decidim/rails"
 
 module Decidim
   module Dev

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

@@ -3,6 +3,7 @@
 require "decidim/dev"
 
 ENV["RAILS_ENV"] ||= "test"
+ENV["RAILS_SYSTEM_TESTING_SCREENSHOT_HTML"] ||= "1"
 
 engine_spec_dir = File.join(Dir.pwd, "spec")
 
@@ -18,7 +19,7 @@ require_relative "rspec_support/authorization"
 
 require "#{Decidim::Dev.dummy_app_path}/config/environment"
 
-Dir["#{engine_spec_dir}/shared/**/*.rb"].sort.each { |f| require f }
+Dir["#{engine_spec_dir}/shared/**/*.rb"].each { |f| require f }
 
 require "paper_trail/frameworks/rspec"
 

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

@@ -2,9 +2,7 @@
 
 shared_examples_for "accessible page" do
   it "passes accessibility tests" do
-    expect(page).to be_axe_clean.excluding(
-      'a[title="Find out more about cookies"]'
-    )
+    expect(page).to be_axe_clean
   end
 
   it "passes HTML validation" do

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

@@ -9,6 +9,36 @@ module AttachmentHelpers
   def attachment_file_size(attachment)
     ActiveSupport::NumberHelper.number_to_human_size(attachment.file_size)
   end
+
+  # Creates ActiveStorage::Blob object and returns its signed_id
+  # Used in non-system tests that need files.
+  def upload_test_file(file, options = {})
+    filename = options[:filename] || upload_test_file_filename(file)
+    content_type = options[:content_type] || upload_test_file_content_type(file)
+
+    blob = ActiveStorage::Blob.create_and_upload!(
+      io: File.open(file),
+      filename: filename,
+      content_type: content_type
+    )
+    return blob if options[:return_blob]
+
+    blob.signed_id
+  end
+
+  private
+
+  def upload_test_file_filename(file)
+    return file.original_filename if file.respond_to? :original_filename
+
+    file.split("/").last
+  end
+
+  def upload_test_file_content_type(file)
+    return file.content_type if file.respond_to? :content_type
+
+    MIME::Types.type_for(file).first.content_type
+  end
 end
 
 RSpec.configure do |config|

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

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module Capybara
+  module AutoCompleteJS
+    def autocomplete_select(value, from:)
+      within("div[data-autocomplete-for='#{from}']") do
+        find(".autocomplete-input").click
+        find(".autocomplete-input").native.send_keys(value[0..4])
+        expect(page).to have_css("#autoComplete_list_1") # select should be open now
+
+        expect(page).to have_css("#autoComplete_result_0", text: value)
+        find("#autoComplete_result_0", text: value).hover
+        expect(page).to have_css("#autoComplete_result_0", text: value)
+        find("#autoComplete_result_0", text: value).click
+        expect(page).to have_css(".autocomplete__selected-item", text: value)
+      end
+    end
+  end
+end
+
+RSpec.configure do |config|
+  config.include Capybara::AutoCompleteJS, type: :system
+end

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

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require "selenium-webdriver"
-require "system_test_html_screenshots"
 
 module Decidim
   # Helpers meant to be used only during capybara test runs.
@@ -32,11 +31,40 @@ Capybara.register_driver :headless_chrome do |app|
                   else
                     "--window-size=1920,1080"
                   end
+  Capybara::Selenium::Driver.new(
+    app,
+    browser: :chrome,
+    capabilities: [options]
+  )
+end
+
+Capybara.server_port = rand(5000..6999)
+
+# In order to work with PWA apps, Chrome can't be run in headless mode, and requires
+# setting up special prefs and flags
+Capybara.register_driver :pwa_chrome do |app|
+  options = ::Selenium::WebDriver::Chrome::Options.new
+  options.args << "--no-sandbox"
+  # Don't limit browser resources
+  options.args << "--disable-dev-shm-usage"
+  # Add pwa.lvh.me host as a secure origin
+  options.args << "--unsafely-treat-insecure-origin-as-secure=http://pwa.lvh.me:#{Capybara.server_port}"
+  # User data flag is mandatory when preferences and locale state is set
+  options.args << "--user-data-dir=/tmp/decidim_tests_user_data_#{rand(1000)}"
+  options.args << if ENV["BIG_SCREEN_SIZE"].present?
+                    "--window-size=1920,3000"
+                  else
+                    "--window-size=1920,1080"
+                  end
+  # Set notifications allowed in http protocol
+  options.local_state["browser.enabled_labs_experiments"] = ["enable-system-notifications@1", "unsafely-treat-insecure-origin-as-secure"]
+  # Mark notification permission as enabled
+  options.prefs["profile.default_content_setting_values.notifications"] = 1
 
   Capybara::Selenium::Driver.new(
     app,
     browser: :chrome,
-    options: options
+    capabilities: [options]
   )
 end
 
@@ -49,11 +77,11 @@ Capybara.register_driver :iphone do |app|
   Capybara::Selenium::Driver.new(
     app,
     browser: :chrome,
-    options: options
+    capabilities: [options]
   )
 end
 
-Capybara.server = :puma, { Silent: true, Threads: "1:1" }
+Capybara.server = :puma, { Silent: true, queue_requests: false }
 
 Capybara.asset_host = "http://localhost:3000"
 
@@ -65,6 +93,16 @@ RSpec.configure do |config|
   config.before :each, type: :system do
     driven_by(:headless_chrome)
     switch_to_default_host
+    domain = (try(:organization) || try(:current_organization))&.host
+    if domain
+      page.driver.browser.execute_cdp(
+        "Network.setCookie",
+        domain: domain,
+        name: Decidim.consent_cookie_name,
+        value: { essential: true }.to_json,
+        path: "/"
+      )
+    end
   end
 
   config.before :each, driver: :rack_test do
@@ -80,7 +118,7 @@ RSpec.configure do |config|
   end
 
   config.after(type: :system) do |example|
-    warn page.driver.browser.manage.logs.get(:browser) unless example.metadata[:driver].eql?(:rack_test)
+    warn page.driver.browser.logs.get(:browser) unless example.metadata[:driver].eql?(:rack_test)
   end
 
   config.include Decidim::CapybaraTestHelpers, type: :system

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(user.name, opts) && have_content(text, opts)
+      have_content(user.name, **opts) && have_content(text, **opts)
     end
   end
 

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

@@ -89,7 +89,7 @@ module Decidim
       include Followable
       include Traceable
       include Publicable
-      include Decidim::DataPortability
+      include Decidim::DownloadYourData
       include Searchable
       include Paddable
       include Amendable
@@ -236,8 +236,8 @@ Decidim.register_component(:dummy) do |component|
     settings.attribute :comments_enabled, type: :boolean, default: true
     settings.attribute :comments_max_length, type: :integer, required: false
     settings.attribute :resources_permissions_enabled, type: :boolean, default: true
-    settings.attribute :dummy_global_attribute_1, type: :boolean
-    settings.attribute :dummy_global_attribute_2, type: :boolean, readonly: ->(_context) { false }
+    settings.attribute :dummy_global_attribute1, type: :boolean
+    settings.attribute :dummy_global_attribute2, type: :boolean, readonly: ->(_context) { false }
     settings.attribute :readonly_attribute, type: :boolean, default: true, readonly: ->(_context) { true }
     settings.attribute :enable_pads_creation, type: :boolean, default: false
     settings.attribute :amendments_enabled, type: :boolean, default: false
@@ -246,8 +246,8 @@ Decidim.register_component(:dummy) do |component|
 
   component.settings(:step) do |settings|
     settings.attribute :comments_blocked, type: :boolean, default: false
-    settings.attribute :dummy_step_attribute_1, type: :boolean
-    settings.attribute :dummy_step_attribute_2, type: :boolean, readonly: ->(_context) { false }
+    settings.attribute :dummy_step_attribute1, type: :boolean
+    settings.attribute :dummy_step_attribute2, type: :boolean, readonly: ->(_context) { false }
     settings.attribute :dummy_step_translatable_text, type: :text, translated: true, editor: true, required: true
     settings.attribute :readonly_step_attribute, type: :boolean, default: true, readonly: ->(_context) { true }
     settings.attribute :amendment_creation_enabled, type: :boolean, default: true

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

@@ -1,25 +1,5 @@
 # frozen_string_literal: true
 
-shared_examples "has mandatory config setting" do |mandatory_field|
-  let(:edit_component_path) do
-    Decidim::EngineRouter.admin_proxy(component.participatory_space).edit_component_path(component.id)
-  end
-
-  before do
-    visit edit_component_path
-    component.update(settings: { mandatory_field => "" })
-    visit edit_component_path
-  end
-
-  it "does not allow updating the component" do
-    click_button "Update"
-
-    within ".#{mandatory_field}_container" do
-      expect(page).to have_content("There's an error in this field")
-    end
-  end
-end
-
 shared_context "with a component" do
   let(:manifest) { Decidim.find_component_manifest(manifest_name) }
   let(:user) { create :user, :confirmed, organization: organization }

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

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+module Decidim
+  module ControllerExampleGroup
+    extend ActiveSupport::Concern
+
+    class_methods do
+      def routes
+        before do
+          routes = yield
+          @orig_default_url_options = routes.default_url_options.dup
+          routes.default_url_options[:script_name] = ""
+
+          self.routes = routes
+        end
+
+        after do
+          routes.default_url_options = @orig_default_url_options
+        end
+      end
+    end
+  end
+end
+
+RSpec.configure do |config|
+  config.include Decidim::ControllerExampleGroup, type: :controller
+end

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

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+module Capybara
+  module Cookies
+    # Update cookie consent
+    def select_cookies(cookies = "all", options = {})
+      visit decidim.root_path if options[:visit_root]
+
+      dialog_present = begin
+        find("#cc-dialog-wrapper")
+      rescue Capybara::ElementNotFound => _e
+        false
+      end
+
+      if dialog_present
+        click_button "Settings"
+      else
+        within ".footer" do
+          click_link "Cookie settings"
+        end
+      end
+
+      if [true, "all"].include?(cookies)
+        click_button "Accept all"
+      elsif cookies.is_a?(Array)
+        cookies.each do |cookie|
+          within "[data-id='#{cookie}']" do
+            find(".switch-paddle").click
+          end
+        end
+        click_button "Save settings"
+      elsif [false, "essential"].include?(cookies)
+        click_button "Accept only essential"
+      end
+    end
+  end
+end
+
+RSpec.configure do |config|
+  config.include Capybara::Cookies, type: :system
+end

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

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+# Adapted from https://github.com/JedWatson/react-select/issues/832#issuecomment-276441836
+
+module Capybara
+  module UploadModal
+    # Replaces attach_file.
+    # Beware that modal does not open within form!
+    def dynamically_attach_file(name, file_location, options = {})
+      find("##{name}_button").click
+      filename = options[:filename] || file_location.to_s.split("/").last
+
+      yield if block_given?
+
+      within ".upload-modal" do
+        find(".remove-upload-item").click if options[:remove_before]
+        input_element = find("input[type='file']", visible: :all)
+        input_element.attach_file(file_location)
+        within "[data-filename='#{filename}']" do
+          expect(page).to have_css("div.progress-bar.filled", wait: 5)
+        end
+        all("input.attachment-title").last.set(options[:title]) if options.has_key?(:title)
+        click_button "Save" unless options[:keep_modal_open]
+      end
+    end
+  end
+end
+
+RSpec.configure do |config|
+  config.include Capybara::UploadModal, type: :system
+end

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

@@ -4,7 +4,7 @@ module FrontendHelpers
   # Thanks to:
   # https://medium.com/@coorasse/catch-javascript-errors-in-your-system-tests-89c2fe6773b1
   def expect_no_js_errors
-    errors = page.driver.browser.manage.logs.get(:browser)
+    errors = page.driver.browser.logs.get(:browser)
     return if errors.blank?
 
     aggregate_failures "javascript errors" do