1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
|
# frozen_string_literal: true
require 'spec_helper'
RSpec.describe 'Query.project.pipelineSchedules', feature_category: :continuous_integration do
include GraphqlHelpers
let_it_be(:user) { create(:user) }
let_it_be(:project) { create(:project, :repository, :public, creator: user, namespace: user.namespace) }
let_it_be(:pipeline_schedule) { create(:ci_pipeline_schedule, project: project, owner: user) }
let(:pipeline_schedule_graphql_data) { graphql_data_at(:project, :pipeline_schedules, :nodes, 0) }
let(:params) { {} }
let(:fields) do
<<~QUERY
nodes {
id
description
active
nextRunAt
realNextRun
lastPipeline {
id
}
refForDisplay
refPath
forTag
cron
cronTimezone
editPath
variables { nodes { #{all_graphql_fields_for('PipelineScheduleVariable')} } }
}
QUERY
end
let(:query) do
%(
query {
project(fullPath: "#{project.full_path}") {
pipelineSchedules {
#{fields}
}
}
}
)
end
describe 'computed graphql fields' do
before do
pipeline_schedule.pipelines << build(:ci_pipeline, project: project)
post_graphql(query, current_user: user)
end
it_behaves_like 'a working graphql query'
it 'returns calculated fields for a pipeline schedule' do
ref_for_display = pipeline_schedule_graphql_data['refForDisplay']
expect(ref_for_display).to eq('master')
expect(pipeline_schedule_graphql_data['refPath']).to eq("/#{project.full_path}/-/commits/#{ref_for_display}")
expect(pipeline_schedule_graphql_data['forTag']).to be(false)
end
it 'returns the edit_path for a pipeline schedule' do
edit_path = pipeline_schedule_graphql_data['editPath']
expect(edit_path).to eq("/#{project.full_path}/-/pipeline_schedules/#{pipeline_schedule.id}/edit")
end
end
describe 'variables' do
let!(:env_vars) { create_list(:ci_pipeline_schedule_variable, 5, pipeline_schedule: pipeline_schedule) }
it 'returns all variables' do
post_graphql(query, current_user: user)
variables = pipeline_schedule_graphql_data['variables']['nodes']
expected = env_vars.map do |var|
a_graphql_entity_for(var, :key, :value, variable_type: var.variable_type.upcase)
end
expect(variables).to match_array(expected)
end
it 'is N+1 safe on the variables level' do
baseline = ActiveRecord::QueryRecorder.new { post_graphql(query, current_user: user) }
create_list(:ci_pipeline_schedule_variable, 2, pipeline_schedule: pipeline_schedule)
expect { post_graphql(query, current_user: user) }.not_to exceed_query_limit(baseline)
end
it 'is N+1 safe on the schedules level' do
baseline = ActiveRecord::QueryRecorder.new { post_graphql(query, current_user: user) }
pipeline_schedule_2 = create(:ci_pipeline_schedule, project: project, owner: user)
create_list(:ci_pipeline_schedule_variable, 2, pipeline_schedule: pipeline_schedule_2)
expect { post_graphql(query, current_user: user) }.not_to exceed_query_limit(baseline)
end
end
describe 'permissions' do
let_it_be(:another_user) { create(:user) }
before do
post_graphql(query, current_user: another_user)
end
it 'does not return the edit_path for a pipeline schedule for a user that does not have permissions' do
edit_path = pipeline_schedule_graphql_data['editPath']
expect(edit_path).to be nil
end
end
it 'avoids N+1 queries' do
create_pipeline_schedules(1)
control = ActiveRecord::QueryRecorder.new { post_graphql(query, current_user: user) }
create_pipeline_schedules(3)
action = ActiveRecord::QueryRecorder.new { post_graphql(query, current_user: user) }
expect(action).not_to exceed_query_limit(control)
end
def create_pipeline_schedules(count)
create_list(:ci_pipeline_schedule, count, project: project)
.each do |pipeline_schedule|
create(:user).tap do |user|
project.add_developer(user)
pipeline_schedule.update!(owner: user)
end
pipeline_schedule.pipelines << build(:ci_pipeline, project: project)
end
end
end
|
