Commit fc51166f authored by Rémy Coutable's avatar Rémy Coutable

Merge branch 'Fix-RSpec/TimecopFreeze-offenses-(Part-1/2)' into 'master'

Fix RSpec/TimecopFreeze offenses (Part 1/2)

See merge request gitlab-org/gitlab!76031
parents 18d6c174 269d158b
--- ---
RSpec/TimecopFreeze: RSpec/TimecopFreeze:
Exclude: Exclude:
- ee/spec/lib/gitlab/analytics/cycle_analytics/data_collector_spec.rb
- ee/spec/lib/gitlab/insights/reducers/count_per_period_reducer_spec.rb
- ee/spec/models/merge_train_spec.rb - ee/spec/models/merge_train_spec.rb
- ee/spec/support/shared_contexts/lib/gitlab/insights/reducers/reducers_shared_contexts.rb - ee/spec/support/shared_contexts/lib/gitlab/insights/reducers/reducers_shared_contexts.rb
- qa/spec/support/repeater_spec.rb - qa/spec/support/repeater_spec.rb
......
...@@ -12,7 +12,9 @@ RSpec.describe Gitlab::Analytics::CycleAnalytics::DataCollector do ...@@ -12,7 +12,9 @@ RSpec.describe Gitlab::Analytics::CycleAnalytics::DataCollector do
end end
around do |example| around do |example|
Timecop.freeze(current_time) { example.run } travel_to(current_time)
example.run
travel_back
end end
def round_to_days(seconds) def round_to_days(seconds)
...@@ -596,13 +598,13 @@ RSpec.describe Gitlab::Analytics::CycleAnalytics::DataCollector do ...@@ -596,13 +598,13 @@ RSpec.describe Gitlab::Analytics::CycleAnalytics::DataCollector do
before do before do
group.add_user(user, GroupMember::MAINTAINER) group.add_user(user, GroupMember::MAINTAINER)
travel_to(Time.new(2019, 6, 1)) do travel_to(Time.new(2019, 6, 1))
mr = create(:merge_request, source_project: project1) mr = create(:merge_request, source_project: project1)
mr.metrics.update!(merged_at: 1.hour.from_now) mr.metrics.update!(merged_at: 1.hour.from_now)
mr = create(:merge_request, source_project: project2) mr = create(:merge_request, source_project: project2)
mr.metrics.update!(merged_at: 1.hour.from_now) mr.metrics.update!(merged_at: 1.hour.from_now)
end travel_back
end end
shared_examples 'filter examples' do shared_examples 'filter examples' do
......
...@@ -26,7 +26,7 @@ RSpec.describe Gitlab::Insights::Reducers::CountPerPeriodReducer do ...@@ -26,7 +26,7 @@ RSpec.describe Gitlab::Insights::Reducers::CountPerPeriodReducer do
context 'with no issues' do context 'with no issues' do
around do |example| around do |example|
Timecop.freeze(Time.utc(2019, 5, 5)) { example.run } travel_to(Time.utc(2019, 5, 5)) { example.run }
end end
let(:project) { create(:project, :public) } let(:project) { create(:project, :public) }
...@@ -144,7 +144,7 @@ RSpec.describe Gitlab::Insights::Reducers::CountPerPeriodReducer do ...@@ -144,7 +144,7 @@ RSpec.describe Gitlab::Insights::Reducers::CountPerPeriodReducer do
merge_request = public_send("issuable#{i}") merge_request = public_send("issuable#{i}")
merge_request_metrics_service = MergeRequestMetricsService.new(merge_request.metrics) merge_request_metrics_service = MergeRequestMetricsService.new(merge_request.metrics)
Event.transaction do Event.transaction do
Timecop.freeze(merge_request.created_at) do travel_to(merge_request.created_at) do
merge_event = EventCreateService.new.merge_mr(merge_request, merge_request.author) merge_event = EventCreateService.new.merge_mr(merge_request, merge_request.author)
merge_request_metrics_service.merge(merge_event) merge_request_metrics_service.merge(merge_event)
end 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