2019-12-03 13:06:49 -05:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
|
|
|
require 'spec_helper'
|
|
|
|
|
|
|
|
describe 'Loading a user avatar' do
|
|
|
|
let(:user) { create(:user, :with_avatar) }
|
|
|
|
|
|
|
|
context 'when logged in' do
|
|
|
|
# The exact query count will vary depending on the 2FA settings of the
|
|
|
|
# instance, group, and user. Removing those extra 2FA queries in this case
|
|
|
|
# may not be a good idea, so we just set up the ideal case.
|
|
|
|
before do
|
|
|
|
stub_application_setting(require_two_factor_authentication: true)
|
|
|
|
|
|
|
|
login_as(create(:user, :two_factor))
|
|
|
|
end
|
|
|
|
|
|
|
|
# One each for: current user, avatar user, and upload record
|
|
|
|
it 'only performs three SQL queries' do
|
|
|
|
get user.avatar_url # Skip queries on first application load
|
|
|
|
|
2020-01-27 10:08:51 -05:00
|
|
|
expect(response).to have_gitlab_http_status(:ok)
|
2019-12-03 13:06:49 -05:00
|
|
|
expect { get user.avatar_url }.not_to exceed_query_limit(3)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when logged out' do
|
|
|
|
# One each for avatar user and upload record
|
|
|
|
it 'only performs two SQL queries' do
|
|
|
|
get user.avatar_url # Skip queries on first application load
|
|
|
|
|
2020-01-27 10:08:51 -05:00
|
|
|
expect(response).to have_gitlab_http_status(:ok)
|
2019-12-03 13:06:49 -05:00
|
|
|
expect { get user.avatar_url }.not_to exceed_query_limit(2)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|