presence: true が設定された model を RSpec でテスト

Question

Rails 初心者です。Rspec を習得すべく、下記のサイトで学習しています。
https://leanpub.com/everydayrailsrspec-jp/read

下記の単純な Contact モデルに対して、

class Contact < ActiveRecord::Base
  has_many :phones
  accepts_nested_attributes_for :phones

  validates :firstname, presence: true
  validates :lastname, presence: true
  validates :email, presence: true, uniqueness: true

  def name
    [firstname, lastname].join(' ')
  end

  def self.by_letter(letter)
    where("lastname LIKE ?", "#{letter}%").order(:lastname)
  end
end

サンプルとして下記のようなスペックが書かれている・・・

describe Contact, type: :model do
it "is invalid without a firstname" do
  contact = Contact.new(firstname: nil)
  expect(contact.errors[:firstname]).to include("can't be blank")
end

のですが、エラーメッセージをチェックする、というのに違和感があり、あれこれ検索したところ、下記でも可能なことを知りました。

it "is invalid without a firstname" do
  should validate_presence_of :firstname
end

または

it "is invalid without a firstname" do
  is_expected.to validate_presence_of :firstname
end

エラーメッセージをチェックする方法のメリットを考えてみたのですが、思い当たりません。

実際、どちらの書き方がよいでしょうか。

Answer 1

エラーメッセージをチェックするメリットは例えば国際化対応をしてエラーメッセージを日本語化しようとした場合に、
きちんと意図したエラーメッセージが返るかどうかをテストできるのがメリットではないでしょうか？

単にエラーが起きることだけを検証したい場合は質問者様の調べられた書き方でもいいと思います。
should を使うよりexpectedを使うことが奨励されています。

用途によって使い分ければよいのはないでしょうか？

Comments

[isaoyoshikoshi]

ご回答をありがとうございました。
日本語化はほぼ必須でしょうから、そのチェックも兼ねることができますね。用途によって使い分ける、ということも、納得しました。

Answer 2

validation のテストでは、次のことをチェックする必要があると思います。

1. valide 判定ができている事。
2. unvalide の場合に、どの項目にどんなエラー(メッセージ)があるのかを確認する事。

従ってエラーメッセージをチェックしないのは、片手落ちと思います。
↓のコード例では、valide 判定と、指定項目に想定したエラーメッセージが設定されていることの２つをチェックしています。

参考情報：

• https://leanpub.com/everydayrailsrspec-jp/read#leanpub-auto-section-15

...
1 # 名がなければ無効な状態であること

2 it "is invalid without a firstname" do
3 contact = Contact.new(firstname: nil)
4 contact.valid?
5 expect(contact.errors[:firstname]).to include("can't be blank")
6 end

...

Comments

[isaoyoshikoshi]

ご回答をありがとうございました。
なるほど、エラーメッセージもテストすることが必要、と考えれば、両方についてチェックできるこの方法が合理的ですね！よく理解できました。