Commit 21eb12d0 authored by Sean Arnold's avatar Sean Arnold

Update spec to test data correctly

parent a0071857
...@@ -261,7 +261,7 @@ RSpec.describe 'getting an issue list for a project' do ...@@ -261,7 +261,7 @@ RSpec.describe 'getting an issue list for a project' do
<<~QUERY <<~QUERY
edges { edges {
node { node {
id iid
alertManagementAlert { alertManagementAlert {
title title
} }
...@@ -270,9 +270,9 @@ RSpec.describe 'getting an issue list for a project' do ...@@ -270,9 +270,9 @@ RSpec.describe 'getting an issue list for a project' do
QUERY QUERY
end end
# Alerts need to reporter and above # Alerts need to have developer permission and above
before do before do
project.add_reporter(current_user) project.add_developer(current_user)
end end
it 'avoids N+1 queries' do it 'avoids N+1 queries' do
...@@ -286,7 +286,10 @@ RSpec.describe 'getting an issue list for a project' do ...@@ -286,7 +286,10 @@ RSpec.describe 'getting an issue list for a project' do
it 'returns the alert data' do it 'returns the alert data' do
post_graphql(query, current_user: current_user) post_graphql(query, current_user: current_user)
issues_data alert_titles = issues_data.map { |issue| issue.dig('node', 'alertManagementAlert', 'title') }
expected_titles = issues.map { |issue| issue.alert_management_alert&.title }
expect(alert_titles).to contain_exactly(*expected_titles)
end end
end end
......
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