rails_workflow 0.2.0 → 0.2.1

data/app/models/{workflow → rails_workflow}/operation_template.rb

@@ -1,18 +1,17 @@
-# todo cover by tests
-module Workflow
+module RailsWorkflow
   class OperationTemplate < ActiveRecord::Base
     include OperationStatus
     include OperationTemplates::Dependencies
     include OperationTemplates::Assignments
     include OperationTemplates::DefaultBuilder
 
-    belongs_to :process_template, class_name: 'Workflow::ProcessTemplate'
-    belongs_to :child_process, class_name: 'Workflow::ProcessTemplate'
+    belongs_to :process_template, class_name: "RailsWorkflow::ProcessTemplate"
+    belongs_to :child_process, class_name: "RailsWorkflow::ProcessTemplate"
 
     class << self
 
       def types
-        Workflow.config.operation_types
+        RailsWorkflow.config.operation_types
       end
 
       # by default system using context data of first dependnecy
@@ -22,7 +21,7 @@ module Workflow
       end
 
       def build_context! operation, dependencies
-        Workflow::Context.new(
+        RailsWorkflow::Context.new(
             parent: operation,
             data: build_context(dependencies) || operation.process.data)
       end
@@ -36,7 +35,7 @@ module Workflow
     end
 
     def default_type
-      Workflow.config.default_operation_template_type
+      RailsWorkflow.config.default_operation_template_type
     end
 
     def resolve_dependency operation
@@ -49,7 +48,7 @@ module Workflow
 
     private
     def default_class kind
-      Workflow.config.operation_types[kind][:class]
+      RailsWorkflow.config.operation_types[kind][:class]
     end
 
     def get_class symb, default

data/app/models/{workflow → rails_workflow}/process.rb

@@ -1,15 +1,14 @@
-# todo cover by tests
-module Workflow
+module RailsWorkflow
   class Process < ActiveRecord::Base
     include ProcessStatus
     include Processes::DependencyResolver
     include Processes::DefaultRunner
 
-    belongs_to :template, class_name: 'Workflow::ProcessTemplate'
-    has_many :operations, class_name: 'Workflow::Operation'
-    has_one :parent_operation, class_name: 'Workflow::Operation', foreign_key: :child_process_id
-    has_one :context, class_name: 'Workflow::Context', as: :parent
-    has_many :workflow_errors, class_name: "Workflow::Error", as: :parent
+    belongs_to :template, class_name: "RailsWorkflow::ProcessTemplate"
+    has_many :operations, class_name: "RailsWorkflow::Operation"
+    has_one :parent_operation, class_name: "RailsWorkflow::Operation", foreign_key: :child_process_id
+    has_one :context, class_name: "RailsWorkflow::Context", as: :parent
+    has_many :workflow_errors, class_name: "RailsWorkflow::Error", as: :parent
 
     delegate :data, to: :context
     scope :by_status, -> (status) { where(status: status) }
@@ -22,13 +21,13 @@ module Workflow
       query = 'select status, cnt from (
                   select row_number() over (partition by status),
                     count(*) over (partition by status) cnt,
-                    status from workflow_processes)t
+                    status from rails_workflow_processes)t
                 where row_number = 1'
 
 
       statuses = connection.select_all(query).rows
 
-      (Workflow::Process::NOT_STARTED..Workflow::Process::ROLLBACK).to_a.map{|status|
+      (RailsWorkflow::Process::NOT_STARTED..RailsWorkflow::Process::ROLLBACK).to_a.map{|status|
         statuses.detect{|s| s.first.to_i == status }.try(:last).to_i
       }
 

data/app/models/{workflow → rails_workflow}/process_template.rb

@@ -1,5 +1,4 @@
-# todo cover by tests
-module Workflow
+module RailsWorkflow
   class ProcessTemplate < ActiveRecord::Base
     include ProcessTemplates::DefaultBuilder
 
@@ -15,12 +14,12 @@ module Workflow
     # and set default Workflow::Process if blank process_class on template
     def process_class
       get_class_for :process_class,
-                    Workflow.config.process_class
+                    RailsWorkflow.config.process_class
     end
 
     def manager_class
       get_class_for(:manager_class,
-                    Workflow.config.manager_class)
+                    RailsWorkflow.config.manager_class)
     end
 
     def independent_operations

data/app/models/{workflow → rails_workflow}/user_by_group_operation.rb

@@ -1,5 +1,4 @@
-# todo cover by tests
-module Workflow
+module RailsWorkflow
   class UserByGroupOperation < Operation
 
     def can_start?

data/app/models/{workflow → rails_workflow}/user_by_role_operation.rb

@@ -1,5 +1,4 @@
-# todo cover by tests
-module Workflow
+module RailsWorkflow
   class UserByRoleOperation < Operation
 
     def can_start?

data/app/views/layouts/{workflow → rails_workflow}/application.html.slim

@@ -7,8 +7,8 @@ head
 
   title Rails Workflow Engine
 
-  = stylesheet_link_tag "workflow/application"
-  = javascript_include_tag "workflow/application"
+  = stylesheet_link_tag "rails_workflow/application"
+  = javascript_include_tag "rails_workflow/application"
   = csrf_meta_tags
 
   a href="#Top"
@@ -48,4 +48,4 @@ body
         |  •
         span<>= mail_to "max.madzhuga@gmail.com", "Contact me", subject: "Workflow Support"
         | &bull;
-        span<>= link_to "Workflow Development Blog", 'http://madzhuga.tumblr.com'
+        span<>= link_to "Rails Workflow Site", 'http://madzhuga.github.io/rails_workflow/'

data/app/views/{workflow → rails_workflow}/operation_templates/index.html.slim

@@ -65,7 +65,7 @@ ol.breadcrumb.alert-warning
           | Add Operation
           span.caret<>
         ul.dropdown-menu role="menu"
-          - Workflow::OperationTemplate.types.each do |type, value|
+          - RailsWorkflow::OperationTemplate.types.each do |type, value|
             li
               = link_to value[:title], new_template_path.call(type)
 

data/app/views/{workflow → rails_workflow}/process_templates/show.html.slim

@@ -48,7 +48,7 @@ ol.breadcrumb.alert-warning
           | Add Operation
           span.caret<>
         ul.dropdown-menu role="menu"
-          - Workflow::OperationTemplate.types.each do |type, value|
+          - RailsWorkflow::OperationTemplate.types.each do |type, value|
             li
               = link_to value[:title], new_template_path.call(type)
 

data/app/views/{workflow → rails_workflow}/processes/index.html.slim

@@ -14,19 +14,19 @@ ol.breadcrumb.alert-warning
             span.badge< = @statistic[:all]
           = link_to processes_path(filter: {status: "in_progress"}), class: "btn btn-primary" do
             | In progress
-            span.badge< = @statistic[:statuses][Workflow::Process::IN_PROGRESS]
+            span.badge< = @statistic[:statuses][RailsWorkflow::Process::IN_PROGRESS]
           = link_to processes_path(filter: {status: "done"}), class: "btn btn-success" do
             | Done
-            span.badge< = @statistic[:statuses][Workflow::Process::DONE]
+            span.badge< = @statistic[:statuses][RailsWorkflow::Process::DONE]
           = link_to processes_path(filter: {status: "waiting"}), class: "btn btn-info" do
             | Waiting
-            span.badge< = @statistic[:statuses][Workflow::Process::WAITING]
+            span.badge< = @statistic[:statuses][RailsWorkflow::Process::WAITING]
           = link_to processes_path(filter: {status: "not_started"}), class: "btn btn-warning" do
             | Not Started
-            span.badge< = @statistic[:statuses][Workflow::Process::NOT_STARTED]
+            span.badge< = @statistic[:statuses][RailsWorkflow::Process::NOT_STARTED]
           = link_to processes_path(filter: {status: "error"}), class: "btn btn-danger" do
             | Error
-            span.badge< = @statistic[:statuses][Workflow::Process::ERROR]
+            span.badge< = @statistic[:statuses][RailsWorkflow::Process::ERROR]
 
         table.table.table-striped.table-hover
           thead
@@ -72,7 +72,7 @@ ol.breadcrumb.alert-warning
           tbody
             - @errors.each do |error|
               tr
-                - if error.parent.is_a? Workflow::Operation
+                - if error.parent.is_a? RailsWorkflow::Operation
                   td = link_to error.message, process_operation_path(error.parent.process, error.parent)
                 - else
                   td = link_to error.message, process_path(error.parent)

data/app/workers/{workflow → rails_workflow}/error_worker.rb

@@ -1,15 +1,15 @@
-module Workflow
+module RailsWorkflow
   class ErrorWorker
     include Sidekiq::Worker
 
     def perform(parent_id, parent_class)
       parent = parent_class.constantize.find(parent_id)
 
-      if parent.is_a? Workflow::Operation
-        parent.status = Workflow::Operation::ERROR
+      if parent.is_a? RailsWorkflow::Operation
+        parent.status = RailsWorkflow::Operation::ERROR
       end
-      if parent.is_a? Workflow::Process
-        parent.status = Workflow::Process::ERROR
+      if parent.is_a? RailsWorkflow::Process
+        parent.status = RailsWorkflow::Process::ERROR
       end
 
       parent.save

data/app/workers/{workflow → rails_workflow}/operation_worker.rb

@@ -1,4 +1,4 @@
-module Workflow
+module RailsWorkflow
   class OperationWorker
     include Sidekiq::Worker
 

data/config/initializers/{workflow.rb → rails_workflow.rb}

@@ -14,7 +14,7 @@ class ActionController::Base
 
       if @current_workflow_operation &&
           @current_workflow_operation.id == operation_id
-          Workflow::Operation.user_ready_statuses.include? @current_workflow_operation.status
+          RailsWorkflow::Operation.user_ready_statuses.include? @current_workflow_operation.status
 
         @current_workflow_operation
 
@@ -22,9 +22,9 @@ class ActionController::Base
 
         @current_workflow_operation = begin
 
-          if Workflow::Operation.exists?(id: operation_id, status: Workflow::Operation::WAITING)
-            operation = Workflow::Operation.find(operation_id)
-            Workflow::OperationHelperDecorator.decorate(operation)
+          if RailsWorkflow::Operation.exists?(id: operation_id, status: RailsWorkflow::Operation::WAITING)
+            operation = RailsWorkflow::Operation.find(operation_id)
+            RailsWorkflow::OperationHelperDecorator.decorate(operation)
           else
             clear_current_operation
           end
@@ -37,14 +37,14 @@ class ActionController::Base
   helper_method :current_operation
 
   def available_operations
-    operations = Workflow::Operation.available_for_user(current_user)
-    Workflow::OperationHelperDecorator.decorate_collection(operations)
+    operations = RailsWorkflow::Operation.available_for_user(current_user)
+    RailsWorkflow::OperationHelperDecorator.decorate_collection(operations)
   end
   helper_method :available_operations
 
   def assigned_operations
-    operations = Workflow::Operation.assigned_to(current_user).waiting
-    Workflow::OperationHelperDecorator.decorate_collection(operations)
+    operations = RailsWorkflow::Operation.assigned_to(current_user).waiting
+    RailsWorkflow::OperationHelperDecorator.decorate_collection(operations)
   end
   helper_method :assigned_operations
 

data/config/routes.rb

@@ -1,4 +1,4 @@
-Workflow::Engine.routes.draw do
+RailsWorkflow::Engine.routes.draw do
 
   resources :operations do
     member do

data/db/migrate/20150210070051_change_namespace.rb

@@ -0,0 +1,10 @@
+class ChangeNamespace < ActiveRecord::Migration
+  def change
+    rename_table :workflow_processes, :rails_workflow_processes
+    rename_table :workflow_operations, :rails_workflow_operations
+    rename_table :workflow_process_templates, :rails_workflow_process_templates
+    rename_table :workflow_operation_templates, :rails_workflow_operation_templates
+    rename_table :workflow_contexts, :rails_workflow_contexts
+    rename_table :workflow_errors, :rails_workflow_errors
+  end
+end

data/lib/generators/{workflow → rails_workflow}/install/install_generator.rb

@@ -1,6 +1,6 @@
 require 'rails/generators/migration'
 
-module Workflow
+module RailsWorkflow
   module Generators
     class InstallGenerator < ::Rails::Generators::Base
       include Rails::Generators::Migration

data/lib/generators/{workflow → rails_workflow}/install/templates/create_workflow_processes.rb

@@ -1,6 +1,6 @@
 class CreateWorkflowProcesses < ActiveRecord::Migration
   def change
-    unless table_exists? :workflow_processes
+    unless table_exists?(:rails_workflow_processes) || table_exists?(:workflow_processes)
       create_table :workflow_processes do |t|
         t.integer :status
         t.boolean :async
@@ -84,22 +84,32 @@ class CreateWorkflowProcesses < ActiveRecord::Migration
       add_column :workflow_operation_templates, :is_background, :boolean, default: true
       add_column :workflow_operations, :is_background, :boolean
 
-    end
+      unless column_exists? :workflow_operation_templates, :type
+        add_column :workflow_operation_templates, :type, :string
+      end
 
-    unless column_exists? :workflow_operation_templates, :type
-      add_column :workflow_operation_templates, :type, :string
-    end
+      unless column_exists? :workflow_errors, :resolved
+        add_column :workflow_errors, :resolved, :boolean
+      end
 
-    unless column_exists? :workflow_errors, :resolved
-      add_column :workflow_errors, :resolved, :boolean
-    end
+      unless column_exists? :workflow_process_templates, :type
+        add_column :workflow_process_templates, :type, :string
+      end
+
+      unless column_exists? :workflow_errors, :resolved, :boolean
+        add_column :workflow_errors, :resolved, :boolean
+      end
 
-    unless column_exists? :workflow_process_templates, :type
-      add_column :workflow_process_templates, :type, :string
     end
 
-    unless column_exists? :workflow_errors, :resolved, :boolean
-      add_column :workflow_errors, :resolved, :boolean
+
+    unless table_exists? :rails_workflow_processes
+      rename_table :workflow_processes, :rails_workflow_processes
+      rename_table :workflow_operations, :rails_workflow_operations
+      rename_table :workflow_process_templates, :rails_workflow_process_templates
+      rename_table :workflow_operation_templates, :rails_workflow_operation_templates
+      rename_table :workflow_contexts, :rails_workflow_contexts
+      rename_table :workflow_errors, :rails_workflow_errors
     end
 
   end

data/lib/{workflow → rails_workflow}/engine.rb

@@ -1,6 +1,6 @@
-module Workflow
+module RailsWorkflow
   class Engine < ::Rails::Engine
-    isolate_namespace Workflow
+    isolate_namespace RailsWorkflow
 
     config.generators do |g|
       g.template_engine :slim
@@ -10,12 +10,6 @@ module Workflow
       g.helper false
     end
 
-    # initializer "workflow.url_helpers" do
-    #   ActiveSupport.on_load(:action_controller) do
-    #     helper Rails.application.routes.url_helpers
-    #   end
-    # end
-
     initializer :append_migrations do |app|
       unless app.root.to_s.match root.to_s
         config.paths["db/migrate"].expanded.each do |expanded_path|

data/lib/rails_workflow/version.rb

@@ -0,0 +1,3 @@
+module RailsWorkflow
+  VERSION = "0.2.1"
+end

data/lib/{workflow.rb → rails_workflow.rb}

@@ -1,11 +1,11 @@
-require "workflow/engine"
+require "rails_workflow/engine"
 require 'singleton'
 require 'inherited_resources'
 require 'devise'
 require 'draper'
 
 
-module Workflow
+module RailsWorkflow
 
   def self.config
     Config.instance
@@ -21,22 +21,22 @@ module Workflow
       @default_operation_types = {
           default: {
               title: "Default Operation",
-              class: "Workflow::Operation"
+              class: "RailsWorkflow::Operation"
           },
           user_role: {
               title: "Operation for User By Role",
-              class: "Workflow::UserByRoleOperation"
+              class: "RailsWorkflow::UserByRoleOperation"
           },
           user_group: {
               title: "Operation by User Group",
-              class: "Workflow::UserByGroupOperation"
+              class: "RailsWorkflow::UserByGroupOperation"
           }
       }
 
-      @default_operation_template_type = "Workflow::OperationTemplate"
-      @default_process_manager = "Workflow::ProcessManager"
-      @default_process_class = "Workflow::Process"
-      @default_process_template_type = "Workflow::ProcessTemplate"
+      @default_operation_template_type = "RailsWorkflow::OperationTemplate"
+      @default_process_manager = "RailsWorkflow::ProcessManager"
+      @default_process_class = "RailsWorkflow::Process"
+      @default_process_template_type = "RailsWorkflow::ProcessTemplate"
       @default_assignment_by = [:group, :role]
 
 

data/spec/controllers/{workflow → rails_workflow}/operation_templates_controller_spec.rb

@@ -1,8 +1,8 @@
 require 'rails_helper'
 
-module Workflow
+module RailsWorkflow
   RSpec.describe OperationTemplatesController, :type => :controller do
-    routes { Workflow::Engine.routes }
+    routes { RailsWorkflow::Engine.routes }
     login_user
 
     before :each do
@@ -13,7 +13,7 @@ module Workflow
       {
           title: 'First Test project',
           process_template_id: @template.id,
-          type: 'Workflow::OperationTemplate'
+          type: 'RailsWorkflow::OperationTemplate'
       }
     }
 
@@ -79,7 +79,7 @@ module Workflow
         it "creates a new CustomOperationTemplate" do
 
           attrs = valid_attributes
-          attrs[:type] = "Workflow::CustomOperationTemplate"
+          attrs[:type] = "RailsWorkflow::CustomOperationTemplate"
           expect {
             post :create, {
                             process_template_id: @template.id,
@@ -88,7 +88,7 @@ module Workflow
           }.to change(OperationTemplate, :count).by(1)
 
           last_operation = OperationTemplate.last
-          expect(last_operation).to be_instance_of(Workflow::CustomOperationTemplate)
+          expect(last_operation).to be_instance_of(RailsWorkflow::CustomOperationTemplate)
         end
 
         it "assigns a newly created operation_template as @operation_template" do
@@ -144,7 +144,7 @@ module Workflow
                          id: operation_template.to_param,
                          operation_template: valid_attributes},
               valid_session, use_route: :workflow
-          # todo: fix to check for changes
+
           # expect(assigns(:operations)).to eq([operation_template])
         end
 

data/spec/controllers/{workflow → rails_workflow}/operations_controller_spec.rb

@@ -18,9 +18,9 @@ require 'rails_helper'
 # Message expectations are only used when there is no simpler way to specify
 # that an instance is receiving a specific message.
 
-module Workflow
+module RailsWorkflow
   RSpec.describe OperationsController, :type => :controller do
-    routes { Workflow::Engine.routes }
+    routes { RailsWorkflow::Engine.routes }
     login_user
 
     let(:valid_attributes) {

data/spec/controllers/{workflow → rails_workflow}/process_templates_controller_spec.rb

@@ -1,8 +1,8 @@
 require 'rails_helper'
 
-module Workflow
+module RailsWorkflow
   RSpec.describe ProcessTemplatesController, :type => :controller do
-    routes { Workflow::Engine.routes }
+    routes { RailsWorkflow::Engine.routes }
     login_user
 
     let(:valid_attributes) {

data/spec/controllers/{workflow → rails_workflow}/processes_controller_spec.rb

@@ -1,8 +1,8 @@
 require 'rails_helper'
 
-module Workflow
+module RailsWorkflow
   RSpec.describe ProcessesController, :type => :controller do
-    routes { Workflow::Engine.routes }
+    routes { RailsWorkflow::Engine.routes }
     login_user
 
     before :each do

data/spec/core/{worklow → rails_workflow}/process_manager_spec.rb

@@ -1,6 +1,6 @@
 require 'rails_helper'
 
-module Workflow
+module RailsWorkflow
   RSpec.describe ProcessManager, :type => :model do
 
     let(:template) {
@@ -10,7 +10,7 @@ module Workflow
       dependencies = [
           {
               "id" => operation.id,
-              "statuses" => [Workflow::OperationStatus::DONE]
+              "statuses" => [RailsWorkflow::OperationStatus::DONE]
           }
       ]
 
@@ -28,7 +28,7 @@ module Workflow
       it 'should create new process' do
         expect(process).to_not be_nil
         expect(process).to be_kind_of(Process)
-        expect(process.status).to eq Workflow::Process::NOT_STARTED
+        expect(process.status).to eq RailsWorkflow::Process::NOT_STARTED
       end
 
       it 'with reference to template' do
@@ -37,7 +37,7 @@ module Workflow
 
       it 'should create new process operations' do
         expect(process.operations.size).to eq 1
-        expect(process.operations.first.status).to eq Workflow::Operation::NOT_STARTED
+        expect(process.operations.first.status).to eq RailsWorkflow::Operation::NOT_STARTED
       end
 
       it 'should create process context' do
@@ -54,30 +54,30 @@ module Workflow
     context 'start process' do
       context 'start process (in progres)' do
         before :each do
-          allow_any_instance_of(Workflow::ProcessManager).to receive(:complete_process)
-          allow_any_instance_of(Workflow::Operation).to receive(:complete)
-          process_manager = Workflow::ProcessManager.new process
+          allow_any_instance_of(RailsWorkflow::ProcessManager).to receive(:complete_process)
+          allow_any_instance_of(RailsWorkflow::Operation).to receive(:complete)
+          process_manager = RailsWorkflow::ProcessManager.new process
           process_manager.start_process
         end
 
         it 'should start new process' do
-          expect(process.status).to eq Workflow::Process::IN_PROGRESS
+          expect(process.status).to eq RailsWorkflow::Process::IN_PROGRESS
         end
 
         it 'should start first operations' do
-          expect(process.operations.first.status).to eq Workflow::Operation::IN_PROGRESS
+          expect(process.operations.first.status).to eq RailsWorkflow::Operation::IN_PROGRESS
         end
       end
     end
 
     context 'complete operations' do
       let(:manager) {
-        Workflow::ProcessManager.new process
+        RailsWorkflow::ProcessManager.new process
       }
 
       context 'dependent operation' do
         it 'should be created when dependencies is sutisfied' do
-          allow_any_instance_of(Workflow::OperationTemplate).to receive(:resolve_dependency).and_return(false)
+          allow_any_instance_of(RailsWorkflow::OperationTemplate).to receive(:resolve_dependency).and_return(false)
           manager.start_process
           process.operations.first.complete
           expect(process.operations.size).to eq 1

data/spec/dummy/app/controllers/sales_contacts_controller.rb

@@ -7,7 +7,7 @@ class SalesContactsController < InheritedResources::Base
         #resource (sales contact) will still be created in the system
         #so it may be better to create process in the same transaction
         #if you need them both to be created or both crashed
-          Workflow::ProcessManager.
+          RailsWorkflow::ProcessManager.
               start_process(
                   18,
                   {

data/spec/dummy/app/models/bad_operation.rb

@@ -1,4 +1,4 @@
-class BadOperation < Workflow::Operation
+class BadOperation < RailsWorkflow::Operation
   def on_complete
     # raise 'ON_COMPLETE EXCEPTION'
   end

data/spec/dummy/app/models/bad_operation_template.rb

@@ -1,4 +1,4 @@
-class BadOperationTemplate < Workflow::OperationTemplate
+class BadOperationTemplate < RailsWorkflow::OperationTemplate
   def build_operation operation
     # raise 'BUILD_OPERATION'
   end