active_interaction 4.1.0 → 5.0.0

data/spec/active_interaction/filters/integer_filter_spec.rb

@@ -4,14 +4,14 @@ describe ActiveInteraction::IntegerFilter, :filter do
   include_context 'filters'
   it_behaves_like 'a filter'
 
-  describe '#cast' do
-    let(:result) { filter.send(:cast, value, nil) }
+  describe '#process' do
+    let(:result) { filter.process(value, nil) }
 
     context 'with an Integer' do
       let(:value) { rand(1 << 16) }
 
       it 'returns the Integer' do
-        expect(result).to eql value
+        expect(result.value).to eql value
       end
     end
 
@@ -19,7 +19,7 @@ describe ActiveInteraction::IntegerFilter, :filter do
       let(:value) { rand(1 << 16) + rand }
 
       it 'returns an Integer' do
-        expect(result).to eql value.to_i
+        expect(result.value).to eql value.to_i
       end
     end
 
@@ -27,17 +27,18 @@ describe ActiveInteraction::IntegerFilter, :filter do
       let(:value) { rand(1 << 16).to_s }
 
       it 'returns an Integer' do
-        expect(result).to eql Integer(value, 10)
+        expect(result.value).to eql Integer(value, 10)
       end
     end
 
     context 'with an invalid String' do
       let(:value) { 'invalid' }
 
-      it 'raises an error' do
-        expect do
-          result
-        end.to raise_error ActiveInteraction::InvalidValueError
+      it 'indicates an error' do
+        error = result.errors.first
+
+        expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+        expect(error.type).to be :invalid_type
       end
     end
 
@@ -51,8 +52,7 @@ describe ActiveInteraction::IntegerFilter, :filter do
       end
 
       it 'returns an Integer' do
-        # jRuby freezes on the implicit string value
-        expect(result).to eql Integer(value.to_str, 10)
+        expect(result.value).to eql Integer(value.to_str, 10)
       end
     end
 
@@ -69,34 +69,36 @@ describe ActiveInteraction::IntegerFilter, :filter do
         include_context 'optional'
 
         it 'returns the default' do
-          expect(result).to eql options[:default]
+          expect(result.value).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
+        it 'indicates an error' do
+          error = result.errors.first
+
+          expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+          expect(error.type).to be :missing
         end
       end
     end
 
     it 'supports different bases' do
       expect(
-        described_class.new(name, base: 8).send(:cast, '071', nil)
-      ).to eql 57
-      expect do
-        described_class.new(name, base: 8).send(:cast, '081', nil)
-      end.to raise_error ActiveInteraction::InvalidValueError
+        described_class.new(name, base: 8).process('071', nil).value
+      ).to be 57
+      error = described_class.new(name, base: 8).process('081', nil).errors.first
+
+      expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+      expect(error.type).to be :invalid_type
     end
   end
 
   describe '#database_column_type' do
     it 'returns :integer' do
-      expect(filter.database_column_type).to eql :integer
+      expect(filter.database_column_type).to be :integer
     end
   end
 end

data/spec/active_interaction/filters/interface_filter_spec.rb

@@ -10,8 +10,8 @@ describe ActiveInteraction::InterfaceFilter, :filter do
     let(:name) { :interface_module }
   end
 
-  describe '#cast' do
-    let(:result) { filter.send(:cast, value, nil) }
+  describe '#process' do
+    let(:result) { filter.process(value, nil) }
 
     context 'with an implicit constant name' do
       context 'passed an instance' do
@@ -24,7 +24,7 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           end
 
           it 'returns a the value' do
-            expect(result).to eql value
+            expect(result.value).to eql value
           end
         end
 
@@ -35,7 +35,7 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           end
 
           it 'returns a the value' do
-            expect(result).to eql value
+            expect(result.value).to eql value
           end
         end
 
@@ -46,7 +46,7 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           end
 
           it 'returns a the value' do
-            expect(result).to eql value
+            expect(result.value).to eql value
           end
         end
 
@@ -54,10 +54,11 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           let(:name) { :interface_module }
           let(:value) { Class.new }
 
-          it 'raises an error' do
-            expect do
-              result
-            end.to raise_error ActiveInteraction::InvalidValueError
+          it 'indicates an error' do
+            error = result.errors.first
+
+            expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+            expect(error.type).to be :invalid_type
           end
         end
 
@@ -65,10 +66,11 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           let(:name) { :interface_module }
           let(:value) { nil }
 
-          it 'raises an error' do
-            expect do
-              result
-            end.to raise_error ActiveInteraction::MissingValueError
+          it 'indicates an error' do
+            error = result.errors.first
+
+            expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+            expect(error.type).to be :missing
           end
         end
 
@@ -78,10 +80,11 @@ describe ActiveInteraction::InterfaceFilter, :filter do
             InterfaceClass.new
           end
 
-          it 'raises an error' do
-            expect do
-              result
-            end.to raise_error ActiveInteraction::InvalidValueError
+          it 'indicates an error' do
+            error = result.errors.first
+
+            expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+            expect(error.type).to be :invalid_type
           end
         end
       end
@@ -94,7 +97,7 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           end
 
           it 'returns a the value' do
-            expect(result).to eql value
+            expect(result.value).to eql value
           end
         end
 
@@ -107,7 +110,7 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           end
 
           it 'returns a the value' do
-            expect(result).to eql value
+            expect(result.value).to eql value
           end
         end
 
@@ -115,10 +118,11 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           let(:name) { :interface_class }
           let(:value) { Class }
 
-          it 'raises an error' do
-            expect do
-              result
-            end.to raise_error ActiveInteraction::InvalidValueError
+          it 'indicates an error' do
+            error = result.errors.first
+
+            expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+            expect(error.type).to be :invalid_type
           end
         end
 
@@ -126,10 +130,11 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           let(:name) { :interface_class }
           let(:value) { InterfaceClass }
 
-          it 'raises an error' do
-            expect do
-              result
-            end.to raise_error ActiveInteraction::InvalidValueError
+          it 'indicates an error' do
+            error = result.errors.first
+
+            expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+            expect(error.type).to be :invalid_type
           end
         end
       end
@@ -144,7 +149,7 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           end
 
           it 'returns a the value' do
-            expect(result).to eql value
+            expect(result.value).to eql value
           end
         end
 
@@ -152,10 +157,11 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           let(:name) { :interface_module }
           let(:value) { Module.new }
 
-          it 'raises an error' do
-            expect do
-              result
-            end.to raise_error ActiveInteraction::InvalidValueError
+          it 'indicates an error' do
+            error = result.errors.first
+
+            expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+            expect(error.type).to be :invalid_type
           end
         end
 
@@ -163,10 +169,11 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           let(:name) { :interface_module }
           let(:value) { InterfaceModule }
 
-          it 'raises an error' do
-            expect do
-              result
-            end.to raise_error ActiveInteraction::InvalidValueError
+          it 'indicates an error' do
+            error = result.errors.first
+
+            expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+            expect(error.type).to be :invalid_type
           end
         end
       end
@@ -187,6 +194,7 @@ describe ActiveInteraction::InterfaceFilter, :filter do
       context 'passed an instance' do
         context 'with the module included' do
           before { options.merge!(from: :interface_module) }
+
           let(:value) do
             Class.new do
               include InterfaceModule
@@ -194,29 +202,31 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           end
 
           it 'returns a the value' do
-            expect(result).to eql value
+            expect(result.value).to eql value
           end
         end
 
         context 'with the class inherited from' do
           before { options.merge!(from: :interface_class) }
+
           let(:value) do
             Class.new(InterfaceClass) {}.new # rubocop:disable Lint/EmptyBlock
           end
 
           it 'returns a the value' do
-            expect(result).to eql value
+            expect(result.value).to eql value
           end
         end
 
         context 'that is extended by the ancestor' do
           before { options.merge!(from: :interface_module) }
+
           let(:value) do
             Class.new {}.new.extend(InterfaceModule) # rubocop:disable Lint/EmptyBlock
           end
 
           it 'returns a the value' do
-            expect(result).to eql value
+            expect(result.value).to eql value
           end
         end
 
@@ -224,10 +234,11 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           let(:name) { :interface_class }
           let(:value) { Class.new }
 
-          it 'raises an error' do
-            expect do
-              result
-            end.to raise_error ActiveInteraction::InvalidValueError
+          it 'indicates an error' do
+            error = result.errors.first
+
+            expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+            expect(error.type).to be :invalid_type
           end
         end
 
@@ -235,10 +246,11 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           let(:name) { :interface_class }
           let(:value) { InterfaceClass.new }
 
-          it 'raises an error' do
-            expect do
-              result
-            end.to raise_error ActiveInteraction::InvalidValueError
+          it 'indicates an error' do
+            error = result.errors.first
+
+            expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+            expect(error.type).to be :invalid_type
           end
         end
       end
@@ -246,17 +258,19 @@ describe ActiveInteraction::InterfaceFilter, :filter do
       context 'passed a class' do
         context 'with the class inherited from' do
           before { options.merge!(from: :interface_class) }
+
           let(:value) do
             Class.new(InterfaceClass) {} # rubocop:disable Lint/EmptyBlock
           end
 
           it 'returns a the value' do
-            expect(result).to eql value
+            expect(result.value).to eql value
           end
         end
 
         context 'that is extended by the ancestor' do
           before { options.merge!(from: :interface_module) }
+
           let(:value) do
             Class.new do
               extend InterfaceModule
@@ -264,7 +278,7 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           end
 
           it 'returns a the value' do
-            expect(result).to eql value
+            expect(result.value).to eql value
           end
         end
 
@@ -272,10 +286,11 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           let(:name) { :interface_class }
           let(:value) { Class }
 
-          it 'raises an error' do
-            expect do
-              result
-            end.to raise_error ActiveInteraction::InvalidValueError
+          it 'indicates an error' do
+            error = result.errors.first
+
+            expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+            expect(error.type).to be :invalid_type
           end
         end
 
@@ -283,10 +298,11 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           let(:name) { :interface_class }
           let(:value) { InterfaceClass }
 
-          it 'raises an error' do
-            expect do
-              result
-            end.to raise_error ActiveInteraction::InvalidValueError
+          it 'indicates an error' do
+            error = result.errors.first
+
+            expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+            expect(error.type).to be :invalid_type
           end
         end
       end
@@ -294,6 +310,7 @@ describe ActiveInteraction::InterfaceFilter, :filter do
       context 'passed a module' do
         context 'that is extended by the ancestor' do
           before { options.merge!(from: :interface_module) }
+
           let(:value) do
             Module.new do
               extend InterfaceModule
@@ -301,7 +318,7 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           end
 
           it 'returns a the value' do
-            expect(result).to eql value
+            expect(result.value).to eql value
           end
         end
 
@@ -309,10 +326,11 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           let(:name) { :interface_module }
           let(:value) { Module.new }
 
-          it 'raises an error' do
-            expect do
-              result
-            end.to raise_error ActiveInteraction::InvalidValueError
+          it 'indicates an error' do
+            error = result.errors.first
+
+            expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+            expect(error.type).to be :invalid_type
           end
         end
 
@@ -320,16 +338,18 @@ describe ActiveInteraction::InterfaceFilter, :filter do
           let(:name) { :interface_module }
           let(:value) { InterfaceModule }
 
-          it 'raises an error' do
-            expect do
-              result
-            end.to raise_error ActiveInteraction::InvalidValueError
+          it 'indicates an error' do
+            error = result.errors.first
+
+            expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+            expect(error.type).to be :invalid_type
           end
         end
       end
 
       context 'given an invalid name' do
         before { options.merge!(from: :invalid) }
+
         let(:value) { Object }
 
         it 'raises an error' do
@@ -353,17 +373,18 @@ describe ActiveInteraction::InterfaceFilter, :filter do
         end
 
         it 'returns a the value' do
-          expect(result).to eql value
+          expect(result.value).to eql value
         end
       end
 
       context 'passed an invalid instance' do
         let(:value) { Class.new }
 
-        it 'raises an error' do
-          expect do
-            result
-          end.to raise_error ActiveInteraction::InvalidValueError
+        it 'indicates an error' do
+          error = result.errors.first
+
+          expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+          expect(error.type).to be :invalid_type
         end
       end
 
@@ -377,17 +398,18 @@ describe ActiveInteraction::InterfaceFilter, :filter do
         end
 
         it 'returns a the value' do
-          expect(result).to eql value
+          expect(result.value).to eql value
         end
       end
 
       context 'passed an invalid class' do
         let(:value) { Class }
 
-        it 'raises an error' do
-          expect do
-            result
-          end.to raise_error ActiveInteraction::InvalidValueError
+        it 'indicates an error' do
+          error = result.errors.first
+
+          expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+          expect(error.type).to be :invalid_type
         end
       end
 
@@ -401,17 +423,18 @@ describe ActiveInteraction::InterfaceFilter, :filter do
         end
 
         it 'returns a the value' do
-          expect(result).to eql value
+          expect(result.value).to eql value
         end
       end
 
       context 'passed an invalid module' do
         let(:value) { Module.new }
 
-        it 'raises an error' do
-          expect do
-            result
-          end.to raise_error ActiveInteraction::InvalidValueError
+        it 'indicates an error' do
+          error = result.errors.first
+
+          expect(error).to be_an_instance_of ActiveInteraction::Filter::Error
+          expect(error.type).to be :invalid_type
         end
       end
     end
@@ -432,7 +455,7 @@ describe ActiveInteraction::InterfaceFilter, :filter do
 
   describe '#database_column_type' do
     it 'returns :string' do
-      expect(filter.database_column_type).to eql :string
+      expect(filter.database_column_type).to be :string
     end
   end
 end