From f6bd1c49eefc77ec26a5ddd3369767a36546d43c Mon Sep 17 00:00:00 2001 From: David Cook Date: Fri, 24 Jan 2025 14:52:54 +1100 Subject: [PATCH] Apply suggestions from code review --- spec/models/spree/ability_spec.rb | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/spec/models/spree/ability_spec.rb b/spec/models/spree/ability_spec.rb index 24b1adae26..1339a39328 100644 --- a/spec/models/spree/ability_spec.rb +++ b/spec/models/spree/ability_spec.rb @@ -34,6 +34,8 @@ RSpec.describe Spree::Ability do let(:fakedispatch_ability) { Spree::Ability.new(fakedispatch_user) } context 'with admin user' do + let(:user) { create(:admin_user) } + it 'should be able to admin' do user.update!(admin: true) expect(subject).to be_able_to :admin, resource @@ -785,7 +787,7 @@ RSpec.describe Spree::Ability do let(:manage_actions) { [:admin, :index, :read, :update, :bulk_update, :bulk_reset] } describe "when admin" do - before { user.update!(admin: true) } + let(:user) { create(:admin_user) } it "should have permission" do is_expected.to have_ability(manage_actions, for: variant_override)