Ошеломленный RSpec

Question

Мне очень жаль, но мне начинает казаться, что я бью себя по голове.Я совершенно сбит с толку RSpec.Я смотрел видео за видео, читал туториал за туториалом, и все равно я просто застрял на исходной точке.

=== вот с чем я работаю

http://github.com/fudgestudios/bort/tree/master

=== Ошибки

F

1)
NoMethodError in 'bidding on an item should work'
You have a nil object when you didn't expect it!
You might have expected an instance of ActiveRecord::Base.
The error occurred while evaluating nil.new_record?
spec/controllers/auction_controller_spec.rb:16:
spec/controllers/auction_controller_spec.rb:6:

Finished in 0.067139 seconds

1 example, 1 failure

=== вот мое действие контроллера

  def bid

      @bid = Bid.new(params[:bid])
      @bid.save

  end

=== вот мой тест

require File.dirname(__FILE__) + '/../spec_helper'
include ApplicationHelper
include UsersHelper
include AuthenticatedTestHelper

describe "bidding on an item" do
  controller_name :items

    before(:each) do
      @user = mock_user
      stub!(:current_user).and_return(@user)
    end

  it "should work" do
    post 'bid', :bid => { :auction_id => 1, :user_id => @user.id, :point => 1 }
    assigns[:bid].should be_new_record
  end

end

=== специальный помощник

http://github.com/fudgestudios/bort/tree/master/spec/spec_helper.rb

Очень печально просыпаться на работу в 3 часа ночи.и ничего не добьешься за день.Пожалуйста, поймите.

Answer 1

У вас есть пара вещей в обратном порядке в before(:each).Поскольку в примере указано, что post должна увеличить количество записей на 1, вы имеете дело с реальными записями, и нет причин вообще что-либо заглушать.Кроме того, на данный момент, поскольку есть только один пример, нет причин иметь блок before .Я бы сделал это таким образом:

describe ItemsController, "bidding on an item" do
  fixtures :users

  it "should create a new Bid" do
    login_as :quentin
    lambda do
      post 'bid', :bid => { :auction_id => 1, :user_id => @user.id, :point => 1 }
    end.should change(Bid, :count).by(1)
  end

end

Одна вещь, которую я бы порекомендовал, - создавать эти вещи ОЧЕНЬ детально на данный момент, пока вы не поймете их лучше.Начните с ожидания (публикация должна изменить количество ставок), запустите спецификацию и позвольте сообщению о сбое подсказать вам, как добавить все, что вам еще нужно, в спецификацию или в код.

Answer 2

Джесси,

Это все равно пройдет, если вы закомментируете две вторые строки before(:each), которые не влияют на пример "следует создать новую ставку".

Ключевое слово lambda создает произвольный блок кода, который не выполняется при его определении, но фактически является объектом, который вы можете присвоить переменной и выполнить позже:

the_post = lambda do
  post 'bid', :bid => { :auction_id => 1, :user_id => @user.id, :point => 1 }
end

На данный момент этот код не выполняется, но мы можем ссылаться на него с помощью переменной 'the_post'.Теперь мы можем отправить ему "должен", за которым следует "изменить ...", вот так:

the_post.should change(Bid, :count).by(1)

Когда эта строка выполняется, происходит несколько вещей.Материал справа от "should" оценивается первым, инициализируя объект сопоставления rspec с некоторыми инструкциями.Этот сопоставитель является аргументом для 'should' - эквивалента этого:

matcher = change(Bid, :count).by(1)
the_post.should(matcher)

Метод 'should' вызывается в the_post, который является блоком кода (который все еще не был выполнен).Под капотом метод 'should' передает self (the_post) сопоставителю, так что у сопоставителя теперь есть все необходимое для оценки примера.

Сопоставитель вызывает Bid.count и записывает значение.Затем он выполняет блок (the_post), а затем вызывает Bid.count во второй раз и сравнивает его со значением, которое он записал ранее.В этом случае, поскольку мы ищем, чтобы Bid.count изменился на 1 (положительное значение здесь неявно - увеличить на 1), если это то, что происходит, сопоставитель замолкает, и пример проходит.

Если значения совпадают или отличаются на какое-то значение, отличное от 1, пример завершится неудачно.Вы можете увидеть, что это работает, если измените математическое ожидание на by(2) вместо by (1).

HTH, Дэвид

Answer 3

Редактировать:вы не должны ожидать увеличения Bid.count при использовании фиктивного объекта.Мантра, которую я забыл:кофеин перед кодированием.

Пока просто закомментирую строки, так что оригинал все еще там.

require File.dirname(__FILE__) + '/../spec_helper'
include ApplicationHelper
include UsersHelper
include AuthenticatedTestHelper

describe "POST to bid_controller" do
  controller_name :items

  before(:each) do
        #@bid = mock_model(Bid)           # create a new mock model so we can verify the appropriate things
        #Bid.stub!(:new).and_return(@bid) # stub the new class method on Bid to return our mock rather than a new ActiveRecord object.
                                         # this separates our controller spec entirely from the database.
  end

  it "should create a new Bid" do
    lambda do
        post 'bid', :bid => { :auction_id => 1, :user_id => @user.id, :point => 1 }
    end.should change(Bid, :count).by(1)
  end

    # ... more specs
end

Постарайтесь написать как можно меньше спецификаций, напишите свои настройки таким образом, чтобы было очевидно, что вы должны проверять в этой спецификации.Например, как я изменил ваш с it "should work" Для it "should create a new Bid".Если в этом контроллере есть что-то еще, напишите новую спецификацию для каждой небольшой части функциональности.

Если вам в конечном итоге понадобятся фиктивные пользователи, есть несколько помощников для restful_authentication, которые упрощают задачу.Сначала создайте пользовательское приспособление в RAILS_ROOT/spec/fixtures/users.yml, вот так:

quentin:
  login: quentin
  email: quentin@example.com
  salt: 7e3041ebc2fc05a40c60028e2c4901a81035d3cd
  crypted_password: 00742970dc9e6319f8019fd54864d3ea740f04b1 # test
  created_at: <%= 5.days.ago.to_s :db %>
  activation_code: 8f24789ae988411ccf33ab0c30fe9106fab32e9b 
  activated_at: <%= 5.days.ago.to_s :db %> 
  name: "Quentin"

Затем в своей спецификации вы сможете написать следующее и иметь свой current_user метод и все остальные части restul_authentication ведут себя так, как вы ожидаете от них во время выполнения.

login_as :quentin
# .... the rest of your spec

В качестве примера еще нескольких спецификаций я мог бы добавить еще пару примеров:

def do_post
    # extracting the method under test, so I don't repeat myself
    post 'bid', :bid => { :auction_id => 1, :user_id => @user.id, :point => 1 }
end

it "should create a new Bid" do
    lambda do
    do_post
    end.should change(Bid, :count).by(1)
end

it "should assign the Bid to the proper auction" do
    @bid.should_receive(:auction_id=).with(1) # test this, I believe doing  Bid.new(params[:bid]) sets the id directly not sets the model
    do_post
end

it "should assign the Bid the proper points" do
    @bid.should_receive(:point=).with(1)
    do_post
end

Answer 4

Пока я не совсем понимаю, что происходит.(с заглушками и лямбдой)....

для

def bid
  @bid = Bid.new params[:bid]
  @bid.save
end

Следующее проходит !!

require File.dirname(__FILE__) + '/../spec_helper'
include ApplicationHelper
include UsersHelper
include AuthenticatedTestHelper

describe "bidding on an item" do
  controller_name :items
  fixtures :users

  before(:each) do
    @user = login_as :quentin
    @bid = mock_model(Bid)           # create a new mock model so we can verify the appropriate things
    @bid.stub!(:new).and_return(@bid) # stub the new class method on Bid to return our mock rather than a new ActiveRecord object.
    #Bid.stub!(:save).and_return(true)# this separates our controller spec entirely from the database.
  end

  it "should create a new Bid" do
    lambda do
      post 'bid', :bid => { :auction_id => 1, :user_id => @user.id, :point => 1 }
    end.should change(Bid, :count).by(1)
  end

end