require File.join(File.dirname(__FILE__) + '/../../spec_helper') describe RailsBestPractices::Checks::ReplaceComplexCreationWithFactoryMethodCheck do before(:each) do @runner = RailsBestPractices::Core::Runner.new(RailsBestPractices::Checks::ReplaceComplexCreationWithFactoryMethodCheck.new) end it "should replace complex creation with factory method" do content = <<-EOF class InvoiceController < ApplicationController def create @invoice = Invoice.new(params[:invoice]) @invoice.address = current_user.address @invoice.phone = current_user.phone @invoice.vip = (@invoice.amount > 1000) if Time.now.day > 15 @invoice.deliver_time = Time.now + 2.month else @invoice.deliver_time = Time.now + 1.month end @invoice.save end end EOF @runner.check('app/controller/invoices_controller.rb', content) errors = @runner.errors errors.should_not be_empty errors[0].to_s.should == "app/controller/invoices_controller.rb:3 - replace complex creation with factory method" end it "should not replace complex creation with factory method with simple creation" do content = <<-EOF class InvoiceController < ApplicationController def create @invoice = Invoice.new(params[:invoice]) @invoice.address = current_user.address @invoice.phone = current_user.phone @invoice.save end end EOF @runner.check('app/controller/invoices_controller.rb', content) errors = @runner.errors errors.should be_empty end it "should not replace complex creation with factory method when attrasgn_count is 5" do content = <<-EOF class InvoiceController < ApplicationController def create @invoice = Invoice.new(params[:invoice]) @invoice.address = current_user.address @invoice.phone = current_user.phone @invoice.vip = (@invoice.amount > 1000) if Time.now.day > 15 @invoice.deliver_time = Time.now + 2.month else @invoice.deliver_time = Time.now + 1.month end @invoice.save end end EOF @runner = RailsBestPractices::Core::Runner.new(RailsBestPractices::Checks::ReplaceComplexCreationWithFactoryMethodCheck.new('attribute_assignment_count' => 5)) @runner.check('app/controller/invoices_controller.rb', content) errors = @runner.errors errors.should be_empty end end