Rspec ユーザー登録時の文字制限のテストの書き方

Question

ユーザーの文字制限のテストを書きたいのですが、うまくいきません。
よろしくお願いいたします！

    it "ユーザー名が５１文字以上の場合は登録できない、かつ、５０文字の場合は登録できる" do
      user = User.new(name: "a" * 51, password: "password")
      expect{ user.name.delete("a") }.to change{ user }.from( false ).to( user.save )
    end

class User < ApplicationRecord
  has_many :timetables, dependent: :destroy
  validates :name, presence: true, length: { maximum: 50 }
  has_secure_password
  validates :password, presence: true, length: { minimum: 6 }

Answer 1

ActiveRecordのバリデーションのテストを書くかどうかはおいておいて、書き方の例を回答します。
（個人的にはRSpecの勉強のために書くのは全然ありだと思います）

AvtiveRecord のinvalid?メソッドを使うとこのようにかけます。
Active Record バリデーション - Railsガイド

it "ユーザー名が５１文字以上の場合は登録できない" do
  user = User.new(name: "a" * 51, password: "password")
  expect(user.invalid?).to be true
end

または、RSpecのbe_invalid?マッチャーを使ってもかけると思います。

it "ユーザー名が５１文字以上の場合は登録できない" do
  user = User.new(name: "a" * 51, password: "password")
  expect(user).to be_invalid
end

どちらでもご自身がリーダブルだと思う方でいいと思います。

RSpecの書き方については、こちらの記事が情報がまとまっていてわかりやすかったです。

• 使えるRSpec入門・その1「RSpecの基本的な構文や便利な機能を理解する」 - Qiita
• 使えるRSpec入門・その2「使用頻度の高いマッチャを使いこなす」 - Qiita

Comments

[muraishi]

ご回答ありがとうございます！
そうなんですよ！まさにこのキータの記事をみながら勉強始めたんです！
invalid?っていうメソッドもあるんですね！今までvalidだけをつかって、not_toでテスト書いてました。

質問を追加して本当に申し訳ないのですが、
実は私も最初はshgtkshruchさんが教えてくださった、この方法でやったのですが（もちろんパスしました）、テストを書いた後に気づいたんです。このテストは５１文字以上名前を入力した時、ユーザー登録できないことはテストできているのですが、５０文字入力した時にユーザー登録が成功することはテストできていないので、また５０文字打った場合の別のテストを書いても良いのですが、なんとか一つのテストで５１文字の場合と５０文字の場合を同時にテストできればと思い

it "ユーザー名が５１文字以上の場合は登録できない" do
user = User.new(name: "a" * 51, password: "password")
expect{ user.name.delete("a") }.to change{ user }.from( false ).to( user.save )
end

こういう形になりました。こんな感じでなんとかテストを書くことってできないでしょうか？
（itを"ユーザー名が５１文字以上の場合は登録できない、かつ５０文字の時は登録できる"とするべきでした。すみません！）

※ちなみに、このchangeメソッドを使ったのは
使えるRSpec入門・その2「使用頻度の高いマッチャを使いこなす」 - Qiitaで、

expect{ X }.to change{ Y }.from(A).to(B) ＝ 「X すると Y が A から B に変わることを期待する」

という一文を自分なりに当てはめて書いてみました！

[maisumakun]

> なんとか一つのテストで５１文字の場合と５０文字の場合を同時にテストできれば

テストは単純にしたほうがいいです。変化したときに動作するか確認する目的のテストでないなら、「51文字で引っかかる」「50文字で引っかからない」ことを別途で検証することをおすすめします。

[muraishi]

わかりました。テストは簡潔にします。伊藤さんのキータでも、「さて、モックやRSpec Railsなどの特殊な用途のマッチャを除けば、ここまでに紹介したマッチャで標準的なテストはほとんどカバーできると思います。」と書いてあったのですね！

[shgtkshruch]

maisumakunさんが書いているように、テストは分けたほうがいいと思います。
各テストで確認するべきことは（関心事）は1つにする、シンプルにするのがいいと思います。
そのほうが失敗した場合にエラーが探しやすくなりますし、コードも読みやすい事が多いです。

RSpecの書き方はこちらのようなガイドラインがあります。
これが正解ではないですが、一つの指針として参考になると思います。
http://www.betterspecs.org/jp/

[muraishi]

はいテストは分けるようにします。なるほど、エラーを特定する時のために簡潔にしておくのがベストなんですね！
ガイドラインを教えていただきありがとうございます！
よんでみます！

Answer 2

そもそも論として、そのようなテストは必要なのでしょうか。

バリデーションを自作するとか、複数の制約の兼ね合いを検証しなければならないのなら別ですが、Rails組み込みのvalidates length:を使うだけの制約を、さらにテストコードで検証するというのは、少し過剰なようにも思えます。

Comments

[muraishi]

ご回答ありがとうございます。
そうなんですね！過剰なんですね！
RSpecを勉強したてで、練習のつもりで小さいコードを書いていたのですが、、、
独学だとRailsの一般常識が把握しづらいので、変なことを質問しがちですが、許してください。
ありがとうございます

[maisumakun]

> RSpecを勉強したてで、練習のつもりで小さいコードを書いていたのですが、、、

Rspec自体の練習だったのですね、それは失礼しました。

[muraishi]

いえいえ、テストの規模が小さすぎるのは省いても良いことを教えてくださりありがとうございます。これからもよろしくお願いいたします。