Use Stoplight::TimeTravel in tests

Author: bolshakov

spec/properties/notifications_spec.rb

@@ -47,7 +47,7 @@ def notifications(name)
         notifications_before_run = notifier.notifications(light.name).count
 
         executions_sequence.each do |(should_fail, time_gap)|
-          Timecop.freeze(Time.now + time_gap)
+          Stoplight::TimeTravel.freeze(Time.now + time_gap)
           suppress(StandardError) { light.run { raise if should_fail } }
 
           color_after_run = light.color

spec/spec_helper.rb

@@ -47,7 +47,7 @@
   end
 
   rspec.after do
-    Timecop.return
+    Stoplight::TimeTravel.return
     redis = Redis.new
     redis.del("stoplight:test_now_ms_stack")
   end

spec/support/data_store/transition_to_color.rb

@@ -40,7 +40,7 @@
       it { expect(transition_to_color(Stoplight::Color::GREEN)).to be(true) }
 
       it "resets timestamps" do
-        Timecop.freeze(current_time) do
+        Stoplight::TimeTravel.freeze(current_time) do
           transition_to_color(Stoplight::Color::GREEN)
         end
 
@@ -73,7 +73,7 @@
 
       it "sets the recovery_started_at timestamp" do
         expect do
-          Timecop.freeze(current_time) do
+          Stoplight::TimeTravel.freeze(current_time) do
             transition_to_color(Stoplight::Color::YELLOW)
           end
         end.to change { state_snapshot }
@@ -83,7 +83,7 @@
 
       context "when cleared" do
         it "looses persisted state" do
-          Timecop.freeze(current_time) do
+          Stoplight::TimeTravel.freeze(current_time) do
             transition_to_color(Stoplight::Color::YELLOW)
           end
           clear
@@ -113,7 +113,7 @@
 
       it "sets the breached_at and recovery_scheduled_after timestamps" do
         expect do
-          Timecop.freeze(current_time) do
+          Stoplight::TimeTravel.freeze(current_time) do
             transition_to_color(Stoplight::Color::RED)
           end
         end.to change { state_snapshot }
@@ -123,7 +123,7 @@
 
       context "when cleared" do
         it "looses persisted state" do
-          Timecop.freeze(current_time) do
+          Stoplight::TimeTravel.freeze(current_time) do
             transition_to_color(Stoplight::Color::YELLOW)
           end
           clear
@@ -143,7 +143,7 @@
 
       it "sets the breached_at and recovery_scheduled_after timestamps" do
         expect do
-          Timecop.freeze(current_time) do
+          Stoplight::TimeTravel.freeze(current_time) do
             transition_to_color(Stoplight::Color::RED)
           end
         end.to change { state_snapshot }
@@ -153,7 +153,7 @@
 
       context "when cleared" do
         it "looses persisted state" do
-          Timecop.freeze(current_time) do
+          Stoplight::TimeTravel.freeze(current_time) do
             transition_to_color(Stoplight::Color::YELLOW)
           end
           clear

spec/support/timecop.rb

@@ -5,6 +5,6 @@
 RSpec.configure do |config|
   config.around(:each, freeze: ->(value) { value.present? }) do |example|
     time = Time.now.to_i
-    Timecop.freeze(time) { example.run }
+    Stoplight::TimeTravel.freeze(time) { example.run }
   end
 end

spec/unit/stoplight/domain/failure_spec.rb

@@ -18,7 +18,7 @@
 
   describe ".from_error" do
     it "creates a failure" do
-      Timecop.freeze do
+      Stoplight::TimeTravel.freeze do
         failure = described_class.from_error(error, time: Time.now)
         expect(failure.error_class).to eql(error_class)
         expect(failure.error_message).to eql(error_message)

spec/unit/stoplight/infrastructure/data_store/metrics_snapshot.rb

@@ -13,15 +13,15 @@
       record_success
 
       expect do
-        Timecop.freeze(last_success_time) do
+        Stoplight::TimeTravel.freeze(last_success_time) do
           record_success
         end
       end.to change { metrics_snapshot.last_success_at }.to(be_within(rounding_error).of(last_success_time))
     end
 
     specify "when first success tracked" do
       expect do
-        Timecop.freeze(last_success_time) do
+        Stoplight::TimeTravel.freeze(last_success_time) do
           record_success
         end
       end.to change { metrics_snapshot.last_success_at }.from(nil).to(be_within(rounding_error).of(last_success_time))
@@ -35,15 +35,15 @@
       record_failure(error)
 
       expect do
-        Timecop.freeze(last_error_time) do
+        Stoplight::TimeTravel.freeze(last_error_time) do
           record_failure(error)
         end
       end.to change { metrics_snapshot.last_error_at }.to(be_within(rounding_error).of(last_error_time))
     end
 
     specify "when first failure tracked" do
       expect do
-        Timecop.freeze(last_error_time) do
+        Stoplight::TimeTravel.freeze(last_error_time) do
           record_failure(error)
         end
       end.to change { metrics_snapshot.last_error_at }.from(nil).to(be_within(rounding_error).of(last_error_time))

spec/unit/stoplight/infrastructure/data_store/recovery_metrics.rb

@@ -13,15 +13,15 @@
       record_success
 
       expect do
-        Timecop.freeze(last_success_time) do
+        Stoplight::TimeTravel.freeze(last_success_time) do
           record_success
         end
       end.to change { get_metrics.last_success_at }.to(be_within(rounding_error).of(last_success_time))
     end
 
     specify "when first success tracked" do
       expect do
-        Timecop.freeze(last_success_time) do
+        Stoplight::TimeTravel.freeze(last_success_time) do
           record_success
         end
       end.to change { get_metrics.last_success_at }.from(nil).to(be_within(rounding_error).of(last_success_time))
@@ -35,15 +35,15 @@
       record_failure(error)
 
       expect do
-        Timecop.freeze(last_error_time) do
+        Stoplight::TimeTravel.freeze(last_error_time) do
           record_failure(error)
         end
       end.to change { get_metrics.last_error_at }.to(be_within(rounding_error).of(last_error_time))
     end
 
     specify "when first failure tracked" do
       expect do
-        Timecop.freeze(last_error_time) do
+        Stoplight::TimeTravel.freeze(last_error_time) do
           record_failure(error)
         end
       end.to change { get_metrics.last_error_at }.to(be_within(rounding_error).of(last_error_time))

spec/unit/stoplight/infrastructure/data_store/window_metrics_snapshot.rb

@@ -9,7 +9,7 @@
         let(:window_size) { 5000 }
 
         before do
-          Timecop.freeze(-window_size - 10) do
+          Stoplight::TimeTravel.freeze(-window_size - 10) do
             record_success
           end
         end
@@ -27,7 +27,7 @@
         let(:window_size) { 5000 }
 
         before do
-          Timecop.freeze(-window_size - 10) do
+          Stoplight::TimeTravel.freeze(-window_size - 10) do
             record_failure(error)
           end
         end
@@ -79,10 +79,10 @@
         let(:window_size) { 300 }
 
         it "excludes it from the count" do
-          Timecop.freeze(Time.now) do
+          Stoplight::TimeTravel.freeze(Time.now) do
             record_success
 
-            Timecop.freeze(window_size) do
+            Stoplight::TimeTravel.freeze(window_size) do
               expect(metrics_snapshot.successes).to eq(0)
             end
           end
@@ -93,10 +93,10 @@
         let(:window_size) { 300 }
 
         it "excludes it from the count" do
-          Timecop.freeze(Time.now) do
+          Stoplight::TimeTravel.freeze(Time.now) do
             record_failure(error)
 
-            Timecop.freeze(window_size) do
+            Stoplight::TimeTravel.freeze(window_size) do
               expect(metrics_snapshot.errors).to eq(0)
             end
           end
@@ -110,10 +110,10 @@
         let(:bucket_count) { window_size }
 
         it "evicts the whole backlog without raising, keeping successes correct" do
-          Timecop.freeze(Time.now) do
-            bucket_count.times { |second| Timecop.freeze(second) { record_success } }
+          Stoplight::TimeTravel.freeze do
+            bucket_count.times { |second| Stoplight::TimeTravel.freeze(second) { record_success } }
 
-            Timecop.freeze(bucket_count + window_size + 10) do
+            Stoplight::TimeTravel.freeze(bucket_count + window_size + 10) do
               expect { record_success }.not_to raise_error
 
               expect(metrics_snapshot.successes).to eq(1)
@@ -122,10 +122,10 @@
         end
 
         it "evicts the whole backlog without raising, keeping errors correct" do
-          Timecop.freeze(Time.now) do
-            bucket_count.times { |second| Timecop.freeze(second) { record_failure(error) } }
+          Stoplight::TimeTravel.freeze do
+            bucket_count.times { |second| Stoplight::TimeTravel.freeze(second) { record_failure(error) } }
 
-            Timecop.freeze(bucket_count + window_size + 10) do
+            Stoplight::TimeTravel.freeze(bucket_count + window_size + 10) do
               expect { record_failure(error) }.not_to raise_error
 
               expect(metrics_snapshot.errors).to eq(1)
@@ -134,12 +134,12 @@
         end
 
         it "subtracts successes and failures independently when buckets mix both" do
-          Timecop.freeze(Time.now) do
+          Stoplight::TimeTravel.freeze do
             bucket_count.times do |second|
-              Timecop.freeze(second) { second.even? ? record_success : record_failure(error) }
+              Stoplight::TimeTravel.freeze(second) { second.even? ? record_success : record_failure(error) }
             end
 
-            Timecop.freeze(bucket_count + window_size + 10) do
+            Stoplight::TimeTravel.freeze(bucket_count + window_size + 10) do
               record_success
 
               expect(metrics_snapshot.successes).to eq(1)

spec/unit/stoplight/infrastructure/redis/storage/unbounded_metrics_spec.rb

@@ -26,7 +26,7 @@ def record_success = unbounded_metrics.record_success
     it "keeps the TTL on the metrics key for a subsequent out-of-order failure" do
       unbounded_metrics.record_failure(StandardError.new)
 
-      Timecop.freeze(Time.now - 30) do
+      Stoplight::TimeTravel.freeze(Time.now - 30) do
         unbounded_metrics.record_failure(StandardError.new)
       end
 

spec/unit/stoplight/infrastructure/system_clock_spec.rb

@@ -7,7 +7,7 @@
     subject(:current_time) { clock.current_time }
 
     around do |example|
-      Timecop.freeze { example.run }
+      Stoplight::TimeTravel.freeze { example.run }
     end
 
     it "returns current wall-clock time" do