transproc 1.0.3 → 1.1.0

data/spec/unit/transformer/dsl_spec.rb

@@ -0,0 +1,15 @@
+RSpec.describe Transproc::Transformer do
+  let(:container) { Module.new { extend Transproc::Registry } }
+  let(:klass) { Transproc::Transformer[container] }
+  let(:transformer) { klass.new }
+
+  context 'when invalid method is used' do
+    it 'raises an error on initialization' do
+      klass.define! do
+        not_valid
+      end
+
+      expect { klass.new }.to raise_error(Transproc::Compiler::InvalidFunctionNameError, /not_valid/)
+    end
+  end
+end

data/spec/unit/transformer/instance_methods_spec.rb

@@ -0,0 +1,25 @@
+RSpec.describe Transproc::Transformer, 'instance methods' do
+  subject(:transformer) do
+    Class.new(Transproc::Transformer[registry]) do
+      define! do
+        map_array(&:capitalize)
+      end
+
+      def capitalize(input)
+        input.upcase
+      end
+    end.new
+  end
+
+  let(:registry) do
+    Module.new do
+      extend Transproc::Registry
+
+      import Transproc::ArrayTransformations
+    end
+  end
+
+  it 'registers a new transformation function' do
+    expect(transformer.call(%w[foo bar])).to eql(%w[FOO BAR])
+  end
+end

data/spec/unit/transformer_spec.rb

@@ -1,4 +1,7 @@
-require 'spec_helper'
+# frozen_string_literal: true
+
+require 'ostruct'
+require 'dry/equalizer'
 
 describe Transproc::Transformer do
   let(:container) { Module.new { extend Transproc::Registry } }
@@ -6,23 +9,76 @@ describe Transproc::Transformer do
   let(:transformer) { klass.new }
 
   describe '.container' do
-    it { expect(klass.container).to eq container }
+    it 'returns the configured container' do
+      expect(klass.container).to be(container)
+    end
 
-    context 'with default transformer' do
-      let(:klass) { described_class }
+    context 'with setter argument' do
+      let(:container) { double(:custom_container) }
+
+      it 'sets and returns the container' do
+        klass.container(container)
 
-      it 'raises exception because there is no container by default' do
-        message = 'Transformer function registry is empty. '\
-                  'Provide your registry via Transproc::Transformer[YourRegistry]'
-        expect { klass.container }.to raise_error(ArgumentError, message)
+        expect(klass.container).to be(container)
       end
     end
+  end
 
-    context 'with setter argument' do
-      subject! { klass.container({}) }
+  describe '.method_missing' do
+    context 'when the method name matches a registered function' do
+      it 'registers a transformation without args' do
+        container.import(Transproc::Coercions)
 
-      it 'sets and returns the container' do
-        expect(klass.container).to eq({})
+        func = klass.t(:to_string)
+
+        result = klass.to_string
+
+        expect(result).to eql(func)
+      end
+
+      it 'registers a transformation with args' do
+        container.import(Transproc::HashTransformations)
+
+        func = klass.t(:rename_keys, id: :user_id)
+
+        result = klass.rename_keys(id: :user_id)
+
+        expect(result).to eql(func)
+      end
+
+      it 'registers a transformation with a block' do
+        container.import(Transproc::ArrayTransformations)
+        container.import(Transproc::HashTransformations)
+
+        func = klass.t(:map_array, klass.t(:rename_keys, id: :user_id))
+
+        result = klass.map_array { rename_keys(id: :user_id) }
+
+        expect(result).to eql(func)
+      end
+
+      it 'registers a transformation with args and a block' do
+        container.import(Transproc::HashTransformations)
+
+        func = klass.t(:map_value, :user, klass.t(:rename_keys, id: :user_id))
+
+        result = klass.map_value(:user) { rename_keys(id: :user_id) }
+
+        expect(result).to eql(func)
+      end
+
+      it 'works with #method' do
+        container.import(Transproc::Coercions)
+
+        func = klass.t(:to_string)
+
+        expect(klass.method(:to_string).()).to eql(func)
+      end
+    end
+
+    context 'when the method name does not match any registered function' do
+      it 'raises NoMethodError' do
+        expect { klass.not_here }.to raise_error(NoMethodError, /not_here/)
       end
     end
   end
@@ -37,11 +93,13 @@ describe Transproc::Transformer do
         end
       end
     end
+
     let(:superclass) do
       Class.new(Transproc::Transformer[container]) do
         arbitrary ->(v) { v + 1 }
       end
     end
+
     let(:subclass) do
       Class.new(superclass) do
         arbitrary ->(v) { v * 2 }
@@ -49,38 +107,53 @@ describe Transproc::Transformer do
     end
 
     it 'inherits container from superclass' do
-      expect(subclass.container).to eq superclass.container
+      expect(subclass.container).to be(superclass.container)
     end
 
-    it 'does not inherit transproc from superclass' do
-      expect(superclass.new.call(2)).to eq 3
-      expect(subclass.new.call(2)).to eq 4
+    it 'inherits transproc from superclass' do
+      expect(superclass.new.call(2)).to be(3)
+      expect(subclass.new.call(2)).to be(6)
     end
   end
 
   describe '.[]' do
+    subject(:subclass) { klass[another_container] }
+
     let(:another_container) { double('Transproc') }
 
-    subject(:subclass) { klass[another_container] }
+    it 'sets a container' do
+      expect(subclass.container).to be(another_container)
+    end
 
-    it { expect(subclass.container).to eq(another_container) }
-    it { is_expected.to be_a(::Class) }
-    it { expect(subclass.ancestors).to include(Transproc::Transformer) }
+    it 'returns a class' do
+      expect(subclass).to be_a(Class)
+    end
+
+    it 'creates a subclass of Transformer' do
+      expect(subclass).to be < Transproc::Transformer
+    end
 
     it 'does not change super class' do
-      expect(klass.container).to eq(container)
+      expect(klass.container).to be(container)
+    end
+
+    it 'does not inherit transproc' do
+      expect(klass[container].transproc).to be_nil
     end
 
     context 'with predefined transformer' do
       let(:klass) do
-        Class.new(Transproc::Transformer) do
+        Class.new(Transproc::Transformer[container]) do
+          container.import Transproc::Coercions
+          container.import Transproc::HashTransformations
+
           map_value :attr, t(:to_symbol)
         end
       end
-    end
 
-    it 'does not inherit transproc' do
-      expect(klass[container].transproc).to be_nil
+      it "inherits parent's transproc" do
+        expect(klass[container].transproc).to eql(klass.transproc)
+      end
     end
   end
 
@@ -96,12 +169,14 @@ describe Transproc::Transformer do
         end
       end
     end
+
     let(:klass) { Transproc::Transformer[container] }
 
     it 'defines anonymous transproc' do
       transproc = klass.define do
         map_value(:attr, t(:to_symbol))
       end
+
       expect(transproc[attr: 'abc']).to eq(attr: :abc)
     end
 
@@ -109,6 +184,7 @@ describe Transproc::Transformer do
       transproc = klass.build do
         map_value(:attr, t(:to_symbol))
       end
+
       expect(transproc[attr: 'abc']).to eq(attr: :abc)
     end
 
@@ -116,6 +192,7 @@ describe Transproc::Transformer do
       klass.define do
         map_value(:attr, :to_sym.to_proc)
       end
+
       expect(klass.transproc).to be_nil
     end
 
@@ -129,12 +206,14 @@ describe Transproc::Transformer do
           end
         end
       end
+
       let(:klass) { described_class[container] }
 
       it 'uses a container from the transformer' do
         transproc = klass.define do
           arbitrary ->(v) { v + 1 }
         end
+
         expect(transproc.call(2)).to eq 3
       end
     end
@@ -155,12 +234,15 @@ describe Transproc::Transformer do
         transproc = klass.define do
           map_value :attr, ->(v) { v * 2 }
         end
+
         expect(transproc.call(attr: 2)).to eq(attr: 4)
       end
     end
   end
 
   describe '.t' do
+    subject(:klass) { Transproc::Transformer[container] }
+
     let(:container) do
       Module.new do
         extend Transproc::Registry
@@ -174,9 +256,9 @@ describe Transproc::Transformer do
       end
     end
 
-    subject!(:klass) { Transproc::Transformer[container] }
-
-    it { expect(klass.t(:custom, '_bar')).to eq container[:custom, '_bar'] }
+    it 'returns a registed function' do
+      expect(klass.t(:custom, '_bar')).to eql(container[:custom, '_bar'])
+    end
 
     it 'is useful in DSL' do
       transproc = Class.new(klass) do
@@ -214,13 +296,16 @@ describe Transproc::Transformer do
           symbolize_keys
           rename_keys user_name: :name
           nest :address, [:city, :street, :zipcode]
+
           map_value :address do
             constructor_inject Test::Address
           end
+
           constructor_inject Test::User
         end
       end
     end
+
     let(:input) do
       [
         { 'user_name' => 'Jane',
@@ -230,7 +315,8 @@ describe Transproc::Transformer do
         }
       ]
     end
-    let(:output) do
+
+    let(:expected_output) do
       [
         Test::User.new(
           name: 'Jane',
@@ -245,38 +331,40 @@ describe Transproc::Transformer do
 
     before do
       module Test
-        class User
-          include Anima.new(:name, :address)
+        class User < OpenStruct
+          include Dry::Equalizer(:name, :address)
         end
 
-        class Address
-          include Anima.new(:city, :street, :zipcode)
+        class Address < OpenStruct
+          include Dry::Equalizer(:city, :street, :zipcode)
         end
       end
     end
 
-    subject! { transformer.call(input) }
-
-    it { is_expected.to eq(output) }
+    it "transforms input" do
+      expect(transformer.(input)).to eql(expected_output)
+    end
 
     context 'with custom registry' do
       let(:klass) do
         Class.new(Transproc::Transformer[registry]) do
-          custom ' is awesome'
+          append ' is awesome'
         end
       end
+
       let(:registry) do
         Module.new do
           extend Transproc::Registry
 
-          def self.custom(value, suffix)
+          def self.append(value, suffix)
             value + suffix
           end
         end
       end
-      let(:input) { 'transproc' }
 
-      it { is_expected.to eq('transproc is awesome') }
+      it 'uses custom functions' do
+        expect(transformer.('transproc')).to eql('transproc is awesome')
+      end
     end
   end
 end

data/spec/unit/transproc_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
 describe Transproc do
   describe 'composition' do

data/transproc.gemspec

@@ -17,8 +17,4 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
   spec.required_ruby_version = '>= 2.3.0'
-
-  spec.add_development_dependency 'bundler', '~> 1.7'
-  spec.add_development_dependency 'rake', '~> 10.0'
-  spec.add_development_dependency 'rspec', '~> 3.3'
 end

metadata

@@ -1,57 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: transproc
 version: !ruby/object:Gem::Version
-  version: 1.0.3
+  version: 1.1.0
 platform: ruby
 authors:
 - Piotr Solnica
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-12-01 00:00:00.000000000 Z
-dependencies:
-- !ruby/object:Gem::Dependency
-  name: bundler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.7'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.7'
-- !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '10.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '10.0'
-- !ruby/object:Gem::Dependency
-  name: rspec
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.3'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.3'
+date: 2019-07-18 00:00:00.000000000 Z
+dependencies: []
 description: Transform Ruby objects in functional style
 email:
 - piotr.solnica@gmail.com
@@ -59,10 +17,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".codeclimate.yml"
 - ".gitignore"
 - ".rspec"
-- ".rubocop.yml"
-- ".rubocop_todo.yml"
 - ".travis.yml"
 - CHANGELOG.md
 - Gemfile
@@ -76,9 +33,11 @@ files:
 - lib/transproc/array/combine.rb
 - lib/transproc/class.rb
 - lib/transproc/coercions.rb
+- lib/transproc/compiler.rb
 - lib/transproc/composer.rb
 - lib/transproc/composite.rb
 - lib/transproc/conditional.rb
+- lib/transproc/constants.rb
 - lib/transproc/error.rb
 - lib/transproc/function.rb
 - lib/transproc/functions.rb
@@ -90,11 +49,10 @@ files:
 - lib/transproc/support/deprecations.rb
 - lib/transproc/transformer.rb
 - lib/transproc/transformer/class_interface.rb
+- lib/transproc/transformer/deprecated/class_interface.rb
+- lib/transproc/transformer/dsl.rb
 - lib/transproc/version.rb
-- rakelib/mutant.rake
-- rakelib/rubocop.rake
 - spec/spec_helper.rb
-- spec/support/mutant.rb
 - spec/unit/array/combine_spec.rb
 - spec/unit/array_transformations_spec.rb
 - spec/unit/class_transformations_spec.rb
@@ -108,6 +66,9 @@ files:
 - spec/unit/recursion_spec.rb
 - spec/unit/registry_spec.rb
 - spec/unit/store_spec.rb
+- spec/unit/transformer/class_interface_spec.rb
+- spec/unit/transformer/dsl_spec.rb
+- spec/unit/transformer/instance_methods_spec.rb
 - spec/unit/transformer_spec.rb
 - spec/unit/transproc_spec.rb
 - transproc.gemspec
@@ -130,14 +91,12 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Transform Ruby objects in functional style
 test_files:
 - spec/spec_helper.rb
-- spec/support/mutant.rb
 - spec/unit/array/combine_spec.rb
 - spec/unit/array_transformations_spec.rb
 - spec/unit/class_transformations_spec.rb
@@ -151,5 +110,8 @@ test_files:
 - spec/unit/recursion_spec.rb
 - spec/unit/registry_spec.rb
 - spec/unit/store_spec.rb
+- spec/unit/transformer/class_interface_spec.rb
+- spec/unit/transformer/dsl_spec.rb
+- spec/unit/transformer/instance_methods_spec.rb
 - spec/unit/transformer_spec.rb
 - spec/unit/transproc_spec.rb