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
|
# frozen_string_literal: true
require 'spec_helper'
RSpec.describe Admin::PlanLimits::UpdateService, feature_category: :shared do
let_it_be(:user) { create(:admin) }
let_it_be(:plan) { create(:plan, name: 'free') }
let_it_be(:limits) { plan.actual_limits }
let_it_be(:params) do
{
ci_pipeline_size: 101,
ci_active_jobs: 102,
ci_project_subscriptions: 104,
ci_pipeline_schedules: 105,
ci_needs_size_limit: 106,
ci_registered_group_runners: 107,
ci_registered_project_runners: 108,
conan_max_file_size: 10,
enforcement_limit: 15,
generic_packages_max_file_size: 20,
helm_max_file_size: 25,
notification_limit: 30,
maven_max_file_size: 40,
npm_max_file_size: 60,
nuget_max_file_size: 60,
pypi_max_file_size: 70,
terraform_module_max_file_size: 80,
storage_size_limit: 90,
pipeline_hierarchy_size: 250
}
end
subject(:update_plan_limits) { described_class.new(params, current_user: user, plan: plan).execute }
context 'when current_user is an admin', :enable_admin_mode do
context 'when the update is successful', :freeze_time do
let(:current_timestamp) { Time.current.utc.to_i }
it 'updates all attributes' do
update_plan_limits
params.each do |key, value|
expect(limits.send(key)).to eq value
end
end
it 'logs the allowed attributes only' do
update_plan_limits
expect(limits.limits_history).to eq(
{ "enforcement_limit" =>
[{ "user_id" => user.id, "username" => user.username,
"timestamp" => current_timestamp, "value" => 15 }],
"notification_limit" =>
[{ "user_id" => user.id, "username" => user.username,
"timestamp" => current_timestamp, "value" => 30 }],
"storage_size_limit" =>
[{ "user_id" => user.id, "username" => user.username,
"timestamp" => current_timestamp, "value" => 90 }] }
)
end
it 'returns success' do
response = update_plan_limits
expect(response[:status]).to eq :success
end
end
context 'when the update is unsuccessful' do
let(:params) { { notification_limit: 'abc' } }
it 'returns an error' do
response = update_plan_limits
expect(response[:status]).to eq :error
expect(response[:message]).to include 'Notification limit is not a number'
end
end
end
context 'when the user is not an admin' do
let(:user) { create(:user) }
it 'returns an error' do
response = update_plan_limits
expect(response[:status]).to eq :error
expect(response[:message]).to eq 'Access denied'
end
end
end
|