モデルの単体テストコードでエラーが起きる

Question

formオブジェクトの単体テストコードで発生しているエラーを解決したい

お世話になっております。

現在、form_objectモデルの単体テストコードを記述しているのですが、実装上で起きたエラー内容が読み取れずどの部分を修正すればいいのかつまずいてしまったため、ご相談させて下さい。

今回エラーが起きている部分は、正常型で値が正しく入力されていれば投稿できるエラー検証になります。

form_objectでは、投稿に関するモデルになっており、

• tagテーブル(タグ付けテーブル)
• uploadテーブル(投稿テーブル)
• upload_tag_relation(タグとの中間テーブル)

今回は、upload_form.rbに記載されているバリデーションを単体テストコードで検証するという実装をしているところで、エラーが生じてしまっている状態です。

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

$ bundle exec rspec spec/models/upload_form_spec.rb　

1) UploadForm 新規投稿 正常に投稿できるとき すべての値が正しく入力されていれば投稿できる
     Failure/Error: expect(@upload_form).to be_valid
       expected #<UploadForm:0x00007fe26c894f08 @upload=#<Upload id: nil, title: nil, text: nil, url: nil, working_day: nil, day_off: nil, cafe_wifi_id: nil, cafe_charging_id: nil, cafe_smoking_id: nil, user_id: nil, created_at: nil, updated_at: nil>, @title="testcafe", @text="content", @url="http", @working_day="平日10:00~20:00", @day_off="不定休", @cafe_wifi_id=0, @cafe_charging_id=0, @cafe_smoking_id=0, @image=#<ActiveStorage::Attached::One:0x00007fe2671f4130 @name="image", @record=#<Upid: nil, title: nil, text: nil, url: nil, working_day: nil, day_off: nil, cafe_wifi_id: nil, cafe_charging_id: nil, cafe_smoking_id: nil, user_id: nil, created_at: nil, updated_at: nil>>, @name="東京", @user_id=552, @validation_context=nil, @error#<ActiveModel::Errors:0x00007fe269733018 @base=#<UploadForm:0x00007fe26c894f08 ...>, @errors=[#<ActiveModel::Error attribute=cafe_wifi_id, type=other_than, options={:value=>0, :count=>0}>, #<ActiveModel::Error attribute=cafe_charging_id, type=other_than, options={:value=>0, :count=>0}>, #<ActiveModel::Error attribute=cafe_smoking_id, type=other_than, options={:value=>0, :count=>0}>]>> to be valid, but got errors: Cafe wifiは0以外の値にしてください, Cafe chargingは0以外の値にしてください, Cafe smokingは0以外の値にしてください
     # ./spec/models/upload_form_spec.rb:13:in `block (4 levels) in <top (required)>'

該当のソースコード

app/models/upload_form.rb

class UploadForm
  include ActiveModel::Model
  attr_accessor :title, :text, :url, :working_day, :day_off, :cafe_wifi_id, :cafe_charging_id, :cafe_smoking_id,
                :user_id, :image, :name

  with_options presence: true do
    validates :title, length: { maximum: 40 }
    validates :text, length: { maximum: 1000 }
    validates :cafe_wifi_id
    validates :cafe_charging_id
    validates :cafe_smoking_id
    validates :user_id
    validates :image
    validates :name
  end

  with_options numericality: { other_than: 0 } do
    validates :cafe_wifi_id
    validates :cafe_charging_id
    validates :cafe_smoking_id
  end

  delegate :persisted?, to: :upload

  def initialize(attributes = nil, upload: Upload.new)
    @upload = upload
    attributes ||= default_attributes
    super(attributes)
  end

  def save(tag_list)

    ActiveRecord::Base.transaction do
      @upload.update(title: title, text: text, url: url, working_day: working_day, day_off: day_off, cafe_wifi_id: cafe_wifi_id, cafe_charging_id: cafe_charging_id, cafe_smoking_id: cafe_smoking_id, user_id: user_id, image: image)

      @upload.upload_tag_relations.each(&:delete)

      tag_list.each do |tag_name|
        tag = Tag.where(name: tag_name).first_or_initialize
        tag.save

        upload_tag_relation = UploadTagRelation.where(upload_id: upload.id, tag_id: tag.id).first_or_initialize
        upload_tag_relation.update(upload_id: upload.id, tag_id: tag.id)

      end
    end
    # upload = Upload.create(title: title, text: text, url: url, working_day: working_day, day_off: day_off, cafe_wifi_id: cafe_wifi_id, cafe_charging_id: cafe_charging_id, cafe_smoking_id: cafe_smoking_id, user_id: user_id, image: image)
    # map = Map.create(address: address, latitude: latitude, longitude: longitude, upload_id: upload_id)
  end

  def to_model
    upload
  end

  private

  attr_reader :upload

  def default_attributes
    {
      title: upload.title,
      text: upload.text,
      url: upload.url,
      working_day: upload.working_day,
      day_off: upload.day_off,
      cafe_wifi_id: upload.cafe_wifi_id,
      cafe_charging_id: upload.cafe_charging_id,
      cafe_smoking_id: upload.cafe_smoking_id,
      image: upload.image,
      name: upload.tags.pluck(:name).join(",")
    }
  end
end

app/models/upload.rb

class Upload < ApplicationRecord
  has_one_attached :image
  has_one :map
  belongs_to :user
  has_many :upload_tag_relations, dependent: :destroy
  has_many :tags, through: :upload_tag_relations
  has_many :favorites, dependent: :destroy
  has_many :comments, dependent: :destroy
  extend ActiveHash::Associations::ActiveRecordExtensions
  belongs_to :cafe_smoking
  belongs_to :cafe_charging
  belongs_to :cafe_wifi
end

app/models/tag.rb

class Tag < ApplicationRecord
  has_many :upload_tag_relations
  has_many :uploads, through: :upload_tag_relations

  validates :name, uniqueness: true

end

app/models/upload_tag_relaiton.rb

class UploadTagRelation < ApplicationRecord
  belongs_to :tag
  belongs_to :upload

  validate :upload_id
  validate :tag_id
end

app/spec/factories/upload_form.rb

FactoryBot.define do
  factory :upload_form do
    title { 'testcafe' }
    text { 'content' }
    url { 'http' }
    working_day { '平日10:00~20:00' }
    day_off { '不定休' }
    cafe_charging_id { 0 }
    cafe_smoking_id { 0 }
    cafe_wifi_id { 0 }
    name { '東京' }
    after(:build) do |upload_form|
      upload_form.image.attach(io: File.open('public/images/test_image2.jpeg'), filename: 'test_image2.jpeg')
    end
  end
end

app/spec/models/upload_form_spec.rb

require 'rails_helper'

RSpec.describe UploadForm, type: :model do
  before do
    @user = FactoryBot.create(:user)
    @upload_form = FactoryBot.build(:upload_form, user_id: @user.id)
    sleep 0.1 # 0.1秒待機
  end

  describe '新規投稿' do
  context '正常に投稿できるとき' do
    it 'すべての値が正しく入力されていれば投稿できる' do
      expect(@upload_form).to be_valid
    end
    it 'urlが空でも投稿できる' do
      @upload_form.url = nil
      expect(@upload_form).to be_valid
    end
    it 'working_dayが空でも投稿できる' do
      @upload_form.working_day = nil
      expect(@upload_form).to be_valid
    end
    it 'day_offが空でも投稿できる' do
      @upload_form.day_off = nil
      expect(@upload_form).to be_valid
    end
  end

  context '正常に投稿できないとき' do
    it 'titleが空だと投稿できない' do
      @upload_form.title = nil
      @upload_form.valid?
      expect(@upload_form.errors.full_messages).to include('Titleを入力してください')
    end
    it 'titleが40文字以上だと投稿できない' do
      @upload_form.title = 'a' * 41
      @upload_form.valid?
      expect(@upload_form.errors.full_messages).to include('Titleは40文字以内で入力してください')
    end
    it 'textが空だと投稿できない' do
      @upload_form.text = nil
      @upload_form.valid?
      expect(@upload_form.errors.full_messages).to include('Textを入力してください')
    end
    it 'textが1000文字以上だと投稿できない' do
      @upload_form.text = 'a' * 1001
      @upload_form.valid?
      expect(@upload_form.errors.full_messages).to include('Textは1000文字以内で入力してください')
    end
    it 'cafe_wifi_idが空だと投稿できない' do
      @upload_form.cafe_wifi_id = nil
      @upload_form.valid?
      expect(@upload_form.errors.full_messages).to include('Cafe wifiを入力してください')
    end
    it 'cafe_wifi_idの値が0だと投稿できない' do
      @upload_form.cafe_wifi_id = '0'
      @upload_form.valid?
      expect(@upload_form.errors.full_messages).to include('Cafe wifiは0以外の値にしてください')
    end
    it 'cafe_charging_idが空だと投稿できない' do
      @upload_form.cafe_charging_id = nil
      @upload_form.valid?
      expect(@upload_form.errors.full_messages).to include('Cafe chargingを入力してください')
    end
    it 'cafe_charging_idの値が0だと投稿できない' do
      @upload_form.cafe_charging_id = '0'
      @upload_form.valid?
      expect(@upload_form.errors.full_messages).to include('Cafe wifiは0以外の値にしてください')
    end
    it 'cafe_smoking_idが空だと投稿できない' do
      @upload_form.cafe_smoking_id = nil
      @upload_form.valid?
      expect(@upload_form.errors.full_messages).to include('Cafe smokingを入力してください')
    end
    it 'cafe_smoking_idの値が0だと投稿できない' do
      @upload_form.cafe_charging_id = '0'
      @upload_form.valid?
      expect(@upload_form.errors.full_messages).to include('Cafe wifiは0以外の値にしてください')
    end
    it 'user_idが空だと投稿できない' do
      @upload_form.user_id = nil
      @upload_form.valid?
      expect(@upload_form.errors.full_messages).to include('Userを入力してください')
    end
    it 'nameが空だと投稿できない' do
      @upload_form.name = nil
      @upload_form.valid?
      expect(@upload_form.errors.full_messages).to include('Nameを入力してください')
    end
    it 'imageが空だと投稿できない' do
      @upload_form.image = nil
      @upload_form.valid?
      expect(@upload_form.errors.full_messages).to include('Imageを入力してください')
    end
  end
end
end

試したこと

• テストコードを実行すると、正常型である正常に投稿できるときのテストコードに対してエラーが起きているため、それ以外の異常型に関しては修正する部分が無い。

• エラーコードを確認してみたところ、@upload=以下~~の部分でnilとなっているためこの部分がテストコードが上手くいかない原因ではないかと考えました。

• @uploadもテストコード上に何かしらの記述をする必要があるのか考えてみましたが、validation自体はupload_form_spec.rbに記述しているためどのように修正すればいいのか分かりませんでした。

補足情報（FW/ツールのバージョンなど）

• rubymine
• ruby(3.0.1)
• Ruby on rails (6.1.3.1)

Answer 1

バリデーションエラーのメッセージのとおりではないでしょうか？

Cafe wifiは0以外の値にしてください, Cafe chargingは0以外の値にしてください, Cafe smokingは0以外の値にしてください

それらの値に0を明記したものをFactoryBotで生成しているように見えます。

Comments

[yuu0000]

ご連絡ありがとうございます！

ご指摘頂いた通り、factoriesで0と表記してしまっておりました。
解決しました！

ありがとうございます。

また、私の知識不足でもありますが、テストコードでエラーが起きた場合は、エラーメッセージのどの部分に着目する必要があるのでしょうか？、、

[maisumakun]

「expected ○○、but got ××」が、「○○になると書いたけど実際には××という結果が得られた」という意味です。