Rspecテストの仕方がわかりません、

Question

Ruby on Rails でWebサイトを作成しており
app/controllers/products_controller.rbのテストを書いています。

class ProductsController < ApplicationController
  def show
    @product = product.find(params[:id])
    @product_properties = @product.product_properties.includes(:property)
  end
end

テストは下記のspec/controllers/products_controller_spec.rbです

require 'rails_helper'

RSpec.describe ProductsController, type: :controller do
  let(:product) { create(:product) }

  describe "show Action successfull" do
    before do
      get :show, params: { id: product.id }
    end

    it "@productが期待される値を持つこと" do
      expect(assigns(:product)).to eq(product)
    end
  end
end

propertyのテストを書きたいのですが、どう書けばいいかわかりません。教えて頂けないでしょうか。

Answer 1

上から説明すると、

let(:product) { create(:product) }

→productを作成するよ！

    before do
      get :show, params: { id: product.id }
    end

→itの前に、showアクションするよ！

    it "@productが期待される値を持つこと" do
      expect(assigns(:product)).to eq(product)
    end

→@productがletで定義したproductと同じだと嬉しいよ！

。。。という流れになっているかと思います。

おそらく、beforeで実施するアクションが、itの中に含まれていないので、it内のassigns(:product)がどの何のアクションの結果の@productなのかが不明なんだと思います。

私たちの会社では、以下のように記載しています。

RSpec.describe ProductsController, type: :controller do
  describe "show Action successfull" do
    subject { get :show, params: params } }
    let(:params) { { id: product.id } }
    let(:product) { create(:product) }

    it "@productが期待される値を持つこと" do
      subject
      expect(assigns(:product)).to eq(product)
    end
  end
end

subjectではメソッドの実行部を書き、これをshowメソッド内のテストを行う時は、最初に叩くようにします。
params: paramsとしている箇所は、今後showのパラメータが増えるケースがある場合に、
it内でlet(:params)を定義してあげれば、そのit内ではそのparamsが使用されるため、it内で完結するため、他のテストに影響が出にくくなります。（specは遅延評価であるため。）

そして、itの中で、subject(showアクション)を実行し、その結果をexpectで比較している。。。という形になります。

こんだけ説明して間違っていたらごめんなさい！！！ｗｗｗ