Skip to content

Commit d42f944

Browse files
authored
Merge pull request rubocop#765 from koic/fix_an_error_for_rails_freeze_time
[Fix rubocop#762] Fix an error for `Rails/FreezeTime`
2 parents 2f68e9f + b56f67d commit d42f944

File tree

3 files changed

+8
-0
lines changed

3 files changed

+8
-0
lines changed
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
* [#762](https://github.com/rubocop/rubocop-rails/issues/762): Fix an error for `Rails/FreezeTime` when using `travel_to` with an argument of `current` method without receiver. ([@koic][])

lib/rubocop/cop/rails/freeze_time.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ class FreezeTime < Base
4444

4545
def on_send(node)
4646
child_node, method_name = *node.first_argument.children
47+
return unless child_node
4748
return unless current_time?(child_node, method_name) || current_time_with_convert?(child_node, method_name)
4849

4950
add_offense(node) { |corrector| corrector.replace(node, 'freeze_time') }

spec/rubocop/cop/rails/freeze_time_spec.rb

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,4 +70,10 @@
7070
travel_to(Time.zone.yesterday.in_time_zone)
7171
RUBY
7272
end
73+
74+
it 'does not register an offense when using `travel_to` with an argument of `current` method without receiver' do
75+
expect_no_offenses(<<~RUBY)
76+
travel_to(current)
77+
RUBY
78+
end
7379
end

0 commit comments

Comments
 (0)