Skip to content
代码片段 群组 项目
未验证 提交 baea9d9e 编辑于 作者: michold's avatar michold
浏览文件

Remove Redis matchers for internal events

Remove Redis matchers for internal events
上级 a44ba813
No related branches found
No related tags found
无相关合并请求
...@@ -49,11 +49,12 @@ ...@@ -49,11 +49,12 @@
end end
it 'creates tracking event' do it 'creates tracking event' do
expect(Gitlab::InternalEvents).to receive(:track_event) expect { response }.to trigger_internal_events('export_runner_usage_by_project_as_csv')
.with('export_runner_usage_by_project_as_csv', user: current_user, .with(
additional_properties: { property: 'instance_type', label: 'instance' }) category: 'InternalEventTracking',
user: current_user,
response additional_properties: { property: 'instance_type', label: 'instance' }
)
end end
it 'creates audit event' do it 'creates audit event' do
...@@ -130,11 +131,13 @@ ...@@ -130,11 +131,13 @@
end end
it 'creates tracking event' do it 'creates tracking event' do
expect(Gitlab::InternalEvents).to receive(:track_event) expect { response }.to trigger_internal_events('export_runner_usage_by_project_as_csv')
.with('export_runner_usage_by_project_as_csv', user: current_user, namespace: specified_scope, .with(
additional_properties: { property: 'instance_type', label: 'group' }) category: 'InternalEventTracking',
user: current_user,
response namespace: specified_scope,
additional_properties: { property: 'instance_type', label: 'group' }
)
end end
end end
...@@ -159,11 +162,13 @@ ...@@ -159,11 +162,13 @@
end end
it 'creates tracking event' do it 'creates tracking event' do
expect(Gitlab::InternalEvents).to receive(:track_event) expect { response }.to trigger_internal_events('export_runner_usage_by_project_as_csv')
.with('export_runner_usage_by_project_as_csv', user: current_user, project: specified_scope, .with(
additional_properties: { property: 'instance_type', label: 'project' }) category: 'InternalEventTracking',
user: current_user,
response project: specified_scope,
additional_properties: { property: 'instance_type', label: 'project' }
)
end end
end end
end end
......
...@@ -131,7 +131,6 @@ def apply_chain_methods(base_matcher, chained_methods) ...@@ -131,7 +131,6 @@ def apply_chain_methods(base_matcher, chained_methods)
[ [
expect_internal_event(event_name), expect_internal_event(event_name),
expect_snowplow(event_name), expect_snowplow(event_name),
expect_redis(event_name),
expect_product_analytics(event_name) expect_product_analytics(event_name)
] ]
end end
...@@ -247,18 +246,6 @@ def expect_no_snowplow_event(event_name) ...@@ -247,18 +246,6 @@ def expect_no_snowplow_event(event_name)
# rubocop:enable RSpec/ExpectGitlabTracking # rubocop:enable RSpec/ExpectGitlabTracking
end end
def expect_redis(event_name)
Gitlab::InternalEvents::EventDefinitions.unique_properties(event_name).map do |property|
expect(Gitlab::Redis::HLL)
.to receive_expected_count_of(:add)
.with(hash_including(
key: a_string_including(event_name),
value: @properties.any? ? id_for(property) : anything,
expiry: 6.weeks
))
end
end
def expect_product_analytics(event_name) def expect_product_analytics(event_name)
return expect_any_product_analytics_call(event_name) if @properties.none? return expect_any_product_analytics_call(event_name) if @properties.none?
......
0% 加载中 .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册