Commit 517dd200 authored by Alex Kalderimis's avatar Alex Kalderimis

Authorization feature spec minor refactor

parent 428a9621
...@@ -13,10 +13,10 @@ RSpec.describe 'Gitlab::Graphql::Authorize' do ...@@ -13,10 +13,10 @@ RSpec.describe 'Gitlab::Graphql::Authorize' do
let(:result) do let(:result) do
schema = empty_schema schema = empty_schema
schema.use(Gitlab::Graphql::Authorize) schema.use(Gitlab::Graphql::Authorize)
execute_query(query_type, schema)['data'] execute_query(query_type, schema)
end end
subject { result['item'] } subject { result.dig('data', 'item') }
shared_examples 'authorization with a single permission' do shared_examples 'authorization with a single permission' do
it 'returns the protected field when user has permission' do it 'returns the protected field when user has permission' do
...@@ -196,7 +196,7 @@ RSpec.describe 'Gitlab::Graphql::Authorize' do ...@@ -196,7 +196,7 @@ RSpec.describe 'Gitlab::Graphql::Authorize' do
end end
end end
subject { result.dig('item', 'edges') } subject { result.dig('data', 'item', 'edges') }
it 'returns only the elements visible to the user' do it 'returns only the elements visible to the user' do
permit(permission_single) permit(permission_single)
...@@ -240,7 +240,7 @@ RSpec.describe 'Gitlab::Graphql::Authorize' do ...@@ -240,7 +240,7 @@ RSpec.describe 'Gitlab::Graphql::Authorize' do
end end
end end
subject { result['item'].first } subject { result['data']['item'].first }
include_examples 'authorization with a single permission' include_examples 'authorization with a single permission'
end end
...@@ -285,7 +285,7 @@ RSpec.describe 'Gitlab::Graphql::Authorize' do ...@@ -285,7 +285,7 @@ RSpec.describe 'Gitlab::Graphql::Authorize' do
end end
it 'renders the issues the user has access to' do it 'renders the issues the user has access to' do
issue_edges = result['testProject']['testIssues']['edges'] issue_edges = result['data']['testProject']['testIssues']['edges']
issue_ids = issue_edges.map { |issue_edge| issue_edge['node']&.fetch('id') } issue_ids = issue_edges.map { |issue_edge| issue_edge['node']&.fetch('id') }
expect(issue_edges.size).to eq(visible_issues.size) expect(issue_edges.size).to eq(visible_issues.size)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment