rspecでのsubjectの使い方について

Question

Rails4 Rspec3でアプリケーション開発をしています。

テストコードとして以下のように記載しています。

    # 有効な属性の場合
    context "with valid attributes" do
      before{
      #blog_imdexを生成
        @blog_image=FactoryGirl.build(:blog_image)
        #ファイルが登録されるディレクトリを指定
        @dir_path = "./public/images/#{Time.now.strftime("%Y")}/#{@blog_image.blog_index_id}/"
        #登録する画像準備
        IMG_FILE_NAME = 'test1.jpg'
        @img_file_path = 'file/'+IMG_FILE_NAME
        #登録する画像準備
        upload_file_path = @dir_path + IMG_FILE_NAME
        #ファイル事前に削除
        File.unlink(upload_file_path) if File.exist?(upload_file_path)
      }

##test1
      subject {
        post :create,
            :blog_images => fixture_file_upload(@img_file_path, 'image/jpg')
      }

##test2
      it { is_expected.to change(BlogImage, :count).by(1) }

      it "saves the new blog_images in the database" do
        expect{
          post :create,
          :blog_images => fixture_file_upload(@img_file_path, 'image/jpg')
        }.to change(BlogImage, :count).by(1)
      end
    end
  end

test1とtest2でsubjectを使用しているだけで、内容としては同じ意味合いとして記載しているつもりです。

ですが、テストを実行するとtest1のみエラーとなります。

Failures:

  1) BlogImagesController POST #create with valid attributes should change #count by 1
     Failure/Error: it { is_expected.to change(BlogImage, :count).by(1) }
       expected #count to have changed by 1, but was not given a block
     # ./spec/controllers/blog_images_controller_spec.rb:33:in `block (4 levels) in <top (required)>'

subjectの使い方が誤っているからこのようになっていると思うのですが、
どのように使うのが正解なのでしょうか。

お教えいただけないでしょうか..
どうぞよろしくお願いします。

Answer 1

原因はエラーメッセージにも書いてありますが but was not given a block です。
change は expect に block が必要です。

test1 について

it { is_expected.to change(BlogImage, :count).by(1) }

は

it { expect(subject).to change(BlogImage, :count).by(1) }

と同じ意味になり、エラーメッセージのとおりexpectにblockがないのでエラーになります

it { expect { subject }.to change(BlogImage, :count).by(1) }

とする事で通ると思います。

test2 について

it "saves the new blog_images in the database" do
  expect{
    post :create,
    :blog_images => fixture_file_upload(@img_file_path, 'image/jpg')
  }.to change(BlogImage, :count).by(1)
end

こちらは expect に block があるので、このままで通ります。

test1と同様の誤りにするのであれば

it "saves the new blog_images in the database" do
  expect(
    post :create,
    :blog_images => fixture_file_upload(@img_file_path, 'image/jpg')
  ).to change(BlogImage, :count).by(1)
end

という感じで、これは test1 と同様のエラーメッセージが出るはずです。