idがランダムに入るshowページリンクのリクエストspecが通らない

Question

お世話になります。

「お勧めはここ！」というリンクをクリックするとランダムにレストランを表示する仕組みを作りましたが、そうするとこれまで成功していたテストが失敗する様になりました。

失敗するテスト

RSpec.describe "静的ページ", type: :request do
  it "正常なレスポンスを返すこと" do
    get root_path
    expect(response).to be_success
    expect(response).to have_http_status "200"
  end
end

エラー表示

     Failure/Error: <%= link_to "お勧めはここ!", restaurant_path(random) %>
     
     ActionView::Template::Error:
       No route matches {:action=>"show", :controller=>"restaurants", :id=>nil}, missing required keys: [:id]

restaurant_path(random)のrandomはヘルパーに書いています。

  def random
    Restaurant.pluck(:id).sample
  end

機能的には問題なく動くのですが、テスト上ではidがnilになってしまうため失敗してしまいます。

test_helperにrandomメソッドを書いたりしてみましたが、idを読み込めない事が問題なため結果は変わりませんでした。

どの様にして解決すればよいでしょうか。
宜しくお願い致します。

Answer 1

単純に、テスト環境に1つもRestaurantが存在しないために、sampleでも取れない、ということではないでしょうか。

つまり、このテストが失敗するということは、「1つもRestaurantが存在しない場合には、エラーでページを表示できない」という問題を見事に拾い出した、とも言えます。

• 「1つもRestaurantが存在しない場合」を想定しなくていいのであれば、テスト環境のデータベースにfixtureなどで追加しておきましょう。
• 「1つもRestaurantが存在しない場合」がありうるのであれば、コード自体が誤りということになりますので書き直しが必要です（その場合は、ランダムリンクそのものを消す、リンクでない文字列にする、などが適切かと思います）。

Comments

[makobe]

ご回答いただきましてありがとうございます。
すごく単純なところを失念してしまっていました。
factoriesに作っていたRestaurantをlet!で呼び出す事でテストは通過する様になりました。
またご指摘いただいた「1つもRestaurantが存在しない場合には、エラーでページを表示できない」についてもよろしくないので、対処致します。
ありがとうございました。