Rspecのコントローラー単体テストでActiveRecord::NotNullViolationが発生する

Question

前提・実現したいこと

この質問で解決したいこと：Rspecでcreateメソッドで保存した値を用いてテストを実行したい

こちらでの質問は初めてのため、質問内容に至らぬ点が多々あるかもしれません。
大変お手数ですが、情報に不備がある場合はご指摘いただけますと幸いです。

開発内容：惣菜投稿サイト。会員情報を登録したユーザーが惣菜を投稿・共有することを目的としたサイト。

エラーが起きている内容：Rspecでの投稿機能のコントローラー単体テスト実装中に、createメソッドで値を保存してテストを実行したいが、データベースのエラーが出てしまう

最終目標：createメソッドで値を保存し、投稿機能のコントローラー単体テストを実行すること

発生している問題・エラーメッセージ

 1) Delis Controller GET #index indexアクションにリクエストすると正常にレスポンスが返ってくる
     Failure/Error: @deli = FactoryBot.create(:deli)
     
     ActiveRecord::NotNullViolation:
       Mysql2::Error: Field 'name' doesn't have a default value
     # ./spec/requests/delis_spec.rb:5:in `block (2 levels) in <top (required)>'
     # ------------------
     # --- Caused by: ---
     # Mysql2::Error:
     #   Field 'name' doesn't have a default value
     #   ./spec/requests/delis_spec.rb:5:in `block (2 levels) in <top (required)>'

該当のソースコード

delis_spec.rb

require 'rails_helper'

RSpec.describe "Delis Controller", type: :request do
  before do
    @deli = FactoryBot.create(:deli)
    sleep(1)
  end

  describe "GET #index" do
    it "indexアクションにリクエストすると正常にレスポンスが返ってくる" do
      get root_path
      expect(response.status).to eq(200)
    end

    it 'indexアクションにリクエストするとレスポンスに投稿済みの惣菜名が存在する' do
      get root_path
      expect(response.body).to include(@deli.name)
    end
  end
end

delis.rb

FactoryBot.define do
  factory :deli do
    name {Faker::Food.dish}
    text {Faker::Lorem.sentence}
    supermarket_id{rand(1..10)}
    association :user
    association :category
    after(:build) do |deli|
      deli.image.attach(io: File.open("#{Rails.root}/public/testimage.jpg"), filename:'testimage.jpg')
    end
  end
end

deli.rb(モデルコントローラー記述内容)

class Deli < ApplicationRecord

  with_options presence:true do
    validates :name
    validates :text
    validates :supermarket_id
    validates :image
  end

  belongs_to :user
  belongs_to :category
  has_one_attached :image
  extend ActiveHash::Associations::ActiveRecordExtensions
  belongs_to_active_hash :supermarket
end

delis_controkker.rb(コントローラー記述内容、new/createメソッドのみ記載しています。)

class DelisController < ApplicationController

〜前略〜  

  def index
    @delis = Deli.all
  end

  def new
    @deli = Deli.new
  end

  def create
    @deli = Deli.new(deli_params)
    if @deli.save
      redirect_to root_path
    else
      render :new
    end
  end

〜中略〜

  private
  def deli_params
    params.require(:deli).permit(:name, :text, :category_id, :supermarket_id, :image).merge(user_id: current_user.id)
  end
end

マイグレーションファイル

class CreateDelis < ActiveRecord::Migration[6.0]
  def change
    create_table :delis do |t|

      t.string :name, null:false
      t.text :text, null:false
      t.integer :supermarket_id, null:false
      t.references :user, null:false, foreign_key: true
      t.references :category, null:false, foreign_key: true
      t.timestamps
    end
  end
end

試したこと

• エラー文からマイグレーションファイルやコントローラーのパラメーター部分に問題があると仮定し、記述内容を再確認

→確認後テスト結果に変化なし。尚、手動で新規投稿すると投稿が完了し、値が保存できている。
下記の記事を参照
https://qiita.com/divclass123/items/34c4a5a7200238a0589c
https://stackoverflow.com/questions/59644863/activerecordnotnullviolation-mysql2-column-cannot-be-null?answertab=active#tab-top

• テストコード内でbuildメソッド・saveメソッドと分けて記述を行う

→確認後のテスト結果に変化なし。なぜ二度手間の記述をしたかだが、buildメソッドでの値の生成のみは成功しているため、この方法を試した。

• FactoryBotによるnameカラムの値の生成を、Fakerを使用せず仮の文字列('test')で試みる

→確認後のテスト結果に変化なし。値の生成には問題なく、あくまで保存に関してデータベースの問題ではないかと考える

補足情報

ruby-version 2.6.5を使用。

長文で読みにくい文章になってしまい大変恐縮ですが、
ご助力いただけますと幸いです。
よろしくお願いいたします。

追記

値の生成についてテストファイル内に記述したところ、テストが成功しました。
そのことから、FactoryBotの記述に問題があるのでは？と考えております。

require 'rails_helper'

RSpec.describe "Delis", type: :request do
  before do
    @user = User.create(name:'sample', email:"[e-mail]", password:"sample00", password_confirmation:'sample00')
    @deli = Deli.create(name:'test', text:'test', category_id:1, supermarket_id:1, user_id:1)
  end

  describe "GET /delis" do
    it "indexアクションにリクエストすると正常にレスポンスが返ってくる" do
      get root_path
      expect(response.status).to eq(200)
    end

    it 'indexアクションにリクエストするとレスポンスに投稿済みの惣菜名が存在する' do
      get root_path
      expect(response.body).to include(@deli.name)
    end

    it 'indexアクションにリクエストするとレスポンスに投稿済みの画像が存在する' do
      
    end

    it 'indexアクションにリクエストするとレスポンスにカテゴリー名が存在する' do
      get root_path
      expect(response.body).to include(@deli.category.name)
    end

    it 'indexアクションにリクエストするとレスポンスに投稿者のユーザー名が存在する' do
      get root_path
      expect(response.body).to include(@deli.user.name)
    end
  end
end

Comments

[maisumakun]

えっと、肝心のindexアクションのコードはどこでしょうか？

[programmingnuoh]

ご確認ありがとうございます。
また、失礼いたしました。
ただいま質問内容のところのコントローラーファイル記述部分にindexアクションの内容を記載いたしました。
何卒よろしくお願いいたします。

Answer 1

そもそも論として、@deli = FactoryBot.create(:deli)はindexアクションの検証には全く不要ではないでしょうか。

Comments

[programmingnuoh]

ご回答いただきありがとうございます。
テストコードの内容に記載漏れがあったのですが、indexアクションの検証の中で投稿した内容が表示されているかという確認をしたいためcreateで値の保存をしていたところ、今回のエラーに遭遇いたしました。
質問内容がわかりづらくなっており申し訳ありません。

[maisumakun]

とりあえず、FactoryBot.create(:deli)が失敗しているようですね。

FactoryBot.build(:deli)のデータを確認してみるとどのようになっていますか？

[programmingnuoh]

下記のようになっております

=> #<Deli:0x00007f919a24eb08
id: nil,
name: "Bruschette with Tomato",
text: "In reprehenderit quae et.",
supermarket_id: 4,
user_id: nil,
category_id: 1,
created_at: nil,
updated_at: nil

[maisumakun]

それをsaveしてnameのNOT NULLを食らうのもおかしいですね…もしかしてですけど、テスト環境のマイグレーションを回し忘れているなどはありませんか？

[programmingnuoh]

下記の記事を参考にマイグレーションしてみたのですが結果変わりませんでした…。
他の方法があるのでしょうか？

https://qiita.com/mm36/items/b47e9cca3555c5a680af

[asm]

> user_id: nil,
user_idにもNOT NULLついてるので、こっちはエラー出て当然ですね

[programmingnuoh]

asmさん
ご指摘いただきありがとうございます。
buildで生成した場合でもuser_idは付くものなのでしょうか？
自分の過去の制作物等も確認したのですが、どれも生成のときにはuser_idはつかなかったのです。
（そもそもRspecの記述方法が根こそぎ間違っているのでしょうか？）

[asm]

buildを引き継ぐのでuser_idがnilになるかもしれません。
(ただ、category_idと何がちがうのか？が疑問です)

save時のsqlをログを探すなりして読み解くしかなさそうですね。

Answer 2

結論としては、アソシエーションを組んでいたcategoryのFactoryBotの記述内容が間違っていたため、エラーが発生していました。

FactoryBot.define do
  factory :category do
    name { 'アレンジレシピ' } ← id {1}
  end
end

idを記述してしまっていました。
nameカラムに値を入れる形で変更したところ、テストが通りました。

maisumakunさん、asmさん、ご助力いただきありがとうございました。