georuby 2.3.0 → 2.5.1

data/spec/geo_ruby/{gpx4r/gpx_spec.rb → gpx_spec.rb}

@@ -1,87 +1,87 @@
-require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+require 'spec_helper'
 
 describe GeoRuby::Gpx4r do
 
   it "should add gpx extension and raise if doesn't exists" do
     expect do
-      expect(File).to receive(:exists?).with("short.gpx").and_return(false)
+      expect(File).to receive(:exist?).with('short.gpx').and_return(false)
       expect(GeoRuby::Gpx4r::GpxFile.open('short')).to be_truthy
     end.to raise_error GeoRuby::Gpx4r::MalformedGpxException
   end
 
-  describe "Waypoints" do
+  describe 'Waypoints' do
 
     before(:all) do
-      @gpxfile = GeoRuby::Gpx4r::GpxFile.open(File.dirname(__FILE__) + '/../../data/gpx/short.gpx', :with_z => true, :with_m => true)
-      @gpxfile2 = GeoRuby::Gpx4r::GpxFile.open(File.dirname(__FILE__) + '/../../data/gpx/fells_loop', :with_z => true, :with_m => true)
-      @gpxfile3 = GeoRuby::Gpx4r::GpxFile.open(File.dirname(__FILE__) + '/../../data/gpx/tracktreks.gpx', :with_z => true)
+      @gpxfile = GeoRuby::Gpx4r::GpxFile.open(File.dirname(__FILE__) + '/../data/gpx/short.gpx', with_z: true, with_m: true)
+      @gpxfile2 = GeoRuby::Gpx4r::GpxFile.open(File.dirname(__FILE__) + '/../data/gpx/fells_loop', with_z: true, with_m: true)
+      @gpxfile3 = GeoRuby::Gpx4r::GpxFile.open(File.dirname(__FILE__) + '/../data/gpx/tracktreks.gpx', with_z: true)
     end
 
-    it "should open and parse" do
+    it 'should open and parse' do
       expect(@gpxfile.record_count).to eql(2724)
     end
 
-    it "should open and parse no trkpt one" do
+    it 'should open and parse no trkpt one' do
       expect(@gpxfile2.record_count).to eql(86)
     end
 
-    it "should open and parse 3" do
+    it 'should open and parse 3' do
       expect(@gpxfile3.record_count).to eql(225)
     end
 
-    it "should read X and Y" do
+    it 'should read X and Y' do
       expect(@gpxfile[0].x).to be_within(0.0001).of(9.093942)
       expect(@gpxfile[0].y).to be_within(0.0001).of(48.731813)
     end
 
-    it "should read Z and M" do
+    it 'should read Z and M' do
       expect(@gpxfile[0].z).to eql(468.0)
-      expect(@gpxfile[0].m).to eql("2008-09-07T17:36:57Z")
+      expect(@gpxfile[0].m).to eql('2008-09-07T17:36:57Z')
     end
 
-    it "should read X and Y 2" do
+    it 'should read X and Y 2' do
       expect(@gpxfile2[0].x).to be_within(0.0001).of(-71.119277)
       expect(@gpxfile2[0].y).to be_within(0.0001).of(42.438878)
     end
 
-    it "should read Z and M 2" do
+    it 'should read Z and M 2' do
       expect(@gpxfile2[0].z).to eql(44.586548)
-      expect(@gpxfile2[0].m).to eql("2001-11-28T21:05:28Z")
+      expect(@gpxfile2[0].m).to eql('2001-11-28T21:05:28Z')
     end
 
-    it "should read X and Y 3" do
+    it 'should read X and Y 3' do
       expect(@gpxfile3[0].x).to be_within(0.0001).of(-149.8358011)
       expect(@gpxfile3[0].y).to be_within(0.0001).of(-17.5326508)
     end
 
-    it "should read Z and M 3" do
+    it 'should read Z and M 3' do
       expect(@gpxfile3[0].z).to eql(88.5787354)
       expect(@gpxfile3[0].m).to eql(0.0)
     end
 
-    it "should return it as a linestring" do
+    it 'should return it as a linestring' do
       expect(@gpxfile.as_line_string).to be_instance_of GeoRuby::SimpleFeatures::LineString
       expect(@gpxfile.as_polyline).to be_instance_of GeoRuby::SimpleFeatures::LineString
     end
 
-    it "should return it as a linestring 3" do
+    it 'should return it as a linestring 3' do
       expect(@gpxfile3.as_line_string).to be_instance_of GeoRuby::SimpleFeatures::LineString
       expect(@gpxfile3.as_polyline).to be_instance_of GeoRuby::SimpleFeatures::LineString
     end
 
-    it "should return a envelope" do
+    it 'should return a envelope' do
       expect(@gpxfile.envelope).to be_instance_of GeoRuby::SimpleFeatures::Envelope
       expect(@gpxfile.envelope.lower_corner.x).to be_within(0.001).of(9.08128)
       expect(@gpxfile.envelope.lower_corner.y).to be_within(0.001).of(48.7169)
     end
 
-    it "should return a envelope 3" do
+    it 'should return a envelope 3' do
       expect(@gpxfile3.envelope).to be_instance_of GeoRuby::SimpleFeatures::Envelope
       expect(@gpxfile3.envelope.lower_corner.x).to be_within(0.001).of(-149.8422613)
       expect(@gpxfile3.envelope.lower_corner.y).to be_within(0.001).of(-17.547636)
     end
 
-    it "should return it as a polygon" do
+    it 'should return it as a polygon' do
       [@gpxfile, @gpxfile2, @gpxfile3].each do |g|
         expect(g.as_polygon).to be_instance_of GeoRuby::SimpleFeatures::Polygon
         expect(g.as_polygon[0]).to be_instance_of GeoRuby::SimpleFeatures::LinearRing
@@ -90,13 +90,13 @@ describe GeoRuby::Gpx4r do
       end
     end
 
-    it "should close the polygon" do
+    it 'should close the polygon' do
       se = GeoRuby::SimpleFeatures::Point.from_x_y(-44, -23)
       sw = GeoRuby::SimpleFeatures::Point.from_x_y(-42, -22)
       nw = GeoRuby::SimpleFeatures::Point.from_x_y(-42, -25)
       ne = GeoRuby::SimpleFeatures::Point.from_x_y(-44, -21)
-      @gpxfile.instance_variable_set(:@points, [se,sw,nw,ne])
-      expect(@gpxfile.as_polygon).to eq(GeoRuby::SimpleFeatures::Polygon.from_points([[se,sw,nw,ne,se]]))
+      @gpxfile.instance_variable_set(:@points, [se, sw, nw, ne])
+      expect(@gpxfile.as_polygon).to eq(GeoRuby::SimpleFeatures::Polygon.from_points([[se, sw, nw, ne, se]]))
     end
   end
 

data/spec/geo_ruby/kml_spec.rb

@@ -2,97 +2,101 @@ require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 
 describe GeoRuby::KmlParser do
   before(:all) do
-    POINT = "<Point><coordinates>-82.4898187291883,34.2473206042649</coordinates></Point>"
-    LINESTRING = "<LineString><coordinates>-122.365662,37.826988 -122.365202,37.826302 -122.364581,37.82655 -122.365038,37.827237</coordinates></LineString>"
-    LINEARRING = "<LinearRing><coordinates>-122.365662,37.826988 -122.365202,37.826302 -122.364581,37.82655 -122.365038,37.827237 -122.365662,37.826988</coordinates></LinearRing>"
-    POLYGON = "<Polygon><outerBoundaryIs><LinearRing><coordinates>-82.5961385808407,34.0134202383713 -82.6029437979289,34.0346366848087 -82.6603553035687,34.1083560439036 -82.7357807829899,34.1697961502507 -82.7425935601244,34.2055536194311 -82.3145921793097,34.4812209701586 -82.2758648198483,34.4347213073308 -82.2405017851073,34.4067761024174 -82.3327002190662,34.3417863447576 -82.2910826671599,34.2708004396966 -82.2497468801283,34.2261551348023 -82.2370438982521,34.1709424545969 -82.2569955519648,34.1119142196088 -82.3237086862075,34.0601294413679 -82.368425596693,34.0533120146082 -82.4455985300521,34.0562556252352 -82.4806178108032,34.0759686807282 -82.5334224196077,34.0620944842448 -82.5961385808407,34.0134202383713</coordinates></LinearRing></outerBoundaryIs></Polygon>"
-    COMPLEX_POLYGON = "<Polygon><outerBoundaryIs><LinearRing><coordinates>-122.366278,37.818844 -122.365248,37.819267 -122.36564,37.819861 -122.366669,37.819429 -122.366278,37.818844</coordinates></LinearRing></outerBoundaryIs><innerBoundaryIs><LinearRing><coordinates>-122.366212,37.818977 -122.365424,37.819294 -122.365704,37.819731 -122.366488,37.819402 -122.366212,37.818977</coordinates></LinearRing></innerBoundaryIs></Polygon>"
-    MULTIGEOMETRY = "<MultiGeometry><Polygon><outerBoundaryIs><LinearRing><coordinates>-82.5961385808407,34.0134202383713 -82.6029437979289,34.0346366848087 -82.6603553035687,34.1083560439036 -82.7357807829899,34.1697961502507 -82.7425935601244,34.2055536194311 -82.3145921793097,34.4812209701586 -82.2758648198483,34.4347213073308 -82.2405017851073,34.4067761024174 -82.3327002190662,34.3417863447576 -82.2910826671599,34.2708004396966 -82.2497468801283,34.2261551348023 -82.2370438982521,34.1709424545969 -82.2569955519648,34.1119142196088 -82.3237086862075,34.0601294413679 -82.368425596693,34.0533120146082 -82.4455985300521,34.0562556252352 -82.4806178108032,34.0759686807282 -82.5334224196077,34.0620944842448 -82.5961385808407,34.0134202383713</coordinates></LinearRing></outerBoundaryIs></Polygon><Point><coordinates>-82.4898187291883,34.2473206042649</coordinates></Point></MultiGeometry>"
-    POINT3D = "<Point><coordinates>-82.4898187291883,34.2473206042649,5</coordinates></Point>"
-    LINESTRING3D = "<LineString><coordinates>-122.365662,37.826988,1 -122.365202,37.826302,2 -122.364581,37.82655,3 -122.365038,37.827237,4</coordinates></LineString>"
+    POINT = '<Point><coordinates>-82.4898187291883,34.2473206042649</coordinates></Point>'
+    LINESTRING = '<LineString><coordinates>-122.365662,37.826988 -122.365202,37.826302 -122.364581,37.82655 -122.365038,37.827237</coordinates></LineString>'
+    LINEARRING = '<LinearRing><coordinates>-122.365662,37.826988 -122.365202,37.826302 -122.364581,37.82655 -122.365038,37.827237 -122.365662,37.826988</coordinates></LinearRing>'
+    POLYGON = '<Polygon><outerBoundaryIs><LinearRing><coordinates>-82.5961385808407,34.0134202383713 -82.6029437979289,34.0346366848087 -82.6603553035687,34.1083560439036 -82.7357807829899,34.1697961502507 -82.7425935601244,34.2055536194311 -82.3145921793097,34.4812209701586 -82.2758648198483,34.4347213073308 -82.2405017851073,34.4067761024174 -82.3327002190662,34.3417863447576 -82.2910826671599,34.2708004396966 -82.2497468801283,34.2261551348023 -82.2370438982521,34.1709424545969 -82.2569955519648,34.1119142196088 -82.3237086862075,34.0601294413679 -82.368425596693,34.0533120146082 -82.4455985300521,34.0562556252352 -82.4806178108032,34.0759686807282 -82.5334224196077,34.0620944842448 -82.5961385808407,34.0134202383713</coordinates></LinearRing></outerBoundaryIs></Polygon>'
+    COMPLEX_POLYGON = '<Polygon><outerBoundaryIs><LinearRing><coordinates>-122.366278,37.818844 -122.365248,37.819267 -122.36564,37.819861 -122.366669,37.819429 -122.366278,37.818844</coordinates></LinearRing></outerBoundaryIs><innerBoundaryIs><LinearRing><coordinates>-122.366212,37.818977 -122.365424,37.819294 -122.365704,37.819731 -122.366488,37.819402 -122.366212,37.818977</coordinates></LinearRing></innerBoundaryIs></Polygon>'
+    MULTIGEOMETRY = '<MultiGeometry><Polygon><outerBoundaryIs><LinearRing><coordinates>-82.5961385808407,34.0134202383713 -82.6029437979289,34.0346366848087 -82.6603553035687,34.1083560439036 -82.7357807829899,34.1697961502507 -82.7425935601244,34.2055536194311 -82.3145921793097,34.4812209701586 -82.2758648198483,34.4347213073308 -82.2405017851073,34.4067761024174 -82.3327002190662,34.3417863447576 -82.2910826671599,34.2708004396966 -82.2497468801283,34.2261551348023 -82.2370438982521,34.1709424545969 -82.2569955519648,34.1119142196088 -82.3237086862075,34.0601294413679 -82.368425596693,34.0533120146082 -82.4455985300521,34.0562556252352 -82.4806178108032,34.0759686807282 -82.5334224196077,34.0620944842448 -82.5961385808407,34.0134202383713</coordinates></LinearRing></outerBoundaryIs></Polygon><Point><coordinates>-82.4898187291883,34.2473206042649</coordinates></Point></MultiGeometry>'
+    POINT3D = '<Point><coordinates>-82.4898187291883,34.2473206042649,5</coordinates></Point>'
+    LINESTRING3D = '<LineString><coordinates>-122.365662,37.826988,1 -122.365202,37.826302,2 -122.364581,37.82655,3 -122.365038,37.827237,4</coordinates></LineString>'
   end
-  
+
   before(:each) do
     @factory = GeoRuby::SimpleFeatures::GeometryFactory.new
     @kml_parser = described_class.new(@factory)
   end
-  
-  it "should parse a GeoRuby::SimpleFeatures::Point correctly" do
+
+  it 'should parse a GeoRuby::SimpleFeatures::Point correctly' do
     @kml_parser.parse(POINT)
     g = @factory.geometry
     expect(g).not_to eql(nil)
     expect(g).to be_an_instance_of(GeoRuby::SimpleFeatures::Point)
-    expect(g.as_kml.gsub(/\n/,'')).to eql(POINT) 
+    expect(g.as_kml.gsub(/\n/, '')).to eql(POINT)
   end
-  
-  it "should parse a GeoRuby::SimpleFeatures::LineString correctly" do
+
+  it 'should parse a GeoRuby::SimpleFeatures::LineString correctly' do
     @kml_parser.parse(LINESTRING)
     g = @factory.geometry
     expect(g).not_to eql(nil)
     expect(g).to be_an_instance_of(GeoRuby::SimpleFeatures::LineString)
-    expect(g.as_kml.gsub(/\n/,'')).to eql(LINESTRING)
+    expect(g.as_kml.gsub(/\n/, '')).to eql(LINESTRING)
 
     @kml_parser.parse(LINEARRING)
     g = @factory.geometry
     expect(g).not_to eql(nil)
     expect(g).to be_an_instance_of(GeoRuby::SimpleFeatures::LinearRing)
-    expect(g.as_kml.gsub(/\n/,'')).to eql(LINEARRING)
+    expect(g.as_kml.gsub(/\n/, '')).to eql(LINEARRING)
   end
-  
-  it "should parse a GeoRuby::SimpleFeatures::Polygon correctly" do
+
+  it 'should parse a GeoRuby::SimpleFeatures::Polygon correctly' do
     @kml_parser.parse(POLYGON)
     g = @factory.geometry
     expect(g).not_to eql(nil)
     expect(g).to be_an_instance_of(GeoRuby::SimpleFeatures::Polygon)
-    expect(g.as_kml.gsub(/\n/,'')).to eql(POLYGON)
+    expect(g.as_kml.gsub(/\n/, '')).to eql(POLYGON)
 
     @kml_parser.parse(COMPLEX_POLYGON)
     g = @factory.geometry
     expect(g).not_to eql(nil)
     expect(g).to be_an_instance_of(GeoRuby::SimpleFeatures::Polygon)
-    expect(g.as_kml.gsub(/\n/,'')).to eql(COMPLEX_POLYGON)
+    expect(g.as_kml.gsub(/\n/, '')).to eql(COMPLEX_POLYGON)
   end
-  
-  it "should parse a MultiGeometry correctly" do
+
+  it 'should parse a MultiGeometry correctly' do
     @kml_parser.parse(MULTIGEOMETRY)
     g = @factory.geometry
     expect(g).not_to eql(nil)
     expect(g.geometries.length).to eql(2)
     expect(g).to be_an_instance_of(GeoRuby::SimpleFeatures::GeometryCollection)
-    expect(g.as_kml.gsub(/\n/,'')).to eql(MULTIGEOMETRY)
+    expect(g.as_kml.gsub(/\n/, '')).to eql(MULTIGEOMETRY)
   end
-  
-  it "should parse 3D geometries correctly" do
-    # not testing generation because GeoRuby kml generation logic currently requires additional
-    # XML nodes to actually output 3D coordinate information.  I might modify that behavior
+
+  it 'should parse 3D geometries correctly' do
+    # not testing generation because GeoRuby kml generation logic currently
+    # requires additional XML nodes to actually output 3D coordinate
+    # information. I might modify that behavior.
     g = @kml_parser.parse(POINT3D)
     expect(g).not_to eql(nil)
     expect(g.with_z).to eql(true)
-    # g.as_kml(:altitude_mode => "clampToGround").gsub(/\n/,'').should eql(POINT3D)
+    # g.as_kml(:altitude_mode => "clampToGround").gsub(/\n/,'')
+    #  .should eql(POINT3D)
 
     g = @kml_parser.parse(LINESTRING3D)
     expect(g).not_to eql(nil)
     expect(g.with_z).to eql(true)
-    # g.as_kml(:altitude_mode => "clampToGround").gsub(/\n/,'').should eql(LINESTRING3D)
+    # g.as_kml(:altitude_mode => "clampToGround").gsub(/\n/,'')
+    #  .should eql(LINESTRING3D)
   end
 
-  it "should yield a geometries with functional bounding boxes" do
-    [LINESTRING, LINEARRING, POLYGON, COMPLEX_POLYGON, MULTIGEOMETRY, LINESTRING3D].each do |kml|
+  it 'should yield a geometries with functional bounding boxes' do
+    [LINESTRING, LINEARRING, POLYGON, COMPLEX_POLYGON,
+      MULTIGEOMETRY, LINESTRING3D].each do |kml|
       @kml_parser.parse(kml)
       g = @factory.geometry
-      expect {
+      expect do
         g.bounding_box
-      }.not_to raise_error
+      end.not_to raise_error
     end
   end
 
-  it "should yield a geometries with functional envelopes" do
+  it 'should yield a geometries with functional envelopes' do
     [LINESTRING, LINEARRING, POLYGON, COMPLEX_POLYGON, MULTIGEOMETRY, LINESTRING3D].each do |kml|
       @kml_parser.parse(kml)
       g = @factory.geometry
-      expect {
+      expect do
         g.envelope.center
-      }.not_to raise_error
+      end.not_to raise_error
     end
   end
 end

data/spec/geo_ruby/shp4r/shp_spec.rb

@@ -2,119 +2,119 @@ require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
 
 describe GeoRuby::Shp4r do
 
-  describe "Point" do
+  describe 'Point' do
     before(:each) do
       @shpfile = GeoRuby::Shp4r::ShpFile.open(File.dirname(__FILE__) + '/../../data/point.shp')
     end
 
-    it "should parse ok" do
+    it 'should parse ok' do
       expect(@shpfile.record_count).to eql(2)
       expect(@shpfile.fields.size).to eq(1)
       expect(@shpfile.shp_type).to eql(GeoRuby::Shp4r::ShpType::POINT)
     end
 
-    it "should parse fields" do
+    it 'should parse fields' do
       field = @shpfile.fields.first
-      expect(field.name).to eql("Hoyoyo")
-      expect(field.type).to eql("N")
+      expect(field.name).to eql('Hoyoyo')
+      expect(field.type).to eql('N')
     end
 
-    it "should parse record 1" do
+    it 'should parse record 1' do
       rec = @shpfile[0]
       expect(rec.geometry).to be_kind_of GeoRuby::SimpleFeatures::Point
       expect(rec.geometry.x).to be_within(0.00001).of(-90.08375)
       expect(rec.geometry.y).to be_within(0.00001).of(34.39996)
-      expect(rec.data["Hoyoyo"]).to eql(6)
+      expect(rec.data['Hoyoyo']).to eql(6)
     end
 
-    it "should parse record 2" do
+    it 'should parse record 2' do
       rec = @shpfile[1]
       expect(rec.geometry).to be_kind_of GeoRuby::SimpleFeatures::Point
       expect(rec.geometry.x).to be_within(0.00001).of(-87.82580)
       expect(rec.geometry.y).to be_within(0.00001).of(33.36416)
-      expect(rec.data["Hoyoyo"]).to eql(9)
+      expect(rec.data['Hoyoyo']).to eql(9)
     end
 
   end
 
-  describe "Polyline" do
+  describe 'Polyline' do
     before(:each) do
       @shpfile = GeoRuby::Shp4r::ShpFile.open(File.dirname(__FILE__) + '/../../data/polyline.shp')
     end
 
-    it "should parse ok" do
+    it 'should parse ok' do
       expect(@shpfile.record_count).to eql(1)
       expect(@shpfile.fields.size).to eq(1)
       expect(@shpfile.shp_type).to eql(GeoRuby::Shp4r::ShpType::POLYLINE)
     end
 
-    it "should parse fields" do
+    it 'should parse fields' do
       field = @shpfile.fields.first
-      expect(field.name).to eql("Chipoto")
+      expect(field.name).to eql('Chipoto')
       # GeoRuby::Shp4r::Dbf now uses the decimal to choose between int and float
       # So here is N instead of F
-      expect(field.type).to eql("N")
+      expect(field.type).to eql('N')
     end
 
-    it "should parse record 1" do
+    it 'should parse record 1' do
       rec = @shpfile[0]
       expect(rec.geometry).to be_kind_of GeoRuby::SimpleFeatures::MultiLineString
       expect(rec.geometry.length).to eql(1)
       expect(rec.geometry[0].length).to eql(6)
-      expect(rec.data["Chipoto"]).to eql(5.678)
+      expect(rec.data['Chipoto']).to eql(5.678)
     end
 
   end
 
-  describe "Polygon" do
+  describe 'Polygon' do
     before(:each) do
       @shpfile = GeoRuby::Shp4r::ShpFile.open(File.dirname(__FILE__) + '/../../data/polygon.shp')
     end
 
-    it "should parse ok" do
+    it 'should parse ok' do
       expect(@shpfile.record_count).to eql(1)
       expect(@shpfile.fields.size).to eq(1)
       expect(@shpfile.shp_type).to eql(GeoRuby::Shp4r::ShpType::POLYGON)
     end
 
-    it "should parse fields" do
+    it 'should parse fields' do
       field = @shpfile.fields.first
-      expect(field.name).to eql("Hello")
-      expect(field.type).to eql("C")
+      expect(field.name).to eql('Hello')
+      expect(field.type).to eql('C')
     end
 
-    it "should parse record 1" do
+    it 'should parse record 1' do
       rec = @shpfile[0]
       expect(rec.geometry).to be_kind_of GeoRuby::SimpleFeatures::MultiPolygon
       expect(rec.geometry.length).to eql(1)
       expect(rec.geometry[0].length).to eql(1)
       expect(rec.geometry[0][0].length).to eql(7)
-      expect(rec.data["Hello"]).to eql("Bouyoul!")
+      expect(rec.data['Hello']).to eql('Bouyoul!')
     end
   end
 
-  describe "Write" do
-    def cp_all_shp(file1,file2)
-      FileUtils.copy(file1 + ".shp",file2 + ".shp")
-      FileUtils.copy(file1 + ".shx",file2 + ".shx")
-      FileUtils.copy(file1 + ".dbf",file2 + ".dbf")
+  describe 'Write' do
+    def cp_all_shp(file1, file2)
+      FileUtils.copy(file1 + '.shp', file2 + '.shp')
+      FileUtils.copy(file1 + '.shx', file2 + '.shx')
+      FileUtils.copy(file1 + '.dbf', file2 + '.dbf')
     end
 
     def rm_all_shp(file)
-      FileUtils.rm(file + ".shp")
-      FileUtils.rm(file + ".shx")
-      FileUtils.rm(file + ".dbf")
+      FileUtils.rm(file + '.shp')
+      FileUtils.rm(file + '.shx')
+      FileUtils.rm(file + '.dbf')
     end
 
-    it "test_point" do
+    it 'test_point' do
       cp_all_shp(File.dirname(__FILE__) + '/../../data/point',
                  File.dirname(__FILE__) + '/../../data/point2')
       shpfile = GeoRuby::Shp4r::ShpFile.open(File.dirname(__FILE__) + '/../../data/point2.shp')
 
       shpfile.transaction do |tr|
         expect(tr).to be_instance_of GeoRuby::Shp4r::ShpTransaction
-        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::Point.from_x_y(123.4,123.4),'Hoyoyo' => 5))
-        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::Point.from_x_y(-16.67,16.41),'Hoyoyo' => -7))
+        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::Point.from_x_y(123.4, 123.4), 'Hoyoyo' => 5))
+        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::Point.from_x_y(-16.67, 16.41), 'Hoyoyo' => -7))
         tr.delete(1)
       end
 
@@ -124,7 +124,7 @@ describe GeoRuby::Shp4r do
       rm_all_shp(File.dirname(__FILE__) + '/../../data/point2')
     end
 
-    it "test_linestring" do
+    it 'test_linestring' do
       cp_all_shp(File.dirname(__FILE__) + '/../../data/polyline',
                  File.dirname(__FILE__) + '/../../data/polyline2')
 
@@ -132,8 +132,8 @@ describe GeoRuby::Shp4r do
 
       shpfile.transaction do |tr|
         expect(tr).to be_instance_of GeoRuby::Shp4r::ShpTransaction
-        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::LineString.from_coordinates([[123.4,123.4],[45.6,12.3]]),'Chipoto' => 5.6778))
-        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::LineString.from_coordinates([[23.4,13.4],[45.6,12.3],[12,-67]]),'Chipoto' => -7.1))
+        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::LineString.from_coordinates([[123.4, 123.4], [45.6, 12.3]]), 'Chipoto' => 5.6778))
+        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::LineString.from_coordinates([[23.4, 13.4], [45.6, 12.3], [12, -67]]), 'Chipoto' => -7.1))
         tr.delete(0)
       end
 
@@ -142,7 +142,7 @@ describe GeoRuby::Shp4r do
       rm_all_shp(File.dirname(__FILE__) + '/../../data/polyline2')
     end
 
-    it "test_polygon" do
+    it 'test_polygon' do
       cp_all_shp(File.dirname(__FILE__) + '/../../data/polygon',
                  File.dirname(__FILE__) + '/../../data/polygon2')
       shpfile = GeoRuby::Shp4r::ShpFile.open(File.dirname(__FILE__) + '/../../data/polygon2.shp')
@@ -150,7 +150,7 @@ describe GeoRuby::Shp4r do
       shpfile.transaction do |tr|
         expect(tr).to be_instance_of GeoRuby::Shp4r::ShpTransaction
         tr.delete(0)
-        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::Polygon.from_coordinates([[[0,0],[40,0],[40,40],[0,40],[0,0]],[[10,10],[10,20],[20,20],[10,10]]]),'Hello' => "oook"))
+        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::Polygon.from_coordinates([[[0, 0], [40, 0], [40, 40], [0, 40], [0, 0]], [[10, 10], [10, 20], [20, 20], [10, 10]]]), 'Hello' => 'oook'))
       end
 
       expect(shpfile.record_count).to eql(1)
@@ -159,14 +159,14 @@ describe GeoRuby::Shp4r do
       rm_all_shp(File.dirname(__FILE__) + '/../../data/polygon2')
     end
 
-    it "test_multipoint" do
+    it 'test_multipoint' do
       cp_all_shp(File.dirname(__FILE__) + '/../../data/multipoint',
                  File.dirname(__FILE__) + '/../../data/multipoint2')
       shpfile = GeoRuby::Shp4r::ShpFile.open(File.dirname(__FILE__) + '/../../data/multipoint2.shp')
 
       shpfile.transaction do |tr|
         expect(tr).to be_instance_of GeoRuby::Shp4r::ShpTransaction
-        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::MultiPoint.from_coordinates([[45.6,-45.1],[12.4,98.2],[51.2,-0.12],[156.12345,56.109]]),'Hello' => 5,"Hoyoyo" => "AEZAE"))
+        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::MultiPoint.from_coordinates([[45.6, -45.1], [12.4, 98.2], [51.2, -0.12], [156.12345, 56.109]]), 'Hello' => 5, 'Hoyoyo' => 'AEZAE'))
       end
 
       expect(shpfile.record_count).to eql(2)
@@ -175,7 +175,7 @@ describe GeoRuby::Shp4r do
       rm_all_shp(File.dirname(__FILE__) + '/../../data/multipoint2')
     end
 
-    it "test_multi_polygon" do
+    it 'test_multi_polygon' do
       cp_all_shp(File.dirname(__FILE__) + '/../../data/polygon',
                  File.dirname(__FILE__) + '/../../data/polygon4')
 
@@ -183,7 +183,7 @@ describe GeoRuby::Shp4r do
 
       shpfile.transaction do |tr|
         expect(tr).to be_instance_of GeoRuby::Shp4r::ShpTransaction
-        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::MultiPolygon.from_polygons([GeoRuby::SimpleFeatures::Polygon.from_coordinates([[[0,0],[40,0],[40,40],[0,40],[0,0]],[[10,10],[10,20],[20,20],[10,10]]])]),'Hello' => "oook"))
+        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::MultiPolygon.from_polygons([GeoRuby::SimpleFeatures::Polygon.from_coordinates([[[0, 0], [40, 0], [40, 40], [0, 40], [0, 0]], [[10, 10], [10, 20], [20, 20], [10, 10]]])]), 'Hello' => 'oook'))
       end
 
       expect(shpfile.record_count).to eql(2)
@@ -193,7 +193,7 @@ describe GeoRuby::Shp4r do
       rm_all_shp(File.dirname(__FILE__) + '/../../data/polygon4')
     end
 
-    it "test_rollback" do
+    it 'test_rollback' do
       cp_all_shp(File.dirname(__FILE__) + '/../../data/polygon',
                  File.dirname(__FILE__) + '/../../data/polygon5')
 
@@ -201,7 +201,7 @@ describe GeoRuby::Shp4r do
 
       shpfile.transaction do |tr|
         expect(tr).to be_instance_of GeoRuby::Shp4r::ShpTransaction
-        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::MultiPolygon.from_polygons([GeoRuby::SimpleFeatures::Polygon.from_coordinates([[[0,0],[40,0],[40,40],[0,40],[0,0]],[[10,10],[10,20],[20,20],[10,10]]])]),'Hello' => "oook"))
+        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::MultiPolygon.from_polygons([GeoRuby::SimpleFeatures::Polygon.from_coordinates([[[0, 0], [40, 0], [40, 40], [0, 40], [0, 0]], [[10, 10], [10, 20], [20, 20], [10, 10]]])]), 'Hello' => 'oook'))
         tr.rollback
       end
       expect(shpfile.record_count).to eql(1)
@@ -212,20 +212,20 @@ describe GeoRuby::Shp4r do
 
     end
 
-    it "test_creation" do
-      shpfile = GeoRuby::Shp4r::ShpFile.create(File.dirname(__FILE__) + '/../../data/point3.shp',GeoRuby::Shp4r::ShpType::POINT,[GeoRuby::Shp4r::Dbf::Field.new("Hoyoyo","C",10,0)])
+    it 'test_creation' do
+      shpfile = GeoRuby::Shp4r::ShpFile.create(File.dirname(__FILE__) + '/../../data/point3.shp', GeoRuby::Shp4r::ShpType::POINT, [GeoRuby::Shp4r::Dbf::Field.new('Hoyoyo', 'C', 10, 0)])
       shpfile.transaction do |tr|
-        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::Point.from_x_y(123,123.4),'Hoyoyo' => "HJHJJ"))
+        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::Point.from_x_y(123, 123.4), 'Hoyoyo' => 'HJHJJ'))
       end
       expect(shpfile.record_count).to eql(1)
       shpfile.close
       rm_all_shp(File.dirname(__FILE__) + '/../../data/point3')
     end
 
-    it "test_creation_multipoint" do
-      shpfile = GeoRuby::Shp4r::ShpFile.create(File.dirname(__FILE__) + '/../../data/multipoint3.shp',GeoRuby::Shp4r::ShpType::MULTIPOINT,[GeoRuby::Shp4r::Dbf::Field.new("Hoyoyo","C",10),GeoRuby::Shp4r::Dbf::Field.new("Hello","N",10)])
+    it 'test_creation_multipoint' do
+      shpfile = GeoRuby::Shp4r::ShpFile.create(File.dirname(__FILE__) + '/../../data/multipoint3.shp', GeoRuby::Shp4r::ShpType::MULTIPOINT, [GeoRuby::Shp4r::Dbf::Field.new('Hoyoyo', 'C', 10), GeoRuby::Shp4r::Dbf::Field.new('Hello', 'N', 10)])
       shpfile.transaction do |tr|
-        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::MultiPoint.from_coordinates([[123,123.4],[345,12.2]]),'Hoyoyo' => "HJHJJ","Hello" => 5))
+        tr.add(GeoRuby::Shp4r::ShpRecord.new(GeoRuby::SimpleFeatures::MultiPoint.from_coordinates([[123, 123.4], [345, 12.2]]), 'Hoyoyo' => 'HJHJJ', 'Hello' => 5))
       end
       expect(shpfile.record_count).to eql(1)
       shpfile.close