sparsam 0.2.5 → 0.2.9

data/spec/sparsam_spec.rb

@@ -227,6 +227,44 @@ describe 'Sparsam' do
       e.field_name.should == "id_i32"
     end
 
+    it "includes additional information on serializing mismatched types" do
+      data = MiniRequired.new
+      data.id_i32 = "not an i32"
+
+      e = nil
+      begin
+        data.serialize
+      rescue Sparsam::TypeMismatch => exception
+        e = exception
+      end
+      e.message.should include("T_I32", "String")
+
+      data = EveryType.new
+      data.a_struct = EveryType.new
+
+      e = nil
+      begin
+        data.serialize
+      rescue Sparsam::TypeMismatch => exception
+        e = exception
+      end
+      e.message.should include("US", "EveryType")
+    end
+
+    it "includes additional information on deserializing mismatched types" do
+      data = NotSS.new
+      data.id_s = "not an i32"
+
+      serialized = data.serialize
+
+      begin
+        Sparsam::Deserializer.deserialize(SS, serialized)
+      rescue Sparsam::TypeMismatch => exception
+        e = exception
+      end
+      e.message.should include("T_I32", "T_STRING")
+    end
+
     it "will throw errors when given junk data" do
       expect {
         Sparsam::Deserializer.deserialize(SS, "wolololololol")
@@ -451,6 +489,23 @@ describe 'Sparsam' do
       end
     end
 
+    it 'handles recursive structs' do
+      recursive_struct = RecursiveStruct.new
+      recursive_struct.id = 1
+      recursive_struct.self_struct = RecursiveStruct.new
+      recursive_struct.self_struct.id = 2
+      recursive_struct.self_list = [RecursiveStruct.new]
+      recursive_struct.self_list[0].id = 3
+
+      data = recursive_struct.serialize
+
+      new_recursive = Sparsam::Deserializer.deserialize(RecursiveStruct, data)
+
+      new_recursive.id.should == 1
+      new_recursive.self_struct.id.should == 2
+      new_recursive.self_list[0].id.should == 3
+    end
+
     it 'handles structs with modified eigenclasses' do
       nested_struct = US.new
 
@@ -491,4 +546,76 @@ describe 'Sparsam' do
       end
     end
   end
+
+  describe 'generated enum types' do
+    let(:enum_module) { Magic }
+
+    it 'includes thrift constants as top level module constants' do
+      enum_module.const_defined?(:Black).should == true
+      enum_module.const_defined?(:White).should == true
+      enum_module.const_defined?(:Red).should == true
+      enum_module.const_defined?(:Blue).should == true
+      enum_module.const_defined?(:Green).should == true
+
+      enum_module.const_get(:Black).should == 0
+      enum_module.const_get(:White).should == 1
+      enum_module.const_get(:Red).should == 2
+      enum_module.const_get(:Blue).should == 3
+      enum_module.const_get(:Green).should == 4
+    end
+
+    it 'contains a VALUE_MAP constant that maps from int value to string' do
+      enum_module.const_defined?(:VALUE_MAP).should == true
+
+      value_map = enum_module.const_get(:VALUE_MAP)
+
+      value_map.should eql({
+        0 => 'Black',
+        1 => 'White',
+        2 => 'Red',
+        3 => 'Blue',
+        4 => 'Green',
+      })
+
+      value_map[enum_module.const_get(:Black)].should == 'Black'
+      value_map[enum_module.const_get(:White)].should == 'White'
+      value_map[enum_module.const_get(:Red)].should == 'Red'
+      value_map[enum_module.const_get(:Blue)].should == 'Blue'
+      value_map[enum_module.const_get(:Green)].should == 'Green'
+    end
+
+    it 'contains an INVERTED_VALUE_MAP constant that maps from name to int value' do
+      enum_module.const_defined?(:INVERTED_VALUE_MAP).should == true
+
+      inverted_value_map = enum_module.const_get(:INVERTED_VALUE_MAP)
+
+      inverted_value_map.should eql({
+        'Black' => 0,
+        'White' => 1,
+        'Red' => 2,
+        'Blue' => 3,
+        'Green' => 4,
+      })
+
+      inverted_value_map['Black'].should == enum_module.const_get(:Black)
+      inverted_value_map['White'].should == enum_module.const_get(:White)
+      inverted_value_map['Red'].should == enum_module.const_get(:Red)
+      inverted_value_map['Blue'].should == enum_module.const_get(:Blue)
+      inverted_value_map['Green'].should == enum_module.const_get(:Green)
+    end
+
+    it 'contains a VALID_VALUES constant that is a set of all enum values' do
+      enum_module.const_defined?(:VALID_VALUES).should == true
+
+      valid_values = enum_module.const_get(:VALID_VALUES)
+
+      valid_values.should be_a(Set)
+      valid_values.should include(enum_module.const_get(:Black))
+      valid_values.should include(enum_module.const_get(:White))
+      valid_values.should include(enum_module.const_get(:Red))
+      valid_values.should include(enum_module.const_get(:Blue))
+      valid_values.should include(enum_module.const_get(:Green))
+      valid_values.length.should == 5
+    end
+  end
 end

data/spec/user.thrift

@@ -82,6 +82,12 @@ struct EveryType {
   15: optional UN a_union
 }
 
+struct RecursiveStruct {
+  1: optional i64 id
+  2: optional RecursiveStruct self_struct
+  3: optional list<RecursiveStruct> self_list
+}
+
 exception SimpleException {
   1: optional string message
 }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sparsam
 version: !ruby/object:Gem::Version
-  version: 0.2.5
+  version: 0.2.9
 platform: ruby
 authors:
 - Airbnb Thrift Developers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-07-07 00:00:00.000000000 Z
+date: 2022-01-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -164,8 +164,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: thrift
-rubygems_version: 2.7.6
+rubygems_version: 3.0.3.1
 signing_key: 
 specification_version: 4
 summary: Ruby bindings for Apache Thrift