gir_ffi 0.14.0 → 0.15.3

data/test/gir_ffi/builders/initializer_builder_test.rb

@@ -1,16 +1,16 @@
 # frozen_string_literal: true
 
-require 'gir_ffi_test_helper'
+require "gir_ffi_test_helper"
 
 describe GirFFI::Builders::InitializerBuilder do
-  describe '#method_definition' do
+  describe "#method_definition" do
     let(:builder) { GirFFI::Builders::InitializerBuilder.new function_info }
     let(:code) { builder.method_definition }
 
-    describe 'for constructors with the default name' do
-      let(:function_info) { get_method_introspection_data 'Regress', 'TestObj', 'new' }
-      it 'builds an initializer' do
-        code.must_equal <<-CODE.reset_indentation
+    describe "for constructors with the default name" do
+      let(:function_info) { get_method_introspection_data "Regress", "TestObj", "new" }
+      it "builds an initializer" do
+        _(code).must_equal <<~CODE
           def initialize(obj)
             _v1 = Regress::TestObj.from(obj)
             _v2 = Regress::Lib.regress_test_obj_new _v1
@@ -20,10 +20,12 @@ describe GirFFI::Builders::InitializerBuilder do
       end
     end
 
-    describe 'for constructors with a custom name' do
-      let(:function_info) { get_method_introspection_data 'Regress', 'TestObj', 'new_from_file' }
-      it 'builds a custom initializer' do
-        code.must_equal <<-CODE.reset_indentation
+    describe "for constructors with a custom name" do
+      let(:function_info) do
+        get_method_introspection_data "Regress", "TestObj", "new_from_file"
+      end
+      it "builds a custom initializer" do
+        _(code).must_equal <<~CODE
           def initialize_from_file(x)
             _v1 = GirFFI::InPointer.from_utf8(x)
             _v2 = FFI::MemoryPointer.new(:pointer).write_pointer nil
@@ -35,13 +37,13 @@ describe GirFFI::Builders::InitializerBuilder do
       end
     end
 
-    describe 'for constructors for boxed types' do
+    describe "for constructors for boxed types" do
       let(:function_info) do
-        get_method_introspection_data 'GIMarshallingTests', 'BoxedStruct', 'new'
+        get_method_introspection_data "GIMarshallingTests", "BoxedStruct", "new"
       end
 
-      it 'builds an initializer that sets owned to true' do
-        code.must_equal <<-CODE.reset_indentation
+      it "builds an initializer that sets owned to true" do
+        _(code).must_equal <<~CODE
           def initialize
             _v1 = GIMarshallingTests::Lib.gi_marshalling_tests_boxed_struct_new
             store_pointer(_v1)

data/test/gir_ffi/builders/interface_builder_test.rb

@@ -1,32 +1,32 @@
 # frozen_string_literal: true
 
-require 'gir_ffi_test_helper'
+require "gir_ffi_test_helper"
 
 describe GirFFI::Builders::InterfaceBuilder do
   let(:interface_builder) do
     GirFFI::Builders::InterfaceBuilder.new(
-      get_introspection_data('Regress', 'TestInterface'))
+      get_introspection_data("Regress", "TestInterface"))
   end
 
-  describe '#build_class' do
+  describe "#build_class" do
     before do
-      info = get_introspection_data 'GObject', 'TypePlugin'
+      info = get_introspection_data "GObject", "TypePlugin"
       @bldr = GirFFI::Builders::InterfaceBuilder.new info
       @iface = @bldr.build_class
     end
 
-    it 'builds an interface as a module' do
+    it "builds an interface as a module" do
       assert_instance_of Module, @iface
     end
 
-    it 'creates methods on the interface' do
+    it "creates methods on the interface" do
       assert_defines_instance_method @iface, :complete_interface_info
     end
   end
 
-  describe '#interface_struct' do
-    it 'returns the interface struct type' do
-      interface_builder.interface_struct.must_equal Regress::TestInterfaceIface
+  describe "#interface_struct" do
+    it "returns the interface struct type" do
+      _(interface_builder.interface_struct).must_equal Regress::TestInterfaceIface
     end
   end
 end

data/test/gir_ffi/builders/module_builder_test.rb

@@ -1,39 +1,39 @@
 # frozen_string_literal: true
 
-require 'gir_ffi_test_helper'
+require "gir_ffi_test_helper"
 
 describe GirFFI::Builders::ModuleBuilder do
   let(:gir) { GObjectIntrospection::IRepository.default }
 
-  describe '#find_namespaced_class_info' do
-    it 'finds the info in the GIR' do
-      allow(gir).to receive(:require).with('Foo', nil)
+  describe "#find_namespaced_class_info" do
+    it "finds the info in the GIR" do
+      allow(gir).to receive(:require).with("Foo", nil)
 
-      builder = GirFFI::Builders::ModuleBuilder.new 'Foo'
+      builder = GirFFI::Builders::ModuleBuilder.new "Foo"
 
-      expect(gir).to receive(:find_by_name).with('Foo', 'Bar').and_return 'gir info'
+      expect(gir).to receive(:find_by_name).with("Foo", "Bar").and_return "gir info"
 
-      builder.find_namespaced_class_info(:Bar).must_equal 'gir info'
+      _(builder.find_namespaced_class_info(:Bar)).must_equal "gir info"
     end
 
-    it 'checks downcased class name as an alternative' do
-      allow(gir).to receive(:require).with('Foo', nil)
+    it "checks downcased class name as an alternative" do
+      allow(gir).to receive(:require).with("Foo", nil)
 
-      builder = GirFFI::Builders::ModuleBuilder.new 'Foo'
+      builder = GirFFI::Builders::ModuleBuilder.new "Foo"
 
-      expect(gir).to receive(:find_by_name).with('Foo', 'Bar').and_return nil
-      expect(gir).to receive(:find_by_name).with('Foo', 'bar').and_return 'gir info'
+      expect(gir).to receive(:find_by_name).with("Foo", "Bar").and_return nil
+      expect(gir).to receive(:find_by_name).with("Foo", "bar").and_return "gir info"
 
-      builder.find_namespaced_class_info(:Bar).must_equal 'gir info'
+      _(builder.find_namespaced_class_info(:Bar)).must_equal "gir info"
     end
 
-    it 'raises a clear error if the named class does not exist' do
-      allow(gir).to receive(:require).with('Foo', nil)
+    it "raises a clear error if the named class does not exist" do
+      allow(gir).to receive(:require).with("Foo", nil)
 
-      builder = GirFFI::Builders::ModuleBuilder.new 'Foo'
+      builder = GirFFI::Builders::ModuleBuilder.new "Foo"
 
-      expect(gir).to receive(:find_by_name).with('Foo', 'Bar').and_return nil
-      expect(gir).to receive(:find_by_name).with('Foo', 'bar').and_return nil
+      expect(gir).to receive(:find_by_name).with("Foo", "Bar").and_return nil
+      expect(gir).to receive(:find_by_name).with("Foo", "bar").and_return nil
 
       assert_raises NameError do
         builder.find_namespaced_class_info :Bar

data/test/gir_ffi/builders/object_builder_test.rb

@@ -1,76 +1,76 @@
 # frozen_string_literal: true
 
-require 'gir_ffi_test_helper'
+require "gir_ffi_test_helper"
 
 describe GirFFI::Builders::ObjectBuilder do
   let(:obj_builder) do
     GirFFI::Builders::ObjectBuilder.new(
-      get_introspection_data('Regress', 'TestObj'))
+      get_introspection_data("Regress", "TestObj"))
   end
   let(:sub_obj_builder) do
     GirFFI::Builders::ObjectBuilder.new(
-      get_introspection_data('Regress', 'TestSubObj'))
+      get_introspection_data("Regress", "TestSubObj"))
   end
   let(:param_spec_builder) do
     GirFFI::Builders::ObjectBuilder.new(
-      get_introspection_data('GObject', 'ParamSpec'))
+      get_introspection_data("GObject", "ParamSpec"))
   end
 
-  describe '#find_signal' do
-    it 'finds the signal "test" for TestObj' do
-      sig = obj_builder.find_signal 'test'
-      sig.name.must_equal 'test'
+  describe "#find_signal" do
+    it 'finds a signal defined on the class itself' do
+      sig = obj_builder.find_signal "test"
+      _(sig.name).must_equal "test"
     end
 
-    it 'finds the signal "test" for TestSubObj' do
-      sig = sub_obj_builder.find_signal 'test'
-      sig.name.must_equal 'test'
+    it 'finds a signal defined on a superclass' do
+      sig = sub_obj_builder.find_signal "test"
+      _(sig.name).must_equal "test"
     end
 
-    it 'finds the signal "changed" for Gtk::Entry' do
-      builder = GirFFI::Builders::ObjectBuilder.new get_introspection_data('Gtk', 'Entry')
-      sig = builder.find_signal 'changed'
-      sig.name.must_equal 'changed'
+    it 'finds signals defined on interfaces' do
+      skip_below "1.57.2"
+      sig = sub_obj_builder.find_signal "interface-signal"
+      _(sig.name).must_equal "interface-signal"
     end
 
     it "returns nil for a signal that doesn't exist" do
-      obj_builder.find_signal('foo').must_be_nil
+      _(obj_builder.find_signal("foo")).must_be_nil
     end
   end
 
-  describe '#find_property' do
-    it 'finds a property specified on the class itself' do
-      prop = obj_builder.find_property('int')
-      prop.name.must_equal 'int'
+  describe "#find_property" do
+    it "finds a property specified on the class itself" do
+      prop = obj_builder.find_property("int")
+      _(prop.name).must_equal "int"
     end
 
-    it 'finds a property specified on the parent class' do
-      prop = sub_obj_builder.find_property('int')
-      prop.name.must_equal 'int'
+    it "finds a property specified on the parent class" do
+      prop = sub_obj_builder.find_property("int")
+      _(prop.name).must_equal "int"
     end
 
-    it 'returns nil if the property is not found' do
-      sub_obj_builder.find_property('this-property-does-not-exist').must_be_nil
+    it "returns nil if the property is not found" do
+      _(sub_obj_builder.find_property("this-property-does-not-exist")).must_be_nil
     end
   end
 
-  describe '#object_class_struct' do
-    it 'returns the class struct type' do
-      obj_builder.object_class_struct.must_equal Regress::TestObjClass
+  describe "#object_class_struct" do
+    it "returns the class struct type" do
+      _(obj_builder.object_class_struct).must_equal Regress::TestObjClass
     end
 
-    it 'returns the parent struct type for classes without their own struct' do
-      binding_info = get_introspection_data 'GObject', 'Binding'
+    it "returns the parent struct type for classes without their own struct" do
+      binding_info = get_introspection_data "GObject", "Binding"
       builder = GirFFI::Builders::ObjectBuilder.new binding_info
-      builder.object_class_struct.must_equal GObject::ObjectClass
+      _(builder.object_class_struct).must_equal GObject::ObjectClass
     end
   end
 
-  describe 'for a struct without defined fields' do
-    let(:info) { get_introspection_data 'GObject', 'Binding' }
+  describe "for a struct without defined fields" do
+    let(:info) { get_introspection_data "GObject", "Binding" }
 
-    it 'uses a single field of the parent struct type as the default layout' do
-      info.n_fields.must_equal 0
+    it "uses a single field of the parent struct type as the default layout" do
+      _(info.n_fields).must_equal 0
 
       builder = GirFFI::Builders::ObjectBuilder.new info
 
@@ -79,20 +79,20 @@ describe GirFFI::Builders::ObjectBuilder do
     end
   end
 
-  describe '#eligible_properties' do
+  describe "#eligible_properties" do
     let(:wi_builder) do
       GirFFI::Builders::ObjectBuilder.new(
-        get_introspection_data('Regress', 'TestWi8021x'))
+        get_introspection_data("Regress", "TestWi8021x"))
     end
 
-    it 'includes properties that do not have a matching getter method' do
+    it "includes properties that do not have a matching getter method" do
       result = obj_builder.eligible_properties
-      result.map(&:name).must_include 'double'
+      _(result.map(&:name)).must_include "double"
     end
 
-    it 'skips properties that have a matching getter method' do
+    it "skips properties that have a matching getter method" do
       result = wi_builder.eligible_properties
-      result.map(&:name).wont_include 'testbool'
+      _(result.map(&:name)).wont_include "testbool"
     end
   end
 end

data/test/gir_ffi/builders/property_builder_test.rb

@@ -1,161 +1,161 @@
 # frozen_string_literal: true
 
-require 'gir_ffi_test_helper'
+require "gir_ffi_test_helper"
 
 describe GirFFI::Builders::PropertyBuilder do
   let(:builder) { GirFFI::Builders::PropertyBuilder.new property_info }
 
-  describe 'for a property of type :glist' do
-    let(:property_info) { get_property_introspection_data('Regress', 'TestObj', 'list') }
-    it 'generates the correct getter definition' do
-      expected = <<-CODE.reset_indentation
-      def list
-        _v1 = get_property('list')
-        _v2 = GLib::List.wrap(:utf8, _v1)
-        _v2
-      end
+  describe "for a property of type :glist" do
+    let(:property_info) { get_property_introspection_data("Regress", "TestObj", "list") }
+    it "generates the correct getter definition" do
+      expected = <<~CODE
+        def list
+          _v1 = get_property('list')
+          _v2 = GLib::List.wrap(:utf8, _v1)
+          _v2
+        end
       CODE
 
-      builder.getter_def.must_equal expected
+      _(builder.getter_def).must_equal expected
     end
 
-    it 'generates the correct setter definition' do
-      expected = <<-CODE.reset_indentation
-      def list= value
-        _v1 = GLib::List.from(:utf8, value)
-        set_property("list", _v1)
-      end
+    it "generates the correct setter definition" do
+      expected = <<~CODE
+        def list= value
+          _v1 = GLib::List.from(:utf8, value)
+          set_property("list", _v1)
+        end
       CODE
 
-      builder.setter_def.must_equal expected
+      _(builder.setter_def).must_equal expected
     end
   end
 
-  describe 'for a property of type :ghash' do
-    let(:property_info) { get_property_introspection_data('Regress', 'TestObj', 'hash-table') }
-    it 'generates the correct getter definition' do
-      expected = <<-CODE.reset_indentation
-      def hash_table
-        _v1 = get_property('hash-table')
-        _v2 = GLib::HashTable.wrap([:utf8, :gint8], _v1)
-        _v2
-      end
+  describe "for a property of type :ghash" do
+    let(:property_info) do
+      get_property_introspection_data("Regress", "TestObj", "hash-table")
+    end
+    it "generates the correct getter definition" do
+      expected = <<~CODE
+        def hash_table
+          _v1 = get_property('hash-table')
+          _v2 = GLib::HashTable.wrap([:utf8, :gint8], _v1)
+          _v2
+        end
       CODE
 
-      builder.getter_def.must_equal expected
+      _(builder.getter_def).must_equal expected
     end
 
-    it 'generates the correct setter definition' do
-      expected = <<-CODE.reset_indentation
-      def hash_table= value
-        _v1 = GLib::HashTable.from([:utf8, :gint8], value)
-        set_property("hash-table", _v1)
-      end
+    it "generates the correct setter definition" do
+      expected = <<~CODE
+        def hash_table= value
+          _v1 = GLib::HashTable.from([:utf8, :gint8], value)
+          set_property("hash-table", _v1)
+        end
       CODE
 
-      builder.setter_def.must_equal expected
+      _(builder.setter_def).must_equal expected
     end
   end
 
-  describe 'for a property of type :strv' do
+  describe "for a property of type :strv" do
     let(:property_info) do
-      get_property_introspection_data('GIMarshallingTests',
-                                      'PropertiesObject',
-                                      'some-strv')
-    end
-    before do
-      skip unless property_info
+      get_property_introspection_data("GIMarshallingTests",
+                                      "PropertiesObject",
+                                      "some-strv")
     end
 
-    it 'generates the correct getter definition' do
-      expected = <<-CODE.reset_indentation
-      def some_strv
-        _v1 = get_property('some-strv')
-        _v1
-      end
+    it "generates the correct getter definition" do
+      expected = <<~CODE
+        def some_strv
+          _v1 = get_property('some-strv')
+          _v1
+        end
       CODE
 
-      builder.getter_def.must_equal expected
+      _(builder.getter_def).must_equal expected
     end
 
-    it 'generates the correct setter definition' do
-      expected = <<-CODE.reset_indentation
-      def some_strv= value
-        _v1 = GLib::Strv.from(value)
-        set_property("some-strv", _v1)
-      end
+    it "generates the correct setter definition" do
+      expected = <<~CODE
+        def some_strv= value
+          _v1 = GLib::Strv.from(value)
+          set_property("some-strv", _v1)
+        end
       CODE
 
-      builder.setter_def.must_equal expected
+      _(builder.setter_def).must_equal expected
     end
   end
 
-  describe 'for a property of type :utf8' do
-    let(:property_info) { get_property_introspection_data('Regress', 'TestObj', 'string') }
-    it 'generates the correct getter definition' do
-      expected = <<-CODE.reset_indentation
-      def string
-        _v1 = get_property('string')
-        _v1
-      end
+  describe "for a property of type :utf8" do
+    let(:property_info) { get_property_introspection_data("Regress", "TestObj", "string") }
+    it "generates the correct getter definition" do
+      expected = <<~CODE
+        def string
+          _v1 = get_property('string')
+          _v1
+        end
       CODE
 
-      builder.getter_def.must_equal expected
+      _(builder.getter_def).must_equal expected
     end
 
-    it 'generates the correct setter definition' do
-      expected = <<-CODE.reset_indentation
-      def string= value
-        _v1 = value
-        set_property("string", _v1)
-      end
+    it "generates the correct setter definition" do
+      expected = <<~CODE
+        def string= value
+          _v1 = value
+          set_property("string", _v1)
+        end
       CODE
 
-      builder.setter_def.must_equal expected
+      _(builder.setter_def).must_equal expected
     end
   end
 
-  describe 'for a property of a callback type' do
+  describe "for a property of a callback type" do
     let(:property_info) do
-      get_property_introspection_data('Regress', 'AnnotationObject', 'function-property')
+      get_property_introspection_data("Regress", "AnnotationObject", "function-property")
     end
 
-    it 'generates the correct getter definition' do
-      expected = <<-CODE.reset_indentation
-      def function_property
-        _v1 = get_property('function-property')
-        _v2 = Regress::AnnotationCallback.wrap(_v1)
-        _v2
-      end
+    it "generates the correct getter definition" do
+      expected = <<~CODE
+        def function_property
+          _v1 = get_property('function-property')
+          _v2 = Regress::AnnotationCallback.wrap(_v1)
+          _v2
+        end
       CODE
 
-      builder.getter_def.must_equal expected
+      _(builder.getter_def).must_equal expected
     end
 
-    it 'generates the correct setter definition' do
-      expected = <<-CODE.reset_indentation
-      def function_property= value
-        _v1 = Regress::AnnotationCallback.from(value)
-        set_property("function-property", _v1)
-      end
+    it "generates the correct setter definition" do
+      expected = <<~CODE
+        def function_property= value
+          _v1 = Regress::AnnotationCallback.from(value)
+          set_property("function-property", _v1)
+        end
       CODE
 
-      builder.setter_def.must_equal expected
+      _(builder.setter_def).must_equal expected
     end
   end
 
-  describe '#container_defines_getter_method?' do
+  describe "#container_defines_getter_method?" do
     let(:property_info) { Object.new.tap { |o| o.extend GirFFI::InfoExt::IPropertyInfo } }
     let(:container_info) { Object.new }
 
     before do
       allow(property_info).to receive(:container).and_return container_info
-      allow(property_info).to receive(:name).and_return 'foo-bar'
-      allow(container_info).to receive(:find_instance_method).with('foo_bar').and_return true
+      allow(property_info).to receive(:name).and_return "foo-bar"
+      allow(container_info)
+        .to receive(:find_instance_method).with("foo_bar").and_return true
     end
 
-    it 'finds methods with underscores for properties with dashes' do
-      builder.container_defines_getter_method?.must_equal true
+    it "finds methods with underscores for properties with dashes" do
+      _(builder.container_defines_getter_method?).must_equal true
     end
   end
 end