rest_my_case 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 24f1ea66d95ea0375eb3a27f226eacd76f615c18
-  data.tar.gz: aaff479de083593b50440e49c84cf68e3911a2ec
+  metadata.gz: 1d0c344e572726b96515f59752258d4073b89937
+  data.tar.gz: aa11435140ce2d8f8b37092b8ad80700194114dc
 SHA512:
-  metadata.gz: 0ee5095a6139908f84ebf44c9e1e665f84dc794a43d08432813cfec5aaedb7de233515f7e64a9eee482f15a5f28002aceb14cba0d39123316c183437903095af
-  data.tar.gz: 0afb19fa366ac967335d80b7bcfe054b1deffaacfb064a00b296e7345d41f59bb7d239998bbb95506226c78ab43815aae0e040e88da7465d77eb8f59fec684d9
+  metadata.gz: 28dfb80a2170e238e2c1298887f3bf7c49b21ff0ea3f2ec99060ea937423cf4d45a79ef9a6aff3fb68eb83da1bdb7041270ec9298c0a71ef02a4b6936628b022
+  data.tar.gz: d18a6ace349f311fcf3ce082fd1bb6608593ccd9019d54ac3b3fc9010a45c8fe5cd80f279de2a3a84f831a319192cec7fb4dda2c9c48901744f46884300dde37

data/lib/rest_my_case.rb

@@ -1,24 +1,24 @@
-require "rest_my_case/configuration"
+require "rest_my_case/configuration/base"
 require "rest_my_case/version"
 require "rest_my_case/errors"
 require "rest_my_case/base"
 
 module RestMyCase
 
-  def self.configure(&block)
-    yield(Config)
+  def self.configure
+    yield config
   end
 
-  def self.configure
-    yield configuration
+  def self.config
+    @config ||= Configuration::Base.new
   end
 
-  def self.configuration
-    @configuration ||= Configuration.new
+  def self.reset_config
+    @config = Configuration::Base.new
   end
 
-  def self.reset_configuration
-    @configuration = Configuration.new
+  def self.get_config(attribute, use_case)
+    config.get(attribute, use_case)
   end
 
 end

data/lib/rest_my_case/base.rb

@@ -1,9 +1,12 @@
+require "rest_my_case/defense_attorney"
 require "rest_my_case/judges/base"
 
 module RestMyCase
 
   class Base
 
+    extend Configuration::Shared
+
     def self.depends(*use_cases)
       dependencies.push *use_cases
     end
@@ -13,11 +16,9 @@ module RestMyCase
     end
 
     def self.perform(attributes = {})
-      unless attributes.respond_to?(:to_hash)
-        raise ArgumentError.new('Must respond_to method #to_hash')
-      end
+      new_trial_case = DefenseAttorney.new(self, attributes).build_trial_case
 
-      Judges::Base.execute_the_sentence(self, attributes.to_hash)
+      Judges::Base.new(trial_case).execute_the_sentence
     end
 
     def self.context_accessor(*methods)

data/lib/rest_my_case/configuration/base.rb

@@ -0,0 +1,24 @@
+require "rest_my_case/configuration/shared"
+
+module RestMyCase
+  module Configuration
+
+    class Base
+
+      include Shared
+
+      def initialize
+        @dependencies_first         = true
+        @parent_dependencies_first  = true
+      end
+
+      def get(attribute, use_case)
+        custom_config = use_case.send(attribute)
+
+        custom_config.nil? ? send(attribute) : custom_config
+      end
+
+    end
+
+  end
+end

data/lib/rest_my_case/configuration/shared.rb

@@ -0,0 +1,12 @@
+module RestMyCase
+  module Configuration
+
+    module Shared
+
+      attr_accessor :dependencies_first,
+                    :parent_dependencies_first
+
+    end
+
+  end
+end

data/lib/rest_my_case/defense_attorney.rb

@@ -2,31 +2,55 @@ require "rest_my_case/context"
 
 module RestMyCase
 
-  module DefenseAttorney
+  class DefenseAttorney
 
-    def self.build_use_cases_for_the(defendant, attributes)
-      shared_context = Context.new attributes
+    def initialize(starting_point_use_case, attributes)
+      unless attributes.respond_to?(:to_hash)
+        raise ArgumentError.new('Must respond_to method #to_hash')
+      end
+
+      @shared_context          = Context.new attributes.to_hash
+      @starting_point_use_case = starting_point_use_case
+    end
 
-      dependencies(defendant).map do |use_case|
-        use_case.new(shared_context)
+    def build_trial_case
+      all_dependencies(@starting_point_use_case).map do |use_case|
+        use_case.new(@shared_context)
       end
     end
 
-    def self.dependencies(use_case)
+    protected ###################### PROTECTED #########################
+
+    def all_dependencies(use_case)
       return [] unless use_case.respond_to?(:dependencies)
 
-      if parent_dependencies_first?(use_case)
-        dependencies(use_case.superclass) | use_case.dependencies
+      if RestMyCase.get_config(:parent_dependencies_first, use_case)
+        all_dependencies(use_case.superclass) | deep_dependencies(use_case)
       else
-        use_case.dependencies | dependencies(use_case.superclass)
+        deep_dependencies(use_case) | all_dependencies(use_case.superclass)
       end
     end
 
-    protected ###################### PROTECTED #########################
+    def deep_dependencies(use_case)
+      deep_dependencies = []
+
+      use_case.dependencies.each do |use_case|
+        deep_dependencies.push *all_dependencies(use_case)
+      end
+
+      include_current_use_case(deep_dependencies, use_case)
+    end
+
+    private ####################### PRIVATE ###########################
 
-    def self.parent_dependencies_first?(use_case)
-      # use_case.parent_dependencies_first ||
-      RestMyCase.configuration.parent_dependencies_first
+    def include_current_use_case(dependencies, use_case)
+      return dependencies unless use_case.superclass.respond_to?(:dependencies)
+
+      if RestMyCase.get_config(:dependencies_first, use_case)
+        dependencies.push(use_case)
+      else
+        dependencies.unshift(use_case)
+      end
     end
 
   end

data/lib/rest_my_case/judges/base.rb

@@ -1,27 +1,21 @@
-require "rest_my_case/defense_attorney"
-
 module RestMyCase
   module Judges
 
     class Base
 
-      def self.execute_the_sentence(defendant, attributes)
-        new_trial = self.new(defendant, attributes)
-
-        new_trial.run_setup_methods
-        new_trial.run_perform_methods
-        new_trial.run_rollback_methods
-        new_trial.run_final_methods
-      end
-
-      def initialize(defendant, attributes)
-        @use_cases =
-          DefenseAttorney.build_use_cases_for_the(defendant, attributes)
-
+      def initialize(use_cases)
+        @use_cases             = use_cases
         @performed_use_cases   = []
         @use_case_that_aborted = false
       end
 
+      def execute_the_sentence
+        run_setup_methods
+        run_perform_methods
+        run_rollback_methods
+        run_final_methods
+      end
+
       protected #################### PROTECTED ####################
 
       def run_setup_methods

data/lib/rest_my_case/version.rb

@@ -1,3 +1,5 @@
 module RestMyCase
-  VERSION = "1.1.0"
+
+  VERSION = "1.2.0"
+
 end

data/spec/rest_my_case/base_spec.rb

@@ -2,27 +2,38 @@ require 'spec_helper'
 
 describe RestMyCase::Base do
 
-  context "When a use case depends on other use cases" do
-
-    it ".dependencies should list the class's dependencies" do
-      expect(Posts::Create::Base.dependencies).to \
-        eq [Posts::Create::BuildPost, Posts::SavePost]
+  module RestMyCaseBase
+    class LogEvents < RestMyCase::Base; end
+    class AnalyseEvents < RestMyCase::Base; end
+    class BuildPost < RestMyCase::Base; end
+    class SavePost < RestMyCase::Base; end
+    class BuildComments < RestMyCase::Base; end
+    class SaveComments < RestMyCase::Base; end
+
+    class UseCaseWrapper < RestMyCase::Base
+      context_writer    :id
+      context_reader    :session
+      context_accessor  :comment
+      depends LogEvents, AnalyseEvents
+    end
+    class CreatePost < UseCaseWrapper
+      depends BuildPost, SavePost
+    end
+    class CreatePostWithComments < CreatePost
+      depends BuildComments, SaveComments
     end
-
   end
 
-  context "When a use case inherits from another that also has its own dependencies" do
-
-    it ".dependencies should only list the class's dependencies" do
-      expect(Comments::Create::SendEmail.dependencies).to eq [Comments::FindOne]
+  describe ".dependencies" do
+    it " should only list the class's dependencies" do
+      expect(RestMyCaseBase::CreatePostWithComments.dependencies).to \
+        eq [RestMyCaseBase::BuildComments, RestMyCaseBase::SaveComments]
     end
-
   end
 
   describe ".context_accessor" do
-
     let(:context)   { RestMyCase::Context.new(id: 1, comment: 'my comment', session: -1) }
-    let(:use_case)  { Comments::FindOne.new(context) }
+    let(:use_case)  { RestMyCaseBase::CreatePostWithComments.new(context) }
 
     it "Should create getters targeting to context" do
       expect(use_case.respond_to?(:comment)).to be true
@@ -40,12 +51,10 @@ describe RestMyCase::Base do
       use_case.comment = 'your comment'
       expect(use_case.context.comment).to eq 'your comment'
     end
-
   end
 
   describe ".context_writer" do
-
-    let(:use_case) { Comments::FindOne.new(RestMyCase::Context.new) }
+    let(:use_case) { RestMyCaseBase::CreatePostWithComments.new(RestMyCase::Context.new) }
 
     it "Should create setters targeting to context" do
       expect(use_case.respond_to?(:id)).to be false
@@ -57,13 +66,11 @@ describe RestMyCase::Base do
 
       expect(use_case.context.id).to eq 2
     end
-
   end
 
   describe ".context_reader" do
-
     let(:context)   { RestMyCase::Context.new(id: 1, comment: 'my comment', session: -1) }
-    let(:use_case)  { Comments::FindOne.new(context) }
+    let(:use_case)  { RestMyCaseBase::CreatePostWithComments.new(context) }
 
     it "Should create getters targeting to context" do
       expect(use_case.respond_to?(:session)).to be true
@@ -73,7 +80,6 @@ describe RestMyCase::Base do
     it "Getter should delegate to context" do
       expect(use_case.session).to eq context.session
     end
-
   end
 
 end

data/spec/rest_my_case/defense_attorney_spec.rb

@@ -2,33 +2,176 @@ require 'spec_helper'
 
 describe RestMyCase::DefenseAttorney do
 
-  describe ".dependencies" do
+  module DefenseAttorney
+    class BuilEvent < RestMyCase::Base; end
+    class SaveEvent < RestMyCase::Base; end
+    class CreateEvent < RestMyCase::Base
+      depends SaveEvent
+    end
+    class LogEvents < RestMyCase::Base
+      depends BuilEvent, CreateEvent
+    end
+    class AnalyseEvents < RestMyCase::Base; end
+    class BuildPost < RestMyCase::Base; end
+    class SavePost < RestMyCase::Base; end
+    class BuildComments < RestMyCase::Base; end
+    class SaveComments < RestMyCase::Base; end
+    class CreateComments < RestMyCase::Base
+      depends SaveComments
+    end
 
-    context "When a use case depends on other use cases" do
+    class UseCaseWrapper < RestMyCase::Base
+      depends LogEvents, AnalyseEvents
+    end
+    class CreatePost < UseCaseWrapper
+      depends BuildPost, SavePost
+    end
+    class CreatePostWithComments < CreatePost
+      depends BuildComments, CreateComments
+    end
+  end
 
-      let(:use_cases) do
-        RestMyCase::DefenseAttorney.dependencies Posts::Create::Base
+  shared_examples "a porper shepherd" do |dependencies|
+    it "use_cases should be in the proper order" do
+      dependencies.each_with_index do |dependency, index|
+        expect(use_cases[index]).to be_a dependency
       end
+    end
 
-      it "use_cases should be in the proper order" do
-        expect(use_cases).to eq [Posts::Create::BuildPost, Posts::SavePost]
+    it "use_cases should share the same context" do
+      use_cases.each do |use_case1|
+        use_cases.each do |use_case2|
+          expect(use_case1.context).to be use_case2.context
+        end
       end
+    end
+  end
+
+  context "When a use case depends on other use cases" do
+    let(:use_cases) do
+      described_class.new(DefenseAttorney::UseCaseWrapper, id: 1).build_trial_case
+    end
 
+    it_behaves_like "a porper shepherd", [
+      DefenseAttorney::BuilEvent,
+      DefenseAttorney::SaveEvent,
+      DefenseAttorney::CreateEvent,
+      DefenseAttorney::LogEvents,
+      DefenseAttorney::AnalyseEvents,
+      DefenseAttorney::UseCaseWrapper
+    ]
+  end
+
+  context "When a use case inherits from another that also has its own dependencies" do
+    let(:use_cases) do
+      described_class.new(DefenseAttorney::CreatePostWithComments, id: 1).build_trial_case
     end
 
-    context "When a use case depends on other use cases" do
+    it_behaves_like "a porper shepherd", [
+      DefenseAttorney::BuilEvent,
+      DefenseAttorney::SaveEvent,
+      DefenseAttorney::CreateEvent,
+      DefenseAttorney::LogEvents,
+      DefenseAttorney::AnalyseEvents,
+      DefenseAttorney::UseCaseWrapper,
+      DefenseAttorney::BuildPost,
+      DefenseAttorney::SavePost,
+      DefenseAttorney::CreatePost,
+      DefenseAttorney::BuildComments,
+      DefenseAttorney::SaveComments,
+      DefenseAttorney::CreateComments,
+      DefenseAttorney::CreatePostWithComments
+    ]
+  end
 
-      let(:use_cases) do
-        RestMyCase::DefenseAttorney.dependencies Comments::Create::SendEmail
+  context "When general configuration has parent_dependencies_first = false" do
+    before do
+      RestMyCase.configure do |config|
+        config.parent_dependencies_first = false
       end
+    end
 
-      it "use_cases should be in the proper order" do
-        expect(use_cases).to \
-          eq [Comments::FindOne, SendEmail::ToAdmins, SendEmail::ToUser, SendEmail::OneMoreDependency]
+    after { RestMyCase.reset_config }
+
+    let(:use_cases) do
+      described_class.new(DefenseAttorney::CreatePostWithComments, id: 1).build_trial_case
+    end
+
+    it_behaves_like "a porper shepherd", [
+      DefenseAttorney::BuildComments,
+      DefenseAttorney::SaveComments,
+      DefenseAttorney::CreateComments,
+      DefenseAttorney::CreatePostWithComments,
+
+      DefenseAttorney::BuildPost,
+      DefenseAttorney::SavePost,
+      DefenseAttorney::CreatePost,
+
+      DefenseAttorney::BuilEvent,
+      DefenseAttorney::SaveEvent,
+      DefenseAttorney::CreateEvent,
+      DefenseAttorney::LogEvents,
+      DefenseAttorney::AnalyseEvents,
+      DefenseAttorney::UseCaseWrapper
+    ]
+  end
+
+  context "When general configuration has dependencies_first = false" do
+    before do
+      RestMyCase.configure do |config|
+        config.dependencies_first = false
       end
+    end
+
+    after { RestMyCase.reset_config }
+
+    let(:use_cases) do
+      described_class.new(DefenseAttorney::CreatePostWithComments, id: 1).build_trial_case
+    end
+
+    it_behaves_like "a porper shepherd", [
+      DefenseAttorney::UseCaseWrapper,
+      DefenseAttorney::LogEvents,
+      DefenseAttorney::BuilEvent,
+      DefenseAttorney::CreateEvent,
+      DefenseAttorney::SaveEvent,
+      DefenseAttorney::AnalyseEvents,
+      DefenseAttorney::CreatePost,
+      DefenseAttorney::BuildPost,
+      DefenseAttorney::SavePost,
+      DefenseAttorney::CreatePostWithComments,
+      DefenseAttorney::BuildComments,
+      DefenseAttorney::CreateComments,
+      DefenseAttorney::SaveComments
+    ]
+  end
+
+  context "When an use case class configuration has dependencies_first = false" do
+    before do
+      DefenseAttorney::UseCaseWrapper.dependencies_first = false
+    end
+
+    after { DefenseAttorney::UseCaseWrapper.dependencies_first = nil }
 
+    let(:use_cases) do
+      described_class.new(DefenseAttorney::CreatePostWithComments, id: 1).build_trial_case
     end
 
+    it_behaves_like "a porper shepherd", [
+      DefenseAttorney::UseCaseWrapper,
+      DefenseAttorney::BuilEvent,
+      DefenseAttorney::SaveEvent,
+      DefenseAttorney::CreateEvent,
+      DefenseAttorney::LogEvents,
+      DefenseAttorney::AnalyseEvents,
+      DefenseAttorney::BuildPost,
+      DefenseAttorney::SavePost,
+      DefenseAttorney::CreatePost,
+      DefenseAttorney::BuildComments,
+      DefenseAttorney::SaveComments,
+      DefenseAttorney::CreateComments,
+      DefenseAttorney::CreatePostWithComments
+    ]
   end
 
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rest_my_case
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - goncalvesjoao
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-04-13 00:00:00.000000000 Z
+date: 2015-04-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pry
@@ -83,7 +83,8 @@ files:
 - Rakefile
 - lib/rest_my_case.rb
 - lib/rest_my_case/base.rb
-- lib/rest_my_case/configuration.rb
+- lib/rest_my_case/configuration/base.rb
+- lib/rest_my_case/configuration/shared.rb
 - lib/rest_my_case/context.rb
 - lib/rest_my_case/defense_attorney.rb
 - lib/rest_my_case/errors.rb
@@ -94,22 +95,6 @@ files:
 - spec/rest_my_case/defense_attorney_spec.rb
 - spec/rest_my_case/judges/base_spec.rb
 - spec/spec_helper.rb
-- spec/support/_send_email/base.rb
-- spec/support/_send_email/fetch_admins.rb
-- spec/support/_send_email/one_more_dependency.rb
-- spec/support/_send_email/to_admins.rb
-- spec/support/_send_email/to_user.rb
-- spec/support/_send_email/use_case.rb
-- spec/support/comments/create/base.rb
-- spec/support/comments/create/build_comment.rb
-- spec/support/comments/create/send_email.rb
-- spec/support/comments/find_one.rb
-- spec/support/comments/save_comment.rb
-- spec/support/comments/use_case.rb
-- spec/support/posts/create/base.rb
-- spec/support/posts/create/build_post.rb
-- spec/support/posts/save_post.rb
-- spec/support/posts/use_case.rb
 homepage: https://github.com/goncalvesjoao/rest_my_case
 licenses:
 - MIT
@@ -139,19 +124,3 @@ test_files:
 - spec/rest_my_case/defense_attorney_spec.rb
 - spec/rest_my_case/judges/base_spec.rb
 - spec/spec_helper.rb
-- spec/support/_send_email/base.rb
-- spec/support/_send_email/fetch_admins.rb
-- spec/support/_send_email/one_more_dependency.rb
-- spec/support/_send_email/to_admins.rb
-- spec/support/_send_email/to_user.rb
-- spec/support/_send_email/use_case.rb
-- spec/support/comments/create/base.rb
-- spec/support/comments/create/build_comment.rb
-- spec/support/comments/create/send_email.rb
-- spec/support/comments/find_one.rb
-- spec/support/comments/save_comment.rb
-- spec/support/comments/use_case.rb
-- spec/support/posts/create/base.rb
-- spec/support/posts/create/build_post.rb
-- spec/support/posts/save_post.rb
-- spec/support/posts/use_case.rb

data/lib/rest_my_case/configuration.rb

@@ -1,13 +0,0 @@
-module RestMyCase
-
-  class Configuration
-
-    attr_accessor :parent_dependencies_first
-
-    def initialize
-      @parent_dependencies_first = false
-    end
-
-  end
-
-end

data/spec/support/_send_email/base.rb

@@ -1,14 +0,0 @@
-require_relative 'use_case'
-require_relative 'to_admins'
-require_relative 'to_user'
-
-module SendEmail
-
-  class Base < UseCase
-
-    depends ToAdmins,
-            ToUser
-
-  end
-
-end

data/spec/support/_send_email/fetch_admins.rb

@@ -1,6 +0,0 @@
-module SendEmail
-
-  class FetchAdmins < RestMyCase::Base
-  end
-
-end

data/spec/support/_send_email/one_more_dependency.rb

@@ -1,6 +0,0 @@
-module SendEmail
-
-  class OneMoreDependency < RestMyCase::Base
-  end
-
-end

data/spec/support/_send_email/to_admins.rb

@@ -1,11 +0,0 @@
-require_relative 'fetch_admins'
-
-module SendEmail
-
-  class ToAdmins < RestMyCase::Base
-
-    depends FetchAdmins
-
-  end
-
-end

data/spec/support/_send_email/to_user.rb

@@ -1,6 +0,0 @@
-module SendEmail
-
-  class ToUser < RestMyCase::Base
-  end
-
-end

data/spec/support/_send_email/use_case.rb

@@ -1,11 +0,0 @@
-require_relative 'one_more_dependency'
-
-module SendEmail
-
-  class UseCase < RestMyCase::Base
-
-    depends OneMoreDependency
-
-  end
-
-end

data/spec/support/comments/create/base.rb

@@ -1,18 +0,0 @@
-require_relative '../use_case'
-require_relative 'build_comment'
-require_relative 'send_email'
-require_relative '../save_comment'
-
-module Comments
-  module Create
-
-    class Base < UseCase
-
-      depends BuildComment,
-              SaveComment,
-              SendEmail
-
-    end
-
-  end
-end

data/spec/support/comments/create/build_comment.rb

@@ -1,8 +0,0 @@
-module Comments
-  module Create
-
-    class BuildComment < UseCase
-    end
-
-  end
-end

data/spec/support/comments/create/send_email.rb

@@ -1,13 +0,0 @@
-require_relative '../find_one'
-
-module Comments
-  module Create
-
-    class SendEmail < ::SendEmail::Base
-
-      depends FindOne
-
-    end
-
-  end
-end

data/spec/support/comments/find_one.rb

@@ -1,8 +0,0 @@
-require_relative 'use_case'
-
-module Comments
-
-  class FindOne < UseCase
-  end
-
-end

data/spec/support/comments/save_comment.rb

@@ -1,6 +0,0 @@
-module Comments
-
-  class SaveComment < UseCase
-  end
-
-end

data/spec/support/comments/use_case.rb

@@ -1,11 +0,0 @@
-module Comments
-
-  class UseCase < RestMyCase::Base
-
-    context_writer    :id
-    context_reader    :session
-    context_accessor  :comment
-
-  end
-
-end

data/spec/support/posts/create/base.rb

@@ -1,16 +0,0 @@
-require_relative '../use_case'
-require_relative 'build_post'
-require_relative '../save_post'
-
-module Posts
-  module Create
-
-    class Base < UseCase
-
-      depends BuildPost,
-              SavePost
-
-    end
-
-  end
-end

data/spec/support/posts/create/build_post.rb

@@ -1,8 +0,0 @@
-module Posts
-  module Create
-
-    class BuildPost < UseCase
-    end
-
-  end
-end

data/spec/support/posts/save_post.rb

@@ -1,6 +0,0 @@
-module Posts
-
-  class SavePost < UseCase
-  end
-
-end

data/spec/support/posts/use_case.rb

@@ -1,6 +0,0 @@
-module Posts
-
-  class UseCase < RestMyCase::Base
-  end
-
-end