ffi-gdal 1.0.0.beta3 → 1.0.0.beta4

data/spec/unit/ogr/field_definition_spec.rb

@@ -0,0 +1,135 @@
+require 'spec_helper'
+require 'ogr/field_definition'
+
+RSpec.describe OGR::FieldDefinition do
+  subject(:field) { described_class.new('test field', :OFTInteger) }
+
+  describe '#set' do
+    before do
+      subject.set('new name', :OFTString, 5, 2, :OJRight)
+    end
+
+    it 'sets the name' do
+      expect(subject.name).to eq 'new name'
+    end
+
+    it 'sets the width' do
+      expect(subject.width).to eq 5
+    end
+
+    it 'sets the precision' do
+      expect(subject.precision).to eq 2
+    end
+
+    it 'sets the justification' do
+      expect(subject.justification).to eq :OJRight
+    end
+  end
+
+  describe '#name' do
+    it 'returns the name given during creation' do
+      expect(subject.name).to eq 'test field'
+    end
+  end
+
+  describe '#name= + #name' do
+    it 'assigns the name' do
+      subject.name = 'new test name'
+      expect(subject.name).to eq 'new test name'
+    end
+  end
+
+  describe '#justification' do
+    context 'default' do
+      it 'returns :OJUndefined' do
+        expect(subject.justification).to eq :OJUndefined
+      end
+    end
+  end
+
+  describe '#justification= + #justification' do
+    it 'assigns the justification' do
+      subject.justification = :OJLeft
+      expect(subject.justification).to eq :OJLeft
+    end
+  end
+
+  describe '#precision' do
+    context 'default' do
+      it 'returns 0' do
+        expect(subject.precision).to be_zero
+      end
+    end
+  end
+
+  describe '#precision= + #precision' do
+    it 'assigns the precision' do
+      subject.precision = 1
+      expect(subject.precision).to eq 1
+    end
+  end
+
+  describe '#type' do
+    context 'default' do
+      it 'returns the value it was created with' do
+        expect(subject.type).to eq :OFTInteger
+      end
+    end
+  end
+
+  describe '#type= + #type' do
+    it 'assigns the type' do
+      subject.type = :OFTString
+      expect(subject.type).to eq :OFTString
+    end
+  end
+
+  describe '#width' do
+    context 'default' do
+      it 'returns 0' do
+        expect(subject.width).to be_zero
+      end
+    end
+  end
+
+  describe '#width= + #width' do
+    it 'assigns the width' do
+      subject.width = 1
+      expect(subject.width).to eq 1
+    end
+  end
+
+  describe '#ignored?' do
+    context 'default' do
+      it 'returns false' do
+        expect(subject).to_not be_ignored
+      end
+    end
+  end
+
+  describe '#ignore= + #ignored?' do
+    it 'assigns the value' do
+      subject.ignore = true
+      expect(subject).to be_ignored
+    end
+  end
+
+  describe '#as_json' do
+    it 'returns a Hash of attributes' do
+      expect(subject.as_json).to eq(
+        is_ignored: false,
+        justification: :OJUndefined,
+        name: 'test field',
+        precision: 0,
+        type: :OFTInteger,
+        width: 0
+      )
+    end
+  end
+
+  describe '#to_json' do
+    it 'returns a string' do
+      expect(subject.to_json).to be_a String
+    end
+  end
+end

data/spec/unit/ogr/field_spec.rb

@@ -0,0 +1,193 @@
+require 'spec_helper'
+require 'ogr/field'
+
+RSpec.describe OGR::Field do
+  describe '#integer' do
+    context 'not set' do
+      it 'returns 0.0' do
+        expect(subject.integer).to eql 0
+      end
+    end
+  end
+
+  describe '#integer= + #integer' do
+    context 'valid int' do
+      it 'sets the value' do
+        subject.integer = 1
+        expect(subject.integer).to eq 1
+      end
+    end
+  end
+
+  describe '#integer64' do
+    context 'not set' do
+      it 'returns 0.0' do
+        expect(subject.integer64).to eql 0
+      end
+    end
+  end
+
+  describe '#integer64= + #integer64' do
+    context 'valid int64' do
+      it 'sets the value' do
+        subject.integer64 = 2**32 + 1
+        expect(subject.integer64).to eq 2**32 + 1
+      end
+    end
+  end
+
+  describe '#real' do
+    context 'not set' do
+      it 'returns 0.0' do
+        expect(subject.real).to eql 0.0
+      end
+    end
+  end
+
+  describe '#real= + #real' do
+    context 'valid float' do
+      it 'sets the value' do
+        subject.real = 1.23
+        expect(subject.real).to eq 1.23
+      end
+    end
+  end
+
+  describe '#string' do
+    context 'not set' do
+      it 'returns an empty string' do
+        expect(subject.string).to eq ''
+      end
+    end
+  end
+
+  describe '#string= + #string' do
+    context 'valid string' do
+      it 'sets the value' do
+        subject.string = 'meow'
+        expect(subject.string).to eq 'meow'
+      end
+    end
+  end
+
+  describe '#integer_list' do
+    context 'not set' do
+      it 'returns an empty array' do
+        expect(subject.integer_list).to eq []
+      end
+    end
+  end
+
+  describe '#integer_list= + #integer_list' do
+    context 'valid int array' do
+      it 'sets the value' do
+        subject.integer_list = [1, 2]
+        expect(subject.integer_list).to eq [1, 2]
+      end
+    end
+  end
+
+  describe '#integer64_list' do
+    context 'not set' do
+      it 'returns an empty array' do
+        expect(subject.integer64_list).to eq []
+      end
+    end
+  end
+
+  describe '#integer64_list= + #integer64_list' do
+    context 'valid int64 array' do
+      it 'sets the value' do
+        subject.integer64_list = [2**32 + 1, 2**32 + 2]
+        expect(subject.integer64_list).to eq [2**32 + 1, 2**32 + 2]
+      end
+    end
+  end
+
+  describe '#real_list' do
+    context 'not set' do
+      it 'returns an empty array' do
+        expect(subject.real_list).to eq []
+      end
+    end
+  end
+
+  describe '#real_list= + #real_list' do
+    context 'valid float array' do
+      it 'sets the value' do
+        subject.real_list = [1.5, 6.9]
+        expect(subject.real_list).to eq [1.5, 6.9]
+      end
+    end
+  end
+
+  describe '#string_list' do
+    context 'not set' do
+      it 'returns an empty array' do
+        expect(subject.string_list).to eq []
+      end
+    end
+  end
+
+  describe '#string_list= + #string_list' do
+    context 'valid string array' do
+      it 'sets the value' do
+        subject.string_list = %w[one two]
+        expect(subject.string_list).to eq %w[one two]
+      end
+    end
+  end
+
+  describe '#binary' do
+    context 'not set' do
+      it 'returns an empty string' do
+        expect(subject.binary).to eq ''
+      end
+    end
+  end
+
+  describe '#binary= + #binary' do
+    context 'valid binary' do
+      it 'sets the value' do
+        subject.binary = [1, 2, 3].pack('C*')
+        expect(subject.binary).to eq [1, 2, 3].pack('C*')
+      end
+    end
+  end
+
+  describe '#set' do
+    context 'not set' do
+      it 'returns a Hash with markers set to 0' do
+        expect(subject.set).to eq(marker1: 0, marker2: 0)
+      end
+    end
+  end
+
+  describe '#set= + #set' do
+    context 'valid set hash' do
+      it 'sets the value' do
+        subject.set = { marker1: 1, marker2: 200 }
+        expect(subject.set).to eq(marker1: 1.0, marker2: 200)
+      end
+    end
+  end
+
+  describe '#date' do
+    context 'not set' do
+      it 'returns nil' do
+        expect(subject.date).to be_nil
+      end
+    end
+  end
+
+  describe '#date= + #date' do
+    context 'valid date' do
+      let(:now) { DateTime.now }
+
+      it 'sets the date' do
+        subject.date = now
+        expect(subject.date.httpdate).to eq now.httpdate
+      end
+    end
+  end
+end

data/spec/unit/ogr/geometries/geometry_collection_spec.rb

@@ -0,0 +1,186 @@
+require 'spec_helper'
+require 'ogr/geometry'
+
+RSpec.describe OGR::GeometryCollection do
+  it_behaves_like 'a geometry' do
+    let(:geometry) { described_class.new }
+  end
+
+  it_behaves_like 'a container geometry' do
+    let(:child_geometry) do
+      OGR::Geometry.create_from_wkt('POLYGON ((0 0,0 1,1 1,1 0,0 0))')
+    end
+  end
+
+  describe '#to_polygon' do
+    let(:polygon) do
+      OGR::Geometry.create_from_wkt('POLYGON ((0 0,0 1,1 1,1 0,0 0))')
+    end
+
+    subject do
+      gc = described_class.new
+      gc.add_geometry(polygon)
+      gc
+    end
+
+    it 'returns a Polygon' do
+      expect(subject.to_polygon).to be_a OGR::Polygon
+    end
+  end
+
+  describe '#to_multi_polygon' do
+    let(:polygon1) do
+      OGR::Geometry.create_from_wkt('POLYGON ((0 0,0 1,1 1,1 0,0 0))')
+    end
+
+    let(:polygon2) do
+      OGR::Geometry.create_from_wkt('POLYGON ((10 10,10 11,11 11,11 10,10 10))')
+    end
+
+    subject do
+      gc = described_class.new
+      gc.add_geometry(polygon1)
+      gc.add_geometry(polygon2)
+      gc
+    end
+
+    it 'returns a MultiPolygon' do
+      expect(subject.to_multi_polygon).to be_a OGR::MultiPolygon
+    end
+  end
+
+  describe '#polygon_from_edges' do
+    context 'auto_close is false' do
+      subject do
+        gc = described_class.new
+        gc.add_geometry(line_string1)
+        gc.add_geometry(line_string2)
+        gc.polygon_from_edges(10, auto_close: false)
+      end
+
+      context 'points are not within tolerance' do
+        let(:line_string1) do
+          OGR::Geometry.create_from_wkt('LINESTRING (0 0,0 1,0 2)')
+        end
+
+        let(:line_string2) do
+          OGR::Geometry.create_from_wkt('LINESTRING (100 100,100 200,100 300)')
+        end
+
+        it 'raises a OGR::Failure error' do
+          expect { subject }.to raise_exception OGR::Failure
+        end
+      end
+
+      context 'points are within tolerance, not closed' do
+        let(:line_string1) do
+          OGR::Geometry.create_from_wkt('LINESTRING (0 0,0 1,0 2)')
+        end
+
+        let(:line_string2) do
+          OGR::Geometry.create_from_wkt('LINESTRING (1 1,1 2,1 3)')
+        end
+
+        it 'returns a Polygon' do
+          expect(subject).to be_a OGR::Polygon
+        end
+
+        it 'returns WKT with the merged strings' do
+          expect(subject.to_wkt).to eq 'POLYGON ((0 0,0 1,0 2,1 1,1 2,1 3))'
+        end
+      end
+
+      context 'points are within tolerance and closed' do
+        let(:line_string1) do
+          OGR::Geometry.create_from_wkt('LINESTRING (0 0,0 1,0 2)')
+        end
+
+        let(:line_string2) do
+          OGR::Geometry.create_from_wkt('LINESTRING (0 2,1 4,0 0)')
+        end
+
+        it 'returns a Polygon' do
+          expect(subject).to be_a OGR::Polygon
+        end
+
+        it 'returns WKT with the merged strings' do
+          expect(subject.to_wkt).to eq 'POLYGON ((0 0,0 1,0 2,1 4,0 0))'
+        end
+      end
+    end
+
+    context 'auto_close is true' do
+      subject do
+        gc = described_class.new
+        gc.add_geometry(line_string1)
+        gc.add_geometry(line_string2)
+        gc.polygon_from_edges(10, auto_close: true)
+      end
+
+      context 'points are not within tolerance' do
+        let(:line_string1) do
+          OGR::Geometry.create_from_wkt('LINESTRING (0 0,0 1,0 2)')
+        end
+
+        let(:line_string2) do
+          OGR::Geometry.create_from_wkt('LINESTRING (100 100,100 200,0 2)')
+        end
+
+        it 'raises a OGR::Failure' do
+          expect { subject }.to raise_exception OGR::Failure
+        end
+      end
+
+      context 'points are within tolerance, not closed' do
+        let(:line_string1) do
+          OGR::Geometry.create_from_wkt('LINESTRING (0 0,0 1,0 2)')
+        end
+
+        let(:line_string2) do
+          OGR::Geometry.create_from_wkt('LINESTRING (1 1,1 2,1 3)')
+        end
+
+        it 'returns a Polygon' do
+          expect(subject).to be_a OGR::Polygon
+        end
+
+        it 'returns WKT with the merged strings' do
+          expect(subject.to_wkt).
+            to eq 'POLYGON ((0 0 0,0 1 0,0 2 0,1 1 0,1 2 0,1 3 0,0 0 0))'
+        end
+      end
+
+      context 'points are within tolerance and already closed' do
+        let(:line_string1) do
+          OGR::Geometry.create_from_wkt('LINESTRING (0 0,0 1,0 2)')
+        end
+
+        let(:line_string2) do
+          OGR::Geometry.create_from_wkt('LINESTRING (0 2,1 4,0 0)')
+        end
+
+        it 'returns a Polygon' do
+          expect(subject).to be_a OGR::Polygon
+        end
+
+        it 'returns WKT with the merged strings' do
+          expect(subject.to_wkt).to eq 'POLYGON ((0 0,0 1,0 2,1 4,0 0))'
+        end
+      end
+
+      context 'unsupported geometries' do
+        subject do
+          gc = described_class.new
+          gc.add_geometry(OGR::MultiPolygon.new)
+          gc
+        end
+
+        it 'raises a GDAL::UnsupportedOperation or OGR::UnsupportedGeometryType' do
+          expect do
+            subject.polygon_from_edges(100, auto_close: false)
+          end.to raise_exception
+        end
+      end
+    end
+  end
+end

data/spec/unit/ogr/geometries/line_string_spec.rb

@@ -0,0 +1,105 @@
+require 'spec_helper'
+require 'ogr/geometry'
+
+RSpec.describe OGR::LineString do
+  let(:open_line_string) do
+    g = described_class.new
+    g.add_point(0, 0)
+    g.add_point(0, 10)
+    g.add_point(10, 10)
+
+    g
+  end
+
+  let(:closed_line_string) do
+    g = described_class.new
+    g.add_point(0, 0)
+    g.add_point(0, 10)
+    g.add_point(10, 10)
+    g.add_point(10, 0)
+    g.add_point(0, 0)
+
+    g
+  end
+
+  it_behaves_like 'a geometry' do
+    let(:geometry) { open_line_string }
+  end
+
+  it_behaves_like 'a line string' do
+    let(:geometry) { open_line_string }
+  end
+
+  describe '#closed' do
+    context 'geometry is closed' do
+      subject { closed_line_string }
+      it { is_expected.to be_closed }
+    end
+
+    context 'geometry is not closed' do
+      subject { open_line_string }
+      it { is_expected.to_not be_closed }
+    end
+  end
+
+  describe '#name' do
+    subject { open_line_string.name }
+    it { is_expected.to eq 'LINESTRING' }
+  end
+
+  describe '#point_count' do
+    subject { open_line_string.point_count }
+    it { is_expected.to eq 3  }
+  end
+
+  describe '#intersects?' do
+    context 'other geometry is a point' do
+      context 'inside the ring' do
+        let(:other_geometry) do
+          OGR::Geometry.create_from_wkt('POINT (0 1)')
+        end
+
+        subject { open_line_string.intersects?(other_geometry) }
+        it { is_expected.to eq true }
+      end
+
+      context 'on a vertex of the ring' do
+        let(:other_geometry) do
+          OGR::Geometry.create_from_wkt('POINT (0 0)')
+        end
+
+        subject { open_line_string.intersects?(other_geometry) }
+        it { is_expected.to eq true }
+      end
+    end
+
+    context 'other geometry is a line string' do
+      context 'outside the ring' do
+        let(:other_geometry) do
+          OGR::Geometry.create_from_wkt('LINESTRING (100 100, 200 200)')
+        end
+
+        subject { open_line_string.intersects?(other_geometry) }
+        it { is_expected.to eq false }
+      end
+
+      context 'ends on a vertex' do
+        let(:other_geometry) do
+          OGR::Geometry.create_from_wkt('LINESTRING (50 50, 0 0)')
+        end
+
+        subject { open_line_string.intersects?(other_geometry) }
+        it { is_expected.to eq true }
+      end
+
+      context 'passes across the boundary' do
+        let(:other_geometry) do
+          OGR::Geometry.create_from_wkt('LINESTRING (15 5, 5 5)')
+        end
+
+        subject { open_line_string.intersects?(other_geometry) }
+        it { is_expected.to eq false }
+      end
+    end
+  end
+end

data/spec/unit/ogr/{linear_ring_spec.rb → geometries/linear_ring_spec.rb}

@@ -1,8 +1,9 @@
 require 'spec_helper'
+require 'ogr/geometry'
 
-describe OGR::LinearRing do
+RSpec.describe OGR::LinearRing do
   let(:linear_ring) do
-    g = OGR::Geometry.create(:wkbLinearRing)
+    g = described_class.new
     g.add_point(0, 0)
     g.add_point(0, 10)
     g.add_point(10, 10)
@@ -12,35 +13,18 @@ describe OGR::LinearRing do
     g
   end
 
-  subject { linear_ring }
-
-  describe '#closed' do
-    it { is_expected.to be_closed }
-  end
-
-  describe '#dimension' do
-    subject { linear_ring.dimension }
-    it { is_expected.to eq 1 }
-  end
-
-  describe '#coordinate_dimension' do
-    subject { linear_ring.coordinate_dimension }
-    it { is_expected.to eq 2 }
+  it_behaves_like 'a geometry' do
+    let(:geometry) { linear_ring }
   end
 
-  describe '#envelope' do
-    subject { linear_ring.envelope }
-    it { is_expected.to be_a OGR::Envelope }
+  it_behaves_like 'a line string' do
+    let(:geometry) { linear_ring }
   end
 
-  describe '#type' do
-    subject { linear_ring.type }
-    it { is_expected.to eq :wkbLineString }
-  end
+  subject { linear_ring }
 
-  describe '#type_to_name' do
-    subject { linear_ring.type }
-    it { is_expected.to eq :wkbLineString }
+  describe '#closed' do
+    it { is_expected.to be_closed }
   end
 
   describe '#name' do
@@ -48,11 +32,6 @@ describe OGR::LinearRing do
     it { is_expected.to eq 'LINEARRING' }
   end
 
-  describe '#count' do
-    subject { linear_ring.count }
-    it { is_expected.to be_zero  }
-  end
-
   describe '#point_count' do
     subject { linear_ring.point_count }
     it { is_expected.to eq 5  }

data/spec/unit/ogr/geometries/multi_line_string_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+require 'ogr/geometry'
+
+RSpec.describe OGR::MultiLineString do
+  it_behaves_like 'a geometry' do
+    let(:geometry) { described_class.new }
+  end
+
+  it_behaves_like 'a container geometry' do
+    let(:child_geometry) do
+      OGR::LineString.new
+    end
+  end
+end

data/spec/unit/ogr/geometries/multi_point_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+require 'ogr/geometry'
+
+RSpec.describe OGR::MultiPoint do
+  it_behaves_like 'a geometry' do
+    let(:geometry) { described_class.new }
+  end
+
+  it_behaves_like 'a container geometry' do
+    let(:child_geometry) { OGR::Point.new }
+    let(:edge_geometry1) { OGR::Point.new }
+    let(:edge_geometry2) { OGR::Point.new }
+  end
+end