2019-07-07 11:18:12 +05:30
|
|
|
# frozen_string_literal: true
|
|
|
|
require 'spec_helper'
|
|
|
|
|
2020-07-28 23:09:34 +05:30
|
|
|
RSpec.describe 'GraphQL' do
|
2019-07-07 11:18:12 +05:30
|
|
|
include GraphqlHelpers
|
2021-04-29 21:17:54 +05:30
|
|
|
include AfterNextHelpers
|
2019-07-07 11:18:12 +05:30
|
|
|
|
2021-04-29 21:17:54 +05:30
|
|
|
let(:query) { graphql_query_for('echo', text: 'Hello world') }
|
2019-07-07 11:18:12 +05:30
|
|
|
|
2021-04-29 21:17:54 +05:30
|
|
|
describe 'logging' do
|
2019-09-04 21:01:54 +05:30
|
|
|
shared_examples 'logging a graphql query' do
|
|
|
|
let(:expected_params) do
|
2021-01-03 14:25:43 +05:30
|
|
|
{
|
|
|
|
query_string: query,
|
|
|
|
variables: variables.to_s,
|
|
|
|
duration_s: anything,
|
2021-04-29 21:17:54 +05:30
|
|
|
operation_name: nil,
|
2021-01-03 14:25:43 +05:30
|
|
|
depth: 1,
|
|
|
|
complexity: 1,
|
|
|
|
used_fields: ['Query.echo'],
|
|
|
|
used_deprecated_fields: []
|
|
|
|
}
|
2019-09-04 21:01:54 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
it 'logs a query with the expected params' do
|
|
|
|
expect(Gitlab::GraphqlLogger).to receive(:info).with(expected_params).once
|
|
|
|
|
|
|
|
post_graphql(query, variables: variables)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not instantiate any query analyzers' do # they are static and re-used
|
|
|
|
expect(GraphQL::Analysis::QueryComplexity).not_to receive(:new)
|
|
|
|
expect(GraphQL::Analysis::QueryDepth).not_to receive(:new)
|
|
|
|
|
|
|
|
2.times { post_graphql(query, variables: variables) }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with no variables' do
|
|
|
|
let(:variables) { {} }
|
|
|
|
|
|
|
|
it_behaves_like 'logging a graphql query'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with variables' do
|
|
|
|
let(:variables) do
|
|
|
|
{ "foo" => "bar" }
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'logging a graphql query'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when there is an error in the logger' do
|
|
|
|
before do
|
2021-04-29 21:17:54 +05:30
|
|
|
logger_analyzer = GitlabSchema.query_analyzers.find do |qa|
|
|
|
|
qa.is_a? Gitlab::Graphql::QueryAnalyzers::LoggerAnalyzer
|
|
|
|
end
|
|
|
|
allow(logger_analyzer).to receive(:process_variables)
|
|
|
|
.and_raise(StandardError.new("oh noes!"))
|
2019-09-04 21:01:54 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
it 'logs the exception in Sentry and continues with the request' do
|
2021-04-29 21:17:54 +05:30
|
|
|
expect(Gitlab::ErrorTracking)
|
|
|
|
.to receive(:track_and_raise_for_dev_exception).at_least(:once)
|
|
|
|
expect(Gitlab::GraphqlLogger)
|
|
|
|
.to receive(:info)
|
2019-09-04 21:01:54 +05:30
|
|
|
|
|
|
|
post_graphql(query, variables: {})
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2021-04-29 21:17:54 +05:30
|
|
|
context 'with invalid variables' do
|
2019-07-07 11:18:12 +05:30
|
|
|
it 'returns an error' do
|
|
|
|
post_graphql(query, variables: "This is not JSON")
|
|
|
|
|
2020-04-08 14:13:33 +05:30
|
|
|
expect(response).to have_gitlab_http_status(:unprocessable_entity)
|
2019-07-07 11:18:12 +05:30
|
|
|
expect(json_response['errors'].first['message']).not_to be_nil
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2021-04-29 21:17:54 +05:30
|
|
|
describe 'authentication', :allow_forgery_protection do
|
2019-07-07 11:18:12 +05:30
|
|
|
let(:user) { create(:user) }
|
|
|
|
|
|
|
|
it 'allows access to public data without authentication' do
|
|
|
|
post_graphql(query)
|
|
|
|
|
|
|
|
expect(graphql_data['echo']).to eq('nil says: Hello world')
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not authenticate a user with an invalid CSRF' do
|
|
|
|
login_as(user)
|
|
|
|
|
|
|
|
post_graphql(query, headers: { 'X-CSRF-Token' => 'invalid' })
|
|
|
|
|
|
|
|
expect(graphql_data['echo']).to eq('nil says: Hello world')
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'authenticates a user with a valid session token' do
|
|
|
|
# Create a session to get a CSRF token from
|
|
|
|
login_as(user)
|
|
|
|
get('/')
|
|
|
|
|
2021-06-08 01:23:25 +05:30
|
|
|
post '/api/graphql', params: { query: query }, headers: { 'X-CSRF-Token' => session['_csrf_token'] }
|
2019-07-07 11:18:12 +05:30
|
|
|
|
|
|
|
expect(graphql_data['echo']).to eq("\"#{user.username}\" says: Hello world")
|
|
|
|
end
|
|
|
|
|
2021-04-29 21:17:54 +05:30
|
|
|
context 'with token authentication' do
|
2019-07-07 11:18:12 +05:30
|
|
|
let(:token) { create(:personal_access_token) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
stub_authentication_activity_metrics(debug: false)
|
|
|
|
end
|
|
|
|
|
2021-02-22 17:27:13 +05:30
|
|
|
it 'authenticates users with a PAT' do
|
2019-07-07 11:18:12 +05:30
|
|
|
expect(authentication_metrics)
|
|
|
|
.to increment(:user_authenticated_counter)
|
|
|
|
.and increment(:user_session_override_counter)
|
|
|
|
.and increment(:user_sessionless_authentication_counter)
|
|
|
|
|
|
|
|
post_graphql(query, headers: { 'PRIVATE-TOKEN' => token.token })
|
|
|
|
|
|
|
|
expect(graphql_data['echo']).to eq("\"#{token.user.username}\" says: Hello world")
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the personal access token has no api scope' do
|
|
|
|
it 'does not log the user in' do
|
2021-04-29 21:17:54 +05:30
|
|
|
token.update!(scopes: [:read_user])
|
2019-07-07 11:18:12 +05:30
|
|
|
|
|
|
|
post_graphql(query, headers: { 'PRIVATE-TOKEN' => token.token })
|
|
|
|
|
2020-04-08 14:13:33 +05:30
|
|
|
expect(response).to have_gitlab_http_status(:ok)
|
2019-07-07 11:18:12 +05:30
|
|
|
|
|
|
|
expect(graphql_data['echo']).to eq('nil says: Hello world')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2019-09-30 21:07:59 +05:30
|
|
|
|
|
|
|
describe 'testing for Gitaly calls' do
|
|
|
|
let(:project) { create(:project, :repository) }
|
|
|
|
let(:user) { create(:user) }
|
|
|
|
|
|
|
|
let(:query) do
|
2021-04-29 21:17:54 +05:30
|
|
|
graphql_query_for(
|
|
|
|
:project,
|
|
|
|
{ full_path: project.full_path },
|
|
|
|
'id'
|
|
|
|
)
|
2019-09-30 21:07:59 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
before do
|
|
|
|
project.add_developer(user)
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'a working graphql query' do
|
|
|
|
before do
|
|
|
|
post_graphql(query, current_user: user)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when Gitaly is called' do
|
|
|
|
before do
|
|
|
|
allow(Gitlab::GitalyClient).to receive(:get_request_count).and_return(1, 2)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "logs a warning that the 'calls_gitaly' field declaration is missing" do
|
2020-01-01 13:55:28 +05:30
|
|
|
expect(Gitlab::ErrorTracking).to receive(:track_and_raise_for_dev_exception).once
|
2019-09-30 21:07:59 +05:30
|
|
|
|
|
|
|
post_graphql(query, current_user: user)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2020-04-08 14:13:33 +05:30
|
|
|
|
|
|
|
describe 'resolver complexity' do
|
|
|
|
let_it_be(:project) { create(:project, :public) }
|
|
|
|
let(:query) do
|
|
|
|
graphql_query_for(
|
|
|
|
'project',
|
|
|
|
{ 'fullPath' => project.full_path },
|
|
|
|
query_graphql_field(resource, {}, 'edges { node { iid } }')
|
|
|
|
)
|
|
|
|
end
|
|
|
|
|
|
|
|
before do
|
|
|
|
stub_const('GitlabSchema::DEFAULT_MAX_COMPLEXITY', 6)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when fetching single resource' do
|
|
|
|
let(:resource) { 'issues(first: 1)' }
|
|
|
|
|
|
|
|
it 'processes the query' do
|
|
|
|
post_graphql(query)
|
|
|
|
|
|
|
|
expect(graphql_errors).to be_nil
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when fetching too many resources' do
|
|
|
|
let(:resource) { 'issues(first: 100)' }
|
|
|
|
|
|
|
|
it 'returns an error' do
|
|
|
|
post_graphql(query)
|
|
|
|
|
|
|
|
expect_graphql_errors_to_include(/which exceeds max complexity/)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2020-06-23 00:09:42 +05:30
|
|
|
|
2021-04-29 21:17:54 +05:30
|
|
|
describe 'complexity limits' do
|
|
|
|
let_it_be(:project) { create(:project, :public) }
|
|
|
|
let!(:user) { create(:user) }
|
|
|
|
|
|
|
|
let(:query_fields) do
|
|
|
|
<<~QUERY
|
|
|
|
id
|
|
|
|
QUERY
|
|
|
|
end
|
|
|
|
|
|
|
|
let(:query) do
|
|
|
|
graphql_query_for(
|
|
|
|
'project',
|
|
|
|
{ 'fullPath' => project.full_path },
|
|
|
|
query_fields
|
|
|
|
)
|
|
|
|
end
|
|
|
|
|
|
|
|
before do
|
|
|
|
stub_const('GitlabSchema::DEFAULT_MAX_COMPLEXITY', 1)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'unauthenticated user' do
|
|
|
|
subject { post_graphql(query) }
|
|
|
|
|
|
|
|
it 'raises a complexity error' do
|
|
|
|
subject
|
|
|
|
|
|
|
|
expect_graphql_errors_to_include(/which exceeds max complexity/)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'authenticated user' do
|
|
|
|
subject { post_graphql(query, current_user: user) }
|
|
|
|
|
|
|
|
it 'does not raise an error as it uses the `AUTHENTICATED_COMPLEXITY`' do
|
|
|
|
subject
|
|
|
|
|
|
|
|
expect(graphql_errors).to be_nil
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2020-06-23 00:09:42 +05:30
|
|
|
describe 'keyset pagination' do
|
|
|
|
let_it_be(:project) { create(:project, :public) }
|
|
|
|
let_it_be(:issues) { create_list(:issue, 10, project: project, created_at: Time.now.change(usec: 200)) }
|
|
|
|
|
|
|
|
let(:page_size) { 6 }
|
2021-04-29 21:17:54 +05:30
|
|
|
let(:issues_edges) { %w[project issues edges] }
|
|
|
|
let(:end_cursor) { %w[project issues pageInfo endCursor] }
|
2020-06-23 00:09:42 +05:30
|
|
|
let(:query) do
|
|
|
|
<<~GRAPHQL
|
|
|
|
query project($fullPath: ID!, $first: Int, $after: String) {
|
|
|
|
project(fullPath: $fullPath) {
|
|
|
|
issues(first: $first, after: $after) {
|
|
|
|
edges { node { iid } }
|
|
|
|
pageInfo { endCursor }
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
GRAPHQL
|
|
|
|
end
|
|
|
|
|
|
|
|
def execute_query(after: nil)
|
2021-04-29 21:17:54 +05:30
|
|
|
post_graphql(
|
2020-06-23 00:09:42 +05:30
|
|
|
query,
|
2021-04-29 21:17:54 +05:30
|
|
|
current_user: nil,
|
2020-06-23 00:09:42 +05:30
|
|
|
variables: {
|
|
|
|
fullPath: project.full_path,
|
|
|
|
first: page_size,
|
|
|
|
after: after
|
|
|
|
}
|
|
|
|
)
|
|
|
|
end
|
|
|
|
|
2021-06-08 01:23:25 +05:30
|
|
|
context 'when new_graphql_keyset_pagination feature flag is off' do
|
|
|
|
before do
|
|
|
|
stub_feature_flags(new_graphql_keyset_pagination: false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'paginates datetimes correctly when they have millisecond data' do
|
|
|
|
# let's make sure we're actually querying a timestamp, just in case
|
|
|
|
expect(Gitlab::Graphql::Pagination::Keyset::QueryBuilder)
|
|
|
|
.to receive(:new).with(anything, anything, hash_including('created_at'), anything).and_call_original
|
|
|
|
|
|
|
|
execute_query
|
|
|
|
first_page = graphql_data
|
|
|
|
edges = first_page.dig(*issues_edges)
|
|
|
|
cursor = first_page.dig(*end_cursor)
|
|
|
|
|
|
|
|
expect(edges.count).to eq(6)
|
|
|
|
expect(edges.last['node']['iid']).to eq(issues[4].iid.to_s)
|
2020-06-23 00:09:42 +05:30
|
|
|
|
2021-06-08 01:23:25 +05:30
|
|
|
execute_query(after: cursor)
|
|
|
|
second_page = graphql_data
|
|
|
|
edges = second_page.dig(*issues_edges)
|
2020-06-23 00:09:42 +05:30
|
|
|
|
2021-06-08 01:23:25 +05:30
|
|
|
expect(edges.count).to eq(4)
|
|
|
|
expect(edges.last['node']['iid']).to eq(issues[0].iid.to_s)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when new_graphql_keyset_pagination feature flag is on' do
|
|
|
|
it 'paginates datetimes correctly when they have millisecond data' do
|
|
|
|
execute_query
|
|
|
|
first_page = graphql_data
|
|
|
|
edges = first_page.dig(*issues_edges)
|
|
|
|
cursor = first_page.dig(*end_cursor)
|
2020-06-23 00:09:42 +05:30
|
|
|
|
2021-06-08 01:23:25 +05:30
|
|
|
expect(edges.count).to eq(6)
|
|
|
|
expect(edges.last['node']['iid']).to eq(issues[4].iid.to_s)
|
2020-06-23 00:09:42 +05:30
|
|
|
|
2021-06-08 01:23:25 +05:30
|
|
|
execute_query(after: cursor)
|
|
|
|
second_page = graphql_data
|
|
|
|
edges = second_page.dig(*issues_edges)
|
|
|
|
|
|
|
|
expect(edges.count).to eq(4)
|
|
|
|
expect(edges.last['node']['iid']).to eq(issues[0].iid.to_s)
|
|
|
|
end
|
2020-06-23 00:09:42 +05:30
|
|
|
end
|
|
|
|
end
|
2019-07-07 11:18:12 +05:30
|
|
|
end
|