Commit f4078760 authored by Toon Claes's avatar Toon Claes

No need to use Gitlab::Geo::LogHelpers in EventGapTracking

parent 9371674b
......@@ -2,7 +2,6 @@ module Gitlab
module Geo
class EventGapTracking
include Utils::StrongMemoize
include ::Gitlab::Geo::LogHelpers
attr_accessor :previous_id
......@@ -61,7 +60,7 @@ module Gitlab
end
def track_gaps(current_id)
log_info("Event log gap detected", previous_event_id: previous_id, current_event_id: current_id)
logger.info("Event log gap detected", previous_event_id: previous_id, current_event_id: current_id)
with_redis do |redis|
expire_time = Time.now.to_i
......
......@@ -104,7 +104,8 @@ describe Gitlab::Geo::EventGapTracking, :clean_gitlab_redis_cache do
describe '#track_gaps' do
it 'logs a message' do
expect(gap_tracking).to receive(:log_info).with(/gap detected/, hash_including(previous_event_id: previous_event_id, current_event_id: event_id_with_gap))
expect(Gitlab::Geo::Logger).to receive(:info)
.with(/gap detected/, hash_including(previous_event_id: previous_event_id, current_event_id: event_id_with_gap))
gap_tracking.track_gaps(event_id_with_gap)
end
......
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