Sha256: 7452479d2f78e48d02d49ebd1fc8b705e62fceee923a6427f0a329fb51c3a7b7

Contents?: true

Size: 1.61 KB

Versions: 11

Compression:

Stored size: 1.61 KB

Contents

require 'spec_helper'

module RailsBestPractices
  module Reviews
    describe MoveCodeIntoHelperReview do
      let(:runner) { Core::Runner.new(reviews: MoveCodeIntoHelperReview.new('array_count' => 2)) }

      it "should move code into helper" do
        content = <<-EOF
        <%= select_tag :state, options_for_select( [[t(:draft), "draft"],
                                                    [t(:published), "published"]],
                                                   params[:default_state] ) %>
        EOF
        runner.review('app/views/posts/show.html.erb', content)
        expect(runner.errors.size).to eq(1)
        expect(runner.errors[0].to_s).to eq("app/views/posts/show.html.erb:1 - move code into helper (array_count >= 2)")
      end

      it "should not move code into helper with simple arguments" do
        content = <<-EOF
        <%= select_tag :state, options_for_select( Post.STATES ) %>
        EOF
        runner.review('app/views/posts/show.html.erb', content)
        expect(runner.errors.size).to eq(0)
      end

      it "should not check ignored files" do
        runner = Core::Runner.new(reviews: MoveCodeIntoControllerReview.new('array_count' => 2, 'ignored_files' => /app\/views\/post/))
        content = <<-EOF
        <%= select_tag :state, options_for_select( [[t(:draft), "draft"],
                                                    [t(:published), "published"]],
                                                   params[:default_state] ) %>
        EOF
        runner.review('app/views/posts/show.html.erb', content)
        expect(runner.errors.size).to eq(0)
      end
    end
  end
end

Version data entries

11 entries across 11 versions & 1 rubygems

Version Path
rails_best_practices-1.19.0 spec/rails_best_practices/reviews/move_code_into_helper_review_spec.rb
rails_best_practices-1.18.1 spec/rails_best_practices/reviews/move_code_into_helper_review_spec.rb
rails_best_practices-1.18.0 spec/rails_best_practices/reviews/move_code_into_helper_review_spec.rb
rails_best_practices-1.17.0 spec/rails_best_practices/reviews/move_code_into_helper_review_spec.rb
rails_best_practices-1.16.0 spec/rails_best_practices/reviews/move_code_into_helper_review_spec.rb
rails_best_practices-1.15.7 spec/rails_best_practices/reviews/move_code_into_helper_review_spec.rb
rails_best_practices-1.15.6 spec/rails_best_practices/reviews/move_code_into_helper_review_spec.rb
rails_best_practices-1.15.4 spec/rails_best_practices/reviews/move_code_into_helper_review_spec.rb
rails_best_practices-1.15.3 spec/rails_best_practices/reviews/move_code_into_helper_review_spec.rb
rails_best_practices-1.15.2 spec/rails_best_practices/reviews/move_code_into_helper_review_spec.rb
rails_best_practices-1.15.1 spec/rails_best_practices/reviews/move_code_into_helper_review_spec.rb