avromatic 5.1.0 → 5.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 820d2553430c52995c62ce458200cabd4bdb16ea386f49ad9eb2a0c25b4d36cc
-  data.tar.gz: 535ae34e72806819c819ab5c9801e402ca8864a70ca0b87ae3bfc5d90273633d
+  metadata.gz: 2619e37540f7e7f6002c5e51c1d252f7ba7b3cca4beefc22d7a6d6c568c73eba
+  data.tar.gz: c1da85904d906f67aa0f26828672080e91e1358f3079b51645a8d123da9f964e
 SHA512:
-  metadata.gz: 5d6fe4c63c5c6476e678febd5688688825abb819ef5374d45f79ede4a75739ebb3066ae6970860ac359080ce661d949ecd17b086e7386fbbbadb252d79c1fcec
-  data.tar.gz: 2cb734de5ba2ab060246c7a619ad8a82b440dc85b31f843db30aaff8860dd740263a6b98d2449e8b315d02aa0cf9896249c6a6329482c5ef70532dea72af98ed
+  metadata.gz: 20ad16e3d4f3f7f36231a1557711df1100ccfd506de38e0bbe8968b8689226805f0a2133c3e8408b03fb44c62844239f88b1b684a720e1bed6fdabc2b9ccea08
+  data.tar.gz: 59a9138f0bb58711800416751cc08858398242251517aa86b8eb86802abe2866e61551714a3eed7622658dbb9a36da60bfdbe521c21d899eb601b104915f84a9

data/lib/avromatic/model/raw_serialization.rb

@@ -15,8 +15,10 @@ module Avromatic
 
         UNSPECIFIED = Object.new
 
+        # rubocop:disable Style/AccessModifierDeclarations
         delegate :datum_writer, :datum_reader, to: :class
         private :datum_writer, :datum_reader
+        # rubocop:enable Style/AccessModifierDeclarations
 
         def avro_raw_value(validate: UNSPECIFIED)
           unless validate == UNSPECIFIED

data/lib/avromatic/model/types/abstract_type.rb

@@ -4,6 +4,7 @@ module Avromatic
   module Model
     module Types
       class AbstractType
+
         EMPTY_ARRAY = [].freeze
         private_constant :EMPTY_ARRAY
 
@@ -23,12 +24,12 @@ module Avromatic
           raise "#{__method__} must be overridden by #{self.class.name}"
         end
 
-        def coercible?(_input)
-          raise "#{__method__} must be overridden by #{self.class.name}"
+        def coercible?(input)
+          input.nil? || input_classes.any? { |input_class| input.is_a?(input_class) }
         end
 
-        def coerced?(_value)
-          raise "#{__method__} must be overridden by #{self.class.name}"
+        def coerced?(value)
+          value.nil? || value_classes.any? { |value_class| value.is_a?(value_class) }
         end
 
         # Note we use positional args rather than keyword args to reduce

data/lib/avromatic/model/types/array_type.rb

@@ -38,7 +38,7 @@ module Avromatic
         end
 
         def coerced?(value)
-          value.nil? || (value.is_a?(::Array) && value.all? { |element| value_type.coerced?(element) })
+          value.nil? || (value.is_a?(::Array) && value.all? { |element_input| value_type.coerced?(element_input) })
         end
 
         def serialize(value, strict)

data/lib/avromatic/model/types/big_int_type.rb

@@ -0,0 +1,49 @@
+# frozen_string_literal: true
+
+require 'avromatic/model/types/abstract_type'
+
+module Avromatic
+  module Model
+    module Types
+      class BigIntType < AbstractType
+        VALUE_CLASSES = [::Integer].freeze
+
+        MAX_RANGE = 2**63
+
+        def self.in_range?(value)
+          value.is_a?(::Integer) && value.between?(-MAX_RANGE, MAX_RANGE - 1)
+        end
+
+        def value_classes
+          VALUE_CLASSES
+        end
+
+        def name
+          'bigint'
+        end
+
+        def coerce(input)
+          if coercible?(input)
+            input
+          else
+            raise ArgumentError.new("Could not coerce '#{input.inspect}' to #{name}")
+          end
+        end
+
+        def coercible?(input)
+          input.nil? || self.class.in_range?(input)
+        end
+
+        alias_method :coerced?, :coercible?
+
+        def serialize(value, _strict)
+          value
+        end
+
+        def referenced_model_classes
+          EMPTY_ARRAY
+        end
+      end
+    end
+  end
+end

data/lib/avromatic/model/types/boolean_type.rb

@@ -24,12 +24,6 @@ module Avromatic
           end
         end
 
-        def coercible?(input)
-          input.nil? || input.is_a?(::TrueClass) || input.is_a?(::FalseClass)
-        end
-
-        alias_method :coerced?, :coercible?
-
         def serialize(value, _strict)
           value
         end

data/lib/avromatic/model/types/date_type.rb

@@ -7,11 +7,16 @@ module Avromatic
     module Types
       class DateType < AbstractType
         VALUE_CLASSES = [::Date].freeze
+        INPUT_CLASSES = [::Date, ::Time].freeze
 
         def value_classes
           VALUE_CLASSES
         end
 
+        def input_classes
+          INPUT_CLASSES
+        end
+
         def name
           'date'
         end
@@ -26,12 +31,6 @@ module Avromatic
           end
         end
 
-        def coercible?(input)
-          input.nil? || input.is_a?(::Date) || input.is_a?(::Time)
-        end
-
-        alias_method :coerced?, :coercible?
-
         def serialize(value, _strict)
           value
         end

data/lib/avromatic/model/types/decimal_type.rb

@@ -42,14 +42,6 @@ module Avromatic
           end
         end
 
-        def coercible?(input)
-          input.nil? || input_classes.any? { |input_class| input.is_a?(input_class) }
-        end
-
-        def coerced?(value)
-          value.nil? || value_classes.any? { |value_class| value.is_a?(value_class) }
-        end
-
         def serialize(value, _strict)
           value
         end

data/lib/avromatic/model/types/enum_type.rb

@@ -38,8 +38,8 @@ module Avromatic
           end
         end
 
-        def coerced?(input)
-          input.nil? || (input.is_a?(::String) && allowed_values.include?(input))
+        def coerced?(value)
+          value.nil? || value.is_a?(::String) && allowed_values.include?(value)
         end
 
         def coercible?(input)

data/lib/avromatic/model/types/float_type.rb

@@ -31,14 +31,6 @@ module Avromatic
           end
         end
 
-        def coercible?(input)
-          input.nil? || input.is_a?(::Float) || input.is_a?(::Integer)
-        end
-
-        def coerced?(input)
-          input.nil? || input.is_a?(::Float)
-        end
-
         def serialize(value, _strict)
           value
         end

data/lib/avromatic/model/types/integer_type.rb

@@ -8,6 +8,12 @@ module Avromatic
       class IntegerType < AbstractType
         VALUE_CLASSES = [::Integer].freeze
 
+        MAX_RANGE = 2**31
+
+        def self.in_range?(value)
+          value.is_a?(::Integer) && value.between?(-MAX_RANGE, MAX_RANGE - 1)
+        end
+
         def value_classes
           VALUE_CLASSES
         end
@@ -17,7 +23,7 @@ module Avromatic
         end
 
         def coerce(input)
-          if input.nil? || input.is_a?(::Integer)
+          if coercible?(input)
             input
           else
             raise ArgumentError.new("Could not coerce '#{input.inspect}' to #{name}")
@@ -25,7 +31,7 @@ module Avromatic
         end
 
         def coercible?(input)
-          input.nil? || input.is_a?(::Integer)
+          input.nil? || self.class.in_range?(input)
         end
 
         alias_method :coerced?, :coercible?

data/lib/avromatic/model/types/record_type.rb

@@ -36,10 +36,6 @@ module Avromatic
           false
         end
 
-        def coerced?(value)
-          value.nil? || value.is_a?(record_class)
-        end
-
         def serialize(value, strict)
           if value.nil?
             value

data/lib/avromatic/model/types/string_type.rb

@@ -31,14 +31,6 @@ module Avromatic
           end
         end
 
-        def coercible?(input)
-          input.nil? || input.is_a?(::String) || input.is_a?(::Symbol)
-        end
-
-        def coerced?(value)
-          value.nil? || value.is_a?(::String)
-        end
-
         def serialize(value, _strict)
           value
         end

data/lib/avromatic/model/types/type_factory.rb

@@ -9,6 +9,7 @@ require 'avromatic/model/types/enum_type'
 require 'avromatic/model/types/fixed_type'
 require 'avromatic/model/types/float_type'
 require 'avromatic/model/types/integer_type'
+require 'avromatic/model/types/big_int_type'
 require 'avromatic/model/types/map_type'
 require 'avromatic/model/types/null_type'
 require 'avromatic/model/types/record_type'
@@ -31,7 +32,7 @@ module Avromatic
           'bytes' => Avromatic::Model::Types::StringType.new,
           'boolean' => Avromatic::Model::Types::BooleanType.new,
           'int' => Avromatic::Model::Types::IntegerType.new,
-          'long' => Avromatic::Model::Types::IntegerType.new,
+          'long' => Avromatic::Model::Types::BigIntType.new,
           'float' => Avromatic::Model::Types::FloatType.new,
           'double' => Avromatic::Model::Types::FloatType.new,
           'null' => Avromatic::Model::Types::NullType.new

data/lib/avromatic/model/types/union_type.rb

@@ -76,7 +76,15 @@ module Avromatic
         private
 
         def find_index(value)
-          # TODO: Cache this?
+          if value.is_a?(::Integer)
+            if Avromatic::Model::Types::IntegerType.in_range?(value)
+              return member_types.index { |member_type| member_type.is_a?(Avromatic::Model::Types::IntegerType) }
+            elsif Avromatic::Model::Types::BigIntType.in_range?(value)
+              return member_types.index { |member_type| member_type.is_a?(Avromatic::Model::Types::BigIntType) }
+            end
+          end
+
+          # TODO: Consider caching the index for each value class
           member_types.find_index do |member_type|
             member_type.value_classes.any? { |value_class| value.is_a?(value_class) }
           end

data/lib/avromatic/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Avromatic
-  VERSION = '5.1.0'
+  VERSION = '5.2.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: avromatic
 version: !ruby/object:Gem::Version
-  version: 5.1.0
+  version: 5.2.0
 platform: ruby
 authors:
 - Salsify Engineering
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-09-11 00:00:00.000000000 Z
+date: 2024-11-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -19,7 +19,7 @@ dependencies:
         version: '5.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '8'
+        version: '8.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -29,7 +29,7 @@ dependencies:
         version: '5.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '8'
+        version: '8.1'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
@@ -39,7 +39,7 @@ dependencies:
         version: '5.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '8'
+        version: '8.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -49,7 +49,7 @@ dependencies:
         version: '5.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '8'
+        version: '8.1'
 - !ruby/object:Gem::Dependency
   name: avro
   requirement: !ruby/object:Gem::Requirement
@@ -298,6 +298,7 @@ files:
 - lib/avromatic/model/types/abstract_timestamp_type.rb
 - lib/avromatic/model/types/abstract_type.rb
 - lib/avromatic/model/types/array_type.rb
+- lib/avromatic/model/types/big_int_type.rb
 - lib/avromatic/model/types/boolean_type.rb
 - lib/avromatic/model/types/custom_type.rb
 - lib/avromatic/model/types/date_type.rb