Commit f016601b authored by Jeremy Jackson's avatar Jeremy Jackson Committed by Lin Jen-Shin

Switch tracking to use the AsyncEmitter

parent e12a355c
...@@ -33,7 +33,7 @@ module Gitlab ...@@ -33,7 +33,7 @@ module Gitlab
def snowplow def snowplow
@snowplow ||= SnowplowTracker::Tracker.new( @snowplow ||= SnowplowTracker::Tracker.new(
SnowplowTracker::Emitter.new(Gitlab::CurrentSettings.snowplow_collector_hostname), SnowplowTracker::AsyncEmitter.new(Gitlab::CurrentSettings.snowplow_collector_hostname, protocol: 'https'),
SnowplowTracker::Subject.new, SnowplowTracker::Subject.new,
SNOWPLOW_NAMESPACE, SNOWPLOW_NAMESPACE,
Gitlab::CurrentSettings.snowplow_site_id Gitlab::CurrentSettings.snowplow_site_id
......
...@@ -49,8 +49,8 @@ describe Gitlab::Tracking do ...@@ -49,8 +49,8 @@ describe Gitlab::Tracking do
it 'can track events' do it 'can track events' do
tracker = double tracker = double
expect(SnowplowTracker::Emitter).to receive(:new).with( expect(SnowplowTracker::AsyncEmitter).to receive(:new).with(
'gitfoo.com' 'gitfoo.com', { protocol: 'https' }
).and_return('_emitter_') ).and_return('_emitter_')
expect(SnowplowTracker::Tracker).to receive(:new).with( expect(SnowplowTracker::Tracker).to receive(:new).with(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment