Skip to content

Commit

Permalink
Fix webpack problem with new foundation version / Fix authorization s…
Browse files Browse the repository at this point in the history
…pecs for redirect
  • Loading branch information
JoonasAapro committed Oct 7, 2024
1 parent 6a5da0e commit bb907af
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 8 deletions.
4 changes: 0 additions & 4 deletions Rakefile
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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
Expand Down Expand Up @@ -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
Expand All @@ -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
Expand Down

0 comments on commit bb907af

Please sign in to comment.