active_interaction 3.8.3 → 4.0.0

data/spec/active_interaction/errors_spec.rb

@@ -1,5 +1,3 @@
-# coding: utf-8
-
 require 'spec_helper'
 require 'active_record'
 unless defined?(JRUBY_VERSION) # rubocop:disable Style/IfUnlessModifier
@@ -28,25 +26,6 @@ describe ActiveInteraction::Errors do
 
   subject(:errors) { described_class.new(klass.new) }
 
-  context 'backports' do
-    describe '#delete' do
-      it 'deletes the detailed error' do
-        errors.add(:attribute)
-        errors.delete(:attribute)
-        expect(errors.details).to_not have_key :attribute
-      end
-    end
-
-    describe '#initialize_dup' do
-      it 'duplicates the detailed errors' do
-        errors.add(:attribute)
-        other = errors.dup
-        expect(other.details).to eql errors.details
-        expect(other.details).to_not be errors.details
-      end
-    end
-  end
-
   describe '#merge!' do
     let(:other) { described_class.new(klass.new) }
 
@@ -128,7 +107,7 @@ describe ActiveInteraction::Errors do
                 klass.name => {
                   attributes: {
                     attribute: {
-                      invalid_type: 'is not a valid %{type}'
+                      invalid_type: 'is not a valid %<type>s'
                     }
                   }
                 }
@@ -173,12 +152,12 @@ describe ActiveInteraction::Errors do
             end
           end
 
-          class A < ActiveRecord::Base
+          class A < ActiveRecord::Base # rubocop:disable Lint/ConstantDefinitionInBlock
             has_one :b
             accepts_nested_attributes_for :b
           end
 
-          class B < ActiveRecord::Base
+          class B < ActiveRecord::Base # rubocop:disable Lint/ConstantDefinitionInBlock
             belongs_to :a
 
             validates :name, presence: true
@@ -189,7 +168,7 @@ describe ActiveInteraction::Errors do
 
         it 'merges the nested errors' do
           a.valid?
-          expect(a.errors.messages).to eql(:'b.name' => ["can't be blank"])
+          expect(a.errors.messages).to eq('b.name': ["can't be blank"])
           expect(a.errors.size).to eql 1
           expect { errors.merge!(a.errors) }.to_not raise_error
           expect(errors.size).to eql 1

data/spec/active_interaction/filter_column_spec.rb

@@ -1,5 +1,3 @@
-# coding: utf-8
-
 require 'spec_helper'
 
 describe ActiveInteraction::FilterColumn do

data/spec/active_interaction/filter_spec.rb

@@ -1,5 +1,3 @@
-# coding: utf-8
-
 require 'spec_helper'
 
 describe ActiveInteraction::Filter, :filter do

data/spec/active_interaction/filters/abstract_date_time_filter_spec.rb

@@ -1,5 +1,3 @@
-# coding: utf-8
-
 require 'spec_helper'
 
 describe ActiveInteraction::AbstractDateTimeFilter, :filter do
@@ -9,7 +7,7 @@ describe ActiveInteraction::AbstractDateTimeFilter, :filter do
     let(:value) { nil }
 
     it 'raises an error' do
-      expect { filter.cast(value, nil) }.to raise_error NameError
+      expect { filter.send(:cast, value, nil) }.to raise_error NameError
     end
   end
 end

data/spec/active_interaction/filters/abstract_numeric_filter_spec.rb

@@ -1,5 +1,3 @@
-# coding: utf-8
-
 require 'spec_helper'
 
 describe ActiveInteraction::AbstractNumericFilter, :filter do
@@ -9,7 +7,7 @@ describe ActiveInteraction::AbstractNumericFilter, :filter do
     let(:value) { nil }
 
     it 'raises an error' do
-      expect { filter.cast(value, nil) }.to raise_error NameError
+      expect { filter.send(:cast, value, nil) }.to raise_error NameError
     end
   end
 end

data/spec/active_interaction/filters/array_filter_spec.rb

@@ -1,5 +1,3 @@
-# coding: utf-8
-
 require 'spec_helper'
 
 describe ActiveInteraction::ArrayFilter, :filter do
@@ -36,7 +34,7 @@ describe ActiveInteraction::ArrayFilter, :filter do
   end
 
   describe '#cast' do
-    let(:result) { filter.cast(value, nil) }
+    let(:result) { filter.send(:cast, value, nil) }
 
     context 'with an Array' do
       let(:value) { [] }
@@ -46,6 +44,20 @@ describe ActiveInteraction::ArrayFilter, :filter do
       end
     end
 
+    context 'with an implicit Array' do
+      let(:value) do
+        Class.new do
+          def to_ary
+            [1, 2, 3]
+          end
+        end.new
+      end
+
+      it 'returns the Array' do
+        expect(result).to eql value.to_ary
+      end
+    end
+
     context 'with a heterogenous Array' do
       let(:value) { [[], false, 0.0, {}, 0, '', :''] }
 
@@ -84,21 +96,35 @@ describe ActiveInteraction::ArrayFilter, :filter do
       end
     end
 
-    context 'with a nested object filter' do
-      let(:block) { proc { object } }
-      let(:name) { :objects }
-      let(:value) { [Object.new] }
+    [
+      %i[object class],
+      %i[record class],
+      %i[interface from]
+    ].each do |(type, option)|
+      context "with a nested #{type} filter" do
+        let(:block) { proc { public_send(type) } }
+        let(:name) { :objects }
+        let(:value) { [''] }
+
+        it 'does not raise an error' do
+          expect { result }.to_not raise_error
+        end
 
-      it 'does not raise an error' do
-        expect { result }.to_not raise_error
-      end
+        it 'has a filter with the right key' do
+          expect(filter.filters).to have_key(:'0')
+        end
 
-      it 'has a filter with the right key' do
-        expect(filter.filters).to have_key(:object)
-      end
+        it 'has a filter with the right option' do
+          expect(filter.filters[:'0'].options).to have_key(option)
+        end
+
+        context 'with a class set' do
+          let(:block) { proc { public_send(type, "#{option}": String) } }
 
-      it 'has a filter with the right name' do
-        expect(filter.filters[:object].name).to eql(:object)
+          it "does not override the #{option}" do
+            expect(filter.filters[:'0'].options[option]).to eql String
+          end
+        end
       end
     end
   end

data/spec/active_interaction/filters/boolean_filter_spec.rb

@@ -1,5 +1,3 @@
-# coding: utf-8
-
 require 'spec_helper'
 
 describe ActiveInteraction::BooleanFilter, :filter do
@@ -10,7 +8,21 @@ describe ActiveInteraction::BooleanFilter, :filter do
     context 'falsey' do
       [false, '0', 'false', 'FALSE', 'off', 'OFF'].each do |value|
         it "returns false for #{value.inspect}" do
-          expect(filter.cast(value, nil)).to be_falsey
+          expect(filter.send(:cast, value, nil)).to be_falsey
+        end
+      end
+
+      context 'with an implicit string' do
+        let(:value) do
+          Class.new do
+            def to_str
+              'false'
+            end
+          end.new
+        end
+
+        it 'returns false' do
+          expect(filter.send(:cast, value, nil)).to be_falsey
         end
       end
     end
@@ -18,7 +30,49 @@ describe ActiveInteraction::BooleanFilter, :filter do
     context 'truthy' do
       [true, '1', 'true', 'TRUE', 'on', 'ON'].each do |value|
         it "returns true for #{value.inspect}" do
-          expect(filter.cast(value, nil)).to be_truthy
+          expect(filter.send(:cast, value, nil)).to be_truthy
+        end
+      end
+
+      context 'with an implicit string' do
+        let(:value) do
+          Class.new do
+            def to_str
+              'true'
+            end
+          end.new
+        end
+
+        it 'returns true' do
+          expect(filter.send(:cast, value, nil)).to be_truthy
+        end
+      end
+    end
+
+    context 'with a blank String' do
+      let(:value) do
+        Class.new do
+          def to_str
+            ' '
+          end
+        end.new
+      end
+
+      context 'optional' do
+        include_context 'optional'
+
+        it 'returns the default' do
+          expect(filter.send(:cast, value, nil)).to eql options[:default]
+        end
+      end
+
+      context 'required' do
+        include_context 'required'
+
+        it 'raises an error' do
+          expect do
+            filter.send(:cast, value, nil)
+          end.to raise_error ActiveInteraction::MissingValueError
         end
       end
     end

data/spec/active_interaction/filters/date_filter_spec.rb

@@ -1,5 +1,3 @@
-# coding: utf-8
-
 require 'spec_helper'
 
 describe ActiveInteraction::DateFilter, :filter do
@@ -15,7 +13,7 @@ describe ActiveInteraction::DateFilter, :filter do
   end
 
   describe '#cast' do
-    let(:result) { filter.cast(value, nil) }
+    let(:result) { filter.send(:cast, value, nil) }
 
     context 'with a Date' do
       let(:value) { Date.new }
@@ -63,6 +61,48 @@ describe ActiveInteraction::DateFilter, :filter do
       end
     end
 
+    context 'with an implicit String' do
+      let(:value) do
+        Class.new do
+          def to_str
+            '2011-12-13'
+          end
+        end.new
+      end
+
+      it 'returns a Date' do
+        expect(result).to eql Date.parse(value)
+      end
+    end
+
+    context 'with a blank String' do
+      let(:value) do
+        Class.new do
+          def to_str
+            ' '
+          end
+        end.new
+      end
+
+      context 'optional' do
+        include_context 'optional'
+
+        it 'returns the default' do
+          expect(result).to eql options[:default]
+        end
+      end
+
+      context 'required' do
+        include_context 'required'
+
+        it 'raises an error' do
+          expect do
+            result
+          end.to raise_error ActiveInteraction::MissingValueError
+        end
+      end
+    end
+
     context 'with a GroupedInput' do
       let(:year) { 2012 }
       let(:month) { 1 }

data/spec/active_interaction/filters/date_time_filter_spec.rb

@@ -1,5 +1,3 @@
-# coding: utf-8
-
 require 'spec_helper'
 
 describe ActiveInteraction::DateTimeFilter, :filter do
@@ -15,7 +13,7 @@ describe ActiveInteraction::DateTimeFilter, :filter do
   end
 
   describe '#cast' do
-    let(:result) { filter.cast(value, nil) }
+    let(:result) { filter.send(:cast, value, nil) }
 
     context 'with a Datetime' do
       let(:value) { DateTime.new }
@@ -63,6 +61,48 @@ describe ActiveInteraction::DateTimeFilter, :filter do
       end
     end
 
+    context 'with an implicit String' do
+      let(:value) do
+        Class.new do
+          def to_str
+            '2011-12-13T14:15:16+17:18'
+          end
+        end.new
+      end
+
+      it 'returns a DateTime' do
+        expect(result).to eql DateTime.parse(value)
+      end
+    end
+
+    context 'with a blank String' do
+      let(:value) do
+        Class.new do
+          def to_str
+            ' '
+          end
+        end.new
+      end
+
+      context 'optional' do
+        include_context 'optional'
+
+        it 'returns the default' do
+          expect(result).to eql options[:default]
+        end
+      end
+
+      context 'required' do
+        include_context 'required'
+
+        it 'raises an error' do
+          expect do
+            result
+          end.to raise_error ActiveInteraction::MissingValueError
+        end
+      end
+    end
+
     context 'with a GroupedInput' do
       let(:year) { 2012 }
       let(:month) { 1 }

data/spec/active_interaction/filters/decimal_filter_spec.rb

@@ -1,5 +1,3 @@
-# coding: utf-8
-
 require 'spec_helper'
 
 describe ActiveInteraction::DecimalFilter, :filter do
@@ -15,7 +13,7 @@ describe ActiveInteraction::DecimalFilter, :filter do
   end
 
   describe '#cast' do
-    let(:result) { filter.cast(value, nil) }
+    let(:result) { filter.send(:cast, value, nil) }
 
     context 'with a Float' do
       let(:value) { rand }
@@ -35,6 +33,20 @@ describe ActiveInteraction::DecimalFilter, :filter do
       end
     end
 
+    context 'with an implicit Integer' do
+      let(:value) do
+        Class.new do
+          def to_int
+            @to_int ||= rand(1 << 16)
+          end
+        end.new
+      end
+
+      it 'returns a BigDecimal' do
+        expect(result).to eql BigDecimal(value.to_int)
+      end
+    end
+
     context 'with a Numeric' do
       let(:value) { rand(1 << 16) }
 
@@ -60,6 +72,48 @@ describe ActiveInteraction::DecimalFilter, :filter do
         end.to raise_error ActiveInteraction::InvalidValueError
       end
     end
+
+    context 'with an implicit String' do
+      let(:value) do
+        Class.new do
+          def to_str
+            '1.1'
+          end
+        end.new
+      end
+
+      it 'returns a BigDecimal' do
+        expect(result).to eql BigDecimal(value)
+      end
+    end
+
+    context 'with a blank String' do
+      let(:value) do
+        Class.new do
+          def to_str
+            ' '
+          end
+        end.new
+      end
+
+      context 'optional' do
+        include_context 'optional'
+
+        it 'returns the default' do
+          expect(result).to eql options[:default]
+        end
+      end
+
+      context 'required' do
+        include_context 'required'
+
+        it 'raises an error' do
+          expect do
+            result
+          end.to raise_error ActiveInteraction::MissingValueError
+        end
+      end
+    end
   end
 
   describe '#database_column_type' do