mirrors
/
rails
mirror of https://github.com/rails/rails
8
0
Fork 0
Code Issues Projects Releases Wiki Activity

Merge pull request #47573 from codergeek121/fix-timewithzone-argumenterror 

Fixes TimeWithZone ArgumentError #47572
This commit is contained in:
Rafael Mendonça França 2023-03-03 18:02:48 -05:00 committed by GitHub
parent 9f60cd8dc7 f9868208b5
commit 5aa84bcea1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
activesupport/lib/active_support/time_with_zone.rb
View File

@ -535,8 +535,8 @@ module ActiveSupport
# Send the missing method to +time+ instance, and wrap result in a new
# TimeWithZone with the existing +time_zone+.
def method_missing(sym, *args, &block)
wrap_with_time_zone time.__send__(sym, *args, &block)
def method_missing(...)
wrap_with_time_zone time.__send__(...)
rescue NoMethodError => e
raise e, e.message.sub(time.inspect, inspect).sub("Time", "ActiveSupport::TimeWithZone"), e.backtrace
end

6
activesupport/test/core_ext/time_with_zone_test.rb
View File

@ -628,6 +628,12 @@ class TimeWithZoneTest < ActiveSupport::TestCase
assert_equal "bar", @twz.foo
end
def test_method_missing_works_with_kwargs
time = @twz.time
def time.method_with_kwarg(foo:); foo; end
assert_equal "bar", @twz.method_with_kwarg(foo: "bar")
end
def test_date_part_value_methods
twz = ActiveSupport::TimeWithZone.new(Time.utc(1999, 12, 31, 19, 18, 17, 500), @time_zone)
assert_not_called(twz, :method_missing) do