kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

Merge pull request #12200 from dchelimsky/simplify-duration-inspect-even-more

Browse source 
Reduce Duration#inspect to a single series of transformations
This commit is contained in:
Rafael Mendonça França 2013-09-11 14:58:46 -07:00
commit bca623bc53
2 changed files with 7 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
activesupport/lib/active_support/duration.rb
View file

@ -70,12 +70,10 @@ module ActiveSupport
alias :until :ago
def inspect #:nodoc:
val_for = parts.inject(::Hash.new(0)) { |h,(l,r)| h[l] += r; h }
[:years, :months, :days, :minutes, :seconds].
select {|unit| val_for[unit].nonzero?}.
tap {|units| units << :seconds if units.empty?}.
map {|unit| [unit, val_for[unit]]}.
map {|unit, val| "#{val} #{val == 1 ? unit.to_s.chop : unit.to_s}"}.
parts.
reduce(::Hash.new(0)) { |h,(l,r)| h[l] += r; h }.
sort_by {|unit, _ | [:years, :months, :days, :minutes, :seconds].index(unit)}.
map {|unit, val| "#{val} #{val == 1 ? unit.to_s.chop : unit.to_s}"}.
to_sentence(:locale => :en)
end

3
activesupport/test/core_ext/duration_test.rb
View file

@ -27,6 +27,7 @@ class DurationTest < ActiveSupport::TestCase
def test_equals
assert 1.day == 1.day
assert 1.day == 1.day.to_i
assert 1.day.to_i == 1.day
assert !(1.day == 'foo')
end
@ -37,6 +38,8 @@ class DurationTest < ActiveSupport::TestCase
assert_equal '6 months and -2 days', (6.months - 2.days).inspect
assert_equal '10 seconds', 10.seconds.inspect
assert_equal '10 years, 2 months, and 1 day', (10.years + 2.months + 1.day).inspect
assert_equal '10 years, 2 months, and 1 day', (10.years + 1.month + 1.day + 1.month).inspect
assert_equal '10 years, 2 months, and 1 day', (1.day + 10.years + 2.months).inspect
assert_equal '7 days', 1.week.inspect
assert_equal '14 days', 1.fortnight.inspect
end