terraformer 0.0.9 → 0.1.0

data/test/geometry_spec.rb

@@ -4,120 +4,6 @@ describe Terraformer::Geometry do
 
   describe 'construction' do
 
-    describe Terraformer::Point do
-
-      it 'constructs from coordinate' do
-        c = Terraformer::Coordinate.new -122.6764, 45.5165
-        p = Terraformer::Point.new c
-        p.must_be_valid_geojson
-      end
-
-    end
-
-    describe Terraformer::MultiPoint do
-
-      it 'constructs from coordinates' do
-        a = Terraformer::Coordinate.new -122.6764, 45.5165
-        b = a + [0.02, 0.02]
-        mp = Terraformer::MultiPoint.new a, b
-        mp.must_be_valid_geojson
-      end
-
-      it 'constructs from Point objects' do
-        a = Terraformer::Coordinate.new -122.6764, 45.5165
-        b = a + [0.02, 0.02]
-        mp = Terraformer::MultiPoint.new a.to_point, b.to_point
-        mp.must_be_valid_geojson
-      end
-
-    end
-
-    describe Terraformer::LineString do
-
-      it 'constructs from coordinates' do
-        a = Terraformer::Coordinate.new -122.6764, 45.5165
-        b = a + [0.02, 0.02]
-        c = b + [0.1, -0.1]
-        ls = Terraformer::LineString.new a, b, c
-        ls.must_be_valid_geojson
-      end
-
-    end
-
-    describe Terraformer::MultiLineString do
-
-      it 'constructs from coordinates' do
-        a = Terraformer::Coordinate.new -122.6764, 45.5165
-        b = a + [0.02, 0.02]
-        c = b + [0.1, -0.1]
-        mls = Terraformer::MultiLineString.new a, b, c
-        mls.must_be_valid_geojson
-      end
-
-      it 'constructs from coordinates arrays' do
-        a = Terraformer::Coordinate.new -122.6764, 45.5165
-        b = a + [0.02, 0.02]
-        c = b + [0.1, -0.1]
-        d = c + [1,1]
-        e = d + [0.02, 0.02]
-        f = e + [0.1, -0.1]
-        mls = Terraformer::MultiLineString.new [a, b, c], [d, e, f]
-        mls.must_be_valid_geojson
-      end
-
-      it 'constructs from LineString objects' do
-        a = Terraformer::Coordinate.new -122.6764, 45.5165
-        b = a + [0.02, 0.02]
-        c = b + [0.1, -0.1]
-        d = c + [1,1]
-        e = d + [0.02, 0.02]
-        f = e + [0.1, -0.1]
-        ls_1 = Terraformer::LineString.new a, b, c
-        ls_2 = Terraformer::LineString.new d, e, f
-        mls = Terraformer::MultiLineString.new ls_1, ls_2
-        mls.must_be_valid_geojson
-      end
-
-    end
-
-    describe Terraformer::Polygon do
-
-      it 'constructs from coordinates' do
-        a = Terraformer::Coordinate.new -122.6764, 45.5165
-        b = a + [0, 0.02]
-        c = b + [0.02, 0]
-        d = c + [0, -0.02]
-        p = Terraformer::Polygon.new a, b, c, d, a
-        p.must_be_valid_geojson
-      end
-
-      it 'constructs from coordinates array' do
-        a = Terraformer::Coordinate.new -122.6764, 45.5165
-        b = a + [0, 0.02]
-        c = b + [0.02, 0]
-        d = c + [0, -0.02]
-        p = Terraformer::Polygon.new [a, b, c, d, a]
-        p.must_be_valid_geojson
-      end
-
-      it 'constructs with holes from coordinates arrays' do
-        a = Terraformer::Coordinate.new -122.6764, 45.5165
-        b = a + [0, 0.02]
-        c = b + [0.02, 0]
-        d = c + [0, -0.02]
-        hole = [
-          [ -122.67072200775145, 45.52438983143154 ],
-          [ -122.67072200775145, 45.53241707548722 ],
-          [ -122.6617956161499, 45.53241707548722 ],
-          [ -122.6617956161499, 45.52438983143154 ],
-          [ -122.67072200775145, 45.52438983143154 ]
-        ].map {|c| Terraformer::Coordinate.new c}
-        p = Terraformer::Polygon.new [a, b, c, d, a], hole
-        p.must_be_valid_geojson
-      end
-
-    end
-
     describe Terraformer::MultiPolygon do
 
       it 'constructs from coordinates' do
@@ -126,6 +12,7 @@ describe Terraformer::Geometry do
         c = b + [0.02, 0]
         d = c + [0, -0.02]
         mp = Terraformer::MultiPolygon.new a, b, c, d, a
+        mp.to_json.must_equal '{"type":"MultiPolygon","coordinates":[[[[-122.6764,45.5165],[-122.6764,45.5365],[-122.6564,45.5365],[-122.6564,45.5165],[-122.6764,45.5165]]]]}'
         mp.must_be_valid_geojson
       end
 
@@ -135,6 +22,13 @@ describe Terraformer::Geometry do
         c = b + [0.02, 0]
         d = c + [0, -0.02]
         mp = Terraformer::MultiPolygon.new [a, b, c, d, a]
+        mp.to_json.must_equal '{"type":"MultiPolygon","coordinates":[[[[-122.6764,45.5165],[-122.6764,45.5365],[-122.6564,45.5365],[-122.6564,45.5165],[-122.6764,45.5165]]]]}'
+        mp.must_be_valid_geojson
+      end
+
+      it 'constructs from array - single polygon' do
+        mp = Terraformer::MultiPolygon.new [[[[-122.6764,45.5165],[-122.6764,45.5365],[-122.6564,45.5365],[-122.6564,45.5165],[-122.6764,45.5165]]]]
+        mp.to_json.must_equal '{"type":"MultiPolygon","coordinates":[[[[-122.6764,45.5165],[-122.6764,45.5365],[-122.6564,45.5365],[-122.6564,45.5165],[-122.6764,45.5165]]]]}'
         mp.must_be_valid_geojson
       end
 
@@ -151,6 +45,13 @@ describe Terraformer::Geometry do
           [ -122.67072200775145, 45.52438983143154 ]
         ].map {|c| Terraformer::Coordinate.new c}
         mp = Terraformer::MultiPolygon.new [a, b, c, d, a], hole
+        mp.to_json.must_equal '{"type":"MultiPolygon","coordinates":[[[[-122.6764,45.5165],[-122.6764,45.5365],[-122.6564,45.5365],[-122.6564,45.5165],[-122.6764,45.5165]],[[-122.67072200775145,45.52438983143154],[-122.67072200775145,45.53241707548722],[-122.6617956161499,45.53241707548722],[-122.6617956161499,45.52438983143154],[-122.67072200775145,45.52438983143154]]]]}'
+        mp.must_be_valid_geojson
+      end
+
+      it 'constructs from array - single polygon with hole' do
+        mp = Terraformer::MultiPolygon.new [[[[-122.6764,45.5165],[-122.6764,45.5365],[-122.6564,45.5365],[-122.6564,45.5165],[-122.6764,45.5165]],[[-122.67072200775145,45.52438983143154],[-122.67072200775145,45.53241707548722],[-122.6617956161499,45.53241707548722],[-122.6617956161499,45.52438983143154],[-122.67072200775145,45.52438983143154]]]]
+        mp.to_json.must_equal '{"type":"MultiPolygon","coordinates":[[[[-122.6764,45.5165],[-122.6764,45.5365],[-122.6564,45.5365],[-122.6564,45.5165],[-122.6764,45.5165]],[[-122.67072200775145,45.52438983143154],[-122.67072200775145,45.53241707548722],[-122.6617956161499,45.53241707548722],[-122.6617956161499,45.52438983143154],[-122.67072200775145,45.52438983143154]]]]}'
         mp.must_be_valid_geojson
       end
 
@@ -173,6 +74,13 @@ describe Terraformer::Geometry do
         ].map {|c| Terraformer::Coordinate.new c}
         p_2 = Terraformer::Polygon.new [a, b, c, d, a], hole
         mp = Terraformer::MultiPolygon.new p_1, p_2
+        mp.to_json.must_equal '{"type":"MultiPolygon","coordinates":[[[[-122.6764,45.5165],[-122.6764,45.5365],[-122.6564,45.5365],[-122.6564,45.5165],[-122.6764,45.5165]]],[[[-122.6764,45.5165],[-122.6764,45.5365],[-122.6564,45.5365],[-122.6564,45.5165],[-122.6764,45.5165]],[[-122.67072200775145,45.52438983143154],[-122.67072200775145,45.53241707548722],[-122.6617956161499,45.53241707548722],[-122.6617956161499,45.52438983143154],[-122.67072200775145,45.52438983143154]]]]}'
+        mp.must_be_valid_geojson
+      end
+
+      it 'constructs from array - multi polygons with hole' do
+        mp = Terraformer::MultiPolygon.new [[[[-122.6764,45.5165],[-122.6764,45.5365],[-122.6564,45.5365],[-122.6564,45.5165],[-122.6764,45.5165]]],[[[-122.6764,45.5165],[-122.6764,45.5365],[-122.6564,45.5365],[-122.6564,45.5165],[-122.6764,45.5165]],[[-122.67072200775145,45.52438983143154],[-122.67072200775145,45.53241707548722],[-122.6617956161499,45.53241707548722],[-122.6617956161499,45.52438983143154],[-122.67072200775145,45.52438983143154]]]]
+        mp.to_json.must_equal '{"type":"MultiPolygon","coordinates":[[[[-122.6764,45.5165],[-122.6764,45.5365],[-122.6564,45.5365],[-122.6564,45.5165],[-122.6764,45.5165]]],[[[-122.6764,45.5165],[-122.6764,45.5365],[-122.6564,45.5365],[-122.6564,45.5165],[-122.6764,45.5165]],[[-122.67072200775145,45.52438983143154],[-122.67072200775145,45.53241707548722],[-122.6617956161499,45.53241707548722],[-122.6617956161499,45.52438983143154],[-122.67072200775145,45.52438983143154]]]]}'
         mp.must_be_valid_geojson
       end
 
@@ -300,7 +208,7 @@ describe Terraformer::Geometry do
         ]
       }'
       b = Terraformer.parse '{
-        "type": "LineString",
+        "type": "MultiLineString",
         "coordinates": [
           [
             [

data/test/helper.rb

@@ -1,9 +1,15 @@
+require 'simplecov'
+SimpleCov.start do
+  add_filter '/test/'
+end
+
 require 'bundler'
 Bundler.require :default, :test
 
 require 'minitest/autorun'
 require 'minitest/pride'
 
+
 lib = File.expand_path '../../lib', __FILE__
 $:.unshift lib unless $:.include? lib
 require 'terraformer'

data/test/line_string_spec.rb

@@ -0,0 +1,154 @@
+require_relative './helper'
+
+describe Terraformer::LineString do
+
+  describe 'construction' do
+
+    it 'constructs from coordinates' do
+      a = Terraformer::Coordinate.new -122.6764, 45.5165
+      b = a + [0.02, 0.02]
+      c = b + [0.1, -0.1]
+      ls = Terraformer::LineString.new a, b, c
+      ls.to_json.must_equal '{"type":"LineString","coordinates":[[-122.6764,45.5165],[-122.6564,45.5365],[-122.5564,45.4365]]}'
+      ls.must_be_valid_geojson
+    end
+
+    it 'constructs from array' do
+      ls = Terraformer::LineString.new [[-122.6764,45.5165],[-122.6564,45.5365],[-122.5564,45.4365]]
+      ls.to_json.must_equal '{"type":"LineString","coordinates":[[-122.6764,45.5165],[-122.6564,45.5365],[-122.5564,45.4365]]}'
+      ls.must_be_valid_geojson
+    end
+
+    it 'constructs from array of points' do
+      ls = Terraformer::LineString.new [-122.6764,45.5165],[-122.6564,45.5365],[-122.5564,45.4365]
+      ls.to_json.must_equal '{"type":"LineString","coordinates":[[-122.6764,45.5165],[-122.6564,45.5365],[-122.5564,45.4365]]}'
+      ls.must_be_valid_geojson
+    end
+  end
+
+  describe 'methods' do
+    before :each do
+      @c1 = Terraformer::Coordinate.new -122.6764, 45.5165
+      @c2 = @c1 + [0.02, 0.02]
+      @c3 = @c2 + [0.1, -0.1]
+      @l = Terraformer::LineString.new @c1, @c2, @c3
+    end
+
+    it 'should have coordinates' do
+      @l.coordinates.must_equal [@c1, @c2, @c3]
+    end
+
+    it 'should have first coordinates' do
+      @l.first_coordinate.must_equal @c1
+    end
+
+    describe 'linear ring' do
+      it 'should not be linear ring' do
+        assert !@l.linear_ring?
+      end
+
+      it 'should be linear ring' do
+        @l.add_vertex @c1
+        assert @l.linear_ring?
+      end
+    end
+
+    it 'should get points' do
+      points = [@c1.to_point, @c2.to_point, @c3.to_point]
+
+      @l.points.must_equal points
+      @l.vertices.must_equal points
+    end
+
+    it 'should get point at' do
+      @l.point_at(1).must_equal Terraformer::Point.new(@c2)
+      @l.vertex_at(1).must_equal Terraformer::Point.new(@c2)
+    end
+
+  end
+
+  describe 'modification methods' do
+    before :each do
+      @c1 = Terraformer::Coordinate.new -122.6764, 45.5165
+      @c2 = @c1 + [0.02, 0.02]
+      @c3 = @c2 + [0.1, -0.1]
+      @l = Terraformer::LineString.new @c1, @c2, @c3
+
+      @c4 = @c3 + [0.5, -0.3]
+    end
+
+    it 'should add coordinate' do
+      @l.add_vertex @c4
+      @l.coordinates.must_equal [ @c1, @c2, @c3, @c4 ]
+    end
+
+    it 'should add point' do
+      p = Terraformer::Point.new @c4
+      @l.add_vertex p
+      @l.coordinates.must_equal [ @c1, @c2, @c3, @c4 ]
+    end
+
+    it 'should return add argument error' do
+      assert_raises ArgumentError do
+        @l.add_vertex 1
+      end
+    end
+
+    it 'should << coordinate' do
+      @l << @c4
+      @l.coordinates.must_equal [ @c1, @c2, @c3, @c4 ]
+    end
+
+    it 'should << point' do
+      p = Terraformer::Point.new @c4
+      @l << p
+      @l.coordinates.must_equal [ @c1, @c2, @c3, @c4 ]
+    end
+
+    it 'should return add argument error' do
+      assert_raises ArgumentError do
+        @l << 1
+      end
+    end
+
+    it 'should insert coordinate' do
+      @l.insert_vertex 1, @c4
+      @l.coordinates.must_equal [ @c1, @c4, @c2, @c3 ]
+    end
+
+    it 'should insert point ' do
+      p = Terraformer::Point.new @c4
+      @l.insert_vertex 1, p
+      @l.coordinates.must_equal [ @c1, @c4, @c2, @c3 ]
+    end
+
+    it 'should return insert argument error' do
+      assert_raises ArgumentError do
+        @l.insert_vertex 1, 1
+      end
+    end
+
+    it 'should remove coordinate' do
+      @l.remove_vertex @c3
+      @l.coordinates.must_equal [ @c1, @c2 ]
+    end
+
+    it 'should remove point' do
+      @l.remove_vertex Terraformer::Point.new(@c3)
+      @l.coordinates.must_equal [ @c1, @c2 ]
+    end
+
+    it 'should return remove argument error' do
+      assert_raises ArgumentError do
+        @l.remove_vertex 1
+      end
+    end
+
+    it 'should remove vertex at index' do
+      @l.remove_vertex_at 1
+      @l.coordinates.must_equal [ @c1, @c3 ]
+    end
+
+  end
+
+end

data/test/multi_line_string_spec.rb

@@ -0,0 +1,78 @@
+require_relative './helper'
+
+describe Terraformer::MultiLineString do
+
+  describe 'construction' do
+
+    it 'constructs from coordinates' do
+      a = Terraformer::Coordinate.new -122.6764, 45.5165
+      b = a + [0.02, 0.02]
+      c = b + [0.1, -0.1]
+      mls = Terraformer::MultiLineString.new a, b, c
+      mls.to_json.must_equal '{"type":"MultiLineString","coordinates":[[[-122.6764,45.5165],[-122.6564,45.5365],[-122.5564,45.4365]]]}'
+      mls.must_be_valid_geojson
+    end
+
+    it 'constructs from array - single line' do
+      ## unlike point, multipoint and linestring, passing in the coordinates array directly into multilinestring doesn't work
+      # mls = Terraformer::MultiLineString.new [[[-122.6764,45.5165],[-122.6564,45.5365],[-122.5564,45.4365]]]
+      mls = Terraformer::MultiLineString.new [[-122.6764,45.5165],[-122.6564,45.5365],[-122.5564,45.4365]]
+      mls.to_json.must_equal '{"type":"MultiLineString","coordinates":[[[-122.6764,45.5165],[-122.6564,45.5365],[-122.5564,45.4365]]]}'
+      mls.must_be_valid_geojson
+    end
+
+    it 'constructs from coordinates arrays' do
+      a = Terraformer::Coordinate.new -122.6764, 45.5165
+      b = a + [0.02, 0.02]
+      c = b + [0.1, -0.1]
+      d = c + [1,1]
+      e = d + [0.02, 0.02]
+      f = e + [0.1, -0.1]
+      mls = Terraformer::MultiLineString.new [a, b, c], [d, e, f]
+      mls.to_json.must_equal '{"type":"MultiLineString","coordinates":[[[-122.6764,45.5165],[-122.6564,45.5365],[-122.5564,45.4365]],[[-121.5564,46.4365],[-121.5364,46.4565],[-121.4364,46.3565]]]}'
+      mls.must_be_valid_geojson
+    end
+    it 'constructs from LineString objects' do
+      a = Terraformer::Coordinate.new -122.6764, 45.5165
+      b = a + [0.02, 0.02]
+      c = b + [0.1, -0.1]
+      d = c + [1,1]
+      e = d + [0.02, 0.02]
+      f = e + [0.1, -0.1]
+      ls_1 = Terraformer::LineString.new a, b, c
+      ls_2 = Terraformer::LineString.new d, e, f
+      mls = Terraformer::MultiLineString.new ls_1, ls_2
+      mls.to_json.must_equal '{"type":"MultiLineString","coordinates":[[[-122.6764,45.5165],[-122.6564,45.5365],[-122.5564,45.4365]],[[-121.5564,46.4365],[-121.5364,46.4565],[-121.4364,46.3565]]]}'
+      mls.must_be_valid_geojson
+    end
+
+    it 'constructs from array - multiple lines' do
+        ## unlike point, multipoint and linestring, passing in the coordinates array directly into multilinestring doesn't work
+        # mls = Terraformer::MultiLineString.new [[[-122.6764,45.5165],[-122.6564,45.5365],[-122.5564,45.4365]],[[-121.5564,46.4365],[-121.5364,46.4565],[-121.4364,46.3565]]]
+        mls = Terraformer::MultiLineString.new [[-122.6764,45.5165],[-122.6564,45.5365],[-122.5564,45.4365]],[[-121.5564,46.4365],[-121.5364,46.4565],[-121.4364,46.3565]]
+      mls.to_json.must_equal '{"type":"MultiLineString","coordinates":[[[-122.6764,45.5165],[-122.6564,45.5365],[-122.5564,45.4365]],[[-121.5564,46.4365],[-121.5364,46.4565],[-121.4364,46.3565]]]}'
+      mls.must_be_valid_geojson
+    end
+
+  end
+
+  describe 'methods' do
+    before :each do
+      @a = Terraformer::Coordinate.new -122.6764, 45.5165
+      @b = @a + [0.02, 0.02]
+      @c = @b + [0.1, -0.1]
+      @d = @c + [1,1]
+      @e = @d + [0.02, 0.02]
+      @f = @e + [0.1, -0.1]
+      @mls = Terraformer::MultiLineString.new [@a, @b, @c], [@d, @e, @f]
+    end
+
+    it 'should have coordinates' do
+      @mls.coordinates.must_equal [[@a, @b, @c], [@d, @e, @f]]
+    end
+
+    it 'should have first coordinates' do
+      @mls.first_coordinate.must_equal @a
+    end
+  end
+end

data/test/multi_point_spec.rb

@@ -0,0 +1,151 @@
+require_relative './helper'
+
+describe Terraformer::MultiPoint do
+
+  describe 'construction' do
+
+    it 'constructs from coordinates' do
+      a = Terraformer::Coordinate.new -122.6764, 45.5165
+      b = a + [0.02, 0.02]
+      mp = Terraformer::MultiPoint.new a, b
+      mp.to_json.must_equal '{"type":"MultiPoint","coordinates":[[-122.6764,45.5165],[-122.6564,45.5365]]}'
+      mp.must_be_valid_geojson
+    end
+
+    it 'constructs from Point objects' do
+      a = Terraformer::Coordinate.new -122.6764, 45.5165
+      b = a + [0.02, 0.02]
+      mp = Terraformer::MultiPoint.new a.to_point, b.to_point
+      mp.to_json.must_equal '{"type":"MultiPoint","coordinates":[[-122.6764,45.5165],[-122.6564,45.5365]]}'
+      mp.must_be_valid_geojson
+    end
+
+    it 'constructs from array' do
+      mp = Terraformer::MultiPoint.new [[-122.6764, 45.5165],[-122.6564, 45.5365]]
+      mp.to_json.must_equal '{"type":"MultiPoint","coordinates":[[-122.6764,45.5165],[-122.6564,45.5365]]}'
+      mp.must_be_valid_geojson
+    end
+
+    it 'constructs from multiple point arrays' do
+      mp = Terraformer::MultiPoint.new [-122.6764, 45.5165],[-122.6564, 45.5365]
+      mp.to_json.must_equal '{"type":"MultiPoint","coordinates":[[-122.6764,45.5165],[-122.6564,45.5365]]}'
+      mp.must_be_valid_geojson
+    end
+
+  end
+  describe 'methods' do
+    before :each do
+      @p1 = Terraformer::Point.new -122.6764, 45.5165
+      @p2 = Terraformer::Point.new -122.6564, 45.5365
+      @mp = Terraformer::MultiPoint.new @p1, @p2
+    end
+
+    it 'should have coordinates' do
+      c = Terraformer::Coordinate.new -122.6764, 45.5165
+      d = Terraformer::Coordinate.new -122.6564, 45.5365
+      @mp.coordinates.must_equal [c,d]
+    end
+
+    it 'should have first_coordinate' do
+      c = Terraformer::Coordinate.new -122.6764, 45.5165
+      @mp.first_coordinate.must_equal c
+    end
+
+    it 'should return points' do
+      @mp.points.must_equal [ @p1, @p2 ]
+    end
+
+    describe 'contains?' do
+      it 'should contain any included points' do
+        assert @mp.contains?(@p1)
+
+      end
+
+      it 'should not contain point' do
+        p = Terraformer::Point.new -122, 45
+        assert !@mp.contains?(p)
+      end
+    end
+  end
+
+  describe 'modification methods' do
+    before :each do
+      @p1 = Terraformer::Point.new -122.6764, 45.5165
+      @p2 = Terraformer::Point.new -122.6564, 45.5365
+      @mp = Terraformer::MultiPoint.new @p1, @p2
+
+      @p3 = Terraformer::Point.new -122.6364, 45.5565
+    end
+
+    it 'should add point' do
+      @mp.add_point @p3
+      @mp.points.must_equal [ @p1, @p2, @p3 ]
+    end
+
+    it 'should add coordinate' do
+      @mp.add_point @p3.first_coordinate
+      @mp.points.must_equal [ @p1, @p2, @p3 ]
+    end
+
+    it 'should return add argument error' do
+      assert_raises ArgumentError do
+        @mp.add_point 1
+      end
+    end
+
+    it 'should << point' do
+      @mp << @p3
+      @mp.points.must_equal [ @p1, @p2, @p3 ]
+    end
+
+    it 'should << coordinate' do
+      @mp << @p3.first_coordinate
+      @mp.points.must_equal [ @p1, @p2, @p3 ]
+    end
+
+    it 'should return << argument error' do
+      assert_raises ArgumentError do
+        @mp << 1
+      end
+    end
+
+    it 'should insert point' do
+      @mp.insert_point 1, @p3
+      @mp.points.must_equal [ @p1, @p3, @p2 ]
+    end
+
+    it 'should insert coordinate' do
+      @mp.insert_point 1, @p3.first_coordinate
+      @mp.points.must_equal [ @p1, @p3, @p2 ]
+    end
+
+    it 'should return insert argument error' do
+      assert_raises ArgumentError do
+        @mp.insert_point 1, 1
+      end
+    end
+
+    it 'should remove point' do
+      @mp.remove_point @p2
+      @mp.points.must_equal [ @p1 ]
+    end
+
+    it 'should remove coordinate' do
+      @mp.remove_point @p2.first_coordinate
+      @mp.points.must_equal [ @p1 ]
+    end
+
+    it 'should return remove argument error' do
+      assert_raises ArgumentError do
+        @mp.remove_point 1
+      end
+    end
+
+    it 'should remove point at index' do
+      @mp.remove_point_at 0
+      @mp.points.must_equal [ @p2 ]
+    end
+
+  end
+
+end