From bb907afe9146433c2a539a768573240480b3c32a Mon Sep 17 00:00:00 2001 From: Joonas Date: Mon, 7 Oct 2024 12:40:55 +0300 Subject: [PATCH] Fix webpack problem with new foundation version / Fix authorization specs for redirect --- Rakefile | 4 ---- .../devise/omniauth_registrations_controller_spec.rb | 9 +++++---- 2 files changed, 5 insertions(+), 8 deletions(-) diff --git a/Rakefile b/Rakefile index cdf4e42..ad90675 100644 --- a/Rakefile +++ b/Rakefile @@ -6,10 +6,6 @@ def install_module(path) Dir.chdir(path) do system("bundle exec rake decidim_privacy:install:migrations") system("bundle exec rake db:migrate") - - # Temporary fix to overcome the issue with sass-embedded, see: - # https://github.com/decidim/decidim/pull/11074 - system("npm i sass-embedded@~1.62.0") end end diff --git a/spec/controllers/decidim/devise/omniauth_registrations_controller_spec.rb b/spec/controllers/decidim/devise/omniauth_registrations_controller_spec.rb index f7ad899..e54e6db 100644 --- a/spec/controllers/decidim/devise/omniauth_registrations_controller_spec.rb +++ b/spec/controllers/decidim/devise/omniauth_registrations_controller_spec.rb @@ -5,7 +5,8 @@ describe Decidim::Devise::OmniauthRegistrationsController, type: :controller do routes { Decidim::Core::Engine.routes } - let(:organization) { create(:organization) } + let(:organization) { create(:organization, available_authorizations: available_authorizations) } + let(:available_authorizations) { ["dummy_authorization_handler"] } before do request.env["decidim.current_organization"] = organization @@ -36,7 +37,7 @@ end it "redirects to the authorizations path" do - expect(subject).to redirect_to("/authorizations") + expect(subject).to redirect_to("/authorizations/first_login") end it "creates a new user" do @@ -64,7 +65,7 @@ end it "redirects to the authorizations path" do - expect(subject).to redirect_to("/authorizations") + expect(subject).to redirect_to("/authorizations/first_login") end it "creates a new user" do @@ -90,7 +91,7 @@ end it "redirects to the authorizations path" do - expect(subject).to redirect_to("/authorizations") + expect(subject).to redirect_to("/authorizations/first_login") end it "creates a new user" do