rest_my_case 1.3.2 → 1.3.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f2d0b9d8de92ec372285332d8dce7720f3224323
-  data.tar.gz: 3bfcd6e611ee75c3ec7ff74cde2246ad80fb2a9e
+  metadata.gz: befb7a3cb43ce0e61ec02c2fc33afe9e1d9f734b
+  data.tar.gz: 18ac6c5fcf8ddb9a3d8534e7697fd1ae7995d299
 SHA512:
-  metadata.gz: a8f3aea034cbf77270ff5a6e9f910959cbd8c536029c66f2b822490947346b1eefea71a960a010f0b8df540d25672bec7cbd61c67aa3caf765d01b94ad2ec745
-  data.tar.gz: b000730a80b5b848b4890154c5993a26d4d11d52a55c63b5bafb0c8ad22b1c7a65399c62b928fb9771f4132255be863c504d0d3b20fc1b3860f16332dfa2ff83
+  metadata.gz: 74deb7063872d6ce4f3a0ed098238b70c27f671f6611ccf291c8facfbe31d5f33f4b18fb368ec04de1b818453c514332422b4e4720fd06d20fa33f90ad6e6529
+  data.tar.gz: 57cce6814654046a2a9700952aff2dace3fceb10abb4d804d788facd9263bd9c17a294b802e255c6341e0426085c01efa47f55d21e015c54311de0c62d5357ad

data/lib/rest_my_case/base.rb

@@ -45,7 +45,7 @@ module RestMyCase
 
     def initialize(context, options = {})
       @context = context
-      @options = options
+      @options = options.dup
     end
 
     def setup;  end
@@ -56,14 +56,6 @@ module RestMyCase
 
     def final; end
 
-    def fail(message = '')
-      abort && @context.errors[self.class.name].push(message)
-    end
-
-    def fail!(message = '')
-      fail(message) && raise(Errors::Abort)
-    end
-
     def abort
       options[:should_abort] = true
     end
@@ -72,6 +64,14 @@ module RestMyCase
       abort && raise(Errors::Abort)
     end
 
+    def fail(message = '')
+      abort && context.errors[self.class.name].push(message)
+    end
+
+    def fail!(message = '')
+      fail(message) && raise(Errors::Abort)
+    end
+
     def skip
       options[:should_skip] = true
     end

data/lib/rest_my_case/judge/base.rb

@@ -35,7 +35,7 @@ module RestMyCase
       def run_rollback_methods
         return nil unless @use_case_that_aborted
 
-        @performed_use_cases.revert.each do |use_case|
+        @performed_use_cases.reverse.each do |use_case|
           run_method(:rollback, use_case)
         end
       end
@@ -62,7 +62,7 @@ module RestMyCase
         begin
           run_method(method_name, use_case)
 
-          use_case.options[:should_abort]
+          use_case.options[:should_abort] && @use_case_that_aborted = use_case
         rescue Errors::Skip => exception
           false
         rescue Errors::Abort => exception

data/lib/rest_my_case/version.rb

@@ -1,5 +1,5 @@
 module RestMyCase
 
-  VERSION = "1.3.2"
+  VERSION = "1.3.5"
 
 end

data/spec/rest_my_case/base_spec.rb

@@ -3,15 +3,15 @@ require 'spec_helper'
 describe RestMyCase::Base do
 
   describe ".dependencies" do
-    it " should only list the class's dependencies" do
+    it "should only list the class's dependencies" do
       expect(RestMyCaseBase::CreatePostWithComments.dependencies).to \
         eq [RestMyCaseBase::BuildComments, RestMyCaseBase::CreateComments]
     end
   end
 
   describe ".context_accessor" do
-    let(:context)   { RestMyCase::TrialCase::Context.new(id: 1, comment: 'my comment', session: -1) }
-    let(:use_case)  { RestMyCaseBase::CreatePostWithComments.new(context) }
+    let(:context) { RestMyCase::TrialCase::Context.new(id: 1, comment: 'my comment', session: -1) }
+    let(:use_case)   { RestMyCaseBase::CreatePostWithComments.new(context) }
 
     it "Should create getters targeting to context" do
       expect(use_case.respond_to?(:comment)).to be true
@@ -22,7 +22,7 @@ describe RestMyCase::Base do
     end
 
     it "Getter should delegate to context" do
-      expect(use_case.comment).to eq context.comment
+      expect(use_case.comment).to eq 'my comment'
     end
 
     it "Setter should delegate to context" do
@@ -32,7 +32,8 @@ describe RestMyCase::Base do
   end
 
   describe ".context_writer" do
-    let(:use_case) { RestMyCaseBase::CreatePostWithComments.new(RestMyCase::TrialCase::Context.new) }
+    let(:context) { RestMyCase::TrialCase::Context.new }
+    let(:use_case)   { RestMyCaseBase::CreatePostWithComments.new(context) }
 
     it "Should create setters targeting to context" do
       expect(use_case.respond_to?(:id)).to be false
@@ -47,8 +48,8 @@ describe RestMyCase::Base do
   end
 
   describe ".context_reader" do
-    let(:context)   { RestMyCase::TrialCase::Context.new(id: 1, comment: 'my comment', session: -1) }
-    let(:use_case)  { RestMyCaseBase::CreatePostWithComments.new(context) }
+    let(:context) { RestMyCase::TrialCase::Context.new(id: 1, comment: 'my comment', session: -1) }
+    let(:use_case)   { RestMyCaseBase::CreatePostWithComments.new(context) }
 
     it "Should create getters targeting to context" do
       expect(use_case.respond_to?(:session)).to be true
@@ -56,21 +57,29 @@ describe RestMyCase::Base do
     end
 
     it "Getter should delegate to context" do
-      expect(use_case.session).to eq context.session
+      expect(use_case.session).to eq -1
     end
   end
 
   describe ".perform" do
 
-    context "When something that doesn't responde to #to_hash is used" do
-      it "should raise an exception" do
-        expect { RestMyCaseBase::CreatePost.perform(Object.new) }.to \
-          raise_error(ArgumentError)
+    context "When a use case #fail during the setup process" do
+      before do
+        class Perform::ValidateName < RestMyCase::Base
+          def setup
+            fail('no name!')
+            context.setup << self.class.name
+          end
+        end
+
+        @context = Perform::CreatePost.perform
       end
-    end
 
-    context "When a use case #abort during the setup process" do
-      before { @context = Perform::CreatePost.perform }
+      after do
+        class Perform::ValidateName < RestMyCase::Base
+          def setup; context.setup << self.class.name; end
+        end
+      end
 
       it "context should reflect an invalid state" do
         expect(@context.valid?).to be false
@@ -80,11 +89,74 @@ describe RestMyCase::Base do
         expect(@context.errors.keys.length).to be 1
       end
 
-      it "context must not be populated by the #perform methods" do
-        # binding.pry
+      it "context prove that only the correct method have ran" do
         expect(@context.setup.length).to be 2
         expect(@context.perform.length).to be 0
-        expect(@context.rollback.length).to be 2
+        expect(@context.rollback.length).to be 0
+        expect(@context.final.length).to be 6
+      end
+    end
+
+    context "When a use case #fail! during the setup process" do
+      before do
+        class Perform::ValidateName < RestMyCase::Base
+          def setup
+            fail!('no name!')
+            context.setup << self.class.name
+          end
+        end
+
+        @context = Perform::CreatePost.perform
+      end
+
+      after do
+        class Perform::ValidateName < RestMyCase::Base
+          def setup; context.setup << self.class.name; end
+        end
+      end
+
+      it "context should reflect an invalid state" do
+        expect(@context.valid?).to be false
+      end
+
+      it "context should contain only one error" do
+        expect(@context.errors.keys.length).to be 1
+      end
+
+      it "context prove that only the correct method have ran" do
+        expect(@context.setup.length).to be 1
+        expect(@context.perform.length).to be 0
+        expect(@context.rollback.length).to be 0
+        expect(@context.final.length).to be 6
+      end
+    end
+
+    context "When a use case #skip during the setup process" do
+      before do
+        class Perform::ValidateBody < RestMyCase::Base
+          def setup
+            skip
+            context.setup << self.class.name
+          end
+        end
+
+        @context = Perform::CreatePost.perform
+      end
+
+      after do
+        class Perform::ValidateName < RestMyCase::Base
+          def setup; context.setup << self.class.name; end
+        end
+      end
+
+      it "context should reflect an valid state" do
+        expect(@context.valid?).to be true
+      end
+
+      it "context prove that only the correct method have ran" do
+        expect(@context.setup.length).to be 6
+        expect(@context.perform.length).to be 5
+        expect(@context.rollback.length).to be 0
         expect(@context.final.length).to be 6
       end
     end

data/spec/support/perform.rb

@@ -1,11 +1,7 @@
 module Perform
 
   class ValidateName < RestMyCase::Base
-    context_reader :name
-
     def setup
-      fail('no name present!') if name.nil?
-
       context.setup << self.class.name
     end
 
@@ -23,11 +19,7 @@ module Perform
   end
 
   class ValidateBody < RestMyCase::Base
-    context_reader :body
-
     def setup
-      fail('no body present!') if body.nil?
-
       context.setup << self.class.name
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rest_my_case
 version: !ruby/object:Gem::Version
-  version: 1.3.2
+  version: 1.3.5
 platform: ruby
 authors:
 - goncalvesjoao