Fix `RSpec/SubjectStub` cop (#25550)

Co-authored-by: Claire <claire.github-309c@sitedethib.com>
pull/25552/head^2
Matt Jankowski 2023-07-12 04:02:19 -04:00 committed by GitHub
parent 7824df0eca
commit 74806deb2c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 9 additions and 11 deletions

View File

@ -409,11 +409,6 @@ RSpec/StubbedMock:
- 'spec/lib/webfinger_resource_spec.rb' - 'spec/lib/webfinger_resource_spec.rb'
- 'spec/services/activitypub/process_collection_service_spec.rb' - 'spec/services/activitypub/process_collection_service_spec.rb'
RSpec/SubjectStub:
Exclude:
- 'spec/services/unallow_domain_service_spec.rb'
- 'spec/validators/blacklisted_email_validator_spec.rb'
# This cop supports unsafe autocorrection (--autocorrect-all). # This cop supports unsafe autocorrection (--autocorrect-all).
Rails/ApplicationController: Rails/ApplicationController:
Exclude: Exclude:

View File

@ -14,7 +14,7 @@ RSpec.describe UnallowDomainService, type: :service do
context 'with limited federation mode' do context 'with limited federation mode' do
before do before do
allow(subject).to receive(:whitelist_mode?).and_return(true) allow(Rails.configuration.x).to receive(:whitelist_mode).and_return(true)
end end
describe '#call' do describe '#call' do
@ -40,7 +40,7 @@ RSpec.describe UnallowDomainService, type: :service do
context 'without limited federation mode' do context 'without limited federation mode' do
before do before do
allow(subject).to receive(:whitelist_mode?).and_return(false) allow(Rails.configuration.x).to receive(:whitelist_mode).and_return(false)
end end
describe '#call' do describe '#call' do

View File

@ -11,14 +11,15 @@ RSpec.describe BlacklistedEmailValidator, type: :validator do
before do before do
allow(user).to receive(:valid_invitation?).and_return(false) allow(user).to receive(:valid_invitation?).and_return(false)
allow_any_instance_of(described_class).to receive(:blocked_email_provider?) { blocked_email } allow(EmailDomainBlock).to receive(:block?) { blocked_email }
end end
context 'when e-mail provider is blocked' do context 'when e-mail provider is blocked' do
let(:blocked_email) { true } let(:blocked_email) { true }
it 'adds error' do it 'adds error' do
expect(subject).to have_received(:add).with(:email, :blocked) described_class.new.validate(user)
expect(errors).to have_received(:add).with(:email, :blocked).once
end end
end end
@ -26,7 +27,8 @@ RSpec.describe BlacklistedEmailValidator, type: :validator do
let(:blocked_email) { false } let(:blocked_email) { false }
it 'does not add errors' do it 'does not add errors' do
expect(subject).to_not have_received(:add).with(:email, :blocked) described_class.new.validate(user)
expect(errors).to_not have_received(:add)
end end
context 'when canonical e-mail is blocked' do context 'when canonical e-mail is blocked' do
@ -37,7 +39,8 @@ RSpec.describe BlacklistedEmailValidator, type: :validator do
end end
it 'adds error' do it 'adds error' do
expect(subject).to have_received(:add).with(:email, :taken) described_class.new.validate(user)
expect(errors).to have_received(:add).with(:email, :taken).once
end end
end end
end end