gir_ffi 0.7.7 → 0.7.8

data/test/ffi-gobject_introspection/i_registered_type_info_test.rb

@@ -1,24 +1,25 @@
 require 'introspection_test_helper'
 
 describe GObjectIntrospection::IRegisteredTypeInfo do
-  describe "#get_type_name" do
-    describe "for an interface" do
+  describe '#get_type_name' do
+    describe 'for an interface' do
       let(:registered_type_info) {
-        get_introspection_data('GIMarshallingTests', 'Interface') }
+        get_introspection_data('GIMarshallingTests', 'Interface')
+      }
 
-      it "returns interface name" do
+      it 'returns interface name' do
         registered_type_info.type_name.must_equal 'GIMarshallingTestsInterface'
       end
     end
 
-    describe "for a type that is not an interface" do
+    describe 'for a type that is not an interface' do
       let(:registered_type_info) {
-        get_introspection_data('GIMarshallingTests', 'Enum') }
+        get_introspection_data('GIMarshallingTests', 'Enum')
+      }
 
-      it "returns nil" do
+      it 'returns nil' do
         registered_type_info.type_name.must_be_nil
       end
     end
   end
 end
-

data/test/ffi-gobject_introspection/i_repository_test.rb

@@ -1,19 +1,19 @@
 require 'introspection_test_helper'
 
 describe GObjectIntrospection::IRepository do
-  describe "an instance" do
-    it "is not created by calling new()" do
+  describe 'an instance' do
+    it 'is not created by calling new()' do
       assert_raises NoMethodError do
         GObjectIntrospection::IRepository.new
       end
     end
 
-    it "is created by calling default()" do
+    it 'is created by calling default()' do
       gir = GObjectIntrospection::IRepository.default
       assert_kind_of GObjectIntrospection::IRepository, gir
     end
 
-    it "is a singleton" do
+    it 'is a singleton' do
       gir = GObjectIntrospection::IRepository.default
       gir2 = GObjectIntrospection::IRepository.default
       assert_equal gir, gir2
@@ -22,40 +22,40 @@ describe GObjectIntrospection::IRepository do
 
   let(:gir) { GObjectIntrospection::IRepository.default }
 
-  describe "#require" do
+  describe '#require' do
     it "raises an error if the namespace doesn't exist" do
       assert_raises RuntimeError do
         gir.require 'VeryUnlikelyGObjectNamespaceName', nil
       end
     end
 
-    it "allows version to be nil" do
+    it 'allows version to be nil' do
       gir.require 'GObject', nil
       pass
     end
 
-    it "allows version to be left out" do
+    it 'allows version to be left out' do
       gir.require 'GObject'
       pass
     end
   end
 
-  describe "#find_by_gtype" do
-    it "raises an error if 0 is passed as the gtype" do
+  describe '#find_by_gtype' do
+    it 'raises an error if 0 is passed as the gtype' do
       proc { gir.find_by_gtype 0 }.must_raise ArgumentError
     end
   end
 
-  describe "enumerating the infos for GObject" do
-    before do
-      gir.require 'GObject', "2.0"
-    end
-
-    it "yields more than one object" do
+  describe '#n_infos' do
+    it 'yields more than one object for the GObject namespace' do
+      gir.require 'GObject', '2.0'
       assert_operator gir.n_infos('GObject'), :>, 0
     end
+  end
 
-    it "yields IBaseInfo objects" do
+  describe '#info' do
+    it 'yields IBaseInfo objects' do
+      gir.require 'GObject', '2.0'
       assert_kind_of GObjectIntrospection::IBaseInfo, gir.info('GObject', 0)
     end
   end

data/test/ffi-gobject_introspection/i_type_info_test.rb

@@ -1,17 +1,31 @@
 require 'introspection_test_helper'
 
 describe GObjectIntrospection::ITypeInfo do
-  describe "#name?" do
+  describe '#name?' do
     let(:object_info) { get_introspection_data('GIMarshallingTests', 'Object') }
     let(:vfunc_info) { object_info.find_vfunc('vfunc_array_out_parameter') }
     let(:arg_info) { vfunc_info.args[0] }
     let(:type_info) { arg_info.argument_type }
 
-    it "raises an error" do
+    it 'raises an error' do
       skip unless vfunc_info
       proc {
         type_info.name
       }.must_raise RuntimeError
     end
   end
+
+  describe '#interface' do
+    describe 'for unresolvable interface types' do
+      let(:function_info) { get_introspection_data 'GObject', 'signal_set_va_marshaller' }
+      let(:argument_info) { function_info.args.last }
+      let(:type_info) { argument_info.argument_type }
+
+      it 'returns an IUnresolvableInfo object' do
+        result = type_info.interface
+        result.info_type.must_equal :unresolved
+        result.must_be_kind_of GObjectIntrospection::IUnresolvedInfo
+      end
+    end
+  end
 end

data/test/ffi-gobject_introspection/lib_test.rb

@@ -1,8 +1,8 @@
 require 'introspection_test_helper'
 
 describe GObjectIntrospection::Lib::GIArgument do
-  describe "its member :v_ssize" do
-    it "is signed" do
+  describe 'its member :v_ssize' do
+    it 'is signed' do
       gia = GObjectIntrospection::Lib::GIArgument.new
       gia[:v_int64] = -1
       assert_equal(-1, gia[:v_ssize])

data/test/ffi-gobject_test.rb

@@ -5,28 +5,28 @@ describe GObject do
     GirFFI.setup :Regress
   end
 
-  describe "::signal_emit" do
-    it "emits a signal" do
+  describe '::signal_emit' do
+    it 'emits a signal' do
       a = 1
       o = Regress::TestSubObj.new
       prc = proc { a = 2 }
       callback = GObject::Callback.from prc
-      ::GObject::Lib.g_signal_connect_data o, "test", callback, nil, nil, 0
-      GObject.signal_emit o, "test"
+      ::GObject::Lib.g_signal_connect_data o, 'test', callback, nil, nil, 0
+      GObject.signal_emit o, 'test'
       assert_equal 2, a
     end
 
-    it "handles return values" do
+    it 'handles return values' do
       s = Gio::SocketService.new
 
       argtypes = [:pointer, :pointer, :pointer, :pointer]
-      callback = FFI::Function.new(:bool, argtypes) { |a,b,c,d| true }
-      ::GObject::Lib.g_signal_connect_data s, "incoming", callback, nil, nil, 0
-      rv = GObject.signal_emit s, "incoming"
+      callback = FFI::Function.new(:bool, argtypes) { |_a, _b, _c, _d| true }
+      ::GObject::Lib.g_signal_connect_data s, 'incoming', callback, nil, nil, 0
+      rv = GObject.signal_emit s, 'incoming'
       assert_equal true, rv.get_value
     end
 
-    it "passes in extra arguments" do
+    it 'passes in extra arguments' do
       o = Regress::TestSubObj.new
       sb = Regress::TestSimpleBoxedA.new
       sb.some_int8 = 31
@@ -35,85 +35,85 @@ describe GObject do
       b2 = nil
 
       argtypes = [:pointer, :pointer, :pointer]
-      callback = FFI::Function.new(:void, argtypes) do |a,b,c|
+      callback = FFI::Function.new(:void, argtypes) do |_a, b, _c|
         b2 = b
       end
-      ::GObject::Lib.g_signal_connect_data o, "test-with-static-scope-arg", callback, nil, nil, 0
-      GObject.signal_emit o, "test-with-static-scope-arg", sb
+      ::GObject::Lib.g_signal_connect_data o, 'test-with-static-scope-arg', callback, nil, nil, 0
+      GObject.signal_emit o, 'test-with-static-scope-arg', sb
 
       sb2 = Regress::TestSimpleBoxedA.wrap b2
       assert sb.equals(sb2)
     end
 
-    it "allows specifying signal detail" do
+    it 'allows specifying signal detail' do
       a = 1
       o = Regress::TestSubObj.new
 
       callback = FFI::Function.new(:void, [:pointer, :pointer, :pointer]) { a = 2 }
-      ::GObject::Lib.g_signal_connect_data o, "notify::detail", callback, nil, nil, 0
+      ::GObject::Lib.g_signal_connect_data o, 'notify::detail', callback, nil, nil, 0
 
-      GObject.signal_emit o, "notify::detail"
+      GObject.signal_emit o, 'notify::detail'
 
       a.must_equal 2
     end
   end
 
-  describe "::signal_connect" do
-    it "installs a signal handler" do
+  describe '::signal_connect' do
+    it 'installs a signal handler' do
       a = 1
       o = Regress::TestSubObj.new
-      GObject.signal_connect(o, "test") { a = 2 }
-      GObject.signal_emit o, "test"
+      GObject.signal_connect(o, 'test') { a = 2 }
+      GObject.signal_emit o, 'test'
       assert_equal 2, a
     end
 
-    it "passes user data to handler" do
+    it 'passes user data to handler' do
       a = 1
       o = Regress::TestSubObj.new
-      GObject.signal_connect(o, "test", 2) { |i, d| a = d }
-      GObject.signal_emit o, "test"
+      GObject.signal_connect(o, 'test', 2) { |_i, d| a = d }
+      GObject.signal_emit o, 'test'
       assert_equal 2, a
     end
 
-    it "passes object to handler" do
+    it 'passes object to handler' do
       o = Regress::TestSubObj.new
       o2 = nil
-      GObject.signal_connect(o, "test") { |i, d| o2 = i }
-      GObject.signal_emit o, "test"
+      GObject.signal_connect(o, 'test') { |i, _d| o2 = i }
+      GObject.signal_emit o, 'test'
       assert_instance_of Regress::TestSubObj, o2
       assert_equal o.to_ptr, o2.to_ptr
     end
 
-    it "does not allow connecting an invalid signal" do
+    it 'does not allow connecting an invalid signal' do
       o = Regress::TestSubObj.new
       assert_raises RuntimeError do
-        GObject.signal_connect(o, "not-really-a-signal") {}
+        GObject.signal_connect(o, 'not-really-a-signal') {}
       end
     end
 
-    it "handles return values" do
+    it 'handles return values' do
       s = Gio::SocketService.new
-      GObject.signal_connect(s, "incoming") { true }
-      rv = GObject.signal_emit s, "incoming"
+      GObject.signal_connect(s, 'incoming') { true }
+      rv = GObject.signal_emit s, 'incoming'
       assert_equal true, rv.get_value
     end
 
-    it "requires a block" do
+    it 'requires a block' do
       o = Regress::TestSubObj.new
       assert_raises ArgumentError do
-        GObject.signal_connect o, "test"
+        GObject.signal_connect o, 'test'
       end
     end
 
-    it "allows specifying signal detail" do
+    it 'allows specifying signal detail' do
       a = 1
       o = Regress::TestSubObj.new
-      GObject.signal_connect(o, "notify::detail", 2) { |i, _, d| a = d }
-      GObject.signal_emit o, "notify::detail"
+      GObject.signal_connect(o, 'notify::detail', 2) { |_i, _, d| a = d }
+      GObject.signal_emit o, 'notify::detail'
       assert_equal 2, a
     end
 
-    describe "connecting a signal with extra arguments" do
+    describe 'connecting a signal with extra arguments' do
       before do
         @a = nil
         @b = 2
@@ -122,21 +122,31 @@ describe GObject do
         sb = Regress::TestSimpleBoxedA.new
         sb.some_int = 23
 
-        GObject.signal_connect(o, "test-with-static-scope-arg", 2) { |instance, object, user_data|
+        GObject.signal_connect(o, 'test-with-static-scope-arg', 2) { |_instance, object, user_data|
           @a = user_data
           @b = object
         }
-        GObject.signal_emit o, "test-with-static-scope-arg", sb
+        GObject.signal_emit o, 'test-with-static-scope-arg', sb
       end
 
-      it "passes on the user data argument" do
+      it 'passes on the user data argument' do
         assert_equal 2, @a
       end
 
-      it "passes on the extra arguments" do
+      it 'passes on the extra arguments' do
         assert_instance_of Regress::TestSimpleBoxedA, @b
         assert_equal 23, @b.some_int
       end
     end
   end
+
+  describe '::signal_connect_after' do
+    it 'installs a signal handler' do
+      a = 1
+      o = Regress::TestSubObj.new
+      GObject.signal_connect_after(o, 'test') { a = 2 }
+      GObject.signal_emit o, 'test'
+      assert_equal 2, a
+    end
+  end
 end

data/test/gir_ffi/arg_helper_test.rb

@@ -1,24 +1,24 @@
 require 'gir_ffi_test_helper'
 
 describe GirFFI::ArgHelper do
-  describe ".cast_from_pointer" do
-    it "handles class types" do
+  describe '.cast_from_pointer' do
+    it 'handles class types' do
       klass = Class.new
       mock(klass).wrap(:pointer_value) { :wrapped_value }
       GirFFI::ArgHelper.cast_from_pointer(klass, :pointer_value).must_equal :wrapped_value
     end
 
-    it "handles negative :gint8" do
+    it 'handles negative :gint8' do
       ptr = FFI::Pointer.new(-127)
       GirFFI::ArgHelper.cast_from_pointer(:gint8, ptr).must_equal(-127)
     end
 
-    it "handles positive :gint8" do
+    it 'handles positive :gint8' do
       ptr = FFI::Pointer.new(128)
       GirFFI::ArgHelper.cast_from_pointer(:gint8, ptr).must_equal(128)
     end
 
-    it "handles :guint32" do
+    it 'handles :guint32' do
       ptr = FFI::Pointer.new(0xffffffff)
       GirFFI::ArgHelper.cast_from_pointer(:guint32, ptr).must_equal(0xffffffff)
     end

data/test/gir_ffi/builder_test.rb

@@ -6,7 +6,7 @@ describe GirFFI::Builder do
   let(:gir) { GObjectIntrospection::IRepository.default }
 
   describe '.build_class' do
-    it "does not replace existing classes" do
+    it 'does not replace existing classes' do
       oldclass = GObject::Object
       GirFFI::Builder.build_class get_introspection_data('GObject', 'Object')
       GObject::Object.must_equal oldclass
@@ -15,44 +15,44 @@ describe GirFFI::Builder do
 
   describe '.attach_ffi_function' do
     let(:lib) { Module.new }
-    it "calls attach_function with the correct types for Regress.test_callback_destroy_notify" do
+    it 'calls attach_function with the correct types for Regress.test_callback_destroy_notify' do
       function_info = get_introspection_data 'Regress', 'test_callback_destroy_notify'
 
       mock(lib).
-        attach_function("regress_test_callback_destroy_notify",
-                        [ Regress::TestCallbackUserData, :pointer, GLib::DestroyNotify ],
+        attach_function('regress_test_callback_destroy_notify',
+                        [Regress::TestCallbackUserData, :pointer, GLib::DestroyNotify],
                         :int32) { true }
 
       GirFFI::Builder.attach_ffi_function(lib, function_info)
     end
 
-    it "calls attach_function with the correct types for Regress::TestObj#torture_signature_0" do
+    it 'calls attach_function with the correct types for Regress::TestObj#torture_signature_0' do
       info = get_method_introspection_data 'Regress', 'TestObj', 'torture_signature_0'
 
       mock(lib).
-        attach_function("regress_test_obj_torture_signature_0",
+        attach_function('regress_test_obj_torture_signature_0',
                         [:pointer, :int32, :pointer, :pointer, :pointer, :pointer, :uint32],
                         :void) { true }
 
       GirFFI::Builder.attach_ffi_function(lib, info)
     end
 
-    it "calls attach_function with the correct types for Regress::TestObj#instance_method" do
+    it 'calls attach_function with the correct types for Regress::TestObj#instance_method' do
       info = get_method_introspection_data 'Regress', 'TestObj', 'instance_method'
-      mock(lib).attach_function("regress_test_obj_instance_method",
+      mock(lib).attach_function('regress_test_obj_instance_method',
                                 [:pointer], :int32) { true }
       GirFFI::Builder.attach_ffi_function(lib, info)
     end
 
-    it "calls attach_function with the correct types for Regress.test_array_gint32_in" do
+    it 'calls attach_function with the correct types for Regress.test_array_gint32_in' do
       info = get_introspection_data 'Regress', 'test_array_gint32_in'
-      mock(lib).attach_function("regress_test_array_gint32_in",
+      mock(lib).attach_function('regress_test_array_gint32_in',
                                 [:int32, :pointer], :int32) { true }
       GirFFI::Builder.attach_ffi_function(lib, info)
     end
   end
 
-  describe "looking at Regress.test_callback_destroy_notify" do
+  describe 'looking at Regress.test_callback_destroy_notify' do
     before do
       save_module :GObject
       save_module :Regress
@@ -61,7 +61,7 @@ describe GirFFI::Builder do
       @go = get_introspection_data 'Regress', 'test_callback_destroy_notify'
     end
 
-    it "defines ffi callback types :Callback and :ClosureNotify" do
+    it 'defines ffi callback types :Callback and :ClosureNotify' do
       Regress.setup_method 'test_callback_destroy_notify'
       tcud = Regress::Lib.find_type :TestCallbackUserData
       dn = GLib::Lib.find_type :DestroyNotify
@@ -78,40 +78,40 @@ describe GirFFI::Builder do
     end
   end
 
-  describe "building Regress::TestBoxed" do
+  describe 'building Regress::TestBoxed' do
     before do
       GirFFI::Builder.build_class get_introspection_data('Regress', 'TestBoxed')
     end
 
-    it "sets up #wrap" do
-      assert Regress::TestBoxed.respond_to? "wrap"
+    it 'sets up #wrap' do
+      assert Regress::TestBoxed.respond_to? 'wrap'
     end
 
-    it "sets up #allocate" do
-      assert Regress::TestBoxed.respond_to? "allocate"
+    it 'sets up #allocate' do
+      assert Regress::TestBoxed.respond_to? 'allocate'
     end
   end
 
-  describe "built Regress module" do
+  describe 'built Regress module' do
     before do
       save_module :Regress
       GirFFI::Builder.build_module 'Regress'
     end
 
-    it "autocreates singleton methods" do
+    it 'autocreates singleton methods' do
       refute_defines_singleton_method Regress, :test_uint
       Regress.test_uint 31
       assert_defines_singleton_method Regress, :test_uint
     end
 
-    it "autocreates the TestObj class on first access" do
+    it 'autocreates the TestObj class on first access' do
       assert !Regress.const_defined?(:TestObj)
       Regress::TestObj.must_be_instance_of Class
       assert Regress.const_defined? :TestObj
     end
 
-    it "knows its own module builder" do
-      assert GirFFI::Builders::ModuleBuilder === Regress.gir_ffi_builder
+    it 'knows its own module builder' do
+      Regress.gir_ffi_builder.must_be_instance_of GirFFI::Builders::ModuleBuilder
     end
 
     after do
@@ -119,35 +119,35 @@ describe GirFFI::Builder do
     end
   end
 
-  describe "having built Regress::TestObj" do
+  describe 'having built Regress::TestObj' do
     before do
       GirFFI::Builder.build_class get_introspection_data('Regress', 'TestObj')
     end
 
-    it "C functions for called instance methods get attached to Regress::Lib" do
-      o = Regress::TestObj.new_from_file("foo")
+    it 'C functions for called instance methods get attached to Regress::Lib' do
+      o = Regress::TestObj.new_from_file('foo')
       o.instance_method
       Regress::Lib.must_respond_to :regress_test_obj_instance_method
     end
 
-    it "the built class knows its own GIR info" do
+    it 'the built class knows its own GIR info' do
       Regress::TestObj.gir_info.name.must_equal 'TestObj'
     end
 
-    it "the built class knows its own class builder" do
+    it 'the built class knows its own class builder' do
       Regress::TestObj.gir_ffi_builder.must_be_instance_of GirFFI::Builders::ObjectBuilder
     end
   end
 
-  describe "built Regress::TestSubObj" do
-    it "inherits #set_bare from its superclass" do
+  describe 'built Regress::TestSubObj' do
+    it 'inherits #set_bare from its superclass' do
       o1 = Regress::TestSubObj.new
       o1.set_bare(nil)
       pass
     end
 
-    it "overrides #instance_method" do
-      obj = Regress::TestObj.new_from_file("foo")
+    it 'overrides #instance_method' do
+      obj = Regress::TestObj.new_from_file('foo')
       subobj = Regress::TestSubObj.new
 
       obj.instance_method.must_equal(-1)
@@ -155,30 +155,30 @@ describe GirFFI::Builder do
     end
   end
 
-  describe "building Regress::TestSubObj" do
+  describe 'building Regress::TestSubObj' do
     before do
       save_module :Regress
       GirFFI::Builder.build_class get_introspection_data('Regress', 'TestSubObj')
     end
 
-    it "sets up the Regress namespace" do
+    it 'sets up the Regress namespace' do
       assert Regress.const_defined? :Lib
       assert Regress.respond_to? :gir_ffi_builder
       assert Regress.const_defined? :GIR_FFI_BUILDER
     end
 
-    it "creates the Regress::Lib module ready to attach functions from the shared library" do
+    it 'creates the Regress::Lib module ready to attach functions from the shared library' do
       expected = [gir.shared_library('Regress')]
       assert_equal expected, Regress::Lib.ffi_libraries.map(&:name)
     end
 
-    it "builds parent classes also" do
+    it 'builds parent classes also' do
       assert Regress.const_defined? :TestObj
       assert Object.const_defined? :GObject
       assert GObject.const_defined? :Object
     end
 
-    it "sets up the inheritance chain" do
+    it 'sets up the inheritance chain' do
       ancestors = Regress::TestSubObj.ancestors
       assert_equal [
         Regress::TestSubObj,
@@ -187,7 +187,7 @@ describe GirFFI::Builder do
       ], ancestors[0..2]
     end
 
-    it "creates a Regress::TestSubObj#to_ptr method" do
+    it 'creates a Regress::TestSubObj#to_ptr method' do
       assert Regress::TestSubObj.public_method_defined? :to_ptr
     end
 
@@ -196,25 +196,25 @@ describe GirFFI::Builder do
     end
   end
 
-  describe "building Regress" do
+  describe 'building Regress' do
     before do
       save_module :Regress
       GirFFI::Builder.build_module 'Regress'
     end
 
-    it "creates a Lib module ready to attach functions from the shared library" do
+    it 'creates a Lib module ready to attach functions from the shared library' do
       gir = GObjectIntrospection::IRepository.default
       expected = [gir.shared_library('Regress')]
       assert_equal expected, Regress::Lib.ffi_libraries.map(&:name)
     end
 
-    it "does not replace existing module" do
+    it 'does not replace existing module' do
       oldmodule = Regress
       GirFFI::Builder.build_module 'Regress'
       assert_equal oldmodule, Regress
     end
 
-    it "does not replace existing Lib module" do
+    it 'does not replace existing Lib module' do
       oldmodule = Regress::Lib
       GirFFI::Builder.build_module 'Regress'
       assert_equal oldmodule, Regress::Lib