SWREI/instrumental
0
0
Fork 0
Code Issues Activity

Remove body_json_ids from api/v2/admin/accounts spec (#32003)

Browse Source
This commit is contained in:
Matt Jankowski 2024-09-23 04:45:05 -04:00 committed by GitHub
parent e0b45b35c9
commit 0ba3ad4a35
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 22 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
spec/requests/api/v2/admin/accounts_spec.rb
View File

@ -36,7 +36,10 @@ RSpec.describe 'API V2 Admin Accounts' do
expect(response).to have_http_status(200) expect(response).to have_http_status(200)
expect(response.content_type) expect(response.content_type)
.to start_with('application/json') .to start_with('application/json')
expect(body_json_ids).to eq([admin_account.id]) expect(response.parsed_body)
.to contain_exactly(
hash_including(id: admin_account.id.to_s)
)
end end
end end
@ -47,8 +50,11 @@ RSpec.describe 'API V2 Admin Accounts' do
expect(response).to have_http_status(200) expect(response).to have_http_status(200)
expect(response.content_type) expect(response.content_type)
.to start_with('application/json') .to start_with('application/json')
expect(body_json_ids).to include(remote_account.id) expect(response.parsed_body)
expect(body_json_ids).to_not include(other_remote_account.id) .to contain_exactly(
hash_including(id: remote_account.id.to_s)
)
.and not_include(hash_including(id: other_remote_account.id.to_s))
end end
end end
@ -59,7 +65,11 @@ RSpec.describe 'API V2 Admin Accounts' do
expect(response).to have_http_status(200) expect(response).to have_http_status(200)
expect(response.content_type) expect(response.content_type)
.to start_with('application/json') .to start_with('application/json')
expect(body_json_ids).to include(suspended_remote.id, suspended_account.id) expect(response.parsed_body)
.to contain_exactly(
hash_including(id: suspended_remote.id.to_s),
hash_including(id: suspended_account.id.to_s)
)
end end
end end
@ -70,7 +80,10 @@ RSpec.describe 'API V2 Admin Accounts' do
expect(response).to have_http_status(200) expect(response).to have_http_status(200)
expect(response.content_type) expect(response.content_type)
.to start_with('application/json') .to start_with('application/json')
expect(body_json_ids).to include(disabled_account.id) expect(response.parsed_body)
.to contain_exactly(
hash_including(id: disabled_account.id.to_s)
)
end end
end end
@ -81,14 +94,13 @@ RSpec.describe 'API V2 Admin Accounts' do
expect(response).to have_http_status(200) expect(response).to have_http_status(200)
expect(response.content_type) expect(response.content_type)
.to start_with('application/json') .to start_with('application/json')
expect(body_json_ids).to include(pending_account.id) expect(response.parsed_body)
.to contain_exactly(
hash_including(id: pending_account.id.to_s)
)
end end
end end
def body_json_ids
response.parsed_body.map { |a| a[:id].to_i }
end
context 'with limit param' do context 'with limit param' do
let(:params) { { limit: 1 } } let(:params) { { limit: 1 } }