<%= t(".description") %>
+ <%= form.check_box :underage, label: t(".label") %> + <%= form.select :underage_limit, ( Decidim::ExtraUserFields::Engine::DEFAULT_UNDERAGE_OPTIONS).to_a, selected: current_organization.extra_user_fields["underage_limit"] || Decidim::ExtraUserFields::Engine::DEFAULT_UNDERAGE_LIMIT, label: t(".limit") %> +<%== @body %>
diff --git a/config/assets.rb b/config/assets.rb new file mode 100644 index 0000000..9be6ca7 --- /dev/null +++ b/config/assets.rb @@ -0,0 +1,29 @@ +# frozen_string_literal: true + +# This file is located at `config/assets.rb` of your module. + +# Define the base path of your module. Please note that `Rails.root` may not be +# used because we are not inside the Rails environment when this file is loaded. +base_path = File.expand_path("..", __dir__) + +# Register an additional load path for webpack. All the assets within these +# directories will be available for inclusion within the Decidim assets. For +# example, if you have `app/packs/src/decidim/foo.js`, you can include that file +# in your JavaScript entrypoints (or other JavaScript files within Decidim) +# using `import "src/decidim/foo"` after you have registered the additional path +# as follows. +Decidim::Webpacker.register_path("#{base_path}/app/packs") + +# Register the entrypoints for your module. These entrypoints can be included +# within your application using `javascript_pack_tag` and if you include any +# SCSS files within the entrypoints, they become available for inclusion using +# `stylesheet_pack_tag`. +Decidim::Webpacker.register_entrypoints( + decidim_extra_user_fields: "#{base_path}/app/packs/entrypoints/decidim_extra_user_fields.js", + decidim_extra_user_fields_css: "#{base_path}/app/packs/entrypoints/decidim_extra_user_fields.scss" +) + +# If you want to import some extra SCSS files in the Decidim main SCSS file +# without adding any extra stylesheet inclusion tags, you can use the following +# method to register the stylesheet import for the main application. +# Decidim::Webpacker.register_stylesheet_import("stylesheets/decidim/homepage_interactive_map/map.scss") diff --git a/config/i18n-tasks.yml b/config/i18n-tasks.yml index ffb4faa..aa01e24 100644 --- a/config/i18n-tasks.yml +++ b/config/i18n-tasks.yml @@ -6,8 +6,10 @@ locales: [en] ignore_unused: - "decidim.components.extra_user_fields.name" - activemodel.attributes.user.* + - activemodel.errors.models.user.* - decidim.admin.extra_user_fields.menu.title - decidim.extra_user_fields.genders.* ignore_missing: - decidim.participatory_processes.scopes.global + - decidim.admin.exports.notice diff --git a/config/locales/de.yml b/config/locales/de.yml new file mode 100644 index 0000000..50d9c8d --- /dev/null +++ b/config/locales/de.yml @@ -0,0 +1,82 @@ +--- +de: + activemodel: + attributes: + user: + country: Land + date_of_birth: Geburtsdatum + gender: Geschlecht + location: Standort + phone_number: Telefonnummer + postal_code: Postleitzahl + errors: + models: + user: + attributes: + date_of_birth: + underage: ungültig. Wenn Sie minderjährig sind, müssen Sie die Erlaubnis der Eltern einholen + decidim: + admin: + actions: + export: Exportieren + exports: + export_as: Exportieren im Format %{export_format} + extra_user_fields: + menu: + title: Benutzerdefinierte Anmeldefelder verwalten + components: + extra_user_fields: + name: Benutzerdefinierte Anmeldefelder + extra_user_fields: + admin: + exports: + users: Teilnehmer + extra_user_fields: + fields: + country: + description: Dieses Feld enthält eine Liste von Ländern. Der Benutzer kann ein Land auswählen. + label: Das Feld Land aktivieren + date_of_birth: + description: Dieses Feld ist ein Feld für das Geburtsdatum. Der Benutzer kann ein Datum auswählen. + label: Das Feld Geburtsdatum aktivieren + gender: + description: Dieses Feld ist ein Feld für die Geschlechtsidentität. Der Benutzer kann ein Geschlecht auswählen. + label: Das Feld Geschlecht aktivieren + location: + description: Dieses Feld ermöglicht das Hinzufügen von Text. Der Benutzer kann einen Ort auswählen. + label: Das Feld Standort aktivieren + phone_number: + description: Dieses Feld ist ein Telefonnummernfeld. Der Benutzer kann eine Nummer auswählen. + label: Das Feld Telefonnummer aktivieren + postal_code: + description: Dieses Feld ist für die Postleitzahl. Der Benutzer kann eine Postleitzahl auswählen. + label: Das Postleitzahlenfeld aktivieren. + form: + callout: + help: Aktivieren Sie die Funktion für benutzerdefinierte Anmeldefelder, um zusätzliche Felder in Ihrem Anmeldeformular zu verwalten. Auch bei aktivierter Option wird das Anmeldeformular nur aktualisiert, wenn mindestens ein zusätzliches Feld aktiviert ist. + extra_user_fields: + extra_user_fields_enabled: Benutzerdefinierte Anmeldefelder aktivieren + section: Verfügbare Anmeldefelder für das Anmeldeformular + global: + title: Aktivieren / Deaktivieren von benutzerdefinierten Anmeldefeldern + index: + save: Speichern + title: Benutzerdefinierte Anmeldefelder verwalten + update: + failure: Bei der Aktualisierung ist ein Fehler aufgetreten. + success: Die Anmeldefelder wurden erfolgreich aktualisiert. + genders: + female: Frau + male: Mann + other: Divers + registration_form: + signup: + legend: Weitere Informationen + statutory_representative: + inform: + body: | + Hallo, + Sie wurden als gesetzlicher Vertreter von %{name} für die Registrierung bei %{organization} benannt. + Beste grüße, + Das %{organization} Team + subject: Sie wurden als gesetzlicher Vertreter benannt diff --git a/config/locales/en.yml b/config/locales/en.yml index d592c05..af86e18 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -9,6 +9,14 @@ en: location: Location phone_number: Phone Number postal_code: Postal code + statutory_representative_email: Representative email + underage: Underage + errors: + models: + user: + attributes: + date_of_birth: + underage: invalid. If you are underage, you must obtain parental authorization decidim: admin: actions: @@ -57,6 +65,11 @@ en: description: This field is a String field. If checked, user will have to fill in a postal code label: Enable postal code field + underage: + description: This field is a Boolean field. User will be able to check + if is underage + label: Enable parental authorization field + limit: This sets the age limit (ex. 18 years old) form: callout: help: Enable custom extra user fields functionality to be able to manage @@ -80,3 +93,12 @@ en: registration_form: signup: legend: More information + underage: I am under %{limit} years old and I agree to get a parental authorization + statutory_representative: + inform: + body: | + Hello, + You have been designated as the legal representative of %{name} for their registration with %{organization}. + Best regards, + The %{organization} Team + subject: You have been designated as the legal representative diff --git a/config/locales/fr.yml b/config/locales/fr.yml index a82b997..fd9beea 100644 --- a/config/locales/fr.yml +++ b/config/locales/fr.yml @@ -9,6 +9,15 @@ fr: location: Localisation phone_number: Numéro de téléphone postal_code: Code postal + statutory_representative_email: Email du représentant légal + underage: Mineur + errors: + models: + user: + attributes: + date_of_birth: + underage: invalide. Si vous êtes mineur, vous devez obtenir une autorisation + parentale decidim: admin: actions: @@ -57,6 +66,11 @@ fr: description: Ce champ est un champ code postal. L'utilisateur pourra choisir un code postal. label: Activer le champ code postal + underage: + description: Ce champ est un champ booléen. L'utilisateur pourra cocher + s'il est mineur. + label: Activer le champ d'autorisation parentale + limit: Cela définit la limite d'âge (ex. 18 ans) form: callout: help: Activez la fonctionnalité des champs d'inscription personnalisés @@ -81,3 +95,13 @@ fr: registration_form: signup: legend: Plus d'information + underage: Je suis âgé de moins de %{limit} ans et j'accepte d'obtenir une + autorisation parentale + statutory_representative: + inform: + body: | + Bonjour, + Vous avez été désigné comme représentant légal de %{name} pour son inscription à %{organization}. + Cordialement, + L'équipe de %{organization} + subject: Vous avez été désigné comme représentant légal diff --git a/lib/decidim/extra_user_fields/engine.rb b/lib/decidim/extra_user_fields/engine.rb index 655fb04..e6a1cb6 100644 --- a/lib/decidim/extra_user_fields/engine.rb +++ b/lib/decidim/extra_user_fields/engine.rb @@ -13,10 +13,15 @@ class Engine < ::Rails::Engine DEFAULT_GENDER_OPTIONS = [:male, :female, :other].freeze + DEFAULT_UNDERAGE_LIMIT = 18 + + DEFAULT_UNDERAGE_OPTIONS = (15..21) + routes do # Add engine routes here # resources :extra_user_fields # root to: "extra_user_fields#index" + get "underage_limit", to: "extra_user_fields#retrieve_underage_limit", as: :retrieve_underage_limit end initializer "decidim_extra_user_fields.registration_additions" do @@ -54,6 +59,12 @@ class Engine < ::Rails::Engine end end end + + initializer "decidim_extra_user_fields.mount_routes" do + Decidim::Core::Engine.routes do + mount Decidim::ExtraUserFields::Engine, at: "/extra_user_fields", as: "decidim_extra_user_fields_engine" + end + end end end end diff --git a/package-lock.json b/package-lock.json new file mode 100644 index 0000000..e83bb48 --- /dev/null +++ b/package-lock.json @@ -0,0 +1,6 @@ +{ + "name": "decidim-module-extra_user_fields", + "lockfileVersion": 3, + "requires": true, + "packages": {} +} diff --git a/spec/commands/decidim/create_omniauth_registration_spec.rb b/spec/commands/decidim/create_omniauth_registration_spec.rb index 3a44850..a87436a 100644 --- a/spec/commands/decidim/create_omniauth_registration_spec.rb +++ b/spec/commands/decidim/create_omniauth_registration_spec.rb @@ -18,6 +18,8 @@ module Comments let(:location) { "Paris" } let(:phone_number) { "0123456789" } let(:postal_code) { "75001" } + let(:underage) { false } + let(:statutory_representative_email) { nil } let(:extended_data) do { country:, @@ -25,7 +27,9 @@ module Comments gender:, location:, phone_number:, - postal_code: + postal_code:, + underage:, + statutory_representative_email: } end @@ -45,7 +49,9 @@ module Comments "date_of_birth" => date_of_birth, "gender" => gender, "phone_number" => phone_number, - "location" => location + "location" => location, + "underage" => underage, + "statutory_representative_email" => statutory_representative_email } } end @@ -147,7 +153,6 @@ module Comments it "links a previously existing user" do user = create(:user, email:, organization:) expect { command.call }.not_to change(User, :count) - expect(user.identities.length).to eq(1) end @@ -227,6 +232,15 @@ module Comments expect(user).not_to be_confirmed end end + + context "when the user is underage and tries to duplicate email" do + let(:underage) { true } + let(:statutory_representative_email) { email } + + it "broadcasts invalid" do + expect { command.call }.to broadcast(:invalid) + end + end end end end diff --git a/spec/commands/decidim/create_registration_spec.rb b/spec/commands/decidim/create_registration_spec.rb index 6760b25..080fde6 100644 --- a/spec/commands/decidim/create_registration_spec.rb +++ b/spec/commands/decidim/create_registration_spec.rb @@ -21,6 +21,8 @@ module Comments let(:location) { "Paris" } let(:phone_number) { "0123456789" } let(:postal_code) { "75001" } + let(:underage) { "0" } + let(:statutory_representative_email) { nil } let(:extended_data) do { country:, @@ -28,7 +30,9 @@ module Comments gender:, location:, phone_number:, - postal_code: + postal_code:, + underage:, + statutory_representative_email: } end @@ -46,7 +50,9 @@ module Comments "date_of_birth" => date_of_birth, "gender" => gender, "phone_number" => phone_number, - "location" => location + "location" => location, + "underage" => underage, + "statutory_representative_email" => statutory_representative_email } } end @@ -118,7 +124,9 @@ module Comments gender:, location:, phone_number:, - postal_code: + postal_code:, + underage:, + statutory_representative_email: } ).and_call_original @@ -140,6 +148,33 @@ module Comments end.to change(User, :count).by(1) end end + + describe "when the user is underage and sends a valid email" do + let(:underage) { "1" } + let(:statutory_representative_email) { "user@example.fr" } + + it "creates a user with the statutory representative email and sends email" do + expect do + expect(Decidim::ExtraUserFields::StatutoryRepresentativeMailer).to receive(:inform).with(instance_of(Decidim::User)).and_call_original + + command.call + + user = User.last + expect(user.extended_data["statutory_representative_email"]).to eq(statutory_representative_email) + end.to change(User, :count).by(1) + end + end + + describe "when the user is underage and tries to duplicate email" do + let(:underage) { "1" } + let(:statutory_representative_email) { email } + + it "broadcasts invalid" do + expect(Decidim::ExtraUserFields::StatutoryRepresentativeMailer).not_to receive(:inform).with(instance_of(Decidim::User)).and_call_original + + expect { command.call }.to broadcast(:invalid) + end + end end end end diff --git a/spec/commands/decidim/extra_user_fields/admin/update_extra_user_fields_spec.rb b/spec/commands/decidim/extra_user_fields/admin/update_extra_user_fields_spec.rb index 1acb5ba..3631000 100644 --- a/spec/commands/decidim/extra_user_fields/admin/update_extra_user_fields_spec.rb +++ b/spec/commands/decidim/extra_user_fields/admin/update_extra_user_fields_spec.rb @@ -18,6 +18,8 @@ module Admin let(:phone_number_pattern) { "^(\\+34)?[0-9 ]{9,12}$" } let(:phone_number_placeholder) { "+34999888777" } let(:location) { true } + let(:underage) { true } + let(:underage_limit) { 18 } # Block ExtraUserFields RspecVar # EndBlock @@ -34,6 +36,8 @@ module Admin "phone_number_pattern" => phone_number_pattern, "phone_number_placeholder" => phone_number_placeholder, "location" => location, + "underage" => underage, + "underage_limit" => underage_limit, # Block ExtraUserFields ExtraUserFields # EndBlock @@ -86,6 +90,8 @@ module Admin expect(extra_user_fields).to include("country" => { "enabled" => true }) expect(extra_user_fields).to include("phone_number" => { "enabled" => true, "pattern" => phone_number_pattern, "placeholder" => phone_number_placeholder }) expect(extra_user_fields).to include("location" => { "enabled" => true }) + expect(extra_user_fields).to include("underage" => { "enabled" => true }) + expect(extra_user_fields).to include("underage_limit" => 18) # Block ExtraUserFields InclusionSpec # EndBlock diff --git a/spec/forms/decidim/account_form_spec.rb b/spec/forms/decidim/account_form_spec.rb index 1b367c0..ad76c2d 100644 --- a/spec/forms/decidim/account_form_spec.rb +++ b/spec/forms/decidim/account_form_spec.rb @@ -3,6 +3,8 @@ require "spec_helper" module Decidim + include ActiveStorage::Blob::Analyzable + describe AccountForm do subject do described_class.new( @@ -21,7 +23,9 @@ module Decidim date_of_birth:, gender:, phone_number:, - location: + location:, + underage:, + statutory_representative_email: ).with_context( current_organization: organization, current_user: user @@ -38,7 +42,9 @@ module Decidim "date_of_birth" => { "enabled" => true }, "gender" => { "enabled" => true }, "phone_number" => { "enabled" => true, "pattern" => phone_number_pattern, "placeholder" => nil }, - "location" => { "enabled" => true } + "location" => { "enabled" => true }, + "underage" => { "enabled" => true }, + "underage_limit" => 18 } end let(:phone_number_pattern) { "^(\\+34)?[0-9 ]{9,12}$" } @@ -59,6 +65,8 @@ module Decidim let(:location) { "Paris" } let(:phone_number) { "0123456789" } let(:postal_code) { "75001" } + let(:underage) { "0" } + let(:statutory_representative_email) { nil } context "with correct data" do it "is valid" do @@ -75,7 +83,7 @@ module Decidim end context "with invalid phone number format" do - let(:phone_number_pattern) { "^(\\+34)?[0-1 ]{9,12}$" } + let(:phone_number) { "ABCDEFGHIJK" } it "is invalid" do expect(subject).not_to be_valid diff --git a/spec/jobs/decidim/extra_user_fields/admin/export_participants_job_spec.rb b/spec/jobs/decidim/extra_user_fields/admin/export_participants_job_spec.rb new file mode 100644 index 0000000..360facb --- /dev/null +++ b/spec/jobs/decidim/extra_user_fields/admin/export_participants_job_spec.rb @@ -0,0 +1,68 @@ +# frozen_string_literal: true + +require "spec_helper" + +module Decidim + module ExtraUserFields + module Admin + describe ExportParticipantsJob do + let(:organization) { create(:organization, extra_user_fields: {}) } + let(:user) { create(:user, :admin, :confirmed, organization:) } + let(:format) { "CSV" } + + it "sends an email with a file attached" do + ExportParticipantsJob.perform_now(organization, user, format) + email = last_email + expect(email.subject).to include("participants") + attachment = email.attachments.first + + expect(attachment.read.length).to be_positive + expect(attachment.mime_type).to eq("application/zip") + expect(attachment.filename).to match(/^participants-[0-9]+-[0-9]+-[0-9]+-[0-9]+\.zip$/) + end + + context "when format is CSV" do + it "uses the csv exporter" do + export_data = double + expect(Decidim::Exporters::CSV).to(receive(:new).with(anything, + Decidim::ExtraUserFields::UserExportSerializer)).and_return(double(export: export_data)) + expect(ExportMailer) + .to(receive(:export).with(user, "participants", export_data)) + .and_return(double(deliver_now: true)) + ExportParticipantsJob.perform_now(organization, user, format) + end + end + + context "when format is JSON" do + let(:format) { "JSON" } + + it "uses the json exporter" do + export_data = double + expect(Decidim::Exporters::JSON) + .to(receive(:new).with(anything, Decidim::ExtraUserFields::UserExportSerializer)) + .and_return(double(export: export_data)) + expect(ExportMailer) + .to(receive(:export).with(user, "participants", export_data)) + .and_return(double(deliver_now: true)) + ExportParticipantsJob.perform_now(organization, user, format) + end + end + + context "when format is excel" do + let(:format) { "Excel" } + + it "uses the excel exporter" do + export_data = double + expect(Decidim::Exporters::Excel) + .to(receive(:new).with(anything, Decidim::ExtraUserFields::UserExportSerializer)) + .and_return(double(export: export_data)) + expect(ExportMailer) + .to(receive(:export).with(user, "participants", export_data)) + .and_return(double(deliver_now: true)) + ExportParticipantsJob.perform_now(organization, user, format) + end + end + end + end + end +end diff --git a/spec/serializers/decidim/extra_user_fields/user_export_serializer_spec.rb b/spec/serializers/decidim/extra_user_fields/user_export_serializer_spec.rb index 9eb39bd..f1aa167 100644 --- a/spec/serializers/decidim/extra_user_fields/user_export_serializer_spec.rb +++ b/spec/serializers/decidim/extra_user_fields/user_export_serializer_spec.rb @@ -15,6 +15,8 @@ country:, phone_number:, location:, + underage:, + statutory_representative_email:, # Block ExtraUserFields ExtraUserFields # EndBlock @@ -28,6 +30,9 @@ let(:country) { "Argentina" } let(:phone_number) { "0123456789" } let(:location) { "Cahors" } + let(:underage) { true } + let(:underage_limit) { 18 } + let(:statutory_representative_email) { "parent@example.org" } # Block ExtraUserFields RspecVar # EndBlock diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb index a30b234..a73f5d2 100644 --- a/spec/spec_helper.rb +++ b/spec/spec_helper.rb @@ -7,3 +7,4 @@ Decidim::Dev.dummy_app_path = File.expand_path(File.join("spec", "decidim_dummy_app")) require "decidim/dev/test/base_spec_helper" + diff --git a/spec/system/account_spec.rb b/spec/system/account_spec.rb index 3033eaa..5c66f68 100644 --- a/spec/system/account_spec.rb +++ b/spec/system/account_spec.rb @@ -10,7 +10,7 @@ end let(:organization) { create(:organization, extra_user_fields:) } - let(:user) { create(:user, :confirmed, organization:, password:, password_confirmation: password) } + let(:user) { create(:user, :confirmed, organization:, password:) } let(:password) { "dqCFgjfDbC7dPbrv" } # rubocop:disable Style/TrailingCommaInHashLiteral let(:extra_user_fields) do @@ -63,15 +63,31 @@ login_as user, scope: :user end + describe "navigation" do + it "shows the account form when clicking on the menu" do + visit decidim.root_path + + within_user_menu do + find("a", text: "account").click + end + + expect(page).to have_css("form.edit_user") + end + end + context "when on the account page" do before do visit decidim.account_path end + it_behaves_like "accessible page" + describe "updating personal data" do + let!(:encrypted_password) { user.encrypted_password } + before do within "form.edit_user" do - select "Castellano", from: :user_locale + select "English", from: :user_locale fill_in :user_name, with: "Nikola Tesla" fill_in :user_personal_url, with: "https://example.org" fill_in :user_about, with: "A Serbian-American inventor, electrical engineer, mechanical engineer, physicist, and futurist." @@ -82,10 +98,6 @@ fill_in :user_postal_code, with: "00000" fill_in :user_phone_number, with: "0123456789" fill_in :user_location, with: "Cahors" - # Block ExtraUserFields FillFieldSpec - - # EndBlock - find("*[type=submit]").click end end @@ -94,6 +106,43 @@ within_flash_messages do expect(page).to have_content("successfully") end + + user.reload + + within_user_menu do + find("a", text: "My public profile").click + end + + expect(page).to have_content("example.org") + expect(page).to have_content("Serbian-American") + + # The user's password should not change when they did not update it + expect(user.reload.encrypted_password).to eq(encrypted_password) + end + + context "when updating avatar" do + it "can update avatar" do + dynamically_attach_file(:user_avatar, Decidim::Dev.asset("avatar.jpg")) + + within "form.edit_user" do + find("*[type=submit]").click + end + + expect(page).to have_css(".flash.success") + end + + it "shows error when image is too big" do + find("#user_avatar_button").click + + within ".upload-modal" do + click_on "Remove" + input_element = find("input[type='file']", visible: :all) + input_element.attach_file(Decidim::Dev.asset("5000x5000.png")) + + expect(page).to have_content("File resolution is too large", count: 1) + expect(page).to have_content("Validation error!") + end + end end context "with phone number pattern blank" do @@ -164,5 +213,360 @@ it_behaves_like "does not display extra user field", "location", "Location" end + + describe "when update password" do + before do + within "form.edit_user" do + select "English", from: :user_locale + fill_in :user_name, with: "Nikola Tesla" + fill_in :user_personal_url, with: "https://example.org" + fill_in :user_about, with: "A Serbian-American inventor, electrical engineer, mechanical engineer, physicist, and futurist." + + fill_in :user_date_of_birth, with: "01/01/2000" + select "Other", from: :user_gender + select "Argentina", from: :user_country + fill_in :user_postal_code, with: "00000" + fill_in :user_phone_number, with: "0123456789" + fill_in :user_location, with: "Cahors" + find("*[type=submit]").click + end + click_on "Change password" + end + + let!(:encrypted_password) { user.encrypted_password } + let(:new_password) { "decidim1234567890" } + + it "toggles old and new password fields" do + within "form.edit_user" do + expect(page).to have_content("must not be too common (e.g. 123456) and must be different from your nickname and your email.") + expect(page).to have_field("user[password]", with: "", type: "password") + expect(page).to have_field("user[old_password]", with: "", type: "password") + click_on "Change password" + expect(page).to have_no_field("user[password]", with: "", type: "password") + expect(page).to have_no_field("user[old_password]", with: "", type: "password") + end + end + + it "shows fields if password is wrong" do + within "form.edit_user" do + fill_in "Password", with: new_password + fill_in "Current password", with: "wrong password12345" + find("*[type=submit]").click + end + expect(page).to have_field("user[password]", with: "decidim1234567890", type: "password") + expect(page).to have_content("is invalid") + end + + it "changes the password with correct password" do + within "form.edit_user" do + fill_in "Password", with: new_password + fill_in "Current password", with: password + find("*[type=submit]").click + end + within_flash_messages do + expect(page).to have_content("successfully") + end + expect(user.reload.encrypted_password).not_to eq(encrypted_password) + expect(page).to have_no_field("user[password]", with: "", type: "password") + expect(page).to have_no_field("user[old_password]", with: "", type: "password") + end + end + + context "when update email" do + let(:pending_email) { "foo@bar.com" } + + before do + within "form.edit_user" do + select "English", from: :user_locale + fill_in :user_name, with: "Nikola Tesla" + fill_in :user_personal_url, with: "https://example.org" + fill_in :user_about, with: "A Serbian-American inventor, electrical engineer, mechanical engineer, physicist, and futurist." + + fill_in :user_date_of_birth, with: "01/01/2000" + select "Other", from: :user_gender + select "Argentina", from: :user_country + fill_in :user_postal_code, with: "00000" + fill_in :user_phone_number, with: "0123456789" + fill_in :user_location, with: "Cahors" + find("*[type=submit]").click + end + end + + context "when typing new email" do + before do + within "form.edit_user" do + fill_in "Your email", with: pending_email + find("*[type=submit]").click + end + end + + it "toggles the current password" do + expect(page).to have_content("In order to confirm the changes to your account, please provide your current password.") + expect(find("#user_old_password")).to be_visible + expect(page).to have_content "Current password" + expect(page).to have_no_content "Password" + end + + it "renders the old password with error" do + within "form.edit_user" do + find("*[type=submit]").click + fill_in :user_old_password, with: "wrong password" + find("*[type=submit]").click + end + within ".flash.alert" do + expect(page).to have_content "There was a problem updating your account." + end + within ".old-user-password" do + expect(page).to have_content "is invalid" + end + end + end + + context "when correct old password" do + before do + within "form.edit_user" do + fill_in "Your email", with: pending_email + find("*[type=submit]").click + fill_in :user_old_password, with: password + + perform_enqueued_jobs { find("*[type=submit]").click } + end + + within_flash_messages do + expect(page).to have_content("You will receive an email to confirm your new email address") + end + # 2 emails generated (confirmation + update) + end + + after do + clear_enqueued_jobs + end + + it "tells user to confirm new email" do + expect(page).to have_content("Email change verification") + expect(page).to have_css("#user_email[disabled='disabled']") + expect(page).to have_content("We have sent an email to #{pending_email} to verify your new email address") + end + + it "resend confirmation" do + within "#email-change-pending" do + click_link_or_button "Send again" + end + expect(page).to have_content("Confirmation email resent successfully to #{pending_email}") + perform_enqueued_jobs + perform_enqueued_jobs + + # the emails include 1 confirmation + 1 update emails added to the 2 previous emails + expect(emails.count).to eq(4) + visit last_email_link + expect(page).to have_content("Your email address has been successfully confirmed") + end + + it "cancels the email change" do + expect(Decidim::User.find(user.id).unconfirmed_email).to eq(pending_email) + within "#email-change-pending" do + click_link_or_button "cancel" + end + + expect(page).to have_content("Email change cancelled successfully") + expect(page).to have_no_content("Email change verification") + expect(Decidim::User.find(user.id).unconfirmed_email).to be_nil + end + end + end + + context "when on the notifications settings page" do + before do + visit decidim.notifications_settings_path + end + + it "updates the user's notifications" do + page.find("[for='newsletter_notifications']").click + + within "form.edit_user" do + find("*[type=submit]").click + end + + within_flash_messages do + expect(page).to have_content("successfully") + end + end + + context "when the user is an admin" do + let!(:user) { create(:user, :confirmed, :admin, password:) } + + before do + login_as user, scope: :user + visit decidim.notifications_settings_path + end + + it "updates the administrator's notifications" do + page.find("[for='email_on_moderations']").click + page.find("[for='user_notification_settings[close_meeting_reminder]']").click + + within "form.edit_user" do + find("*[type=submit]").click + end + + within_flash_messages do + expect(page).to have_content("successfully") + end + end + end + end + + context "when on the interests page" do + before do + visit decidim.user_interests_path + end + + it "does not find any scopes" do + expect(page).to have_content("My interests") + expect(page).to have_content("This organization does not have any scope yet") + end + + context "when scopes are defined" do + let!(:scopes) { create_list(:scope, 3, organization:) } + let!(:subscopes) { create_list(:subscope, 3, parent: scopes.first) } + + before do + visit decidim.user_interests_path + end + + it "display translated scope name" do + expect(page).to have_content("My interests") + within "label[for='user_scopes_#{scopes.first.id}_checked']" do + expect(page).to have_content(translated(scopes.first.name)) + end + end + + it "allows to choose interests" do + label_field = "label[for='user_scopes_#{scopes.first.id}_checked']" + expect(page).to have_content("My interests") + find(label_field).click + click_on "Update my interests" + + within_flash_messages do + expect(page).to have_content("Your interests have been successfully updated.") + end + end + end + end + + context "when on the delete my account page" do + before do + visit decidim.delete_account_path + end + + it "does not display the authorizations message by default" do + expect(page).to have_no_content("Some data bound to your authorization will be saved for security.") + end + + it "the user can delete their account" do + fill_in :delete_user_delete_account_delete_reason, with: "I just want to delete my account" + + within ".form__wrapper-block" do + click_on "Delete my account" + end + + click_on "Yes, I want to delete my account" + + within_flash_messages do + expect(page).to have_content("successfully") + end + + click_link_or_button("Log in", match: :first) + + within ".new_user" do + fill_in :session_user_email, with: user.email + fill_in :session_user_password, with: password + find("*[type=submit]").click + end + + expect(page).to have_no_content("Signed in successfully") + expect(page).to have_no_content(user.name) + end + + context "when the user has an authorization" do + let!(:authorization) { create(:authorization, :granted, user:) } + + it "displays the authorizations message" do + visit decidim.delete_account_path + + expect(page).to have_content("Some data bound to your authorization will be saved for security.") + end + end + end + end + + context "when on the notifications page in a PWA browser" do + let(:organization) { create(:organization, host: "pwa.lvh.me") } + let(:user) { create(:user, :confirmed, password:, organization:) } + let(:password) { "dqCFgjfDbC7dPbrv" } + let(:vapid_keys) do + { + enabled: true, + public_key: "BKmjw_A8tJCcZNQ72uG8QW15XHQnrGJjHjsmoUILUUFXJ1VNhOnJLc3ywR3eZKibX4HSqhB1hAzZFj__3VqzcPQ=", + private_key: "TF_MRbSSs_4BE1jVfOsILSJemND8cRMpiznWHgdsro0=" + } + end + + context "when VAPID keys are set" do + before do + Rails.application.secrets[:vapid] = vapid_keys + driven_by(:pwa_chrome) + switch_to_host(organization.host) + login_as user, scope: :user + visit decidim.notifications_settings_path + end + + context "when on the account page" do + it "enables push notifications if supported browser" do + sleep 2 + page.find("[for='allow_push_notifications']").click + + # Wait for the browser to be subscribed + sleep 5 + + within "form.edit_user" do + find("*[type=submit]").click + end + + within_flash_messages do + expect(page).to have_content("successfully") + end + + find(:css, "#allow_push_notifications", visible: false).execute_script("this.checked = true") + end + end + end + + context "when VAPID is disabled" do + before do + Rails.application.secrets[:vapid] = { enabled: false } + driven_by(:pwa_chrome) + switch_to_host(organization.host) + login_as user, scope: :user + visit decidim.notifications_settings_path + end + + it "does not show the push notifications switch" do + expect(page).to have_no_selector(".push-notifications") + end + end + + context "when VAPID keys are not set" do + before do + Rails.application.secrets.delete(:vapid) + driven_by(:pwa_chrome) + switch_to_host(organization.host) + login_as user, scope: :user + visit decidim.notifications_settings_path + end + + it "does not show the push notifications switch" do + expect(page).to have_no_selector(".push-notifications") + end + end end end diff --git a/spec/system/admin_manages_officializations_spec.rb b/spec/system/admin_manages_officializations_spec.rb index 3c9f367..93a1b69 100644 --- a/spec/system/admin_manages_officializations_spec.rb +++ b/spec/system/admin_manages_officializations_spec.rb @@ -19,13 +19,25 @@ within ".layout-nav" do click_on "Participants" end - end - - it "includes export dropdown button" do within ".sidebar-menu" do click_on "Participants" end + end + it "includes export dropdown button" do expect(page).to have_content("Export") end + + context "when clicking on export csv button" do + before do + find("span.exports").click + click_on "Export CSV" + end + + it "redirects to officialization index page and display a flash message" do + expect(page).to have_title("Participants") + expect(page).to have_content("Export") + expect(page).to have_content("Your export is currently in progress. You will receive an email when it is complete") + end + end end