sevenwire-forgery 0.2.0 → 0.2.1

data/VERSION.yml

@@ -1,4 +1,4 @@
 --- 
-patch: 0
-major: 0
-minor: 2
+:minor: 2
+:patch: 1
+:major: 0

data/lib/extensions/range.rb

@@ -1,8 +1,8 @@
 class Range
   def random
-    return nil unless self.max
-    Integer(min) && Integer(max)
-    Kernel.rand(self.max - self.min + 1) + self.min
+    Integer(first) && Integer(last)
+    raise ArgumentError if first > last
+    Kernel.rand(last - first + 1) + first
   rescue ArgumentError
     self.to_a.random
   end

data/spec/extensions/range_spec.rb

@@ -1,4 +1,5 @@
 require File.dirname(__FILE__) + '/../spec_helper'
+require 'timeout'
 
 describe Range do
   it "should get a random number out of the range" do
@@ -6,6 +7,14 @@ describe Range do
     10.times { range.should include(range.random) }
   end
 
+  it "should not take a long time when the range is huge" do
+    Timeout.timeout(1){(1234567890..12345678901234567890).random}.should_not raise_error(Timeout::Error)
+  end
+
+  it "should return nil for a random number from a reverse range" do
+    10.times { (9..0).random.should be_nil }
+  end
+
   it "should get a random string our of the range" do
     range = ("a".."z")
     10.times { range.should include(range.random) }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: sevenwire-forgery
 version: !ruby/object:Gem::Version 
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors: 
 - Nathan Sutton