Могу ли я получить некоторую обратную связь с rspec при написании спецификаций контроллеров? - PullRequest
Новая
       26

Могу ли я получить некоторую обратную связь с rspec при написании спецификаций контроллеров?

2 голосов
/ 08 июня 2011

Мне было интересно, смогу ли я получить какие-либо отзывы со спецификацией контроллера ниже.На самом деле, я новичок, когда пишу спецификации и спецификации контроллера, которые сильно отличаются от спецификации модели!Поэтому мне интересно, если я не могу пойти в неправильном направлении ...

subject_controller.rb 

def show
  @subject = Subject.find(params[:id])

  if @subject.trusted?(current_user)
    @messages = @subject.messages
  else
    @messages = @subject.messages.public
    @messages = @messages + @subject.messages.where(:user_ids => current_user.id)
    @messages.uniq!
  end

  # sort the list
  @messages = @messages.sort_by(&:created_at).reverse

  if !@subject.company.id == current_user.company.id
    redirect_to(subjects_path, :notice => "Invalid subject")
  end
end

subject_controller_spec.rb 

require 'spec_helper'

describe SubjectsController do  
  before(:each) do
    @subject = mock_model(Subject)
  end

  context "for signed users" do
    before(:each) do
      @current_user = sign_in Factory(:user)
    end

    context "GET #show" do
      before(:each) do
        Subject.stub!(:find, @subject).and_return(@subject)
      end

      context "when current_user is trusted" do
        before(:each) do
          messages = []
          company = mock_model(Company)

          @subject.should_receive(:trusted?).and_return(true)
          @subject.should_receive(:messages).and_return(messages)
          @subject.should_receive(:company).and_return(company)
        end

        it "should render success" do
          get :show, :id => @subject
          response.should be_success
        end
      end

      context "when current_user is not trusted" do
        before(:each) do
          messages = []
          company = mock_model(Company)

          @subject.should_receive(:trusted?).and_return(false)
          @subject.should_receive(:messages).and_return(messages)
          messages.should_receive(:public).and_return(messages)
          @subject.should_receive(:messages).and_return(messages)
          messages.should_receive(:where).and_return(messages)
          @subject.should_receive(:company).and_return(company)
        end

        it "should render success" do
          get :show, :id => @subject
          response.should be_success
        end
      end

      context "when subject's company is not equal to current_user's company" do
        # I have no idea of how to implement ==
      end
    end
  end
end

Factories.rb

Factory.define :user do |u|
  u.first_name 'Test User'                          #
  u.username 'Test User'                            #
  u.surname 'TheTest'                               #
  u.email 'foo@foobar.com'                          #
  u.password 'please'                               #
  u.confirmed_at Time.now                           #
end

Ответы [ 2 ]

0 голосов
/ 26 октября 2011

Не пишите спецификации контроллера RSpec вообще. Вместо этого используйте истории с огурцами. Гораздо проще, и вы получите лучшее освещение.

0 голосов
/ 08 июня 2011

Насколько я могу судить, вы на правильном пути. Основная идея состоит в том, чтобы полностью изолировать код вашего контроллера от модели и представления в этих тестах. Вы, кажется, делаете это - заглушка и насмешливая модель взаимодействия.

...