ffi-gdal 0.0.4 → 1.0.0.beta1

data/lib/ogr/spatial_reference_extensions.rb

@@ -0,0 +1,32 @@
+require 'json'
+
+module OGR
+  module SpatialReferenceExtensions
+
+    # @return [Hash]
+    def as_json
+      {
+        angular_units: angular_units,
+        epsg_treats_as_lat_long: epsg_treats_as_lat_long?,
+        epsg_treats_as_northing_easting: epsg_treats_as_northing_easting?,
+        is_compound: compound?,
+        is_geocentric: geocentric?,
+        is_geographic: geographic?,
+        is_local: local?,
+        is_projected: projected?,
+        is_vertical: vertical?,
+        linear_units: linear_units,
+        prime_meridian: prime_meridian,
+        semi_major: semi_major,
+        semi_minor: semi_minor,
+        spheroid_inverse_flattening: spheroid_inverse_flattening,
+        utm_zone: utm_zone
+      }
+    end
+
+    # @return [String]
+    def to_json
+      as_json.to_json
+    end
+  end
+end

data/lib/ogr/style_table.rb

@@ -0,0 +1,17 @@
+require_relative '../ffi/ogr'
+require_relative 'style_table_extensions'
+
+module OGR
+  class StyleTable
+    include StyleTableExtensions
+
+    # @param style_table [OGR::StyleTable, FFI::Pointer]
+    def initialize(style_table)
+      @ogr_style_table_pointer = GDAL._pointer(OGR::StyleTable, style_table)
+    end
+
+    def c_pointer
+      @ogr_style_table_pointer
+    end
+  end
+end

data/lib/ogr/style_table_extensions.rb

@@ -0,0 +1,16 @@
+require 'json'
+
+module OGR
+  module StyleTableExtensions
+
+    # @return [String]
+    def as_json
+      'StyleTable interface not yet wrapped with ffi-ruby'
+    end
+
+    # @return [String]
+    def to_json
+      as_json.to_json
+    end
+  end
+end

data/spec/{ffi-gdal/integration → integration}/color_table_info_spec.rb

@@ -13,7 +13,7 @@ TIF_FILES.each do |file|
 
     # TODO: Test against each raster band
     subject do
-      band = GDAL::RasterBand.new(dataset.c_pointer, band_id: 1)
+      band = dataset.raster_band(1)
       band.color_table
     end
 

data/spec/{ffi-gdal/integration → integration}/driver_info_spec.rb

@@ -5,7 +5,7 @@ require 'ffi-gdal'
 TIF_FILES.each do |file|
   describe 'Driver Info' do
     subject do
-      GDAL::Driver.new(file_path: file)
+      GDAL::Driver.open_by_file(file)
     end
 
     it_behaves_like 'a major object'

data/spec/{ffi-gdal/integration → integration}/raster_attribute_table_info_spec.rb

@@ -12,7 +12,7 @@ TIF_FILES.each do |file|
     end
 
     subject do
-      band = GDAL::RasterBand.new(dataset.c_pointer, band_id: 1)
+      band = dataset.raster_band(1)
       band.default_raster_attribute_table
     end
 

data/spec/{ffi-gdal/integration → integration}/raster_band_info_spec.rb

@@ -5,8 +5,8 @@ require 'ffi-gdal'
 
 TIF_FILES.each do |file|
   dataset = GDAL::Dataset.open(file, 'r')
-  dataset.each_band do |band_under_test|
 
+  dataset.each_band do |band_under_test|
     describe "Raster Band Info" do
       after :suite do
         dataset.close
@@ -14,7 +14,7 @@ TIF_FILES.each do |file|
 
       # TODO: Test against each raster band
       subject do
-        GDAL::RasterBand.new(dataset.c_pointer, raster_band_pointer: band_under_test.c_pointer)
+        GDAL::RasterBand.new(band_under_test.c_pointer)
       end
 
       it_behaves_like 'a major object'
@@ -37,10 +37,10 @@ TIF_FILES.each do |file|
         specify { expect(subject.access_flag).to eq :GA_ReadOnly }
       end
 
-      describe '#band_number' do
+      describe '#number' do
         it 'is a non-zero Integer' do
-          expect(subject.band_number).to be_a Fixnum
-          expect(subject.band_number).to be > 0
+          expect(subject.number).to be_a Fixnum
+          expect(subject.number).to be > 0
         end
       end
 

data/spec/spec_helper.rb

@@ -1,5 +1,6 @@
 $:.unshift File.expand_path('../lib', __FILE__)
-$:.unshift File.expand_path('../spec', __FILE__)
+#$:.unshift File.expand_path('../spec', __FILE__)
+require 'ffi-gdal'
 
 Dir['./spec/support/shared_examples/**/*.rb'].sort.each { |f| require f}
 
@@ -11,3 +12,5 @@ RSpec.configure do |config|
   #     --seed 1234
   config.order = 'random'
 end
+
+OGR::Geometry.logging_enabled = true

data/spec/support/shapefiles/states_21basic/states.prj

@@ -0,0 +1 @@
+GEOGCS["GCS_North_American_1983",DATUM["D_North_American_1983",SPHEROID["GRS_1980",6378137.0,298.257222101]],PRIMEM["Greenwich",0.0],UNIT["Degree",0.0174532925199433]]

data/spec/support/worldfiles/SR_50M/SR_50M.VERSION.txt

@@ -0,0 +1 @@
+2.0.0

data/spec/support/worldfiles/SR_50M/SR_50M.prj

@@ -0,0 +1 @@
+GEOGCS["GCS_WGS_1984",DATUM["D_WGS_1984",SPHEROID["WGS_1984",6378137.0,298.257223563]],PRIMEM["Greenwich",0.0],UNIT["Degree",0.017453292519943295]]

data/spec/support/worldfiles/SR_50M/SR_50M.tfw

@@ -0,0 +1,6 @@
+          0.03333333333333 
+          0.00000000000000 
+          0.00000000000000 
+          -0.03333333333333 
+          -179.98333333333333 
+          89.98333333333333 

data/spec/{ext/cpl_error_symbols_spec.rb → unit/ext/error_symbols_spec.rb}

@@ -1,5 +1,5 @@
 require 'spec_helper'
-require 'ext/cpl_error_symbols'
+require 'ext/error_symbols'
 
 describe Symbol do
   describe '#to_ruby' do

data/spec/unit/gdal/color_table_spec.rb

@@ -0,0 +1,146 @@
+require 'spec_helper'
+
+
+describe GDAL::ColorTable do
+  describe '.create' do
+    context 'with a valid PaletteInterpretation' do
+      it 'creates a new ColorTable' do
+        expect(described_class.create(:GPI_RGB)).to be_a(described_class)
+      end
+    end
+
+    context 'with an invalid PaletteInterpretation' do
+      it 'raises an ArgumentError' do
+        expect {
+          described_class.create(:MEOW)
+        }.to raise_exception(ArgumentError)
+      end
+    end
+  end
+
+  describe '#initialize' do
+    context ':GPI_RGB' do
+      it 'extends the new object with the ColorTableTypes::RGB module' do
+        expect_any_instance_of(described_class).to receive(:extend).
+          with(GDAL::ColorTableTypes::RGB)
+        described_class.create(:GPI_RGB)
+      end
+    end
+
+    context ':GPI_Gray' do
+      it 'extends the new object with the ColorTableTypes::Gray module' do
+        expect_any_instance_of(described_class).to receive(:extend).
+          with(GDAL::ColorTableTypes::Gray)
+        described_class.create(:GPI_Gray)
+      end
+    end
+
+    context ':GPI_CMYK' do
+      it 'extends the new object with the ColorTableTypes::CMYK module' do
+        expect_any_instance_of(described_class).to receive(:extend).
+          with(GDAL::ColorTableTypes::CMYK)
+        described_class.create(:GPI_CMYK)
+      end
+    end
+
+    context ':GPI_HLS' do
+      it 'extends the new object with the ColorTableTypes::HLS module' do
+        expect_any_instance_of(described_class).to receive(:extend).
+          with(GDAL::ColorTableTypes::HLS)
+        described_class.create(:GPI_HLS)
+      end
+    end
+  end
+
+  subject do
+    described_class.create(:GPI_RGB)
+  end
+
+  describe '#palette_interpretation' do
+    it 'equals the symbol that was given during .create' do
+      expect(subject.palette_interpretation).to eq :GPI_RGB
+    end
+  end
+
+  describe '#color_entry_count' do
+    it 'defaults to 0' do
+      expect(subject.color_entry_count).to be_zero
+    end
+  end
+
+  describe '#color_entry' do
+    context 'when no entries' do
+      it 'returns nil' do
+        expect(subject.color_entry(900)).to be_nil
+      end
+    end
+
+    context 'when entries' do
+      before do
+        subject.add_color_entry(0)
+      end
+
+      it 'returns the ColorEntry' do
+        expect(subject.color_entry(0)).to be_a GDAL::ColorEntry
+      end
+    end
+  end
+
+  describe '#color_entry_as_rgb' do
+    context 'when no entries' do
+      it 'returns nil' do
+        expect(subject.color_entry_as_rgb(900)).to be_nil
+      end
+    end
+
+    context 'when entries' do
+      before do
+        subject.add_color_entry(0)
+      end
+
+      it 'returns the ColorEntry' do
+        expect(subject.color_entry_as_rgb(0)).to be_a GDAL::ColorEntry
+      end
+    end
+  end
+
+  describe '#add_color_entry' do
+    context 'no values given' do
+      it 'creates and returns the entry' do
+        expect(subject.add_color_entry(0)).to be_a GDAL::ColorEntry
+      end
+    end
+
+    context 'all values given' do
+      it 'creates and returns the entry' do
+        expect(subject.add_color_entry(0, 75, 250, 255)).to be_a GDAL::ColorEntry
+      end
+    end
+
+    context 'bad values given' do
+      it 'creates and returns the entry' do
+        expect(subject.add_color_entry(0, 300, 250, 255)).to be_a GDAL::ColorEntry
+      end
+    end
+  end
+
+  describe '#create_color_ramp!' do
+    context 'no color entries' do
+      it 'returns nil' do
+        entry0 = GDAL::ColorEntry.new
+        entry1 = GDAL::ColorEntry.new
+        expect(subject.create_color_ramp!(0, entry0, 1, entry1)).to be_nil
+      end
+    end
+
+    context 'color entries that exist' do
+      it 'returns nil' do
+        entry0 = subject.add_color_entry(0, 0, 0, 0, 0)
+        entry1 = subject.add_color_entry(1, 10, 10, 10, 10)
+        entry2 = subject.add_color_entry(2, 100, 100, 100, 100)
+
+        expect(subject.create_color_ramp!(0, entry0, 2, entry2)).to be_nil
+      end
+    end
+  end
+end

data/spec/unit/ogr/layer_spec.rb

@@ -0,0 +1,97 @@
+require 'spec_helper'
+
+describe OGR::Layer do
+  let(:data_source) do
+    OGR::DataSource.open('spec/support/shapefiles/states_21basic/states.shp', 'r')
+  end
+
+  let(:layer0) do
+    data_source.layer(0)
+  end
+
+  subject do
+    layer0
+  end
+
+  describe '#name' do
+    subject { layer0.name }
+    it { is_expected.to eq 'states' }
+  end
+
+  describe '#geometry_type' do
+    subject { layer0.geometry_type }
+    it { is_expected.to eq :wkbPolygon }
+  end
+
+  describe '#feature_count' do
+    subject { layer0.feature_count }
+    it { is_expected.to eq 51 }
+  end
+
+  describe '#feature' do
+    subject { layer0.feature(0) }
+    it { is_expected.to be_a OGR::Feature }
+  end
+
+  describe '#next_feature' do
+    subject { layer0.next_feature }
+    it { is_expected.to be_a OGR::Feature }
+  end
+
+  describe '#features_read' do
+    subject { layer0.features_read }
+    it { is_expected.to be >= 0 }
+  end
+
+  describe '#features' do
+    subject { layer0.features }
+    it { is_expected.to be_an Array }
+    specify { expect(subject.first).to be_a OGR::Feature }
+    specify { expect(subject.size).to eq layer0.feature_count }
+  end
+
+  describe '#feature_definition' do
+    subject { layer0.feature_definition }
+    it { is_expected.to be_a OGR::FeatureDefinition }
+  end
+
+  describe '#spatial_reference' do
+    subject { layer0.spatial_reference }
+    it { is_expected.to be_a OGR::SpatialReference }
+  end
+
+  describe '#extent' do
+    subject { layer0.extent }
+    it { is_expected.to be_a OGR::Envelope }
+  end
+
+  describe '#fid_column' do
+    subject { layer0.fid_column }
+    it { is_expected.to be_a String }
+    it { is_expected.to be_empty }
+  end
+
+  describe '#geometry_column' do
+    subject { layer0.geometry_column }
+    it { is_expected.to be_a String }
+    it { is_expected.to be_empty }
+  end
+
+  describe '#style_table' do
+    subject { layer0.style_table }
+    it { is_expected.to be_nil }
+  end
+
+  describe '#geometry_from_extent' do
+    it 'is a Polygon' do
+      geometry = subject.geometry_from_extent
+      expect(geometry).to be_a OGR::Polygon
+    end
+  end
+
+  describe '#as_json' do
+    specify do
+      expect { subject.as_json }.to_not raise_exception
+    end
+  end
+end

data/spec/unit/ogr/linear_ring_spec.rb

@@ -0,0 +1,111 @@
+require 'spec_helper'
+
+describe OGR::LinearRing do
+  let(:linear_ring) do
+    g = OGR::Geometry.create(:wkbLinearRing)
+    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
+
+  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 }
+  end
+
+  describe '#envelope' do
+    subject { linear_ring.envelope }
+    it { is_expected.to be_a OGR::Envelope }
+  end
+
+  describe '#type' do
+    subject { linear_ring.type }
+    it { is_expected.to eq :wkbLineString }
+  end
+
+  describe '#type_to_name' do
+    subject { linear_ring.type }
+    it { is_expected.to eq :wkbLineString }
+  end
+
+  describe '#name' do
+    subject { linear_ring.name }
+    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  }
+  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 { linear_ring.intersects?(other_geometry) }
+        it { is_expected.to eq false }
+      end
+
+      context 'on a vertex of the ring' do
+        let(:other_geometry) do
+          OGR::Geometry.create_from_wkt('POINT (0 0)')
+        end
+
+        subject { linear_ring.intersects?(other_geometry) }
+        it { is_expected.to eq false }
+      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 { linear_ring.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 { linear_ring.intersects?(other_geometry) }
+        it { is_expected.to eq false }
+      end
+
+      context 'passes across the boundary' do
+        let(:other_geometry) do
+          OGR::Geometry.create_from_wkt('LINESTRING (15 5, 5 5)')
+        end
+
+        subject { linear_ring.intersects?(other_geometry) }
+        it { is_expected.to eq false }
+      end
+    end
+  end
+end