Allow passed points to be any object that is enumerable

Previous to this change the `raw_points` passed to `Simplifier#process` had to be an array.
This commit loosens the requirement by checking wether the object is enumerable / iterable, thus allowing non-arrays to be passed (e.g. ActiveRecord relations!).

Author: nTraum

lib/simplify_rb.rb

@@ -6,7 +6,7 @@
 module SimplifyRb
   class Simplifier
     def process(raw_points, tolerance = 1, highest_quality = false)
-      raise ArgumentError.new('Points must be an array') unless raw_points.is_a? Array
+      raise ArgumentError.new('raw_points must be enumerable') unless raw_points.is_a? Enumerable
 
       return raw_points if raw_points.length <= 1
 

spec/simplify_rb_spec.rb

@@ -49,10 +49,19 @@
       end
     end
 
-    describe 'unexpected argument' do
-      it 'raises an error if the points are not passsed as an array' do
-        data = { x: 1, y: 2 }
-        expect { subject.process(data) }.to raise_error(ArgumentError, 'Points must be an array')
+    describe 'unexpected arguments' do
+      context 'when raw_points is not enumerable' do
+        it 'raises an ArgumentError' do
+          data = Object.new
+          expect { subject.process(data) }.to raise_error(ArgumentError, 'raw_points must be enumerable')
+        end
+      end
+
+      context "when raw_points is enumerable, but doesn't respond to x/y" do
+        it 'raises an ArgumentError' do
+          data = [{ foo: :bar }, { foo: :bar }]
+          expect { subject.process(data) }.to raise_error(ArgumentError, 'Points must have :x and :y values')
+        end
       end
     end
   end