Skip to content

Commit 97e7e7a

Browse files
authored
Fix: Allow non-block TimeTravel.freeze and remove redundant test
1 parent 79bdd4b commit 97e7e7a

2 files changed

Lines changed: 1 addition & 11 deletions

File tree

spec/support/time_travel.rb

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,13 +16,11 @@ def freeze(time = Time.now)
1616
redis.rpop(STACK_KEY)
1717
end
1818
end
19-
elsif redis.exists?(STACK_KEY)
19+
else
2020
redis.del(STACK_KEY)
2121
Timecop.freeze(time)
2222
redis_time_ms = (Time.now.to_f * 1000).to_i
2323
redis.rpush(STACK_KEY, redis_time_ms)
24-
else
25-
raise LocalJumpError, "freeze requires a block"
2624
end
2725
end
2826

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

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -14,14 +14,6 @@
1414
end
1515
end
1616

17-
it "requires a block" do
18-
frozen_time = Time.new(2025, 6, 15, 14, 30, 45)
19-
20-
expect do
21-
Stoplight::TimeTravel.freeze(frozen_time)
22-
end.to raise_error(LocalJumpError, /requires a block/)
23-
end
24-
2517
it "preserves outer context after nested freeze block" do
2618
start_time = Time.new(2025, 1, 1, 12, 0, 0)
2719
inner_time = Time.new(2025, 6, 15, 14, 30, 45)

0 commit comments

Comments
 (0)