usecasing 0.1.1 → 0.1.2

data/lib/usecasing/base.rb

@@ -20,15 +20,13 @@ module UseCase
       end
 
       def perform(ctx = { })
-        any_ciclic, ciclic = CyclicFinder.new(self).cyclic?
-        raise StandardError.new("cyclic detected: #{ciclic}") if any_ciclic
-        execution_order = [] 
-        build_execution_order(self, execution_order)
-        tx(execution_order, ctx) do |usecase, context| 
+        tx(ExecutionOrder.run(self), ctx) do |usecase, context| 
           usecase.new(context).perform 
         end
       end
 
+      private
+
       def tx(execution_order, context)
         ctx = Context.new(context)
         executed = []
@@ -48,35 +46,6 @@ module UseCase
         context
       end
 
-      def build_execution_order(node, result)
-        return result.push(node) if node.dependencies.empty?
-        
-        node.dependencies.each do |item|
-          build_execution_order(item, result)
-        end
-
-        result.push(node)
-      end
-
-      # def build_execution_order
-      #   stack = [self]
-      #   result  = []
-      #   visited = {}
-        
-      #   until stack.empty?
-      #     node = stack.last
-      #     if(node.dependencies.empty? || visited[node.name])
-      #       result.push(stack.pop)
-      #     else
-      #       stack.push(*(node.dependencies.reverse))
-      #       visited[node.name] = true
-      #     end
-      #   end
-
-      #   return result
-
-      # end
-
     end #ClassMethods
   end #BaseClassMethod
 

data/lib/usecasing/execution_order.rb

@@ -0,0 +1,23 @@
+module UseCase
+
+  class ExecutionOrder
+
+    def self.run(start_node)
+      any_ciclic, ciclic = CyclicFinder.new(start_node).cyclic?
+      raise StandardError.new("cyclic detected: #{ciclic}") if any_ciclic
+      post_order(start_node, [])
+    end
+
+    private
+    def self.post_order(node, result)
+      return result.push(node) if node.dependencies.empty?
+        
+      node.dependencies.each do |item|
+        post_order(item, result)
+      end
+
+      result.push(node)
+    end
+
+  end
+end

data/lib/usecasing/version.rb

@@ -1,3 +1,3 @@
 module Usecasing
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end

data/lib/usecasing.rb

@@ -1,7 +1,8 @@
 require "usecasing/version"
 
 module UseCase
-  autoload :Context,       'usecasing/context'
-  autoload :Base,          'usecasing/base'
-  autoload :CyclicFinder,  'usecasing/cyclic_finder'
+  autoload :Context,        'usecasing/context'
+  autoload :Base,           'usecasing/base'
+  autoload :CyclicFinder,   'usecasing/cyclic_finder'
+  autoload :ExecutionOrder, 'usecasing/execution_order'
 end

data/spec/execution_order_spec.rb

@@ -0,0 +1,40 @@
+require 'spec_helper'
+
+describe UseCase::ExecutionOrder do 
+
+  context "with only one node" do 
+    it 'returns post order' do 
+      EOFirst = Class.new(UseCase::Base)
+      expect(UseCase::ExecutionOrder.run(EOFirst)).to eql([EOFirst])
+    end
+  end
+
+  context "with two nodes" do 
+    it 'returns the dependency first' do 
+      EODepdency = Class.new(UseCase::Base)
+      EODependent = Class.new(UseCase::Base) do 
+        depends EODepdency
+      end
+
+      expect(UseCase::ExecutionOrder.run(EODependent)).to eql([EODepdency, EODependent])
+
+    end
+  end
+
+  context 'with repeated nodes' do 
+    it 'returns duplicated nodes'  do
+      EORepeatedSMS = Class.new(UseCase::Base)
+      
+      EOAlert = Class.new(UseCase::Base) do 
+        depends EORepeatedSMS
+      end
+
+      EOCreate = Class.new(UseCase::Base) do 
+        depends EOAlert, EORepeatedSMS
+      end
+
+      expect(UseCase::ExecutionOrder.run(EOCreate)).to eql([EORepeatedSMS, EOAlert, EORepeatedSMS, EOCreate])
+    end
+  end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: usecasing
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
   prerelease: 
 platform: ruby
 authors:
@@ -77,8 +77,10 @@ files:
 - lib/usecasing/base.rb
 - lib/usecasing/context.rb
 - lib/usecasing/cyclic_finder.rb
+- lib/usecasing/execution_order.rb
 - lib/usecasing/version.rb
 - spec/context_spec.rb
+- spec/execution_order_spec.rb
 - spec/spec_helper.rb
 - spec/usecase_base_spec.rb
 - usecasing.gemspec
@@ -96,7 +98,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1663280317763873983
+      hash: 183179680279514934
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -105,7 +107,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1663280317763873983
+      hash: 183179680279514934
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.24
@@ -114,5 +116,6 @@ specification_version: 3
 summary: UseCase Driven Approach
 test_files:
 - spec/context_spec.rb
+- spec/execution_order_spec.rb
 - spec/spec_helper.rb
 - spec/usecase_base_spec.rb