georuby 1.9.9 → 2.0.0

data/lib/geo_ruby.rb

@@ -1,21 +1,7 @@
 # $:.unshift(File.dirname(__FILE__)) #unless
 #  $:.include?(File.dirname(__FILE__)) || $:.include?(File.expand_path(File.dirname(__FILE__)))
 
-require 'geo_ruby/simple_features/helper'
-require 'geo_ruby/simple_features/ewkt_parser'
-require 'geo_ruby/simple_features/ewkb_parser'
-require 'geo_ruby/simple_features/geometry'
-require 'geo_ruby/simple_features/point'
-require 'geo_ruby/simple_features/line_string'
-require 'geo_ruby/simple_features/linear_ring'
-require 'geo_ruby/simple_features/circle'
-require 'geo_ruby/simple_features/polygon'
-require 'geo_ruby/simple_features/multi_point'
-require 'geo_ruby/simple_features/multi_line_string'
-require 'geo_ruby/simple_features/multi_polygon'
-require 'geo_ruby/simple_features/geometry_collection'
-require 'geo_ruby/simple_features/envelope'
-require 'geo_ruby/simple_features/geometry_factory'
+require 'geo_ruby/simple_features'
 
 # Require if you need
 # require 'geo_ruby/shp4r/shp'

data/lib/geo_ruby/geojson.rb

@@ -8,6 +8,7 @@ rescue LoadError
 end
 
 module GeoRuby
+  
   #Raised when an error in the GeoJSON string is detected
   class GeojsonFormatError < StandardError
   end
@@ -73,6 +74,7 @@ module GeoRuby
 
 
   class GeojsonParser
+    include GeoRuby::SimpleFeatures
     attr_reader :geometry
 
     def parse(geojson, srid=DEFAULT_SRID)

data/lib/geo_ruby/georss.rb

@@ -21,6 +21,7 @@ module GeoRuby
   #Parses GeoRSS strings
   #You can also use directly the static method Geometry.from_georss
   class GeorssParser
+    include GeoRuby::SimpleFeatures
     attr_reader :georss_tags, :geometry
 
     #Parses the georss geometry  passed as argument and notifies the factory of events

data/lib/geo_ruby/kml.rb

@@ -61,7 +61,7 @@ module GeoRuby
     def accumulate_end(e); @buffer << "</#{e[0]}>"; end
     
     def parse_coordinates(buffer)
-      if(buffer =~ /<coordinates>(.+)<\/coordinates>/)
+      if(buffer =~ /<coordinates>(.+)<\/coordinates>/m)
         $1.gsub(/\n/, " ").strip.split(/\s+/).each do |coord|
           x,y,z = coord.split(',')
           if(x.nil? || y.nil?) 

data/lib/geo_ruby/simple_features.rb

@@ -0,0 +1,25 @@
+module GeoRuby
+  module SimpleFeatures
+    
+    %w[
+      geometry
+      circle
+      envelope
+      ewkb_parser
+      ewkt_parser
+      geometry_collection
+      geometry_factory
+      helper
+      line_string
+      linear_ring
+      multi_line_string
+      multi_point
+      multi_polygon
+      point
+      polygon
+    ].each do |rel_file|
+      require File.join(File.dirname(__FILE__), 'simple_features', rel_file)
+    end
+    
+  end
+end

data/lib/geo_ruby/version.rb

@@ -1,3 +1,3 @@
 module GeoRuby
-  VERSION = '1.9.9'
+  VERSION = '2.0.0'
 end

data/spec/geo_ruby/geojson_spec.rb

@@ -7,25 +7,25 @@ DATA_DIR = File.dirname(__FILE__) + '/../data/geojson/'
 #
 # TODO Refactor comon test approaches into methods
 # TODO Add use of contexts?
-describe GeojsonParser do
+describe GeoRuby::GeojsonParser do
 
-  it "should create a specified Point" do
+  it "should create a specified GeoRuby::SimpleFeatures::Point" do
     point_json = %{ { "type": "Point", "coordinates": [100.0, 0.0] } }
-    point = Geometry.from_geojson(point_json)
-    point.class.should eql(Point)
+    point = GeoRuby::SimpleFeatures::Geometry.from_geojson(point_json)
+    point.class.should eql(GeoRuby::SimpleFeatures::Point)
     point_hash = JSON.parse(point_json)
     point.to_coordinates.should eql(point_hash['coordinates'])
   end
 
-  it "should create a specified LineString" do
+  it "should create a specified GeoRuby::SimpleFeatures::LineString" do
     ls_json = %{ { "type": "LineString", "coordinates": [ [100.0, 0.0], [101.0, 1.0] ]} }
-    line_string = Geometry.from_geojson(ls_json)
-    line_string.class.should eql(LineString)
+    line_string = GeoRuby::SimpleFeatures::Geometry.from_geojson(ls_json)
+    line_string.class.should eql(GeoRuby::SimpleFeatures::LineString)
     ls_hash = JSON.parse(ls_json)
     line_string.to_coordinates.should eql(ls_hash['coordinates'])
   end
 
-  it "should create a specified Polygon" do
+  it "should create a specified GeoRuby::SimpleFeatures::Polygon" do
     poly_json = <<-EOJ
       { "type": "Polygon",
         "coordinates": [
@@ -34,26 +34,26 @@ describe GeojsonParser do
           ]
       }
     EOJ
-    polygon = Geometry.from_geojson(poly_json)
-    polygon.class.should eql(Polygon)
+    polygon = GeoRuby::SimpleFeatures::Geometry.from_geojson(poly_json)
+    polygon.class.should eql(GeoRuby::SimpleFeatures::Polygon)
     polygon.rings.size.should eql(2)
     poly_hash = JSON.parse(poly_json)
     polygon.to_coordinates.should eql(poly_hash['coordinates'])
   end
 
-  it "should create a specified MultiPoint" do
+  it "should create a specified GeoRuby::SimpleFeatures::MultiPoint" do
     mp_json = <<-EOJ
       { "type": "MultiPoint",
         "coordinates": [ [100.0, 0.0], [101.0, 1.0] ]
         }
     EOJ
-    multi_point = Geometry.from_geojson(mp_json)
-    multi_point.class.should eql(MultiPoint)
+    multi_point = GeoRuby::SimpleFeatures::Geometry.from_geojson(mp_json)
+    multi_point.class.should eql(GeoRuby::SimpleFeatures::MultiPoint)
     mp_hash = JSON.parse(mp_json)
     multi_point.to_coordinates.should eql(mp_hash['coordinates'])
   end
 
-  it "should create a specified MultiLineString" do
+  it "should create a specified GeoRuby::SimpleFeatures::MultiLineString" do
     mls_json = <<-EOJ
       { "type": "MultiLineString",
         "coordinates": [
@@ -62,13 +62,13 @@ describe GeojsonParser do
           ]
       }
     EOJ
-    multi_ls = Geometry.from_geojson(mls_json)
-    multi_ls.class.should eql(MultiLineString)
+    multi_ls = GeoRuby::SimpleFeatures::Geometry.from_geojson(mls_json)
+    multi_ls.class.should eql(GeoRuby::SimpleFeatures::MultiLineString)
     mls_hash = JSON.parse(mls_json)
     multi_ls.to_coordinates.should eql(mls_hash['coordinates'])
   end
 
-  it "should create a specifiead MultiPolygon" do
+  it "should create a specifiead GeoRuby::SimpleFeatures::MultiPolygon" do
     mpoly_json = <<-EOJ
       { "type": "MultiPolygon",
         "coordinates": [
@@ -78,13 +78,13 @@ describe GeojsonParser do
           ]
         }
     EOJ
-    mpoly = Geometry.from_geojson(mpoly_json)
-    mpoly.class.should eql(MultiPolygon)
+    mpoly = GeoRuby::SimpleFeatures::Geometry.from_geojson(mpoly_json)
+    mpoly.class.should eql(GeoRuby::SimpleFeatures::MultiPolygon)
     mpoly_hash = JSON.parse(mpoly_json)
     mpoly.to_coordinates.should eql(mpoly_hash['coordinates'])
   end
 
-  it "should create a specified GeometryCollection" do
+  it "should create a specified GeoRuby::SimpleFeatures::GeometryCollection" do
     gcol_json = <<-EOJ
       { "type": "GeometryCollection",
         "geometries": [
@@ -97,8 +97,8 @@ describe GeojsonParser do
         ]
       }
     EOJ
-    gcol = Geometry.from_geojson(gcol_json)
-    gcol.class.should eql(GeometryCollection)
+    gcol = GeoRuby::SimpleFeatures::Geometry.from_geojson(gcol_json)
+    gcol.class.should eql(GeoRuby::SimpleFeatures::GeometryCollection)
     gcol_hash = JSON.parse(gcol_json)
     gcol.geometries.each_with_index do |g,i|
       gh = gcol_hash['geometries'][i]
@@ -120,8 +120,8 @@ describe GeojsonParser do
         }
       }
     EOJ
-    f = Geometry.from_geojson(feature_json)
-    f.class.should eql(GeojsonFeature)
+    f = GeoRuby::SimpleFeatures::Geometry.from_geojson(feature_json)
+    f.class.should eql(GeoRuby::GeojsonFeature)
     feature_hash = JSON.parse(feature_json)
     f.id.should eql(feature_hash['id'])
     f.properties.should eql(feature_hash['properties'])
@@ -131,8 +131,8 @@ describe GeojsonParser do
 
   it "should create a specified FeatureCollection" do
     fcol_json = File.read(DATA_DIR + 'feature_collection.json')
-    fcol = Geometry.from_geojson(fcol_json)
-    fcol.class.should eql(GeojsonFeatureCollection)
+    fcol = GeoRuby::SimpleFeatures::Geometry.from_geojson(fcol_json)
+    fcol.class.should eql(GeoRuby::GeojsonFeatureCollection)
     fcol_hash = JSON.parse(fcol_json)
     fcol.features.each_with_index do |f,i|
       fgh = fcol_hash['features'][i]['geometry']

data/spec/geo_ruby/georss_spec.rb

@@ -2,13 +2,225 @@ require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 
 RSS_DATA_DIR = File.dirname(__FILE__) + '/../data/georss/'
 
-describe GeorssParser do
+describe GeoRuby::GeorssParser do
 
   it "should parse an rss file" do
-    geo = GeorssParser.new.parse(File.read(RSS_DATA_DIR + "/w3c.xml"))
-    geo.should be_a Point
+    geo = subject.parse(File.read(RSS_DATA_DIR + "/w3c.xml"))
+    geo.should be_a GeoRuby::SimpleFeatures::Point
   end
 
+  it "test_point_creation" do
+    point = GeoRuby::SimpleFeatures::Point.from_x_y(3,4)
 
+    point.as_georss(:dialect => :simple, :elev => 45.7, :featuretypetag => "hoyoyo").gsub("\n","").should eql("<georss:point featuretypetag=\"hoyoyo\" elev=\"45.7\">4 3</georss:point>")
+    point.as_georss(:dialect => :w3cgeo).gsub("\n","").should eql("<geo:lat>4</geo:lat><geo:long>3</geo:long>")
+    point.as_georss(:dialect => :gml).gsub("\n","").should eql("<georss:where><gml:Point><gml:pos>4 3</gml:pos></gml:Point></georss:where>")
+
+    point.as_kml(:id => "HOYOYO-42").gsub("\n","").should eql("<Point id=\"HOYOYO-42\"><coordinates>3,4</coordinates></Point>")
+  end
+
+  it "test_line_string" do
+    ls = GeoRuby::SimpleFeatures::LineString.from_points([GeoRuby::SimpleFeatures::Point.from_lon_lat_z(12.4,-45.3,56),GeoRuby::SimpleFeatures::Point.from_lon_lat_z(45.4,41.6,45)],123,true)
+
+    ls.as_georss.gsub("\n","").should eql("<georss:line>-45.3 12.4 41.6 45.4</georss:line>")
+    ls.as_georss(:dialect => :w3cgeo).gsub("\n","").should eql("<geo:lat>-45.3</geo:lat><geo:long>12.4</geo:long>")
+    ls.as_georss(:dialect => :gml).gsub("\n","").should eql("<georss:where><gml:LineString><gml:posList>-45.3 12.4 41.6 45.4</gml:posList></gml:LineString></georss:where>")
+
+    ls.as_kml(:extrude => 1, :altitude_mode => "absolute").gsub("\n","").should eql("<LineString><extrude>1</extrude><altitudeMode>absolute</altitudeMode><coordinates>12.4,-45.3,56 45.4,41.6,45</coordinates></LineString>")
+  end
+
+  it "test_polygon" do
+    linear_ring1 = GeoRuby::SimpleFeatures::LinearRing.from_coordinates([[12.4,-45.3],[45.4,41.6],[4.456,1.0698],[12.4,-45.3]],256)
+    linear_ring2 = GeoRuby::SimpleFeatures::LinearRing.from_coordinates([[2.4,5.3],[5.4,1.4263],[14.46,1.06],[2.4,5.3]],256)
+    polygon = GeoRuby::SimpleFeatures::Polygon.from_linear_rings([linear_ring1,linear_ring2],256)
+
+    polygon.as_georss(:georss_ns => "hoyoyo").gsub("\n","").should eql("<hoyoyo:polygon>-45.3 12.4 41.6 45.4 1.0698 4.456 -45.3 12.4</hoyoyo:polygon>")
+    polygon.as_georss(:dialect => :w3cgeo, :w3cgeo_ns => "bouyoul").gsub("\n","").should eql("<bouyoul:lat>-45.3</bouyoul:lat><bouyoul:long>12.4</bouyoul:long>")
+    polygon.as_georss(:dialect => :gml).gsub("\n","").should eql("<georss:where><gml:Polygon><gml:exterior><gml:LinearRing><gml:posList>-45.3 12.4 41.6 45.4 1.0698 4.456 -45.3 12.4</gml:posList></gml:LinearRing></gml:exterior></gml:Polygon></georss:where>")
+
+    polygon.as_kml.gsub("\n","").should eql("<Polygon><outerBoundaryIs><LinearRing><coordinates>12.4,-45.3 45.4,41.6 4.456,1.0698 12.4,-45.3</coordinates></LinearRing></outerBoundaryIs><innerBoundaryIs><LinearRing><coordinates>2.4,5.3 5.4,1.4263 14.46,1.06 2.4,5.3</coordinates></LinearRing></innerBoundaryIs></Polygon>")
+  end
+
+  it "test_geometry_collection" do
+    gc = GeoRuby::SimpleFeatures::GeometryCollection.from_geometries([GeoRuby::SimpleFeatures::Point.from_x_y(4.67,45.4,256),GeoRuby::SimpleFeatures::LineString.from_coordinates([[5.7,12.45],[67.55,54]],256)],256)
+
+    #only the first geometry is output
+    gc.as_georss(:dialect => :simple,:floor => 4).gsub("\n","").should eql("<georss:point floor=\"4\">45.4 4.67</georss:point>")
+    gc.as_georss(:dialect => :w3cgeo).gsub("\n","").should eql("<geo:lat>45.4</geo:lat><geo:long>4.67</geo:long>")
+    gc.as_georss(:dialect => :gml).gsub("\n","").should eql("<georss:where><gml:Point><gml:pos>45.4 4.67</gml:pos></gml:Point></georss:where>")
+
+    gc.as_kml(:id => "HOYOYO-42").gsub("\n","").should eql("<MultiGeometry id=\"HOYOYO-42\"><Point><coordinates>4.67,45.4</coordinates></Point><LineString><coordinates>5.7,12.45 67.55,54</coordinates></LineString></MultiGeometry>")
+  end
+
+  it "test_envelope" do
+    linear_ring1 = GeoRuby::SimpleFeatures::LinearRing.from_coordinates([[12,-45,5],[45,41,6],[4,1,8],[12.4,-45,3]],256,true)
+    linear_ring2 = GeoRuby::SimpleFeatures::LinearRing.from_coordinates([[2,5,9],[5.4,1,-5.4],[14,1,34],[2,5,3]],256,true)
+    polygon = GeoRuby::SimpleFeatures::Polygon.from_linear_rings([linear_ring1,linear_ring2],256,true)
+
+    e = polygon.envelope
+
+    e.as_georss(:dialect => :simple).gsub("\n","").should eql("<georss:box>-45 4 41 45</georss:box>")
+    #center
+    e.as_georss(:dialect => :w3cgeo).gsub("\n","").should eql("<geo:lat>-2</geo:lat><geo:long>24</geo:long>")
+    e.as_georss(:dialect => :gml).gsub("\n","").should eql("<georss:where><gml:Envelope><gml:LowerCorner>-45 4</gml:LowerCorner><gml:UpperCorner>41 45</gml:UpperCorner></gml:Envelope></georss:where>")
+
+    e.as_kml.gsub("\n","").should eql("<LatLonAltBox><north>41</north><south>-45</south><east>45</east><west>4</west><minAltitude>-5.4</minAltitude><maxAltitude>34</maxAltitude></LatLonAltBox>")
+  end
+
+  it "test_point_georss_read" do
+    #W3CGeo
+    str = "   <geo:lat >12.3</geo:lat >\n\t  <geo:long>   4.56</geo:long> "
+    geom = GeoRuby::SimpleFeatures::Geometry.from_georss(str)
+    geom.class.should eql(GeoRuby::SimpleFeatures::Point)
+    geom.lat.should eql(12.3)
+    geom.lon.should eql(4.56)
+
+    str = " <geo:Point> \n \t  <geo:long>   4.56</geo:long> \n\t  <geo:lat >12.3</geo:lat > </geo:Point>  "
+    geom = GeoRuby::SimpleFeatures::Geometry.from_georss(str)
+    geom.class.should eql(GeoRuby::SimpleFeatures::Point)
+    geom.lat.should eql(12.3)
+    geom.lon.should eql(4.56)
+
+    #gml
+    str = " <georss:where> \t\r  <gml:Point  > \t <gml:pos> 4 \t 3 </gml:pos> </gml:Point> </georss:where>"
+    geom = GeoRuby::SimpleFeatures::Geometry.from_georss(str)
+    geom.class.should eql(GeoRuby::SimpleFeatures::Point)
+    geom.lat.should eql(4.0)
+    geom.lon.should eql(3.0)
+
+    #simple
+    str = "<georss:point > 4 \r\t  3 \t</georss:point >"
+    geom  = GeoRuby::SimpleFeatures::Geometry.from_georss(str)
+    geom.class.should eql(GeoRuby::SimpleFeatures::Point)
+    geom.lat.should eql(4.0)
+    geom.lon.should eql(3.0)
+
+    #simple with tags
+    str = "<georss:point featuretypetag=\"hoyoyo\"  elev=\"45.7\" \n floor=\"2\" relationshiptag=\"puyopuyo\" radius=\"42\" > 4 \n 3 \t</georss:point >"
+    geom,tags = GeoRuby::SimpleFeatures::Geometry.from_georss_with_tags(str)
+    geom.class.should eql(GeoRuby::SimpleFeatures::Point)
+    geom.lat.should eql(4.0)
+    geom.lon.should eql(3.0)
+    tags.featuretypetag.should eql("hoyoyo")
+    tags.elev.should eql(45.7)
+    tags.relationshiptag.should eql("puyopuyo")
+    tags.floor.should eql(2)
+    tags.radius.should eql(42.0)
+  end
+
+  it "test_line_string_georss_read" do
+    ls = GeoRuby::SimpleFeatures::LineString.from_points([GeoRuby::SimpleFeatures::Point.from_lon_lat(12.4,-45.3),GeoRuby::SimpleFeatures::Point.from_lon_lat(45.4,41.6)])
+
+    str = "<georss:line > -45.3 12.4 \n \r41.6\t 45.4</georss:line>"
+    geom  = GeoRuby::SimpleFeatures::Geometry.from_georss(str)
+    geom.class.should eql(GeoRuby::SimpleFeatures::LineString)
+    ls.should == geom
+
+    str = "<georss:where><gml:LineString><gml:posList>-45.3 12.4 41.6 45.4</gml:posList></gml:LineString></georss:where>"
+    geom = GeoRuby::SimpleFeatures::Geometry.from_georss(str)
+    geom.class.should eql(GeoRuby::SimpleFeatures::LineString)
+    ls.should == geom
+  end
+
+  it "test_polygon_georss_read" do
+    linear_ring = GeoRuby::SimpleFeatures::LinearRing.from_coordinates([[12.4,-45.3],[45.4,41.6],[4.456,1.0698],[12.4,-45.3]])
+    polygon = GeoRuby::SimpleFeatures::Polygon.from_linear_rings([linear_ring])
+
+    str = "<hoyoyo:polygon featuretypetag=\"42\"  > -45.3 12.4 41.6 \n\r 45.4 1.0698 \r 4.456 -45.3 12.4 </hoyoyo:polygon>"
+    geom = GeoRuby::SimpleFeatures::Geometry.from_georss(str)
+    geom.class.should eql(GeoRuby::SimpleFeatures::Polygon)
+    polygon.should == geom
+
+    str = "<georss:where>\r\t \n  <gml:Polygon><gml:exterior>   <gml:LinearRing><gml:posList> -45.3 \n\r 12.4 41.6 \n\t 45.4 1.0698 4.456 -45.3 12.4</gml:posList></gml:LinearRing></gml:exterior></gml:Polygon></georss:where>"
+    geom = GeoRuby::SimpleFeatures::Geometry.from_georss(str)
+    geom.class.should eql(GeoRuby::SimpleFeatures::Polygon)
+    polygon.should == geom
+  end
+
+  it "test_envelope_georss_read" do
+    e = GeoRuby::SimpleFeatures::Envelope.from_coordinates([[4.456,-45.3],[45.4,41.6]])
+
+    str = "<georss:box  >-45.3 4.456 \n41.6 45.4</georss:box>"
+    geom = GeoRuby::SimpleFeatures::Geometry.from_georss(str)
+    geom.class.should eql(GeoRuby::SimpleFeatures::Envelope)
+    geom.should == e
+
+    str = "<georss:where><gml:Envelope><gml:lowerCorner>-45.3 \n 4.456</gml:lowerCorner><gml:upperCorner>41.6 \t\n 45.4</gml:upperCorner></gml:Envelope></georss:where>"
+    geom = GeoRuby::SimpleFeatures::Geometry.from_georss(str)
+    geom.class.should eql(GeoRuby::SimpleFeatures::Envelope)
+    geom.should == e
+  end
+
+  it "test_kml_read" do
+    g = GeoRuby::SimpleFeatures::Geometry.from_kml("<Point><coordinates>45,12,25</coordinates></Point>")
+    g.should be_a GeoRuby::SimpleFeatures::Point
+    g.should == GeoRuby::SimpleFeatures::Point.from_x_y_z('45','12','25')
+
+    g = GeoRuby::SimpleFeatures::Geometry.from_kml("<LineString>
+      <extrude>1</extrude>
+      <tessellate>1</tessellate>
+      <coordinates>
+        -122.364383,37.824664,0 -122.364152,37.824322,0
+      </coordinates>
+    </LineString>")
+    g.should be_a GeoRuby::SimpleFeatures::LineString
+    g.length.should eql(2)
+    g.should == GeoRuby::SimpleFeatures::LineString.from_points([GeoRuby::SimpleFeatures::Point.from_x_y_z('-122.364383','37.824664','0'),GeoRuby::SimpleFeatures::Point.from_x_y_z('-122.364152','37.824322','0')],4326,true)
+
+    g = GeoRuby::SimpleFeatures::Geometry.from_kml("<Polygon>
+      <extrude>1</extrude>
+      <altitudeMode>relativeToGround</altitudeMode>
+      <outerBoundaryIs>
+        <LinearRing>
+          <coordinates>
+            -122.366278,37.818844,30
+            -122.365248,37.819267,30
+            -122.365640,37.819861,30
+            -122.366669,37.819429,30
+            -122.366278,37.818844,30
+          </coordinates>
+        </LinearRing>
+      </outerBoundaryIs>
+      <innerBoundaryIs>
+        <LinearRing>
+          <coordinates>
+            -122.366212,37.818977,30
+            -122.365424,37.819294,30
+            -122.365704,37.819731,30
+            -122.366488,37.819402,30
+            -122.366212,37.818977,30
+          </coordinates>
+        </LinearRing>
+      </innerBoundaryIs>
+      <innerBoundaryIs>
+        <LinearRing>
+          <coordinates>
+            -122.366212,37.818977,30
+            -122.365424,37.819294,30
+            -122.365704,37.819731,30
+            -122.366488,37.819402,30
+            -122.366212,37.818977,30
+          </coordinates>
+        </LinearRing>
+      </innerBoundaryIs>
+    </Polygon>")
+    g.should be_a GeoRuby::SimpleFeatures::Polygon
+    g.length.should eql(3)
+  end
+
+  it "test_to_kml_for_point_does_not_raise_type_error_if_geom_data_not_provided" do
+    point = GeoRuby::SimpleFeatures::Point.from_coordinates([1.6,2.8],123)
+    lambda { point.kml_representation }.should_not raise_error(TypeError)
+  end
+
+  it "test_to_kml_for_polygon_does_not_raise_type_error_if_geom_data_not_provided" do
+    polygon =  GeoRuby::SimpleFeatures::Polygon.from_coordinates([[[12.4,-45.3],[45.4,41.6],[4.456,1.0698],[12.4,-45.3]],[[2.4,5.3],[5.4,1.4263],[14.46,1.06],[2.4,5.3]]],256)
+    lambda { polygon.kml_representation }.should_not raise_error(TypeError)
+  end
+
+  it "test_to_kml_for_line_string_does_not_raise_type_error_if_geom_data_not_provided" do
+    ls = GeoRuby::SimpleFeatures::LineString.from_coordinates([[5.7,12.45],[67.55,54]],256)
+    lambda { ls.kml_representation }.should_not raise_error(TypeError)
+  end
 
 end