rules_engine 0.1.4 → 0.1.5

data/VERSION

@@ -1 +1 @@
-0.1.4
+0.1.5

data/lib/rules_engine/rule/definition.rb

@@ -63,10 +63,13 @@ module RulesEngine
 
       ##################################################################
       # callbacks when the rule is added and removed from a workflow
-      def after_add_to_workflow(workflow_code)
+      def before_create()
       end
     
-      def before_remove_from_workflow(workflow_code)
+      def before_update()
+      end
+      
+      def before_destroy()
       end
     
       ##################################################################

data/rails_generators/manifests/complex.rb

@@ -17,13 +17,10 @@ class ComplexManifest
 
    m.template "app/rules/complex.rb",  "app/rules/#{rule_name}.rb"
    m.template "app/views/re_rule_definitions/complex/_edit.html.erb",  "app/views/re_rule_definitions/#{rule_name}/_edit.html.erb"
+   m.template "app/views/re_rule_definitions/complex/_form.html.erb",  "app/views/re_rule_definitions/#{rule_name}/_form.html.erb"
+   m.template "app/views/re_rule_definitions/complex/_form_word.html.erb",  "app/views/re_rule_definitions/#{rule_name}/_form_word.html.erb"
    m.template "app/views/re_rule_definitions/complex/_help.html.erb",  "app/views/re_rule_definitions/#{rule_name}/_help.html.erb"
    m.template "app/views/re_rule_definitions/complex/_new.html.erb",  "app/views/re_rule_definitions/#{rule_name}/_new.html.erb"
-   m.template "app/views/re_rule_definitions/complex/_script.html.erb",  "app/views/re_rule_definitions/#{rule_name}/_script.html.erb"
-   m.template "app/views/re_rule_definitions/complex/_title.html.erb",  "app/views/re_rule_definitions/#{rule_name}/_title.html.erb"
-   m.template "app/views/re_rule_definitions/complex/_word.html.erb",  "app/views/re_rule_definitions/#{rule_name}/_word.html.erb"
-   m.template "app/views/re_rule_definitions/complex/_words.html.erb",  "app/views/re_rule_definitions/#{rule_name}/_words.html.erb"
-   m.template "app/views/re_rule_definitions/complex/_workflow.html.erb",  "app/views/re_rule_definitions/#{rule_name}/_workflow.html.erb"
    m.template "spec/lib/rules/complex_spec.rb",  "spec/lib/rules/#{rule_name}_spec.rb"
 
   end

data/rails_generators/manifests/complex.yml

@@ -14,11 +14,8 @@ templates :
   files : 
     - app/rules/complex.rb
     - app/views/re_rule_definitions/complex/_edit.html.erb
+    - app/views/re_rule_definitions/complex/_form_word.html.erb
+    - app/views/re_rule_definitions/complex/_form.html.erb
     - app/views/re_rule_definitions/complex/_help.html.erb
-    - app/views/re_rule_definitions/complex/_workflow.html.erb
-    - app/views/re_rule_definitions/complex/_script.html.erb
-    - app/views/re_rule_definitions/complex/_title.html.erb
     - app/views/re_rule_definitions/complex/_new.html.erb
-    - app/views/re_rule_definitions/complex/_word.html.erb
-    - app/views/re_rule_definitions/complex/_words.html.erb
     - spec/lib/rules/complex_spec.rb

data/rails_generators/manifests/simple.yml

@@ -14,8 +14,8 @@ templates :
   files : 
     - app/rules/simple.rb
     - app/views/re_rule_definitions/simple/_edit.html.erb
+    - app/views/re_rule_definitions/simple/_form.html.erb
     - app/views/re_rule_definitions/simple/_help.html.erb
     - app/views/re_rule_definitions/simple/_new.html.erb
-    - app/views/re_rule_definitions/simple/_form.html.erb
     - spec/lib/rules/simple_spec.rb
   

data/rails_generators/templates/app/controllers/re_plans_controller.rb

@@ -31,7 +31,7 @@ class RePlansController < ApplicationController
       
       respond_to do |format|
         format.html do
-          redirect_to(re_plans_path)
+          redirect_to(change_re_plan_path(@re_plan))
         end  
         format.js do
           render :action => "create"

data/rails_generators/templates/app/controllers/re_workflow_rules_controller.rb

@@ -19,9 +19,7 @@ class ReWorkflowRulesController < ApplicationController
     if @rule_class.nil?
       flash[:error] = "#{params[:rule_class]} : class not found."
       render :error
-    else
-      @rule = @rule_class.new  
-    end      
+    end
   end  
   
   def error

data/rails_generators/templates/app/models/re_rule.rb

@@ -8,12 +8,18 @@ class ReRule < ActiveRecord::Base
 
   default_scope :order => 're_rules.position ASC'
   
-  before_save :before_save_rule
-  after_create :after_create_rule
+  before_save    :before_save_rule
   before_destroy :before_destroy_rule
 
   def before_save_rule
     return if rule.nil?
+    
+    if self.new_record?
+      rule.before_create()
+    else  
+      rule.before_update()
+    end  
+    
     self.title = rule.title
     self.summary = rule.summary
     self.data = rule.data
@@ -23,19 +29,17 @@ class ReRule < ActiveRecord::Base
     re_workflow.changed! if changes.detect { |change| !ignore_attributes.include?(change[0])}    
   end
 
-  def after_create_rule
-    rule.after_add_to_workflow(self.re_workflow.code) unless rule.nil?
-  end
-
   def before_destroy_rule
-    rule.before_remove_from_workflow(self.re_workflow.code) unless rule.nil?
+    rule.before_destroy() unless rule.nil?
   end
 
   def validate
     if self.rule.nil?
-      errors.add("rule_class", "not found") 
+      errors.add("rule_class", "not found")
     elsif !self.rule.valid?
-      errors.add(self.rule_class_name, "not valid") 
+      errors.add(self.rule_class_name, "not valid")
+    else
+      true  
     end
   end
 

data/rails_generators/templates/app/models/re_rule_expected_outcome.rb

@@ -7,6 +7,8 @@ class ReRuleExpectedOutcome < ActiveRecord::Base
   def validate    
     if outcome == RulesEngine::Rule::Outcome::START_WORKFLOW && workflow_code.blank?
       errors.add(:workflow_code, "workflow code required")
+    else
+      true  
     end
   end
 end

data/rails_generators/templates/app/rules/complex.rb

@@ -76,7 +76,7 @@ module RulesEngine
         end
     
         @workflow_action = param_hash[:<%=rule_name%>_workflow_action] || 'continue'
-        @workflow = param_hash[:<%=rule_name%>_workflow] || ''
+        @workflow = param_hash[:<%=rule_name%>_workflow]
       end
   
       ##################################################################
@@ -91,10 +91,10 @@ module RulesEngine
 
       ##################################################################
       # callbacks when the rule is added and removed from a workflow
-      def after_add_to_workflow(workflow_code)
+      def before_add_to_workflow()
       end
   
-      def before_remove_from_workflow(workflow_code)
+      def before_remove_from_workflow()
       end
   
       ##################################################################
@@ -110,7 +110,7 @@ module RulesEngine
     
         words.each do |word|
           if /#{word}/i =~ tweet        
-            RulesEngine::Process.auditor.audit(process_id, "#{title} Found #{word}", RulesEngine::Process::AUDIT_INFO)
+            RulesEngine::Process.auditor.audit(process_id, "Found #{word}", RulesEngine::Process::AUDIT_INFO)
             data[:match] = word
         
             case workflow_action

data/rails_generators/templates/app/rules/simple.rb

@@ -9,9 +9,9 @@ module RulesEngine
         {
           :group => 'General',
           :display_name => 'Simple',    
-          :help_partial => '/re_rule_definitions/rule_<%=rule_name%>/help',
-          :new_partial => '/re_rule_definitions/rule_<%=rule_name%>/new',
-          :edit_partial => '/re_rule_definitions/rule_<%=rule_name%>/edit'
+          :help_partial => '/re_rule_definitions/<%=rule_name%>/help',
+          :new_partial => '/re_rule_definitions/<%=rule_name%>/new',
+          :edit_partial => '/re_rule_definitions/<%=rule_name%>/edit'
         } 
   
       ##################################################################
@@ -48,24 +48,24 @@ module RulesEngine
       def attributes=(params)
         param_hash = params.symbolize_keys
 
-        @title = param_hash[:rule_<%=rule_name%>_title]
-        @description = param_hash[:rule_<%=rule_name%>_description]
+        @title = param_hash[:<%=rule_name%>_title]
+        @description = param_hash[:<%=rule_name%>_description]
       end
   
       ##################################################################
       # validation and errors
       def valid?
         @errors = {}
-        @errors[:rule_<%=rule_name%>_title] = "Title required" if title.blank?    
+        @errors[:<%=rule_name%>_title] = "Title required" if title.blank?    
         return @errors.empty?
       end
 
       ##################################################################
       # callbacks when the rule is added and removed from a workflow
-      def after_add_to_workflow(workflow_code)
+      def before_add_to_workflow()
       end
   
-      def before_remove_from_workflow(workflow_code)
+      def before_remove_from_workflow()
       end
   
       ##################################################################

data/rails_generators/templates/app/views/re_rule_definitions/complex/_edit.html.erb

@@ -1,7 +1 @@
-<%%= render '/re_rule_definitions/<%=rule_name%>/script' %>
-
-<%%= render '/re_rule_definitions/<%=rule_name%>/title' %>
-
-<%%= render '/re_rule_definitions/<%=rule_name%>/words', :f => f %>
-
-<%%= render '/re_rule_definitions/<%=rule_name%>/workflow' %>
+<%%= render '/re_rule_definitions/<%=rule_name%>/form', :f => f %>

data/rails_generators/templates/app/views/re_rule_definitions/complex/_form.html.erb

@@ -0,0 +1,76 @@
+<%% javascript_tag do %>
+  var new_<%=rule_name%> = '<%%=escape_javascript(render("/re_rule_definitions/<%=rule_name%>/form_word"))%>'  
+  
+  var update<%=rule_class%>WorkflowAction = function() {
+    if ($('#<%=rule_name%>_workflow_action').attr('value') == "start_workflow")
+    {
+      $('#<%=rule_name%>_workflow').re_form_enable();
+      $('#<%=rule_name%>_workflow').removeAttr("disabled");      
+    } else {
+      $('#<%=rule_name%>_workflow').re_form_disable();
+      $('#<%=rule_name%>_workflow').attr("disabled", true); 
+    }      
+  }
+
+  $(document).ready(function() {	
+    $('#<%=rule_name%>_title').focus();	
+    
+    $('#<%=rule_name%>_workflow_action').change(function() {
+      update<%=rule_class%>WorkflowAction();
+    });    
+    
+    update<%=rule_class%>WorkflowAction();
+  });
+
+<%% end %>
+
+<!-- TITLE -->
+<%%= re_text_field "Title", 
+                        "<%=rule_name%>_title", 
+                        params[:<%=rule_name%>_title] || @re_rule.rule.title,
+                        :size => 30, 
+                        :required => true, 
+												:error => @re_rule.rule.errors[:<%=rule_name%>_title],
+                        :span => '4x13' %>
+
+<!-- WORDS -->
+<%% position = 0 %>
+<%% (@re_rule.rule.words || []).each do | word | %>  
+  <%% f.fields_for :rule_data, {:first => false} do |frd| %>
+    <%%= render '/re_rule_definitions/<%=rule_name%>/form_word', :f => frd, :position => position, :word => word %>
+    <%% position += 1 %>        
+  <%% end %>
+<%% end %>  
+<%% if position == 0 %>
+  <%% f.fields_for :rule_data, {:first => false} do |frd| %>
+    <%%= render '/re_rule_definitions/<%=rule_name%>/form_word', :f => frd, :position => position %>      
+  <%% end %>
+<%% end %>
+
+<div id="new_position_<%=rule_name%>"></div>
+<%%= re_form_text "", re_add_link('Add another word to match', '<%=rule_name%>') %>    
+
+
+<!-- ACTION -->
+<div class="clear top-10"></div>
+<hr/>
+
+<div class="span-8">
+  <%%= re_select "Match Outcome", '<%=rule_name%>_workflow_action', 
+                                    options_for_select([
+                                                        ["Continue Next", 'next'], 
+                                                        ["Stop Success", 'stop_success'],
+                                                        ["Stop Failure", 'stop_failure'],
+                                                        ["Start another Workflow", 'start_workflow']
+                                                        ], 
+                                                        @re_rule.rule.workflow_action || params['<%=rule_name%>_workflow_action']),
+																		:required => true %>
+</div>
+<div class="prepend-1 span-7 last">
+  <%%= re_text_field "Workflow Code", "<%=rule_name%>_workflow", @re_rule.rule.workflow || params[:<%=rule_name%>_workflow], 
+																		:size => 20, 
+																		:required => true, 
+																		:span => '3x5',
+																		:error => @re_rule.rule.errors[:<%=rule_name%>_workflow] %>
+</div>
+

data/rails_generators/templates/app/views/re_rule_definitions/complex/_help.html.erb

@@ -1 +1 @@
-<p><%=rule_class%> : This will look for the first match in a given tweet and when found execute the required outcome</p>
+<p>This will look for the first match in a given tweet and when found execute the required outcome</p>

data/rails_generators/templates/app/views/re_rule_definitions/complex/_new.html.erb

@@ -1,7 +1 @@
-<%%= render '/re_rule_definitions/<%=rule_name%>/script' %>
-
-<%%= render '/re_rule_definitions/<%=rule_name%>/title' %>
-
-<%%= render '/re_rule_definitions/<%=rule_name%>/words', :f => f %>
-
-<%%= render '/re_rule_definitions/<%=rule_name%>/workflow' %>
+<%%= render '/re_rule_definitions/<%=rule_name%>/form', :f => f %>

data/rails_generators/templates/app/views/re_rule_definitions/simple/_edit.html.erb

@@ -1 +1 @@
-<%%= render '/re_rule_definitions/rule_<%=rule_name%>/form' %>
+<%%= render '/re_rule_definitions/<%=rule_name%>/form' %>

data/rails_generators/templates/app/views/re_rule_definitions/simple/_form.html.erb

@@ -1,16 +1,25 @@
-<%%= re_text_field "Simple Title", 
-                        "rule_<%=rule_name%>_title", 
-                        params[:rule_<%=rule_name%>_title] || @re_rule.rule.title,
+<%% javascript_tag do %>
+
+  $(document).ready(function() {	
+    $('#<%=rule_name%>_title').focus();	
+  });
+
+<%% end %>
+
+
+<%%= re_text_field "Title", 
+                        "<%=rule_name%>_title", 
+                        params[:<%=rule_name%>_title] || @re_rule.rule.title,
                         :size => 30, 
                         :required => true, 
-												:error => @re_rule.rule.errors[:rule_<%=rule_name%>_title],
+												:error => @re_rule.rule.errors[:<%=rule_name%>_title],
                         :span => '4x13' %>
 
 
 <%%= re_text_field "Description", 
-                        "rule_<%=rule_name%>_description", 
-                        params[:rule_<%=rule_name%>_description] || @re_rule.rule.description,
+                        "<%=rule_name%>_description", 
+                        params[:<%=rule_name%>_description] || @re_rule.rule.description,
                         :size => 40, 
                         :required => false, 
-												:error => @re_rule.rule.errors[:rule_<%=rule_name%>_description],
+												:error => @re_rule.rule.errors[:<%=rule_name%>_description],
                         :span => '4x13' %>

data/rails_generators/templates/app/views/re_rule_definitions/simple/_help.html.erb

@@ -1 +1 @@
-<p> Simple : This rule does nothing </p>
+<p>This rule does nothing </p>

data/rails_generators/templates/app/views/re_rule_definitions/simple/_new.html.erb

@@ -1 +1 @@
-<%%= render '/re_rule_definitions/rule_<%=rule_name%>/form' %>
+<%%= render '/re_rule_definitions/<%=rule_name%>/form' %>

data/rails_generators/templates/app/views/re_workflows/_rule_change.html.erb

@@ -18,16 +18,16 @@
 	<div class="clear"></div>
 </div>
 <div class="<%= re_rule_error.blank? ? 're-bluebox' : 're-redbox' %> no-top"> 
-	<div class="span-8">
+	<div class="span-7">
 		<p class="top-5">
 			<%=h re_rule.summary %>
 		</p>
 	</div>
-	<div class="prepend-8">
+	<div class="span-5 float-right">
 		<div class="float-right red smalltext right-5">
 			<%=h re_rule_error %>
 		</div>
-		<div class="clear"></div>
+		<div class="top-5 clear"></div>
 		<% if re_rule.re_rule_expected_outcomes.empty? %>
 			<div class="re-xtra-rule-continue">Continue</div>
 		<% end %>        

data/rails_generators/templates/app/views/re_workflows/_rule_show.html.erb

@@ -9,16 +9,16 @@
   <div class="clear"></div>    
 </div>
 <div class="<%= re_rule_error.blank? ? 're-bluebox' : 're-redbox' %> no-top"> 
-	<div class="span-8">
+	<div class="span-7">
      <p class="top-5">
 			<%=h re_rule.summary %>
 		</p>	
 	</div>      
-	<div class="prepend-8">
+	<div class="span-5 float-right">
 		<div class="float-right red smalltext right-5">
 			<%=h re_rule_error %>
 		</div>
-		<div class="clear"></div>
+		<div class="top-5 clear"></div>
 
 	  <% if re_rule.re_rule_expected_outcomes.empty? %>
 	    <div class="re-xtra-rule-continue">Continue</div>

data/rails_generators/templates/doc/README.rules_engine

@@ -146,7 +146,7 @@ class ApplicationController < ActionController::Base
 
 ######################################
 # 8. Create a Simple Rule
-  ./script/generate rules_engine rule_simple does_nothing
+  ./script/generate rules_engine simple does_nothing
 
 ######################################
 # 9. Create a Complex Rule

data/rails_generators/templates/spec/controllers/re_plans_controller_spec.rb

@@ -31,151 +31,264 @@ describe RePlansController do
     end
   end
   
-  # describe "new" do
-  #   it_should_require_rules_engine_editor_access(:new, :id => 123)
-  #   
-  #   it "should assign a new plan record" do
-  #     re_plan = mock_model(RePlan)
-  #     RePlan.should_receive(:new).and_return(re_plan)
-  #     get :new, :id => 123
-  #     assigns[:re_plan].should == re_plan
-  #   end
-  #   
-  #   it "should render the 'new' template" do
-  #     get :new, :id => 123
-  #     response.should render_template(:new)
-  #   end
-  # end
-  #   
-  # describe "create" do
-  #   it_should_require_rules_engine_editor_access(:create, :re_plan => {})
-  # 
-  #   before do
-  #     @re_plan = mock_model(RePlan, :save => false)
-  #     RePlan.stub!(:new).and_return(@re_plan) 
-  #   end
-  #   
-  #   it "should assign the re_plan parameters" do
-  #     RePlan.should_receive(:new).with("name" => "name", "value" => "value")
-  #     post :create, :re_plan => { :name => "name", :value => "value" }
-  #   end
-  # 
-  #   it "should save the re_plan" do
-  #     @re_plan.should_receive(:save)
-  #     post :create, :re_plan => { :key => "value" }
-  #   end
-  #       
-  #   describe "save failed" do
-  #     it "should render the 'new' template" do
-  #       post :create, :re_plan => { :key => "value" }
-  #       response.should render_template(:new)
-  #     end
-  #   end
-  # 
-  #   describe "save succeeded" do
-  #     before do
-  #       @re_plan.stub!(:save).and_return(true)
-  #     end
-  #     
-  #     it "should display a flash success message" do
-  #       post :create, :re_plan => { :key => "value" }
-  #       flash[:success].should_not be_blank
-  #     end
-  #     
-  #     it "should redirect to the change re_plan page for HTML" do
-  #       post :create, :re_plan => { :key => "value" }
-  #       response.should redirect_to(change_re_plan_path(@re_plan))
-  #     end
-  # 
-  #     it "should render 'create' template for JAVASCRIPT" do
-  #       xhr :post, :create, :re_plan => { :key => "value" }
-  #       response.should render_template(:create)
-  #     end
-  #   end
-  # end
-  #   
-  # describe "change" do
-  #   it_should_require_rules_engine_editor_access(:change, :id => 123)
-  #   
-  #   it "should get the plan record with the ID" do
-  #     re_plan = mock_model(RePlan)
-  #     RePlan.should_receive(:find).with("123").and_return(re_plan)
-  #     get :change, :id => 123
-  #     assigns[:re_plan].should == re_plan
-  #   end
-  # end
-  # 
-  # describe "edit" do
-  #   it_should_require_rules_engine_editor_access(:edit, :id => 123)
-  #   
-  #   it "should get the plan record with the ID" do
-  #     re_plan = mock_model(RePlan)
-  #     RePlan.should_receive(:find).with("123").and_return(re_plan)
-  #     get :edit, :id => 123
-  #     assigns[:re_plan].should == re_plan
-  #   end
-  # end
-  # 
-  # describe "update" do
-  #   it_should_require_rules_engine_editor_access(:update, :id => 123, :re_plan => {})
-  # 
-  #   before do
-  #     @re_plan = mock_model(RePlan, :save => false)
-  #     @re_plan.stub!(:attributes=)
-  #     RePlan.stub!(:find).and_return(@re_plan) 
-  #   end
-  # 
-  #   it "should get the plan record with the ID" do
-  #     RePlan.should_receive(:find).with("123").and_return(@re_plan)
-  #     put :update, :id => 123, :re_plan => { :key => "value" }
-  #     assigns[:re_plan].should == @re_plan
-  #   end
-  #   
-  #   it "should assign the re_plan parameters" do
-  #     @re_plan.should_receive(:attributes=).with("name" => "name", "value" => "value")
-  #     put :update, :id => 123, :re_plan => { :name => "name", :value => "value" }
-  #   end
-  # 
-  #   it "should not assign the re_plan parameters :code" do
-  #     @re_plan.should_receive(:attributes=).with("name" => "name", "value" => "value")
-  #     put :update, :id => 123, :re_plan => { :name => "name", :value => "value", :code => "code" }
-  #   end
-  # 
-  #   it "should save the re_plan" do
-  #     @re_plan.should_receive(:save)
-  #     put :update, :id => 123, :re_plan => { :key => "value" }
-  #   end
-  #       
-  #   describe "save failed" do
-  #     it "should render the 'edit' template" do
-  #       put :update, :id => 123, :re_plan => { :key => "value" }
-  #       response.should render_template(:edit)
-  #     end
-  #   end
-  # 
-  #   describe "save succeeded" do
-  #     before do
-  #       @re_plan.stub!(:save).and_return(true)
-  #     end
-  #     
-  #     it "should display a flash success message" do
-  #       put :update, :id => 123, :re_plan => { :key => "value" }
-  #       flash[:success].should_not be_blank
-  #     end
-  #     
-  #     it "should redirect to the change re_plan page for HTML" do
-  #       put :update, :id => 123, :re_plan => { :key => "value" }
-  #       response.should redirect_to(change_re_plan_path(@re_plan))
-  #     end
-  # 
-  #     it "should render 'update' template for JAVASCRIPT" do
-  #       xhr :put, :update, :id => 123, :re_plan => { :key => "value" }
-  #       response.should render_template(:update)
-  #     end
-  #   end
-  # end
-  # 
-  # 
+  describe "new" do
+    it_should_require_rules_engine_editor_access(:new, :id => 123)
+    
+    it "should assign a new plan record" do
+      re_plan = RePlan.make
+      RePlan.should_receive(:new).and_return(re_plan)
+      get :new, :id => 123
+      assigns[:re_plan].should == re_plan
+    end
+    
+    it "should render the 'new' template" do
+      get :new, :id => 123
+      response.should render_template(:new)
+    end
+  end
+    
+  describe "create" do
+    it_should_require_rules_engine_editor_access(:create, :re_plan => {})
+  
+    before do
+      @re_plan = RePlan.make
+      RePlan.stub!(:new).and_return(@re_plan) 
+    end
+    
+    it "should assign the re_plan parameters" do
+      RePlan.should_receive(:new).with("code" => "name", "title" => "value")
+      post :create, :re_plan => { :code => "name", :title => "value" }
+    end
+  
+    it "should save the re_plan" do
+      @re_plan.should_receive(:save)
+      post :create, :re_plan => { :title => "name" }
+    end
+        
+    describe "save failed" do
+      before(:each) do
+        @re_plan.stub!(:save).and_return(false)
+      end
+      
+      it "should render the 'new' template" do
+        post :create, :re_plan => { :title => "name" }
+        response.should render_template(:new)
+      end
+    end
+  
+    describe "save succeeded" do
+      before(:each) do
+        @re_plan.stub!(:save).and_return(true)
+      end
+      
+      it "should display a flash success message" do
+        post :create, :re_plan => { :title => "name" }
+        flash[:success].should_not be_blank
+      end
+      
+      it "should redirect to the change re_plan page for HTML" do
+        post :create, :re_plan => { :title => "name" }
+        response.should redirect_to(change_re_plan_path(@re_plan))
+      end
+  
+      it "should render 'create' template for JAVASCRIPT" do
+        xhr :post, :create, :re_plan => { :title => "name" }
+        response.should render_template(:create)
+      end
+    end
+  end
+    
+  describe "edit" do
+    it_should_require_rules_engine_editor_access(:edit, :id => 123)
+    
+    it "should get the plan record with the ID" do
+      re_plan = RePlan.make
+      RePlan.should_receive(:find).with("123").and_return(re_plan)
+      get :edit, :id => 123
+      assigns[:re_plan].should == re_plan
+    end
+  end
+  
+  describe "update" do
+    it_should_require_rules_engine_editor_access(:update, :id => 123, :re_plan => {})
+  
+    before do
+      @re_plan = RePlan.make
+      RePlan.stub!(:find).and_return(@re_plan) 
+    end
+  
+    it "should get the plan record with the ID" do
+      RePlan.should_receive(:find).with("123").and_return(@re_plan)
+      put :update, :id => 123, :re_plan => { :title => "value" }
+      assigns[:re_plan].should == @re_plan
+    end
+    
+    it "should assign the re_plan parameters" do
+      @re_plan.should_receive(:attributes=).with("title" => "name")
+      put :update, :id => 123, :re_plan => { :title => "name" }
+    end
+  
+    it "should not assign the re_plan parameters :code" do
+      @re_plan.should_receive(:attributes=).with("title" => "name")
+      put :update, :id => 123, :re_plan => { :title => "name", :code => "code" }
+    end
+  
+    it "should save the re_plan" do
+      @re_plan.should_receive(:save)
+      put :update, :id => 123, :re_plan => { :title => "name" }
+    end
+        
+    describe "save failed" do
+      before(:each) do
+        @re_plan.stub!(:save).and_return(false)
+      end
+      
+      it "should render the 'edit' template" do
+        put :update, :id => 123, :re_plan => { :title => "name" }
+        response.should render_template(:edit)
+      end
+    end
+  
+    describe "save succeeded" do
+      before do
+        @re_plan.stub!(:save).and_return(true)
+      end
+      
+      it "should display a flash success message" do
+        put :update, :id => 123, :re_plan => { :title => "name" }
+        flash[:success].should_not be_blank
+      end
+      
+      it "should redirect to the change re_plan page for HTML" do
+        put :update, :id => 123, :re_plan => { :title => "name" }
+        response.should redirect_to(change_re_plan_path(@re_plan))
+      end
+  
+      it "should render 'update' template for JAVASCRIPT" do
+        xhr :put, :update, :id => 123, :re_plan => { :title => "name" }
+        response.should render_template(:update)
+      end
+    end
+  end
+  
+  describe "destroy" do
+    it_should_require_rules_engine_editor_access(:destroy, :id => 123)
+  
+    before do
+      @re_plan = RePlan.make
+      RePlan.stub!(:find).and_return(@re_plan) 
+    end
+  
+    it "should get the plan record with the ID" do
+      RePlan.should_receive(:find).with("123").and_return(@re_plan)
+      delete :destroy, :id => 123
+      assigns[:re_plan].should == @re_plan
+    end
+    
+    it "should destroy the re_plan" do
+      @re_plan.should_receive(:destroy)
+      delete :destroy, :id => 123
+    end
+   
+    it "should display a flash success message" do
+      delete :destroy, :id => 123
+      flash[:success].should_not be_blank
+    end
+    
+    it "should redirect to the re_plans page for HTML" do
+      delete :destroy, :id => 123
+      response.should redirect_to(re_plans_path)
+    end
+  
+    it "should redirect to the re_plans page for JAVASCRIPT" do
+      xhr :delete, :destroy, :id => 123
+      response.body.should == "window.location.href = '#{re_plans_path}';"
+    end    
+  end
+  
+  describe "change" do
+    it_should_require_rules_engine_editor_access(:change, :id => 123)
+    
+    it "should get the plan record with the ID" do
+      re_plan = RePlan.make
+      RePlan.should_receive(:find).with("123").and_return(re_plan)
+      get :change, :id => 123
+      assigns[:re_plan].should == re_plan
+    end
+  end
+  
+  describe "preview" do
+    it_should_require_rules_engine_reader_access(:preview, :id => 123)
+    
+    it "should get the plan record with the ID" do
+      re_plan = RePlan.make
+      RePlan.should_receive(:find).with("123").and_return(re_plan)
+      get :preview, :id => 123
+      assigns[:re_plan].should == re_plan
+    end
+  end
+  
+  describe "publish" do
+    it_should_require_rules_engine_editor_access(:publish, :id => 123)
+
+    before do
+      @re_plan = RePlan.make
+      @re_plan.stub!(:plan_error).and_return(nil)
+      RePlan.stub!(:find).and_return(@re_plan) 
+    end
+    
+    describe "plan has errors" do
+      it "should display a flash error message" do
+        @re_plan.should_receive(:plan_error).and_return('error')
+        put :publish, :id => 123, :tag => "publish tag"
+        flash[:error].should_not be_blank
+      end
+    end
+
+    describe "tag is blank" do
+      it "should display a flash error message" do
+        put :publish, :id => 123, :tag => ""
+        flash[:error].should_not be_blank
+      end
+    end
+
+    describe "plan is valid" do
+      before(:each) do
+        @publisher = mock('publisher')
+        @publisher.stub!(:publish)
+        RulesEngine::Publish.stub!(:publisher).and_return(@publisher)
+      end
+      
+      it "should publish the plan" do
+        @publisher.should_receive(:publish).with(@re_plan.code, "publish tag", @re_plan.publish)
+        put :publish, :id => 123, :tag => "publish tag"
+      end
+
+      it "should update the plan to published" do
+        put :publish, :id => 123, :tag => "publish tag"
+        @re_plan.plan_status.should == RePlan::PLAN_STATUS_PUBLISHED
+      end
+
+      it "should display a flash success message" do
+        put :publish, :id => 123, :tag => "publish tag"
+        flash[:success].should_not be_blank
+      end
+    end
+    
+    it "should redirect to the change_re_plan page for HTML" do
+      put :publish, :id => 123, :tag => "publish tag"
+      response.should redirect_to(change_re_plan_path(@re_plan))
+    end
+  
+    it "should render the update page for JAVASCRIPT" do
+      xhr :put, :publish, :id => 123, :tag => "publish tag"
+      response.should render_template(:update)
+    end    
+  end
+end
+
+
+
   # describe "activate_all" do
   #   it_should_require_rules_engine_editor_access(:activate_all)
   # 
@@ -369,41 +482,4 @@ describe RePlansController do
   #     response.should render_template(:update)
   #   end    
   # end
-  # 
-  # describe "destroy" do
-  #   it_should_require_rules_engine_editor_access(:destroy, :id => 123)
-  # 
-  #   before do
-  #     @re_plan = mock_model(RePlan)
-  #     @re_plan.stub!(:destroy)
-  #     RePlan.stub!(:find).and_return(@re_plan) 
-  #   end
-  # 
-  #   it "should get the plan record with the ID" do
-  #     RePlan.should_receive(:find).with("123").and_return(@re_plan)
-  #     delete :destroy, :id => 123
-  #     assigns[:re_plan].should == @re_plan
-  #   end
-  #   
-  #   it "should destroy the re_plan" do
-  #     @re_plan.should_receive(:destroy)
-  #     delete :destroy, :id => 123
-  #   end
-  #  
-  #   it "should display a flash success message" do
-  #     delete :destroy, :id => 123
-  #     flash[:success].should_not be_blank
-  #   end
-  #   
-  #   it "should redirect to the re_plans page for HTML" do
-  #     delete :destroy, :id => 123
-  #     response.should redirect_to(re_plans_path)
-  #   end
-  # 
-  #   it "should redirect to the re_plans page for JAVASCRIPT" do
-  #     xhr :delete, :destroy, :id => 123
-  #     response.body.should == "window.location.href = '#{re_plans_path}';"
-  #   end    
-  # end
-
-end
+  #