rest_my_case 1.10.9 → 1.11.0

data/lib/rest_my_case/status.rb

@@ -1,28 +1,29 @@
+require 'rest_my_case/context/status'
+
 module RestMyCase
 
   module Status
 
-    module ClassMethods
-      def trial_court
-        @trial_court ||= Trial::Court.new \
-          Judge::Base, DefenseAttorney::Base, Base, Context::Status
-      end
-    end
-
     def self.included(parent_class)
-      parent_class.extend ClassMethods
+      return unless parent_class.respond_to? :trial_court
+
+      parent_class.trial_court.context_class = Context::Status
     end
 
     def status
       context.status
     end
 
-    def failure(status, error = nil)
-      error = { message: error } unless error.is_a?(Hash)
+    def failure(status, error_message = nil)
+      if error_message.is_a?(Hash)
+        error_data = error_message
+      else
+        error_data = { message: error_message }
+      end
 
-      error[:status] = status
+      error_data[:status] = status
 
-      error(error)
+      error(error_data)
     end
 
     def failure!(status, error = nil)

data/lib/rest_my_case/trial/case.rb

@@ -12,10 +12,10 @@ module RestMyCase
                   :defendant_class
 
       def initialize(last_ancestor, context_class, use_case_classes, attributes)
-        @context         = build_context attributes, context_class
-        @last_ancestor   = last_ancestor
+        @context = build_context attributes, context_class
+        @last_ancestor = last_ancestor
         @defendant_class = build_defendant(last_ancestor, use_case_classes)
-        @defendant       = @defendant_class.new @context
+        @defendant = @defendant_class.new @context
       end
 
       def aborted

data/lib/rest_my_case/validator.rb

@@ -6,11 +6,6 @@ module RestMyCase
 
       attr_writer :validators
 
-      def trial_court
-        @trial_court ||= Trial::Court.new \
-          Judge::Base, DefenseAttorney::Base, Validator, Context::Base
-      end
-
       def target_options
         @target_options || Helpers.super_method(self, :target_options)
       end
@@ -59,6 +54,8 @@ module RestMyCase
 
     extend ClassMethods
 
+    trial_court.last_ancestor = Validator
+
     self.silence_dependencies_abort = true
 
     extend AccusationAttorneys::HelperMethods
@@ -91,9 +88,7 @@ module RestMyCase
 
       return if Helpers.blank?(targets) || all_validations_green?(targets)
 
-      if parent_target
-        parent_target.errors.add(target_options[:name], :invalid)
-      end
+      parent_target.errors.add(target_options[:name], :invalid) if parent_target
 
       error('unprocessable_entity')
     end

data/lib/rest_my_case/version.rb

@@ -1,5 +1,5 @@
 module RestMyCase
 
-  VERSION = '1.10.9'
+  VERSION = '1.11.0'
 
 end

data/lib/rest_my_case.rb

@@ -6,11 +6,7 @@ require 'rest_my_case/errors/base'
 require 'rest_my_case/config/base'
 require 'rest_my_case/config/general'
 require 'rest_my_case/defense_attorney/base'
-require 'rest_my_case/context/errors/base'
-require 'rest_my_case/context/errors/status'
 require 'rest_my_case/context/base'
-require 'rest_my_case/context/status'
-require 'rest_my_case/context/http_status'
 require 'rest_my_case/judge/base'
 require 'rest_my_case/trial/case'
 require 'rest_my_case/trial/court'

data/rest_my_case.gemspec

@@ -9,7 +9,7 @@ Gem::Specification.new do |gem|
   gem.authors       = ['goncalvesjoao']
   gem.email         = ['goncalves.joao@gmail.com']
   gem.summary       = %q{Quick and light "The Clean Architecture" use case implementation.}
-  gem.description   = %q{Very light Ruby gem with everything you need in a "The Clean Architecture" use case scenario}
+  gem.description   = %q{Light Ruby gem with everything you need in a "The Clean Architecture" use case scenario}
   gem.homepage      = 'https://github.com/goncalvesjoao/rest_my_case'
   gem.license       = 'MIT'
 

data/spec/rest_my_case/accusation_attorneys/base_spec.rb

@@ -9,7 +9,7 @@ describe RestMyCase::AccusationAttorneys::Base do
       before { @instance = AccusationAttorneysTestCase1.new({ attributes: { id: '1' } }) }
 
       it 'should raise an error' do
-        expect { @instance.validate(nil) }.to raise_error
+        expect { @instance.validate(nil) }.to raise_error NotImplementedError
       end
     end
   end

data/spec/rest_my_case/accusation_attorneys/each_spec.rb

@@ -9,7 +9,7 @@ describe RestMyCase::AccusationAttorneys::Each do
       before { @instance = AccusationAttorneysTestCase2.new({ attributes: { id: '1' } }) }
 
       it 'should raise an error' do
-        expect { @instance.validate_each(nil, nil, nil) }.to raise_error
+        expect { @instance.validate_each(nil, nil, nil) }.to raise_error NotImplementedError
       end
     end
   end

data/spec/rest_my_case/accusation_attorneys/format_spec.rb

@@ -64,8 +64,8 @@ describe RestMyCase::AccusationAttorneys::Format do
 
   describe "without options" do
     it 'should raise an error' do
-      expect { Class.new(RestMyCase::Validator) { validates_format_of :body } }.to raise_error
+      expect { Class.new(RestMyCase::Validator) { validates_format_of :body } }.to raise_error ArgumentError
     end
   end
 
-end
+end

data/spec/rest_my_case/base/context_accessor_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe RestMyCase::Base do
+
+  describe ".context_accessor" do
+
+    let(:context)  { RestMyCase::Context::Base.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
+    end
+
+    it "Should create setters targeting to context" do
+      expect(use_case.respond_to?(:comment=)).to be true
+    end
+
+    it "Getter should delegate to context" do
+      expect(use_case.comment).to eq 'my comment'
+    end
+
+    it "Setter should delegate to context" do
+      use_case.comment = 'your comment'
+      expect(use_case.context.comment).to eq 'your comment'
+    end
+
+  end
+
+end

data/spec/rest_my_case/base/context_reader_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+describe RestMyCase::Base do
+
+  describe ".context_reader" do
+
+    let(:context)  { RestMyCase::Context::Base.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
+      expect(use_case.respond_to?(:session=)).to be false
+    end
+
+    it "Getter should delegate to context" do
+      expect(use_case.session).to eq -1
+    end
+
+  end
+
+end

data/spec/rest_my_case/base/context_writer_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+describe RestMyCase::Base do
+
+  describe ".context_writer" do
+
+    let(:context)  { RestMyCase::Context::Base.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
+      expect(use_case.respond_to?(:id=)).to be true
+    end
+
+    it "Setter should delegate to context" do
+      use_case.id = 2
+
+      expect(use_case.context.id).to eq 2
+    end
+
+  end
+
+end

data/spec/rest_my_case/base/dependencies_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+
+describe RestMyCase::Base do
+
+  describe ".dependencies" do
+
+    it "should only list the class's dependencies" do
+      expect(RestMyCaseBase::CreatePostWithComments.dependencies).to \
+        eq [RestMyCaseBase::BuildComments, RestMyCaseBase::CreateComments]
+    end
+
+  end
+
+end

data/spec/rest_my_case/base/invoke!_spec.rb

@@ -0,0 +1,140 @@
+require 'spec_helper'
+
+describe RestMyCase::Base do
+
+  describe "#invoke!" do
+
+    context "when the invokees don't abort" do
+
+      class Perform::ScreammingInvoker < Perform::UseCaseWrapper
+        def perform
+          super
+          invoke! Perform::BuildPost, Perform::Validations, Perform::SavePost
+        end
+      end
+
+      it "should invoke other use cases with the invoker's context and actually alter it" do
+        @context = Perform::ScreammingInvoker.perform
+
+        expect(@context.setup).to eq [
+          "Perform::Logger",
+          "Perform::ScreammingInvoker",
+          "Perform::Logger",
+          "Perform::BuildPost",
+          "Perform::Logger",
+          "Perform::ValidateName",
+          "Perform::Logger",
+          "Perform::ValidateBody",
+          "Perform::Logger",
+          "Perform::Validations",
+          "Perform::Logger",
+          "Perform::SavePost"
+        ]
+        expect(@context.perform).to eq [
+          "Perform::Logger",
+          "Perform::ScreammingInvoker",
+          "Perform::Logger",
+          "Perform::BuildPost",
+          "Perform::Logger",
+          "Perform::ValidateName",
+          "Perform::Logger",
+          "Perform::ValidateBody",
+          "Perform::Logger",
+          "Perform::Validations",
+          "Perform::Logger",
+          "Perform::SavePost"
+        ]
+        expect(@context.rollback).to eq []
+        expect(@context.final).to eq [
+          "Perform::Logger",
+          "Perform::BuildPost",
+          "Perform::Logger",
+          "Perform::ValidateName",
+          "Perform::Logger",
+          "Perform::ValidateBody",
+          "Perform::Logger",
+          "Perform::Validations",
+          "Perform::Logger",
+          "Perform::SavePost",
+          "Perform::Logger",
+          "Perform::ScreammingInvoker"
+        ]
+      end
+
+    end
+
+    context "when the invokees abort" do
+
+      class Perform::Validations2 < Perform::UseCaseWrapper
+        def perform
+          super
+
+          invoke! Perform::ValidateName, Perform::ValidateBody
+
+          context.should_not_be_true = true
+        end
+      end
+
+      class Perform::CreatePost2 < Perform::UseCaseWrapper
+        depends Perform::BuildPost, Perform::Validations2, Perform::SavePost
+      end
+
+      before do
+        @context = Perform::CreatePost2.perform \
+          error: ['Perform::ValidateName_perform', 'Perform::ValidateBody_perform']
+      end
+
+      it "invoker should abort the process it his invokees have also aborted" do
+        expect(@context.setup).to eq [
+          "Perform::Logger",
+          "Perform::BuildPost",
+          "Perform::Logger",
+          "Perform::Validations2",
+          "Perform::Logger",
+          "Perform::SavePost",
+          "Perform::Logger",
+          "Perform::CreatePost2",
+          "Perform::Logger",
+          "Perform::ValidateName",
+          "Perform::Logger",
+          "Perform::ValidateBody"
+        ]
+        expect(@context.perform).to eq [
+          "Perform::Logger",
+          "Perform::BuildPost",
+          "Perform::Logger",
+          "Perform::Validations2",
+          "Perform::Logger",
+          "Perform::ValidateName"
+        ]
+        expect(@context.rollback).to eq [
+          "Perform::ValidateName",
+          "Perform::Logger",
+          "Perform::Validations2",
+          "Perform::Logger",
+          "Perform::BuildPost",
+          "Perform::Logger"
+        ]
+        expect(@context.final).to eq [
+          "Perform::Logger",
+          "Perform::ValidateName",
+          "Perform::Logger",
+          "Perform::ValidateBody",
+          "Perform::Logger",
+          "Perform::BuildPost",
+          "Perform::Logger",
+          "Perform::Validations2",
+          "Perform::Logger",
+          "Perform::SavePost",
+          "Perform::Logger",
+          "Perform::CreatePost2"
+        ]
+
+        expect(@context.should_not_be_true).to be_falsey
+      end
+
+    end
+
+  end
+
+end

data/spec/rest_my_case/base/invoke_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe RestMyCase::Base do
+
+  describe "#invoke" do
+
+    class Perform::Invoker < Perform::UseCaseWrapper
+      def perform
+        super
+        invoke Perform::BuildPost, Perform::Validations, Perform::SavePost
+
+        context.should_be_true = true
+      end
+    end
+
+    it "should invoke other use cases with the invoker's cloned context and they can't alter the invoker's context" do
+      context = Perform::Invoker.perform
+
+      expect(context.setup.length).to be 2
+      expect(context.perform.length).to be 2
+      expect(context.rollback.length).to be 0
+      expect(context.final.length).to be 2
+
+      expect(context.should_be_true).to be true
+    end
+
+  end
+
+end

data/spec/rest_my_case/{base_spec.rb → base/perform_spec.rb}

@@ -2,73 +2,6 @@ require 'spec_helper'
 
 describe RestMyCase::Base do
 
-  describe ".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::Context::Base.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
-    end
-
-    it "Should create setters targeting to context" do
-      expect(use_case.respond_to?(:comment=)).to be true
-    end
-
-    it "Getter should delegate to context" do
-      expect(use_case.comment).to eq 'my comment'
-    end
-
-    it "Setter should delegate to context" do
-      use_case.comment = 'your comment'
-      expect(use_case.context.comment).to eq 'your comment'
-    end
-
-  end
-
-  describe ".context_writer" do
-
-    let(:context)  { RestMyCase::Context::Base.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
-      expect(use_case.respond_to?(:id=)).to be true
-    end
-
-    it "Setter should delegate to context" do
-      use_case.id = 2
-
-      expect(use_case.context.id).to eq 2
-    end
-
-  end
-
-  describe ".context_reader" do
-
-    let(:context)  { RestMyCase::Context::Base.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
-      expect(use_case.respond_to?(:session=)).to be false
-    end
-
-    it "Getter should delegate to context" do
-      expect(use_case.session).to eq -1
-    end
-
-  end
-
   describe ".perform" do
 
     context "When something that doesn't responds to #to_hash is passed down" do
@@ -305,163 +238,4 @@ describe RestMyCase::Base do
 
   end
 
-  describe "#invoke" do
-
-    class Perform::Invoker < Perform::UseCaseWrapper
-      def perform
-        super
-        invoke Perform::BuildPost, Perform::Validations, Perform::SavePost
-
-        context.should_be_true = true
-      end
-    end
-
-    it "should invoke other use cases with the invoker's cloned context and they can't alter the invoker's context" do
-      context = Perform::Invoker.perform
-
-      expect(context.setup.length).to be 2
-      expect(context.perform.length).to be 2
-      expect(context.rollback.length).to be 0
-      expect(context.final.length).to be 2
-
-      expect(context.should_be_true).to be true
-    end
-
-  end
-
-  describe "#invoke!" do
-
-    context "when the invokees don't abort" do
-
-      class Perform::ScreammingInvoker < Perform::UseCaseWrapper
-        def perform
-          super
-          invoke! Perform::BuildPost, Perform::Validations, Perform::SavePost
-        end
-      end
-
-      it "should invoke other use cases with the invoker's context and actually alter it" do
-        @context = Perform::ScreammingInvoker.perform
-
-        expect(@context.setup).to eq [
-          "Perform::Logger",
-          "Perform::ScreammingInvoker",
-          "Perform::Logger",
-          "Perform::BuildPost",
-          "Perform::Logger",
-          "Perform::ValidateName",
-          "Perform::Logger",
-          "Perform::ValidateBody",
-          "Perform::Logger",
-          "Perform::Validations",
-          "Perform::Logger",
-          "Perform::SavePost"
-        ]
-        expect(@context.perform).to eq [
-          "Perform::Logger",
-          "Perform::ScreammingInvoker",
-          "Perform::Logger",
-          "Perform::BuildPost",
-          "Perform::Logger",
-          "Perform::ValidateName",
-          "Perform::Logger",
-          "Perform::ValidateBody",
-          "Perform::Logger",
-          "Perform::Validations",
-          "Perform::Logger",
-          "Perform::SavePost"
-        ]
-        expect(@context.rollback).to eq []
-        expect(@context.final).to eq [
-          "Perform::Logger",
-          "Perform::BuildPost",
-          "Perform::Logger",
-          "Perform::ValidateName",
-          "Perform::Logger",
-          "Perform::ValidateBody",
-          "Perform::Logger",
-          "Perform::Validations",
-          "Perform::Logger",
-          "Perform::SavePost",
-          "Perform::Logger",
-          "Perform::ScreammingInvoker"
-        ]
-      end
-
-    end
-
-    context "when the invokees abort" do
-
-      class Perform::Validations2 < Perform::UseCaseWrapper
-        def perform
-          super
-
-          invoke! Perform::ValidateName, Perform::ValidateBody
-
-          context.should_not_be_true = true
-        end
-      end
-
-      class Perform::CreatePost2 < Perform::UseCaseWrapper
-        depends Perform::BuildPost, Perform::Validations2, Perform::SavePost
-      end
-
-      before do
-        @context = Perform::CreatePost2.perform \
-          error: ['Perform::ValidateName_perform', 'Perform::ValidateBody_perform']
-      end
-
-      it "invoker should abort the process it his invokees have also aborted" do
-        expect(@context.setup).to eq [
-          "Perform::Logger",
-          "Perform::BuildPost",
-          "Perform::Logger",
-          "Perform::Validations2",
-          "Perform::Logger",
-          "Perform::SavePost",
-          "Perform::Logger",
-          "Perform::CreatePost2",
-          "Perform::Logger",
-          "Perform::ValidateName",
-          "Perform::Logger",
-          "Perform::ValidateBody"
-        ]
-        expect(@context.perform).to eq [
-          "Perform::Logger",
-          "Perform::BuildPost",
-          "Perform::Logger",
-          "Perform::Validations2",
-          "Perform::Logger",
-          "Perform::ValidateName"
-        ]
-        expect(@context.rollback).to eq [
-          "Perform::ValidateName",
-          "Perform::Logger",
-          "Perform::Validations2",
-          "Perform::Logger",
-          "Perform::BuildPost",
-          "Perform::Logger"
-        ]
-        expect(@context.final).to eq [
-          "Perform::Logger",
-          "Perform::ValidateName",
-          "Perform::Logger",
-          "Perform::ValidateBody",
-          "Perform::Logger",
-          "Perform::BuildPost",
-          "Perform::Logger",
-          "Perform::Validations2",
-          "Perform::Logger",
-          "Perform::SavePost",
-          "Perform::Logger",
-          "Perform::CreatePost2"
-        ]
-
-        expect(@context.should_not_be_true).to be_falsey
-      end
-
-    end
-
-  end
-
 end

data/spec/rest_my_case/base/required_context_spec.rb

@@ -0,0 +1,33 @@
+require 'spec_helper'
+
+describe RestMyCase::Base do
+
+  describe ".required_context" do
+
+    context "When #setup odly populates the required field" do
+      before { @context = RequiredContext::Users::GetCurrentUser.perform }
+
+      it "context should be ok" do
+        expect(@context.ok?).to be true
+      end
+    end
+
+    context "When dependencies also have a required_context" do
+      before { @context = RequiredContext::Posts::Update.perform }
+
+      it "context should NOT be ok" do
+        expect(@context.ok?).to be false
+      end
+
+      it "context errors should include 3 context_errors" do
+        expect(@context.errors.length).to be 3
+        expect(@context.errors[0][:context_errors][:id]).to be_truthy
+        expect(@context.errors[1][:context_errors][:post]).to be_truthy
+        expect(@context.errors[1][:context_errors][:post_attributes]).to be_truthy
+        expect(@context.errors[2][:context_errors][:post]).to be_truthy
+      end
+    end
+
+  end
+
+end