require 'spec_helper'

module RailsBestPractices
  module Reviews
    describe UseSayWithTimeInMigrationsReview do
      let(:runner) { Core::Runner.new(reviews: UseSayWithTimeInMigrationsReview.new) }

      it 'should use say with time in migrations' do
        content =<<-EOF
        def self.up
          User.find_each do |user|
            user.first_name, user.last_name = user.full_name.split(' ')
            user.save
          end
        end
        EOF
        runner.review('db/migrate/20101010080658_update_users.rb', content)
        expect(runner.errors.size).to eq(1)
        expect(runner.errors[0].to_s).to eq('db/migrate/20101010080658_update_users.rb:2 - use say with time in migrations')
      end

      it 'should use say with time in migrations with create_table' do
        content =<<-EOF
        def self.up
          create_table :users do |t|
            t.string :login
            t.timestamps
          end

          User.find_each do |user|
            user.first_name, user.last_name = user.full_name.split(' ')
            user.save
          end
        end
        EOF
        runner.review('db/migrate/20101010080658_update_users.rb', content)
        expect(runner.errors.size).to eq(1)
        expect(runner.errors[0].to_s).to eq('db/migrate/20101010080658_update_users.rb:7 - use say with time in migrations')
      end

      it 'should not use say with time in migrations' do
        content =<<-EOF
        def self.up
          say_with_time("Initialize first_name and last_name for users") do
            User.find_each do |user|
              user.first_name, user.last_name = user.full_name.split(' ')
              user.save
              say(user.id + " was updated.")
            end
          end
        end
        EOF
        runner.review('db/migrate/20101010080658_update_users.rb', content)
        expect(runner.errors.size).to eq(0)
      end

      it 'should not use say with time in migrations when not first code line' do
        content =<<-EOF
        def self.up
          User.find_each do |user|
            say_with_time 'Updating user with latest data' do
              user.do_time_consuming_stuff
            end
          end
        end
        EOF
        runner.review('db/migrate/20101010080658_update_users.rb', content)
        expect(runner.errors.size).to eq(0)
      end

      it 'should not use say with time when default migration message' do
        content =<<-EOF
        def self.up
          create_table :users do |t|
            t.string :login
            t.string :email
            t.timestamps
          end
        end
        EOF
        runner.review('db/migrate/20101010080658_create_users.rb', content)
        expect(runner.errors.size).to eq(0)
      end

      it 'should not raise an error' do
        content =<<-EOF
        class AddAdmin < ActiveRecord::Migration

          class Person < ActiveRecord::Base
          end

          class Blog < ActiveRecord::Base
          end

          def self.up
            add_column :people, :admin, :boolean, default: false, null: false
            add_column :people, :deactivated, :boolean,
                                default: false, null: false

            key = Crypto::Key.from_file("\#{RAILS_ROOT}/rsa_key.pub")
            person = Person.new(email: "admin@example.com",
                                name: "admin",
                                crypted_password: key.encrypt("admin"),
                                description: "")
            person.admin = true
            person.save!
            Blog.create(person_id: person.id)
          end

          def self.down
            remove_column :people, :deactivated
            Person.delete(Person.find_by_name("admin"))
            remove_column :people, :admin
          end
        end
        EOF
        runner.review('db/migrate/20101010080658_create_users.rb', content)
        expect(runner.errors.size).to eq(3)
      end

      it 'should not check ignored files' do
        runner = Core::Runner.new(reviews: UseSayWithTimeInMigrationsReview.new(ignored_files: /20101010080658_update_users/))
        content =<<-EOF
        def self.up
          User.find_each do |user|
            user.first_name, user.last_name = user.full_name.split(' ')
            user.save
          end
        end
        EOF
        runner.review('db/migrate/20101010080658_update_users.rb', content)
        expect(runner.errors.size).to eq(0)
      end
    end
  end
end