sorbet-coerce 0.2.4 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8c5679d22c95a68cf624bef8a9d3f0c62b9b33f158b439b4115b2cb2f18dc508
-  data.tar.gz: f74881e16d72bb4b07b5de8072ef1f10d7794e8e6af249f9edcf132628a3a0e2
+  metadata.gz: 7503ed812a03fc5478701f1821be08195b10945d700dd27cda21088aa2e01f55
+  data.tar.gz: 6fd2a30012fad3af9f5fceee3de2a60c03bc7fb3f7fccdf42223b669c0e8c409
 SHA512:
-  metadata.gz: 5f0a4df69764c02a35aba2101740beb6445bc77dec5f5edea12c31c7db18837696153e4f41ab5604ba374a610a203377ec38f621c2d93f5e58a57d72e6e05b4b
-  data.tar.gz: 334a49158e79661994d826dbd5845b071260d66f5ffeae49d45d22c6c62a2deeeccb11fe3efe8e616856c2eb3bf60c12031795c4cace5817b87e6ca7ebfa633f
+  metadata.gz: ebbeb777352a39fe67575ee44c12ccb4ad1433ff7504bd76a9f9b86d91aef09a6e120496763254f433af783e2ab6dfec17b7822651db1bf563f9b7f223e8149c
+  data.tar.gz: 7d536c441a095d32af634b7ad362750e9c6d4f95789cf9cfdeb3d77aa0a90485ca4622517d3138cf7815c9eadc0e282ef7e75711b5fde27d4bfcca75b561dba8

data/lib/bundled_rbi/sorbet-coerce.rbi

@@ -1,4 +1,4 @@
-# typed: true
+# typed: false
 module SafeType
   class CoercionError < StandardError; end
 end

data/lib/sorbet-coerce.rb

@@ -1,23 +1,8 @@
-# typed: strict
-require_relative 'configuration'
-require 'private/converter'
-require 'safe_type'
+# typed: ignore
+require 'sorbet-coerce/converter'
 
 module TypeCoerce
-  class CoercionError < SafeType::CoercionError; end
-  class ShapeError < SafeType::CoercionError; end
-
-  define_singleton_method(:[]) do |type|
-    Class.new(TypeCoerce::Private::Converter) do
-      define_method(:to_s) { "#{name}#[#{type.to_s}]" }
-
-      define_method(:from) do |args, raise_coercion_error: nil|
-        if raise_coercion_error.nil?
-          raise_coercion_error = TypeCoerce::Configuration.raise_coercion_error
-        end
-
-        T.send('let', send('_convert', args, type, raise_coercion_error), type)
-      end
-    end
+  def self.[](type)
+    TypeCoerce::Converter.new(type)
   end
 end

data/lib/sorbet-coerce/converter.rb

@@ -0,0 +1,183 @@
+# typed: ignore
+require 'polyfill'
+require 'safe_type'
+require 'set'
+require 'sorbet-runtime'
+require 'sorbet-coerce/configuration'
+
+using Polyfill(Hash: %w[#slice])
+
+module TypeCoerce
+  class CoercionError < SafeType::CoercionError; end
+  class ShapeError < SafeType::CoercionError; end
+end
+
+class TypeCoerce::Converter
+  def initialize(type)
+    @type = type
+  end
+
+  def new
+    self
+  end
+
+  def to_s
+    "#{name}#[#{@type.to_s}]"
+  end
+
+  def from(args, raise_coercion_error: nil)
+    if raise_coercion_error.nil?
+      raise_coercion_error = TypeCoerce::Configuration.raise_coercion_error
+    end
+
+    T.let(_convert(args, @type, raise_coercion_error), @type)
+  end
+
+  private
+
+  PRIMITIVE_TYPES = T.let(::Set[
+    Date,
+    DateTime,
+    Float,
+    Integer,
+    String,
+    Symbol,
+    Time,
+  ], T.untyped)
+
+  def _convert(value, type, raise_coercion_error)
+    if type.is_a?(T::Types::Untyped)
+      value
+    elsif type.is_a?(T::Types::TypedArray)
+      _convert_to_a(value, type.type, raise_coercion_error)
+    elsif type.is_a?(T::Types::TypedSet)
+      Set.new(_convert_to_a(value, type.type, raise_coercion_error))
+    elsif type.is_a?(T::Types::Simple)
+      _convert(value, type.raw_type, raise_coercion_error)
+    elsif type.is_a?(T::Types::Union)
+      true_idx = T.let(nil, T.nilable(Integer))
+      false_idx = T.let(nil, T.nilable(Integer))
+      nil_idx = T.let(nil, T.nilable(Integer))
+
+      type.types.each_with_index do |t, i|
+        nil_idx = i if t.is_a?(T::Types::Simple) && t.raw_type == NilClass
+        true_idx = i if t.is_a?(T::Types::Simple) && t.raw_type == TrueClass
+        false_idx = i if t.is_a?(T::Types::Simple) && t.raw_type == FalseClass
+      end
+
+      raise ArgumentError.new(
+        'the only supported union types are T.nilable and T::Boolean',
+      ) unless (
+        (!true_idx.nil? && !false_idx.nil? && !nil_idx.nil?) || # T.nilable(T::Boolean)
+        (type.types.length == 2 && (
+          !nil_idx.nil? || (!true_idx.nil? && !false_idx.nil?) # T.nilable || T::Boolean
+        ))
+      )
+
+      if !true_idx.nil? && !false_idx.nil?
+        _convert_simple(value, T::Boolean, raise_coercion_error)
+      else
+        _convert(value, type.types[nil_idx == 0 ? 1 : 0], raise_coercion_error)
+      end
+    elsif type.is_a?(T::Types::TypedHash)
+      return {} if _nil_like?(value, type)
+
+      unless value.respond_to?(:map)
+        raise TypeCoerce::ShapeError.new(value, type)
+      end
+
+      value.map do |k, v|
+        [
+          _convert(k, type.keys, raise_coercion_error),
+          _convert(v, type.values, raise_coercion_error),
+        ]
+      end.to_h
+    elsif Object.const_defined?('T::Private::Types::TypeAlias') &&
+          type.is_a?(T::Private::Types::TypeAlias)
+      _convert(value, type.aliased_type, raise_coercion_error)
+    elsif type.respond_to?(:<) && type < T::Struct
+      return value if value.is_a?(type)
+
+      args = _build_args(value, type, raise_coercion_error)
+      type.new(args)
+    elsif type.respond_to?(:<) && type < T::Enum
+      return value if value.is_a?(type)
+
+      _convert_enum(value, type, raise_coercion_error)
+    else
+      return value if value.is_a?(type)
+
+      _convert_simple(value, type, raise_coercion_error)
+    end
+  end
+
+  def _convert_enum(value, type, raise_coercion_error)
+    if raise_coercion_error
+      type.deserialize(value)
+    else
+      type.try_deserialize(value)
+    end
+  rescue KeyError
+    raise TypeCoerce::CoercionError.new(value, type)
+  end
+
+  def _convert_simple(value, type, raise_coercion_error)
+    return nil if _nil_like?(value, type)
+
+    safe_type_rule = T.let(nil, T.untyped)
+
+    if type == T::Boolean
+      safe_type_rule = SafeType::Boolean.strict
+    elsif value.is_a?(type)
+      return value
+    elsif PRIMITIVE_TYPES.include?(type)
+      safe_type_rule = SafeType.const_get(type.name).strict
+    else
+      safe_type_rule = type
+    end
+
+    if safe_type_rule.is_a?(SafeType::Rule)
+      SafeType::coerce(value, safe_type_rule)
+    else
+      type.new(value)
+    end
+  rescue SafeType::EmptyValueError, SafeType::CoercionError
+    if raise_coercion_error
+      raise TypeCoerce::CoercionError.new(value, type)
+    else
+      nil
+    end
+  end
+
+  def _convert_to_a(ary, type, raise_coercion_error)
+    return [] if _nil_like?(ary, type)
+
+    unless ary.respond_to?(:map)
+      raise TypeCoerce::ShapeError.new(ary, type)
+    end
+
+    ary.map { |value| _convert(value, type, raise_coercion_error) }
+  end
+
+  def _build_args(args, type, raise_coercion_error)
+    return {} if _nil_like?(args, Hash)
+
+    unless args.respond_to?(:each_pair)
+      raise TypeCoerce::ShapeError.new(args, type)
+    end
+
+    props = type.props
+    args.map { |name, value|
+      key = name.to_sym
+      [
+        key,
+        (!props.include?(key) || value.nil?) ?
+          nil : _convert(value, props[key][:type], raise_coercion_error),
+      ]
+    }.to_h.slice(*props.keys)
+  end
+
+  def _nil_like?(value, type)
+    value.nil? || (value == '' && type != String)
+  end
+end

data/spec/coerce_spec.rb

@@ -35,6 +35,17 @@ describe TypeCoerce do
       const :myhash, T::Hash[String, Integer], default: Hash['a' => 1]
     end
 
+    class TestEnum < T::Enum
+      enums do
+        Test = new
+        Other = new
+      end
+    end
+
+    class WithEnum < T::Struct
+      const :myenum, TestEnum
+    end
+
     class CustomType
       attr_reader :a
 
@@ -99,6 +110,7 @@ describe TypeCoerce do
       expect(param.info.name).to eql 'mango'
       expect(param.info.skill_ids).to eql [123, 456]
       expect(param.opt.notes).to eql []
+      expect(TypeCoerce[Param].new.from(param)).to eq(param)
 
       expect(param2.id).to eql 2
       expect(param2.info.name).to eql 'honeydew'
@@ -163,8 +175,10 @@ describe TypeCoerce do
 
   context 'when given custom types' do
     it 'coerces correctly' do
-      T.assert_type!(TypeCoerce[CustomType].new.from(a: 1), CustomType)
-      expect(TypeCoerce[CustomType].new.from(1).a).to be 1
+      obj = TypeCoerce[CustomType].new.from(1)
+      T.assert_type!(obj, CustomType)
+      expect(obj.a).to be 1
+      expect(TypeCoerce[CustomType].new.from(obj)).to be obj
 
       expect{TypeCoerce[UnsupportedCustomType].new.from(1)}.to raise_error(ArgumentError)
       # CustomType2.new(anything) returns Integer 1; 1.is_a?(CustomType2) == false
@@ -251,6 +265,24 @@ describe TypeCoerce do
     end
   end
 
+  context 'when dealing with enums' do
+    it 'coerces a serialized enum correctly' do
+      coerced = TypeCoerce[WithEnum].new.from(myenum: "test")
+      expect(coerced.myenum).to eq(TestEnum::Test)
+    end
+
+    it 'handles a real enum correctly' do
+      coerced = TypeCoerce[WithEnum].new.from(myenum: TestEnum::Test)
+      expect(coerced.myenum).to eq(TestEnum::Test)
+    end
+
+    it 'handles bad enum' do
+      expect {
+        TypeCoerce[WithEnum].new.from(myenum: "bad_key")
+      }.to raise_error(TypeCoerce::CoercionError)
+    end
+  end
+
   it 'works with T.untyped' do
     expect(TypeCoerce[T.untyped].new.from(1)).to eql 1
 

data/spec/soft_error_spec.rb

@@ -4,6 +4,12 @@ require 'sorbet-runtime'
 
 describe TypeCoerce do
   context 'when type errors are soft errors' do
+    class SoftErrorTestEnum < T::Enum
+      enums do
+        Other = new
+      end
+    end
+
     let(:ignore_error) { Proc.new {} }
 
     before(:each) do
@@ -48,6 +54,7 @@ describe TypeCoerce do
 
     it 'works as expected' do
       expect(TypeCoerce[Integer].new.from(invalid_arg)).to eql(nil)
+      expect(TypeCoerce[SoftErrorTestEnum].new.from('bad_key')).to eql(nil)
 
       expect{TypeCoerce[T::Array[Integer]].new.from(1)}.to raise_error(TypeCoerce::ShapeError)
       expect(TypeCoerce[T::Array[Integer]].new.from({a: 1})).to eql([nil])

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sorbet-coerce
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.4.0
 platform: ruby
 authors:
 - Chan Zuckerberg Initiative
@@ -10,20 +10,6 @@ bindir: bin
 cert_chain: []
 date: 2019-10-04 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: sorbet
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.4.4704
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.4.4704
 - !ruby/object:Gem::Dependency
   name: polyfill
   requirement: !ruby/object:Gem::Requirement
@@ -72,44 +58,58 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.4.4704
+- !ruby/object:Gem::Dependency
+  name: sorbet
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.4.4704
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.4.4704
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3.8'
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '3.8'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3.8'
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '3.8'
 - !ruby/object:Gem::Dependency
   name: byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: 11.0.1
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 11.0.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: 11.0.1
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 11.0.1
 description: 
@@ -119,10 +119,9 @@ extensions: []
 extra_rdoc_files: []
 files:
 - lib/bundled_rbi/sorbet-coerce.rbi
-- lib/configuration.rb
-- lib/private/converter.rb
-- lib/private/coverter.rbi
 - lib/sorbet-coerce.rb
+- lib/sorbet-coerce/configuration.rb
+- lib/sorbet-coerce/converter.rb
 - spec/coerce_spec.rb
 - spec/nested_spec.rb
 - spec/soft_error_spec.rb
@@ -147,8 +146,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.7
+rubygems_version: 3.0.8
 signing_key: 
 specification_version: 4
 summary: A type coercion lib works with Sorbet's static type checker and type definitions;

data/lib/private/converter.rb

@@ -1,149 +0,0 @@
-# typed: strict
-require 'safe_type'
-require 'set'
-require 'sorbet-runtime'
-require 'polyfill'
-
-using Polyfill(Hash: %w[#slice])
-
-module TypeCoerce; end
-
-module TypeCoerce::Private
-  class Converter
-    extend T::Sig
-
-    PRIMITIVE_TYPES = T.let(::Set[
-      Date,
-      DateTime,
-      Float,
-      Integer,
-      String,
-      Symbol,
-      Time,
-    ], T.untyped)
-
-    protected
-    sig { params(value: T.untyped, type: T.untyped, raise_coercion_error: T::Boolean).returns(T.untyped) }
-    def _convert(value, type, raise_coercion_error)
-      if type.is_a?(T::Types::Untyped)
-        value
-      elsif type.is_a?(T::Types::TypedArray)
-        _convert_to_a(value, type.type, raise_coercion_error)
-      elsif type.is_a?(T::Types::TypedSet)
-        Set.new(_convert_to_a(value, type.type, raise_coercion_error))
-      elsif type.is_a?(T::Types::Simple)
-        _convert(value, type.raw_type, raise_coercion_error)
-      elsif type.is_a?(T::Types::Union)
-        true_idx = T.let(nil, T.nilable(Integer))
-        false_idx = T.let(nil, T.nilable(Integer))
-        nil_idx = T.let(nil, T.nilable(Integer))
-
-        type.types.each_with_index do |t, i|
-          nil_idx = i if t.is_a?(T::Types::Simple) && t.raw_type == NilClass
-          true_idx = i if t.is_a?(T::Types::Simple) && t.raw_type == TrueClass
-          false_idx = i if t.is_a?(T::Types::Simple) && t.raw_type == FalseClass
-        end
-
-        raise ArgumentError.new(
-          'the only supported union types are T.nilable and T::Boolean',
-        ) unless (
-          (!true_idx.nil? && !false_idx.nil? && !nil_idx.nil?) || # T.nilable(T::Boolean)
-          (type.types.length == 2 && (
-            !nil_idx.nil? || (!true_idx.nil? && !false_idx.nil?) # T.nilable || T::Boolean
-          ))
-        )
-
-        if !true_idx.nil? && !false_idx.nil?
-          _convert_simple(value, T::Boolean, raise_coercion_error)
-        else
-          _convert(value, type.types[nil_idx == 0 ? 1 : 0], raise_coercion_error)
-        end
-      elsif type.is_a?(T::Types::TypedHash)
-        return {} if _nil_like?(value, type)
-
-        unless value.respond_to?(:map)
-          raise TypeCoerce::ShapeError.new(value, type)
-        end
-
-        value.map do |k, v|
-          [
-            _convert(k, type.keys, raise_coercion_error),
-            _convert(v, type.values, raise_coercion_error),
-          ]
-        end.to_h
-      elsif Object.const_defined?('T::Private::Types::TypeAlias') &&
-            type.is_a?(T::Private::Types::TypeAlias)
-        _convert(value, type.aliased_type, raise_coercion_error)
-      elsif type.respond_to?(:<) && type < T::Struct
-        args = _build_args(value, type, raise_coercion_error)
-        type.new(args)
-      else
-        _convert_simple(value, type, raise_coercion_error)
-      end
-    end
-
-    sig { params(value: T.untyped, type: T.untyped, raise_coercion_error: T::Boolean).returns(T.untyped) }
-    def _convert_simple(value, type, raise_coercion_error)
-      return nil if _nil_like?(value, type)
-
-      safe_type_rule = T.let(nil, T.untyped)
-
-      if type == T::Boolean
-        safe_type_rule = SafeType::Boolean.strict
-      elsif value.is_a?(type)
-        return value
-      elsif PRIMITIVE_TYPES.include?(type)
-        safe_type_rule = SafeType.const_get(type.name).strict
-      else
-        safe_type_rule = type
-      end
-
-      if safe_type_rule.is_a?(SafeType::Rule)
-        SafeType::coerce(value, safe_type_rule)
-      else
-        type.new(value)
-      end
-    rescue SafeType::EmptyValueError, SafeType::CoercionError
-      if raise_coercion_error
-        raise TypeCoerce::CoercionError.new(value, type)
-      else
-        nil
-      end
-    end
-
-    sig { params(ary: T.untyped, type: T.untyped, raise_coercion_error: T::Boolean).returns(T.untyped) }
-    def _convert_to_a(ary, type, raise_coercion_error)
-      return [] if _nil_like?(ary, type)
-
-      unless ary.respond_to?(:map)
-        raise TypeCoerce::ShapeError.new(ary, type)
-      end
-
-      ary.map { |value| _convert(value, type, raise_coercion_error) }
-    end
-
-    sig { params(args: T.untyped, type: T.untyped, raise_coercion_error: T::Boolean).returns(T.untyped) }
-    def _build_args(args, type, raise_coercion_error)
-      return {} if _nil_like?(args, Hash)
-
-      unless args.respond_to?(:each_pair)
-        raise TypeCoerce::ShapeError.new(args, type)
-      end
-
-      props = type.props
-      args.map { |name, value|
-        key = name.to_sym
-        [
-          key,
-          (!props.include?(key) || value.nil?) ?
-            nil : _convert(value, props[key][:type], raise_coercion_error),
-        ]
-      }.to_h.slice(*props.keys)
-    end
-
-    sig { params(value: T.untyped, type: T.untyped).returns(T::Boolean) }
-    def _nil_like?(value, type)
-      value.nil? || (value == '' && type != String)
-    end
-  end
-end

data/lib/private/coverter.rbi

@@ -1,10 +0,0 @@
-# typed: true
-module T
-  module Private
-    module Types
-      class TypeAlias
-        def aliased_type; end
-      end
-    end
-  end
-end