github/rails
1
1
Fork 0
mirror of https://github.com/github/rails.git synced 2026-04-26 03:00:59 -04:00
Code Issues Packages Projects Releases Wiki Activity

Let Integer#multiple_of? accept zero as argument

Browse Source 
[#2982 state:committed]

Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
This commit is contained in:
Xavier Noria
2009-11-15 01:22:26 +01:00
committed by Jeremy Kemper
parent 8571aa613f
commit 7a2554d9a3
2 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
activesupport/lib/active_support/core_ext/integer/multiple.rb
View File

@@ -1,6 +1,6 @@
class Integer class Integer
# Check whether the integer is evenly divisible by the argument. # Check whether the integer is evenly divisible by the argument.
def multiple_of?(number) def multiple_of?(number)
self % number == 0 number != 0 ? self % number == 0 : zero?
end end
end end

5
activesupport/test/core_ext/integer_ext_test.rb
View File

@@ -5,6 +5,11 @@ class IntegerExtTest < Test::Unit::TestCase
def test_multiple_of def test_multiple_of
[ -7, 0, 7, 14 ].each { |i| assert i.multiple_of?(7) } [ -7, 0, 7, 14 ].each { |i| assert i.multiple_of?(7) }
[ -7, 7, 14 ].each { |i| assert ! i.multiple_of?(6) } [ -7, 7, 14 ].each { |i| assert ! i.multiple_of?(6) }
# test the 0 edge case
assert 0.multiple_of?(0)
assert !5.multiple_of?(0)
# test with a prime # test with a prime
assert !22953686867719691230002707821868552601124472329079.multiple_of?(2) assert !22953686867719691230002707821868552601124472329079.multiple_of?(2)
assert !22953686867719691230002707821868552601124472329079.multiple_of?(3) assert !22953686867719691230002707821868552601124472329079.multiple_of?(3)