Remove double subject call in `api/v1/admin/ip_blocks` spec (#28696)
This commit is contained in:
parent
b68f185573
commit
95bd46d32a
|
@ -177,7 +177,9 @@ RSpec.describe 'IP Blocks' do
|
||||||
let(:params) { { severity: 'sign_up_requires_approval', comment: 'Decreasing severity' } }
|
let(:params) { { severity: 'sign_up_requires_approval', comment: 'Decreasing severity' } }
|
||||||
|
|
||||||
it 'returns the correct ip block', :aggregate_failures do
|
it 'returns the correct ip block', :aggregate_failures do
|
||||||
subject
|
expect { subject }
|
||||||
|
.to change_severity_level
|
||||||
|
.and change_comment_value
|
||||||
|
|
||||||
expect(response).to have_http_status(200)
|
expect(response).to have_http_status(200)
|
||||||
expect(body_as_json).to match(hash_including({
|
expect(body_as_json).to match(hash_including({
|
||||||
|
@ -187,12 +189,12 @@ RSpec.describe 'IP Blocks' do
|
||||||
}))
|
}))
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'updates the severity correctly' do
|
def change_severity_level
|
||||||
expect { subject }.to change { ip_block.reload.severity }.from('no_access').to('sign_up_requires_approval')
|
change { ip_block.reload.severity }.from('no_access').to('sign_up_requires_approval')
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'updates the comment correctly' do
|
def change_comment_value
|
||||||
expect { subject }.to change { ip_block.reload.comment }.from('Spam').to('Decreasing severity')
|
change { ip_block.reload.comment }.from('Spam').to('Decreasing severity')
|
||||||
end
|
end
|
||||||
|
|
||||||
context 'when ip block does not exist' do
|
context 'when ip block does not exist' do
|
||||||
|
|
Loading…
Reference in New Issue