require 'spec_helper'

module RailsBestPractices
  module Reviews
    describe LawOfDemeterReview do
      let(:runner) { Core::Runner.new(prepares: [Prepares::ModelPrepare.new, Prepares::SchemaPrepare.new], reviews: LawOfDemeterReview.new) }

      describe "belongs_to" do
        before(:each) do
          content = <<-EOF
          class Invoice < ActiveRecord::Base
            belongs_to :user
          end
          EOF
          runner.prepare('app/models/invoice.rb', content)

          content = <<-EOF
          ActiveRecord::Schema.define(version: 20110216150853) do
            create_table "users", force => true do |t|
              t.string :name
              t.string :address
              t.string :cellphone
            end
          end
          EOF
          runner.prepare('db/schema.rb', content)
        end

        it "should law of demeter with erb" do
          content = <<-EOF
          <%= @invoice.user.name %>
          <%= @invoice.user.address %>
          <%= @invoice.user.cellphone %>
          EOF
          runner.review('app/views/invoices/show.html.erb', content)
          expect(runner.errors.size).to eq(3)
          expect(runner.errors[0].to_s).to eq("app/views/invoices/show.html.erb:1 - law of demeter")
        end

        it "should law of demeter with haml" do
          content = <<-EOF
= @invoice.user.name
= @invoice.user.address
= @invoice.user.cellphone
          EOF
          runner.review('app/views/invoices/show.html.haml', content)
          expect(runner.errors.size).to eq(3)
          expect(runner.errors[0].to_s).to eq("app/views/invoices/show.html.haml:1 - law of demeter")
        end

        it "should law of demeter with slim" do
          content = <<-EOF
= @invoice.user.name
= @invoice.user.address
= @invoice.user.cellphone
          EOF
          runner.review('app/views/invoices/show.html.slim', content)
          expect(runner.errors.size).to eq(3)
          expect(runner.errors[0].to_s).to eq("app/views/invoices/show.html.slim:1 - law of demeter")
        end

        it "should no law of demeter" do
          content = <<-EOF
          <%= @invoice.user_name %>
          <%= @invoice.user_address %>
          <%= @invoice.user_cellphone %>
          EOF
          runner.review('app/views/invoices/show.html.erb', content)
          expect(runner.errors.size).to eq(0)
        end
      end

      describe "has_one" do
        before(:each) do
          content = <<-EOF
          class Invoice < ActiveRecord::Base
            has_one :price
          end
          EOF
          runner.prepare('app/models/invoice.rb', content)

          content = <<-EOF
          ActiveRecord::Schema.define(version: 20110216150853) do
            create_table "prices", force => true do |t|
              t.string :currency
              t.integer :number
            end
          end
          EOF
          runner.prepare('db/schema.rb', content)
        end

        it "should law of demeter" do
          content = <<-EOF
          <%= @invoice.price.currency %>
          <%= @invoice.price.number %>
          EOF
          runner.review('app/views/invoices/show.html.erb', content)
          expect(runner.errors.size).to eq(2)
          expect(runner.errors[0].to_s).to eq("app/views/invoices/show.html.erb:1 - law of demeter")
        end
      end

      context "polymorphic association" do
        before :each do
          content = <<-EOF
          class Comment < ActiveRecord::Base
            belongs_to :commentable, polymorphic: true
          end
          EOF
          runner.prepare('app/models/comment.rb', content)

          content = <<-EOF
          class Post < ActiveRecord::Base
            has_many :comments
          end
          EOF
          runner.prepare('app/models/comment.rb', content)

          content = <<-EOF
          ActiveRecord::Schema.define(version: 20110216150853) do
            create_table "posts", force => true do |t|
              t.string :title
            end
          end
          EOF
          runner.prepare('db/schema.rb', content)
        end

        it "should law of demeter" do
          content = <<-EOF
          <%= @comment.commentable.title %>
          EOF
          runner.review('app/views/comments/index.html.erb', content)
          expect(runner.errors.size).to eq(1)
          expect(runner.errors[0].to_s).to eq("app/views/comments/index.html.erb:1 - law of demeter")
        end
      end

      it "should no law of demeter with method call" do
        content = <<-EOF
        class Question < ActiveRecord::Base
          has_many :answers, dependent: :destroy
        end
        EOF
        runner.prepare('app/models/question.rb', content)
        content = <<-EOF
        class Answer < ActiveRecord::Base
          belongs_to :question, counter_cache: true, touch: true
        end
        EOF
        runner.prepare('app/models/answer.rb', content)
        content = <<-EOF
        class CommentsController < ApplicationController
          def comment_url
            question_path(@answer.question)
          end
        end
        EOF
        runner.review('app/controllers/comments_controller.rb', content)
        expect(runner.errors.size).to eq(0)
      end

      it "should not check ignored files" do
        runner = Core::Runner.new(prepares: [Prepares::ModelPrepare.new, Prepares::SchemaPrepare.new],
                                  reviews: LawOfDemeterReview.new(ignored_files: /app\/views\/invoices/))
        content = <<-EOF
          <%= @invoice.user.name %>
          <%= @invoice.user.address %>
          <%= @invoice.user.cellphone %>
        EOF
        runner.review('app/views/invoices/show.html.erb', content)
        expect(runner.errors.size).to eq(0)
      end
    end
  end
end