ffi-gdal 1.0.0.beta3 → 1.0.0.beta4

data/spec/unit/ffi/gdal_spec.rb

@@ -0,0 +1,70 @@
+require 'spec_helper'
+require 'fakefs/safe'
+
+RSpec.describe FFI::GDAL do
+  describe '.find_lib' do
+    context 'ENV["GDAL_LIBRARY_PATH"] is set' do
+      before do
+        allow(ENV).to receive(:[]).with('GDAL_LIBRARY_PATH').and_return '/pants'
+      end
+
+      it 'returns GDAL_LIBRARY_PATH + libgdal file name' do
+        expect(described_class.find_lib('stuff')).to match %r{\A/pants/stuff\.+}
+      end
+    end
+
+    context 'ENV["GDAL_LIBRARY_PATH"] is not set' do
+      before do
+        allow(described_class).to receive(:search_paths).and_return %w[/pants]
+        FakeFS.activate!
+        FileUtils.mkdir 'pants'
+        FileUtils.touch("/pants/stuff.#{FFI::Platform::LIBSUFFIX}")
+      end
+
+      after { FakeFS.deactivate! }
+
+      it 'returns the search path + libgdal file name' do
+        expect(described_class.find_lib('stuff')).to match %r{\A/pants/stuff\.+}
+      end
+    end
+  end
+
+  describe '.search_paths' do
+    context 'ENV["GDAL_LIBRARY_PATH"] is set' do
+      before do
+        allow(ENV).to receive(:[]).with('GDAL_LIBRARY_PATH').and_return '/pants'
+      end
+
+      it 'returns GDAL_LIBRARY_PATH' do
+        expect(described_class.search_paths).to match /\A\/pants\z/
+      end
+    end
+
+    context 'ENV["GDAL_LIBRARY_PATH"] is not set' do
+      it 'returns a bunch of directories' do
+        expect(described_class.search_paths.size).to be > 0
+      end
+    end
+  end
+
+  describe '._files_with_constants' do
+    it 'returns a non-empty Array' do
+      expect(described_class._files_with_constants).to be_an Array
+      expect(described_class._files_with_constants).to_not be_empty
+    end
+  end
+
+  describe '_file_with_constants' do
+    context 'valid file' do
+      it 'returns the path to that file' do
+        expect(described_class._file_with_constants('gdal.h')).to match '.+gdal\.h\z'
+      end
+    end
+
+    context 'invalid file' do
+      it 'returns nil' do
+        expect(described_class._file_with_constants('derp')).to be_nil
+      end
+    end
+  end
+end

data/spec/unit/gdal/color_entry_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+require 'gdal/color_entry'
+
+RSpec.describe GDAL::ColorEntry do
+end

data/spec/unit/gdal/color_interpretation_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+require 'gdal/color_interpretation'
+
+RSpec.describe GDAL::ColorInterpretation do
+end

data/spec/unit/gdal/{color_table_extensions_spec.rb → color_table_mixins/extensions_spec.rb}

@@ -1,9 +1,9 @@
 require 'spec_helper'
+require 'gdal/color_table'
 
-
-describe GDAL::ColorTable do
+RSpec.describe GDAL::ColorTable do
   subject do
-    described_class.create(:GPI_RGB)
+    described_class.new(:GPI_RGB)
   end
 
   describe '#color_entries_for' do

data/spec/unit/gdal/color_table_spec.rb

@@ -1,29 +1,27 @@
 require 'spec_helper'
+require 'gdal/color_table'
 
-
-describe GDAL::ColorTable do
-  describe '.create' do
+RSpec.describe GDAL::ColorTable do
+  describe '#initialize' do
     context 'with a valid PaletteInterpretation' do
       it 'creates a new ColorTable' do
-        expect(described_class.create(:GPI_RGB)).to be_a(described_class)
+        expect(described_class.new(: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)
+      it 'raises an GDAL::InvalidColorTable' do
+        expect do
+          described_class.new(:MEOW)
+        end.to raise_exception(GDAL::InvalidColorTable)
       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)
+        described_class.new(:GPI_RGB)
       end
     end
 
@@ -31,7 +29,7 @@ describe GDAL::ColorTable 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)
+        described_class.new(:GPI_Gray)
       end
     end
 
@@ -39,7 +37,7 @@ describe GDAL::ColorTable 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)
+        described_class.new(:GPI_CMYK)
       end
     end
 
@@ -47,13 +45,13 @@ describe GDAL::ColorTable 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)
+        described_class.new(:GPI_HLS)
       end
     end
   end
 
   subject do
-    described_class.create(:GPI_RGB)
+    described_class.new(:GPI_RGB)
   end
 
   describe '#palette_interpretation' do
@@ -136,7 +134,7 @@ describe GDAL::ColorTable do
     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)
+        _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

data/spec/unit/gdal/data_type_spec.rb

@@ -1,6 +1,7 @@
 require 'spec_helper'
+require 'gdal/data_type'
 
-describe GDAL::DataType do
+RSpec.describe GDAL::DataType do
   describe '.size' do
     context 'with valid data type' do
       it 'returns the size' do
@@ -10,9 +11,9 @@ describe GDAL::DataType do
 
     context 'with invalid data type' do
       it 'raises an ArgumentError' do
-        expect {
+        expect do
           described_class.size(:Bob)
-        }.to raise_exception(ArgumentError)
+        end.to raise_exception(ArgumentError)
       end
     end
   end
@@ -32,9 +33,9 @@ describe GDAL::DataType do
 
     context 'with invalid data type' do
       it 'raises an ArgumentError' do
-        expect {
+        expect do
           described_class.complex?(:Bob)
-        }.to raise_exception(ArgumentError)
+        end.to raise_exception(ArgumentError)
       end
     end
   end
@@ -48,11 +49,9 @@ describe GDAL::DataType do
 
     context 'with invalid data type' do
       it 'raises an ArgumentError' do
-        skip 'Figure out why this causes a segfault'
-
-        expect {
+        expect do
           described_class.name(:Bob)
-        }.to raise_exception(ArgumentError)
+        end.to raise_exception(ArgumentError)
       end
     end
   end
@@ -60,19 +59,13 @@ describe GDAL::DataType do
   describe '.by_name' do
     context 'with valid data type name' do
       it 'returns the data type' do
-        skip 'Figure out why this causes a segfault'
-
         expect(described_class.by_name('Byte')).to eq :GDT_Byte
       end
     end
 
     context 'with invalid data type name' do
-      it 'raises an ArgumentError' do
-        skip 'Figure out why this causes a segfault'
-
-        expect {
-          described_class.by_name('Bob')
-        }.to raise_exception(ArgumentError)
+      it 'returns :GDT_Unknown' do
+        expect(described_class.by_name('Bob')).to eq :GDT_Unknown
       end
     end
   end

data/spec/unit/gdal/dataset_spec.rb

@@ -1,6 +1,7 @@
 require 'spec_helper'
+require 'gdal/dataset'
 
-describe GDAL::Dataset do
+RSpec.describe GDAL::Dataset do
   let(:file_path) do
     File.expand_path('../../support/images/osgeo/geotiff/GeogToWGS84GeoKey/GeogToWGS84GeoKey5.tif', __dir__)
   end
@@ -59,12 +60,8 @@ describe GDAL::Dataset do
   end
 
   describe '#add_band' do
-    it 'raises a GDAL::CPLErrFailure' do
-      skip 'Figure out why this causes a segfault'
-
-      expect {
-        subject.add_band(:GDT_Byte)
-      }.to raise_exception(GDAL::CPLErrFailure)
+    it 'raises a GDAL::UnsupportedOperation' do
+      expect { subject.add_band(:GDT_Byte) }.to raise_exception(GDAL::UnsupportedOperation)
     end
   end
 

data/spec/unit/gdal/driver_mixins/extensions_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+require 'gdal/driver'
+
+RSpec.describe GDAL::Driver do
+  describe '.short_names' do
+    subject { described_class.short_names }
+
+    it 'is an Array of Strings' do
+      expect(subject).to be_an Array
+      expect(subject.first).to be_a String
+    end
+  end
+
+  describe '.long_names' do
+    subject { described_class.long_names }
+
+    it 'is an Array of Strings' do
+      expect(subject).to be_an Array
+      expect(subject.first).to be_a String
+    end
+  end
+end

data/spec/unit/gdal/driver_spec.rb

@@ -0,0 +1,49 @@
+require 'spec_helper'
+require 'gdal/driver'
+
+RSpec.describe GDAL::Driver do
+  describe '.count' do
+    subject { described_class.count }
+    it { is_expected.to be > 0 }
+  end
+
+  describe '.by_name' do
+    context 'not a driver' do
+      it 'raises a GDAL::InvalidDriverName' do
+        expect { described_class.by_name('blargh') }.
+          to raise_exception GDAL::InvalidDriverName
+      end
+    end
+
+    context 'valid driver name' do
+      subject { described_class.by_name('MEM') }
+      it { is_expected.to be_an_instance_of GDAL::Driver }
+    end
+  end
+
+  describe '.at_index' do
+    context 'invalid index' do
+      it 'raises a GDAL::InvalidDriverIndex' do
+        expect { described_class.at_index 123_456 }.
+          to raise_exception GDAL::InvalidDriverIndex
+      end
+    end
+
+    context 'valid index' do
+      subject { described_class.at_index(0) }
+      it { is_expected.to be_an_instance_of GDAL::Driver }
+    end
+  end
+
+  describe '.identify_driver' do
+    context 'file is not supported' do
+      subject { described_class.identify_driver(__FILE__) }
+      it { is_expected.to be_nil }
+    end
+
+    context 'file is supported' do
+      subject { described_class.identify_driver('spec/support/worldfiles/SR_50M/SR_50M.tif') }
+      it { is_expected.to be_an_instance_of GDAL::Driver }
+    end
+  end
+end

data/spec/unit/gdal/environment_methods_spec.rb

@@ -0,0 +1,6 @@
+require 'spec_helper'
+require 'gdal/environment_methods'
+
+RSpec.describe GDAL::EnvironmentMethods do
+  subject { Object.new.extend(described_class) }
+end

data/spec/unit/gdal/geo_transform_spec.rb

@@ -0,0 +1,276 @@
+require 'spec_helper'
+require 'gdal/geo_transform'
+
+RSpec.describe GDAL::GeoTransform do
+  let(:world_file_path) do
+    'spec/support/worldfiles/SR_50M/SR_50M.tfw'
+  end
+
+  describe '.from_world_file' do
+    context 'a valid file' do
+      it 'reads in the file' do
+        result = described_class.from_world_file(world_file_path)
+        expect(result).to be_a GDAL::GeoTransform
+      end
+    end
+
+    context 'an invalid file' do
+      it 'reads in the file' do
+        expect do
+          described_class.from_world_file('things')
+        end.to raise_exception GDAL::OpenFailure
+      end
+    end
+  end
+
+  context 'an imported World file' do
+    subject(:world) do
+      described_class.from_world_file(world_file_path)
+    end
+
+    describe '#x_origin' do
+      subject { world.x_origin }
+      it { is_expected.to eq(-179.99999999999997) }
+    end
+
+    describe '#pixel_width' do
+      subject { world.pixel_width }
+      it { is_expected.to eq 0.03333333333333 }
+    end
+
+    describe '#x_rotation' do
+      subject { world.x_rotation }
+      it { is_expected.to eq 0.0 }
+    end
+
+    describe '#y_origin' do
+      subject { world.y_origin }
+      it { is_expected.to eq 90.0 }
+    end
+
+    describe '#pixel_height' do
+      subject { world.pixel_height }
+      it { is_expected.to eq(-0.03333333333333) }
+    end
+
+    describe '#y_rotation' do
+      subject { world.y_rotation }
+      it { is_expected.to eq 0.0 }
+    end
+  end
+
+  context 'a new GeoTransform' do
+    describe '#initialize' do
+      it 'sets all attributes to 0.0' do
+        expect(subject.x_origin).to eq 0.0
+        expect(subject.pixel_width).to eq 0.0
+        expect(subject.x_rotation).to eq 0.0
+        expect(subject.y_origin).to eq 0.0
+        expect(subject.pixel_height).to eq 0.0
+        expect(subject.y_rotation).to eq 0.0
+      end
+    end
+
+    describe '#x_origin= + #x_origin' do
+      it 'sets the value' do
+        subject.x_origin = 123
+        expect(subject.x_origin).to eq 123.0
+      end
+    end
+
+    describe '#pixel_width= + #pixel_width' do
+      it 'sets the value' do
+        subject.pixel_width = 123
+        expect(subject.pixel_width).to eq 123.0
+      end
+    end
+
+    describe '#x_rotation= + #x_rotation' do
+      it 'sets the value' do
+        subject.x_rotation = 123
+        expect(subject.x_rotation).to eq 123.0
+      end
+    end
+
+    describe '#y_origin= + #y_origin' do
+      it 'sets the value' do
+        subject.y_origin = 123
+        expect(subject.y_origin).to eq 123.0
+      end
+    end
+
+    describe '#pixel_width= + #pixel_height' do
+      it 'sets the value' do
+        subject.pixel_height = 123
+        expect(subject.pixel_height).to eq 123.0
+      end
+    end
+
+    describe '#y_rotation= + #y_rotation' do
+      it 'sets the value' do
+        subject.y_rotation = 123
+        expect(subject.y_rotation).to eq 123.0
+      end
+    end
+
+    describe '#apply_geo_transform' do
+      subject do
+        gt = described_class.new
+        gt.x_origin = 1000
+        gt.y_origin = -2000
+        gt.pixel_height = 0.1
+        gt.pixel_width = 0.1
+        gt
+      end
+
+      it "returns the GeoTransform's origins for (0, 0)" do
+        projected = subject.apply_geo_transform(0, 0)
+        expect(projected[:x_geo]).to eq 1000
+        expect(projected[:y_geo]).to eq(-2000)
+      end
+
+      it 'returns calculated points' do
+        projected = subject.apply_geo_transform(100, 100)
+        expect(projected[:x_geo]).to eq 1010.0
+        expect(projected[:y_geo]).to eq(-1990.0)
+      end
+    end
+
+    describe '#compose' do
+      context 'given parameter is not a GeoTransform' do
+        it 'raises a GDAL::NullObject' do
+          expect { subject.compose('not a pointer') }.to raise_exception GDAL::NullObject
+        end
+      end
+
+      context 'given parameter is a GeoTransform' do
+        let(:other_geo_transform) do
+          gt = described_class.new
+          gt.x_origin = 1000
+          gt.y_origin = 1000
+          gt.pixel_width = 0.5
+          gt.pixel_height = 0.25
+          gt.x_rotation = 1.0
+          gt.y_rotation = -1.0
+          gt
+        end
+
+        subject do
+          gt = described_class.new
+          gt.x_origin = 500
+          gt.y_origin = 500
+          gt.pixel_width = 0.25
+          gt.pixel_height = 0.5
+          gt.x_rotation = -1.0
+          gt.y_rotation = 1.0
+
+          gt.compose(other_geo_transform)
+        end
+
+        it 'is a new GeoTransform' do
+          expect(subject).to be_a GDAL::GeoTransform
+        end
+
+        it 'has a combined x_origin' do
+          expect(subject.x_origin).to eq 1750.0
+        end
+
+        it 'has a combined pixel_width' do
+          expect(subject.pixel_width).to eq 1.125
+        end
+
+        it 'has a combined x_rotation' do
+          expect(subject.x_rotation).to eq 0.0
+        end
+
+        it 'has a combined y_origin' do
+          expect(subject.y_origin).to eq 625.0
+        end
+
+        it 'has a combined pixel_height' do
+          expect(subject.pixel_height).to eq 1.125
+        end
+
+        it 'has a combined y_rotation' do
+          expect(subject.y_rotation).to eq 0.0
+        end
+      end
+    end
+
+    describe '#invert' do
+      subject do
+        gt = described_class.new
+        gt.x_origin = 1000
+        gt.y_origin = 2000
+        gt.pixel_width = 0.5
+        gt.pixel_height = 0.5
+        gt.x_rotation = 1.0
+        gt.y_rotation = -1.0
+
+        gt.invert
+      end
+
+      it 'returns a new GeoTransform' do
+        expect(subject).to be_a GDAL::GeoTransform
+      end
+
+      it 'returns an inverted x_origin' do
+        expect(subject.x_origin).to eq 1200
+      end
+
+      it 'returns an inverted pixel_width' do
+        expect(subject.pixel_width).to eq 0.4
+      end
+
+      it 'returns an inverted x_rotation' do
+        expect(subject.x_rotation).to eq(-0.8)
+      end
+
+      it 'returns an inverted y_origin' do
+        expect(subject.y_origin).to eq (-1600.0)
+      end
+
+      it 'returns an inverted pixel_height' do
+        expect(subject.pixel_height).to eq 0.4
+      end
+
+      it 'returns an inverted y_rotation' do
+        expect(subject.y_rotation).to eq 0.8
+      end
+    end
+
+    describe '#world_to_pixel' do
+      subject do
+        gt = described_class.new
+        gt.x_origin = 1000
+        gt.y_origin = -2000
+        gt.pixel_height = 0.1
+        gt.pixel_width = 0.1
+        gt
+      end
+
+      it 'converts (0, 0)' do
+        result = subject.world_to_pixel(0, 0)
+        expect(result).to eq pixel: -10_000, line: -20_000
+      end
+
+      it 'converts (100, -100)' do
+        result = subject.world_to_pixel(100, -100)
+        expect(result).to eq pixel: -9000, line: -19_000
+      end
+    end
+
+    describe '#as_json' do
+      it 'returns the attributes as a Hash' do
+        expect(subject.as_json).to eq(
+          x_origin: 0.0,
+          x_rotation: 0.0,
+          pixel_width: 0.0,
+          y_origin: 0.0,
+          y_rotation: 0.0,
+          pixel_height: 0.0
+        )
+      end
+    end
+  end
+end

data/spec/unit/gdal/grid_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+require 'gdal/grid'
+
+RSpec.describe GDAL::Grid do
+end