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
|
# frozen_string_literal: true
require "spec_helper"
RSpec.describe Gitlab::Metrics::Subscribers::Ldap, :request_store, feature_category: :logging do
let(:transaction) { Gitlab::Metrics::WebTransaction.new({}) }
let(:subscriber) { described_class.new }
let(:attributes) do
[
:altServer, :namingContexts, :supportedCapabilities, :supportedControl,
:supportedExtension, :supportedFeatures, :supportedLdapVersion, :supportedSASLMechanisms
]
end
let(:event_1) do
instance_double(
ActiveSupport::Notifications::Event,
name: "open.net_ldap",
payload: {
ignore_server_caps: true,
base: "",
scope: 0,
attributes: attributes,
result: nil
},
time: Time.current,
duration: 0.321
)
end
let(:event_2) do
instance_double(
ActiveSupport::Notifications::Event,
name: "search.net_ldap",
payload: {
ignore_server_caps: true,
base: "",
scope: 0,
attributes: attributes,
result: nil
},
time: Time.current,
duration: 0.12
)
end
let(:event_3) do
instance_double(
ActiveSupport::Notifications::Event,
name: "search.net_ldap",
payload: {
ignore_server_caps: true,
base: "",
scope: 0,
attributes: attributes,
result: nil
},
time: Time.current,
duration: 5.3
)
end
around do |example|
freeze_time { example.run }
end
describe ".payload" do
context "when SafeRequestStore is empty" do
it "returns an empty array" do
expect(described_class.payload).to eql(net_ldap_count: 0, net_ldap_duration_s: 0.0)
end
end
context "when LDAP recorded some values" do
before do
Gitlab::SafeRequestStore[:net_ldap_count] = 7
Gitlab::SafeRequestStore[:net_ldap_duration_s] = 1.2
end
it "returns the populated payload" do
expect(described_class.payload).to eql(net_ldap_count: 7, net_ldap_duration_s: 1.2)
end
end
end
describe "#observe_event" do
before do
allow(subscriber).to receive(:current_transaction).and_return(transaction)
end
it "tracks LDAP request count" do
expect(transaction).to receive(:increment)
.with(:gitlab_net_ldap_total, 1, { name: "open" })
expect(transaction).to receive(:increment)
.with(:gitlab_net_ldap_total, 1, { name: "search" })
subscriber.observe_event(event_1)
subscriber.observe_event(event_2)
end
it "tracks LDAP request duration" do
expect(transaction).to receive(:observe)
.with(:gitlab_net_ldap_duration_seconds, 0.321, { name: "open" })
expect(transaction).to receive(:observe)
.with(:gitlab_net_ldap_duration_seconds, 0.12, { name: "search" })
expect(transaction).to receive(:observe)
.with(:gitlab_net_ldap_duration_seconds, 5.3, { name: "search" })
subscriber.observe_event(event_1)
subscriber.observe_event(event_2)
subscriber.observe_event(event_3)
end
it "stores per-request counters" do
subscriber.observe_event(event_1)
subscriber.observe_event(event_2)
subscriber.observe_event(event_3)
expect(Gitlab::SafeRequestStore[:net_ldap_count]).to eq(3)
expect(Gitlab::SafeRequestStore[:net_ldap_duration_s]).to eq(5.741) # 0.321 + 0.12 + 5.3
end
end
end
|
