質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.59%

Rails ユーザー登録の単体テスト、バリデーションでのエラーを解決したい

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 3,610

score 5

解決したいこと

フリマアプリの作成中です。
ユーザー登録で、userモデルとaddressモデルの単体テストをクリアしたい。

テスト結果(user登録)

イメージ説明

テスト結果(address登録)

イメージ説明

spec/factories/user.rb

FactoryBot.define do
  password = Faker::Internet.password(min_length: 7, max_length: 100)

  factory :user do
    nickname              { "yamada" }
    email                 { Faker::Internet.email }
    password              { password }
    password_confirmation { password }
    first_name            { "ぜんかく" }
    last_name             { "ぜんかく" }
    first_kana            { "ゼンカクカナ" }
    last_kana             { "ゼンカクカナ" }
    birthday              { "2020-01-01" }
  end
end

spec/models/user_spec.rb

require 'rails_helper'

# RSpec.describe User, type: :model do
#   pending "add some examples to (or delete) #{__FILE__}"
# end

describe User do
  describe '#create' do

    it "nickname、first_name、last_name、first_kana、last_kana、email、password、password_confirmation、birthdayが存在すれば登録できること" do
      user = build(:user)
      expect(user).to be_valid
    end

    it "nicknameがない場合は登録できないこと" do
      user = build(:user, nickname: nil)
      user.valid?
      expect(user.errors[:nickname]).to include("can't be blank")
    end

    it "emailがない場合は登録できないこと" do
      user = build(:user, email: nil)
      user.valid?
      expect(user.errors[:email]).to include("can't be blank")
    end

    it "emailに@がない場合は登録できないこと " do
      user = build(:user, email: "aaaaaa")
      user.valid?
      expect(user.errors[:email]).to include("is invalid")
    end

    it "重複したemailが存在する場合登録できないこと" do
      user = create(:user)
      another_user = build(:user, email: user.email)
      another_user.valid?
      expect(another_user.errors[:email]).to include("has already been taken")
    end

    it "passwordがない場合は登録できないこと" do
      user = build(:user, password: nil)
      user.valid?
      expect(user.errors[:password]).to include("can't be blank")
    end

    it "passwordが7文字以上であれば登録できること" do
      password = Faker::Internet.password(min_length: 7, max_length: 7)
      user = build(:user, password: password, password_confirmation: password)
      user.valid?
      expect(user).to be_valid
    end

    it "passwordが6文字以下であれば登録できないこと" do
      password = Faker::Internet.password(min_length: 6, max_length: 6)
      user = build(:user, password: password, password_confirmation: password)
      user.valid?
      expect(user.errors[:password]).to include("is too short (minimum is 7 characters)")
    end

    it "passwordが7文字以上で、英数字の組み合わせであれば登録できること" do
      user = build(:user, password: "abcd123", password_confirmation: "abcd123")
      user.valid?
      expect(user).to be_valid
    end

    it "passwordが数字のみの場合は登録できないこと" do
      user = build(:user, password: "1234567", password_confirmation: "1234567")
      user.valid?
      expect(user.errors[:password]).to include("can't register with only numbers")
    end

    it "passwordが英文字のみの場合は登録できないこと" do
      user = build(:user, password: "abcdefg", password_confirmation: "abcdefg")
      user.valid?
      expect(user.errors[:password]).to include("can't register with only letters")
    end

    it "password_confirmationがない場合は登録できないこと" do
      user = build(:user, password_confirmation: nil)
      user.valid?
      expect(user.errors[:password_confirmation]).to include("can't be blank")
    end

    it "passwordとpassword_confirmationが一致していない場合は登録できないこと" do
      user = build(:user, password: "abcd123", password_confirmation: "1234abc")
      user.valid?
      expect(user.errors[:password_confirmation]).to include("doesn't match Password")
    end

    it "first_nameがない場合は登録できないこと" do
      user = build(:user, first_name: nil)
      user.valid?
      expect(user.errors[:first_name]).to include("can't be blank")
    end

    it "last_nameがない場合は登録できないこと" do
      user = build(:user, last_name: nil)
      user.valid?
      expect(user.errors[:last_name]).to include("can't be blank")
    end

    it "first_kanaがない場合は登録できないこと" do
      user = build(:user, first_kana: nil)
      user.valid?
      expect(user.errors[:first_kana]).to include("can't be blank")
    end

    it "last_kanaがない場合は登録できないこと" do
      user = build(:user, last_kana: nil)
      user.valid?
      expect(user.errors[:last_kana]).to include("can't be blank")
    end

    it "birthdayがない場合は登録できないこと" do
      user = build(:user, birthday: nil)
      user.valid?
      expect(user.errors[:birthday]).to include("can't be blank")
    end
  end

  describe '#ぜんかく' do

    it "first_nameが全角文字であれば登録できること" do
      user = build(:user, first_name: "ぜんかく")
      user.valid?
      expect(user).to be_valid
    end

    it "last_nameが全角文字であれば登録できること" do
      user = build(:user, last_name: "ぜんかく")
      user.valid?
      expect(user).to be_valid
    end

  end

  describe '#ゼンカクカナ' do

    it "first_kanaが全角カナ文字であれば登録できること" do
      user = build(:user, first_kana: "ゼンカクカナ")
      user.valid?
      expect(user).to be_valid
    end

    it "last_kanaが全角カナ文字であれば登録できること" do
      user = build(:user, last_kana: "ゼンカクカナ")
      user.valid?
      expect(user).to be_valid
    end

  end
end

app/models/address.rb

class Address < ApplicationRecord
  extend ActiveHash::Associations::ActiveRecordExtensions

  belongs_to :user, optional: true
  belongs_to_active_hash :prefecture

  VALID_ZIPCODE_REGEX = /\A\d{3}-?\d{4}\z/i
  validates :zipcode, presence: true,
            # 数字(3)文字、ハイフン(-)省略可、数字(4)文字になっているか
            format: { with: VALID_ZIPCODE_REGEX, message: "ハイフンを入れて半角英数字で入力してください" }
  validates :prefecture_id, :city, :address, presence: true
end

spec/factories/addresses.rb

FactoryBot.define do
  factory :address do
    zipcode               { "123-4567" }
    prefecture_id         { 1 }
    city                  { "大阪市北区" }
    address               { "中崎3-1" }
  end
end

spec/models/address_spec.rb

require 'rails_helper'

# RSpec.describe Address, type: :model do
#   pending "add some examples to (or delete) #{__FILE__}"
# end

describe Address do
  describe '#create' do

    it "zipcode、prefecture_id、city、addressが存在すれば登録できること" do
      address = build(:address)
      expect(address).to be_valid
    end

    it "zipcodeがない場合は登録できないこと" do
      address = build(:address, zipcode: nil)
      address.valid?
      expect(address.errors[:zipcode]).to include("can't be blank")
    end

    it "zipcodeは7桁でないと登録できないこと" do
      address = build(:address, zipcode: "123-4567")
      address.valid?
      expect(address.errors[:zipcode]).to include("doesn't enter in 7 digits")
    end

    it "prefecture_idがない場合は登録できないこと" do
      address = build(:address, prefecture_id: nil)
      address.valid?
      expect(address.errors[:prefecture_id]).to include("can't be blank")
    end

    it "cityがない場合は登録できないこと" do
      address = build(:address, city: nil)
      address.valid?
      expect(address.errors[:city]).to include("can't be blank")
    end

    it "addressがない場合は登録できないこと" do
      address = build(:address, address: nil)
      address.valid?
      expect(address.errors[:address]).to include("can't be blank")
    end

  end
end

正規表現、バリデーション関連にご有識のある方、ご助力お願いします。
蛇足な記述等あればそちらもご指摘いただけると幸いです。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

画像貼り付けでなく、text貼り付けにしてもらえないかな。コピペできないし検索できないし不便。

半角にしろというエラーメッセージですね。そういうvalidationかけてませんか?
Faker::Internet.passwordは(未確認ですが)大文字、英記号を含むんじゃないかな

あと、
expectでは英語を期待し、戻ってくるのは日本語ですね、locale調整してください

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.59%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る