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

受付中

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 2,080

GOROGORO

score 69

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の使い方が誤っているからこのようになっていると思うのですが、
どのように使うのが正解なのでしょうか。

お教えいただけないでしょうか..
どうぞよろしくお願いします。
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

0

原因はエラーメッセージにも書いてありますが 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) }

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

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 と同様のエラーメッセージが出るはずです。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.22%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる