sparsam 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 321b7965fbbc375a4cc91aa22881aa516674353f
-  data.tar.gz: 4fca4a60305a3ad495f80e8a5280c13541ece0dc
+  metadata.gz: 80bcbb2225d5e0f24f9a295afd99be378fca3fcc
+  data.tar.gz: cdfd6e2214719008a2482693bcf60ca662a22609
 SHA512:
-  metadata.gz: f2395063ac5bcecdcbd9764006318f08c859344b366b0df8808ef369dacc094bdc1e2c81b5d6506adeb49f645810889d61c589cf4ddd1586f9d7691cbc0c128b
-  data.tar.gz: a9964ead299058bdd1667a2389da48f0c14d5a60ab0e66fbd357ec8d3e193b02cda5f89752b12f7533359defb88c99045852fb4f1842baa373f7bbb541e5caf9
+  metadata.gz: 41eff67315ab200edfab3501d6997136fd1a0642e0b1cbe0a46e47e5bccd0f3a4926ffb2ed6a04f97e757736058e0c76d6dc875a7a7bbb58c132c9665d06cef0
+  data.tar.gz: a7d5aef8bd988ec4535f54c7dbd73f3352742ed2574e5cb1c79b51508a2529201121adb9d15726a9508347c458d80142e643b50a8fadc29a16881895849cad86

data/ext/serializer.cpp

@@ -518,7 +518,7 @@ void ThriftSerializer::writeAny(TType ttype, FieldInfo *field_info,
   }
 
   case protocol::T_SET: {
-    if (CLASS_OF(actual) != SetClass) {
+    if (rb_class_real(CLASS_OF(actual)) != SetClass) {
       raise_type_mismatch(outer_struct, field_sym);
     }
 
@@ -553,7 +553,7 @@ void ThriftSerializer::writeAny(TType ttype, FieldInfo *field_info,
   }
 
   case protocol::T_STRUCT: {
-    if (CLASS_OF(actual) != field_info->klass) {
+    if (rb_class_real(CLASS_OF(actual)) != field_info->klass) {
       raise_type_mismatch(outer_struct, field_sym);
     }
 
@@ -616,7 +616,7 @@ void ThriftSerializer::writeStruct(VALUE klass, VALUE data) {
 VALUE serializer_writeStruct(VALUE self, VALUE klass, VALUE data) {
   watch_for_texcept() get_ts();
 
-  if (CLASS_OF(data) != klass) {
+  if (rb_class_real(CLASS_OF(data)) != klass) {
     VALUE expected_name = rb_class_name(klass);
     VALUE actual_name = rb_class_name(CLASS_OF(data));
 

data/spec/sparsam_spec.rb

@@ -442,5 +442,23 @@ describe 'Sparsam' do
         end
       end
     end
+
+    it 'handles structs with modified eigenclasses' do
+      nested_struct = US.new
+
+      class << nested_struct
+        def foo
+        end
+      end
+
+      data = SS.new(us_i: nested_struct)
+
+      class << data
+        def foo
+        end
+      end
+
+      expect { data.serialize }.not_to raise_error
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sparsam
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - Airbnb Thrift Developers
@@ -161,7 +161,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: thrift
-rubygems_version: 2.6.12
+rubygems_version: 2.6.11
 signing_key: 
 specification_version: 4
 summary: Ruby bindings for Apache Thrift