Проблемы с validates_inclusion_of, acts_as_tree и rspec

Question

У меня проблемы с правильной работой rspec для проверки validates_inclusion_ моей миграции выглядит следующим образом:

class CreateCategories < ActiveRecord::Migration
  def self.up
    create_table :categories do |t|
      t.string :name
      t.integer :parent_id
      t.timestamps
    end
  end

  def self.down
    drop_table :categories
  end
end

моя модель выглядит так:

class Category < ActiveRecord::Base
  acts_as_tree

  validates_presence_of :name
  validates_uniqueness_of :name
  validates_inclusion_of :parent_id, :in => Category.all.map(&:id), :unless => Proc.new { |c| c.parent_id.blank? }
end

мои заводы:

Factory.define :category do |c|
  c.name "Category One"
end

Factory.define :category_2, :class => Category do |c|
  c.name "Category Two"
end

моя модель выглядит так:

require 'spec_helper'

describe Category do
  before(:each) do
    @valid_attributes = {
      :name => "Category"
    }
  end

  it "should create a new instance given valid attributes" do
    Category.create!(@valid_attributes)
  end

  it "should have a name and it shouldn't be empty" do
    c = Category.new :name => nil
    c.should be_invalid
    c.name = ""
    c.should be_invalid
  end

  it "should not create a duplicate names" do
    Category.create!(@valid_attributes)
    Category.new(@valid_attributes).should be_invalid
  end

  it "should not save with invalid parent" do
    parent = Factory(:category)
    child = Category.new @valid_attributes
    child.parent_id = parent.id + 100
    child.should be_invalid
  end

  it "should save with valid parent" do
    child = Factory.build(:category_2)
    child.parent = Factory(:category)
    # FIXME: make it pass, it works on cosole, but I don't know why the test is failing
    child.should be_valid
  end
end

Я получаю следующую ошибку:

'Категория должна сохраняться с действительными parent 'FAILED Ожидается # быть действительным, но не было ошибок:

Родитель отсутствует

На консоли все вроде нормально и работает как положено:

c1 = Category.new :name => "Parent Category"
c1.valid? #=> true
c1.save #=> true
c1.id #=> 1
c2 = Category.new :name => "Child Category"
c2.valid? #=> true
c2.parent_id = 100
c2.valid? #=> false
c2.parent_id = 1
c2.valid? #=> true

Я использую рельсы 2.3.5, rspec 1.3.0 и rspec-rails 1.3.2

Кто-нибудь, есть идеи?

Answer 1

Проблема в том, что вы не можете позвонить на Category.all.map(&:id) внутри вызываемого на validates_inclusion_of.

Первое указание на то, что это так, станет очевидным при попытке запустить

rake db:migrate:down VERSION=<n>
rake db:migrate:up VERSOIN=<n>

где <n> - номер версии миграции, которая создает модель категории.

Вы получите что-то вроде:

in /Users/sseefried/tmp/so)
==  CreateCategories: reverting ===============================================
-- drop_table(:categories)
    -> 0.0032s
==  CreateCategories: reverted (0.0034s) ======================================

(in /Users/sseefried/tmp/so)
rake aborted!
SQLite3::SQLException: no such table: categories: SELECT * FROM "categories" 

(See full trace by running task with --trace)

Это потому, что rake пытается загрузить app/models/category.rb перед запуском миграции. Поскольку модель Category не существует, она терпит неудачу.

Другой способ увидеть проблему - это сделать tail -f log/development.log, а затем попытаться открыть консоль с помощью script/console. Вы увидите SQL-запрос в форме:

SELECT * FROM "categories"

на выходе. Это соответствует звонку на Category.all.map(:&id). Однако, как только вы начнете набирать команды, такие как:

c1 = Category.new, :name => "Category 1"

вы увидите, что запрос SELECT * from "categories" не появляется в журнале. Мораль этой истории такова: только константы могут появляться в вызовах validations_inclusion_of, потому что код там будет оцениваться только один раз. .

Единственная причина, по которой ваш консольный код работал, заключается в том, что в предыдущем сеансе консоли вы создали объект Category с id=1

Вы можете написать пользовательскую проверку, которая делает то, что вы хотите:

validate :parent_exists

protected

def parent_exists
  ids = Category.all.map(&:id)
  if !parent_id.blank? && !ids.member?(parent_id)
    errors.add(:parent_id, "does not point to a valid parent record")
  end
end

Ваши тесты rspec пройдут, как только вы добавите этот код.

Answer 2

На самом деле, вы можете отложить перечислимые вычисления, просто поместив Category.all.map(&:id) в proc / lambda. Запись

  validates_inclusion_of :parent_id,
                         in: proc{ Category.all.map(&:id) },
                         unless: proc{ |c| c.parent_id.blank? }

будет извлекать идентификаторы категорий во время проверки, а не во время объявления класса.