numeric_array 1.0.0 → 1.0.1

data/README.md

@@ -13,3 +13,13 @@ An extension for the native Array class with support for mathematical/statistica
     - Array#std_dev
 
 
+Usage
+-
+
+    gem install numeric_array
+  
+    require 'numeric_array'
+    
+If using bundler
+
+    gem 'numeric_array', :require => 'numeric_array'

data/VERSION

@@ -1 +1 @@
-1.0.0
+1.0.1

data/lib/numeric_array.rb

@@ -3,10 +3,13 @@ module NumericArray
     
     NUMERIC_REGEX = /^-?\d+.?\d*e?-?\d*$/
     
-    def sum
-      check_numeric_array!
-      a = numerify
-      a.inject(0) {|sum, value| sum + value}
+    # Don't require numeric for this Array#sum. ActiveSupport uses it as well: ["foo", "bar"].sum #=> "foobar"
+    def sum(identity = 0, &block)
+      if block_given?
+        map(&block).sum(identity)
+      else
+        inject { |sum, element| sum + element } || identity
+      end
     end
 
     #  average of an array of numbers

data/spec/numeric_array_spec.rb

@@ -1,15 +1,14 @@
 require 'spec_helper'
 
 describe NumericArray do
-  describe ["1", 2.52, "foo", :bar, "-1.25e45" ] do
-    let(:array) { ["1", 2.52, "foo", :bar, "-1.25e45" ] }    
+  describe [1, "2.52", "foo", 'bar', "-1.25e45" ] do
+    let(:array) { [1, "2.52", "foo", 'bar', "-1.25e45" ] }    
     it { should_not be_numeric }
     it { should_not be_a_numeric_array }
-    its(:numerify!) { lambda{ array.numerify! }.should raise_error(ArgumentError) }
-    its(:sum) { lambda{ array.sum }.should raise_error(ArgumentError) }
     its(:avg) { lambda{ array.avg }.should raise_error(ArgumentError) }
     its(:variance) { lambda{ array.variance }.should raise_error(ArgumentError) }
     its(:std_dev) { lambda{ array.std_dev }.should raise_error(ArgumentError) }
+    its(:numerify!) { lambda{ array.numerify! }.should raise_error(ArgumentError) }    
   end
   
   describe [1, 2.52, -8.65, 20/50.0] do

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 1
   - 0
-  - 0
-  version: 1.0.0
+  - 1
+  version: 1.0.1
 platform: ruby
 authors: 
 - Evan Sagge