苦手なテストについてふりかえろうと思い書いております。
今回は単体テストについて
単体テストコード
モデルやコントローラーなどの機能ごとに問題がないか確かめます.
バリデーションの挙動を確認します。
Ruby on Rails内にも、アプリケーションのテストコードを記述する機能を設けることができます。その機能はRSpecというGemを使うことで便利に実装できます。
Gemを追加しましょう
Gemfile
group :development, :test do
# Call 'byebug' anywhere in the code to stop execution and get a debugger console
gem 'byebug', platforms: [:mri, :mingw, :x64_mingw]
gem 'rspec-rails', '~> 4.0.0'
end
注意するのが、group :development, :testというグループの中に記述しましょう。
一番下ではないです。
bundle installを実行しましょう
% bundle install
インストールが完了すると、以下のようにディレクトリやファイルが生成されます。
create .rspec
create spec
create spec/spec_helper.rb
create spec/rails_helper.rb
.rspecに設定を追加しましょう
--require spec_helper
--format documentation
ここで、FactoryBotというものも導入
FactoryBot
インスタンスをまとめることができるGemです。他のファイルであらかじめ各クラスのインスタンスに定める値を設定しておき、各テストコードで使用します。
Gemfileを編集しましょう
group :development, :test do
# Call 'byebug' anywhere in the code to stop execution and get a debugger console
gem 'byebug', platforms: [:mri, :mingw, :x64_mingw]
gem 'rspec-rails', '~> 4.0.0'
gem 'factory_bot_rails'
end
この時も、group :development, :test doの中に記述するように注意してください。
Gemを導入するために、bundle installを実行しましょう。
% bundle install
コマンドを実行して、テストファイルを作成しましょう
rails g rspec:model user
FactoryBot導入後はテストコードを記述するファイルを生成すると同時に、自動生成されます。
先にテストコードのファイルを作成してると、Factory.Bot用のファイルは自分で作成します。
factoriesの下にファイル名.rbで作る
インスタンスの各値を設定しましょう
FactoryBot.define do
factory :user do
nickname {'test'}
email {'test@example'}
password {'000000'}
password_confirmation {password}
end
end
設定したインスタンスを生成するためには、FactoryBot.build(:user)という記述をテストコードの中に記述します。
require 'rails_helper'
RSpec.describe User, type: :model do
describe '#create' do
before do
@user = FactoryBot.build(:user)
end
context 'ユーザ新規登録ができる時' do
it 'nicknameとemail,passwordとpassword_confirmation,family_nameとfist_name,family_name_kanaとfist_name_kana,birthdateが存在すれば登録できる' do
expect(@user).to be_valid
end
it 'passwordが6文字以上であれば登録できること' do
@user.password = '123abc'
@user.password_confirmation = '123abc'
expect(@user).to be_valid
end
end
context 'ユーザ新規登録ができない時' do
it 'nicknameが空では登録できない' do
@user.nickname = ''
@user.valid?
expect(@user.errors.full_messages).to include("Nickname can't be blank")
end
〜中略〜
context 'ユーザ新規登録ができる時' do~endを使って、正常系と異常系を分けると見やすくなる
インスタンスを生成する際に、FactoryBotで設定した値は、'test@example'など固有のものでした。しかし、これから複数のテストを行う際に、「すでにemailの重複したインスタンスが存在する」といったような意図しない形で弾かれてしまうことがあります。
そんな時に使うのが、
Faker
ランダムな値を生成するGemです。メールアドレス、人名、パスワードなど、さまざまな意図に応じたランダムな値を生成してくれます。
Gemfileに以下の様に追記します。今回はFakerをテストコードのために使用するので、これまで同様group :development, :test doの中に記述しましょう。
group :development, :test do
# Call 'byebug' anywhere in the code to stop execution and get a debugger console
gem 'byebug', platforms: [:mri, :mingw, :x64_mingw]
gem 'rspec-rails', '~> 4.0.0'
gem 'factory_bot_rails'
gem 'faker'
end
そして
% bundle install
Fakerを使う場合
FactoryBot.define do
factory :user do
nickname {Faker::Name.initials(number: 2)}
email {Faker::Internet.free_email}
password {Faker::Internet.password(min_length: 6)}
password_confirmation {password}
end
end
ただ、複雑なバリデーションの場合は自分で仮に作った方が簡単な時がある。
例えばパスワードを英数字混合とかにしてたら、たまに数字のみになったりしてテストがエラーになったりする
最後にテストの実行コード
bundle exec rspec spec/models/テストしたいファイル名.spec.rb
はじめに
他に便利な使い方があれば教えてもらえると幸いです
間違っている箇所があれば教えてもらえると幸いです
レシピ
条件付き attribute
条件でキーの有無が変わるときは下記のように書けます。
app/serializers/user_serializer.rb
class UserSerializer < ApplicationSerializer
attribute :posts_count, if: -> { object.posts.present? } do
object.posts.count
end
end
シリアライザ内で使えるパラメーターを渡す
シリアライザにキー、バリューを渡すことで
app/controllers/users_controller.rb
class UsersController < Api::ApplicationController
def show
render json: current_user,
serializer: UserSerializer,
hoge: 'test'
end
end
or
ActiveModelSerializers::SerializableResource.new(
user,
serializer: UserSerializer,
adapter: :attributes,
hoge: 'test'
).serializable_hash
シリアライザ内において instance_options からその値を参照することができます。
app/serializers/user_serializer.rb
class UserSerializer < ApplicationSerializer
attribute :test_val do
instance_options[:hoge]
end
end
個人的ベストプラクティス
シリアライザは明示的に指定した方が良さそう
シリアライザを明示的に指定なくてもモデルのクラス名から推測して UserSerializer を使ってくれますが、
app/controllers/users_controller.rb
class UsersController < Api::ApplicationController
def show
render json: current_user
end
end
UserSerializer を変更したときの影響範囲を調べるときにシリアライザ名で grep しにくい
一目でどのシリアライザを使っているかわかりにくい
ことからシリアライザを明示的に指定した方が良さそうです。
app/controllers/users_controller.rb
class UsersController < Api::ApplicationController
def show
render json: current_user,
serializer: UserSerializer
end
end
attribute とメソッドを2つ使うよりは attribute のみでキーを出力した方が良さそう
attribute とメソッド定義を2つ書くより
app/serializers/user_serializer.rb
class UserSerializer < ApplicationSerializer
attribute :name_hoge
def name_hoge
"#{object.name}_hoge"
end
end
attribute ブロックのみを使った方がシンプルになります。
app/serializers/user_serializer.rb
class UserSerializer < ApplicationSerializer
attribute :name_hoge do
"#{object.name}_hoge"
end
end
proc のため return は使えないことに注意してください
メソッドの処理そのままでメソッド名と異なるキー名を使いたいときは attribute を使った方が良さそう
attribute ブロックを書くより
attribute :is_admin do
object.admin?
end
attribute メソッドの key オプションを使った方がシンプルに書けます。
app/serializers/user_serializer.rb
class UserSerializer < ApplicationSerializer
attribute :admin?, key: :is_admin
end
リソースのハッシュ出力では adapter: :attributes オプションを使ってルートキーを出力させない方が良さそう
serializable_hash[:post] のようにキー名を指定する書き方をするとシリアライザの名前空間が変わったり、type メソッドに渡す値が変わるとキー名が変わってしまいデグレが起こる可能性があるので、ルートキーが必要ない場合は adapter: :attributes を使用してキー名を記載しない方が良いと思います。
単一リソース
ActiveModelSerializers::SerializableResource#serializable_hash を使います。
serializer オプションにシリアライザを指定します。
app/serializers/user_serializer.rb
class UserSerializer < ApplicationSerializer
attribute :single_post do
ActiveModelSerializers::SerializableResource.new(
object.single_post,
serializer: PostSerializer,
adapter: :attributes
).serializable_hash
end
end
複数リソース
each_serializer オプションにシリアライザを指定します。
app/serializers/user_serializer.rb
class UserSerializer < ApplicationSerializer
attribute :posts do
ActiveModelSerializers::SerializableResource.new(
object.posts,
each_serializer: PostSerializer,
adapter: :attributes
).serializable_hash
end
end
アソシエーションをシリアライズするときは has_many, has_one, belongs_to を使うよりも手動でシリアライズした方がパラメーターを渡すことができるので良さそう
アソシエーションも返却するときは has_many などのメソッドを使えばシンプルに書くことができますが、
app/serializers/user_serializer.rb
class UserSerializer < ApplicationSerializer
has_many :posts, serializer: PostSerializer
end
パラメーターを渡すことはできません
app/serializers/user_serializer.rb
class UserSerializer < ApplicationSerializer
# hoge_param は PostSerializer に渡らない
has_many :posts, serializer: PostSerializer, hoge_param: 'hoge'
end
アソシエーションのシリアライザにパラメーターを渡したいときは手動でシリアライザを展開しなければなりません
app/serializers/user_serializer.rb
class UserSerializer < ApplicationSerializer
attribute :posts do
ActiveModelSerializers::SerializableResource.new(
object.posts,
each_serializer: PostSerializer,
hoge_param:'hoge',
adapter: :attributes
).serializable_hash
end
end
ルートキーにメタ情報を追加したいときは meta オプションを使う方が良さそう
キー名にこだわりがなくルートキーと同じ階層にメタ情報を追加したいときは meta を使うと楽です。
app/controllers/users_controller.rb
class UsersController < Api::ApplicationController
def show
render json: current_user,
serializer: UserSerializer
meta: {time: Time.zone.now}
end
end
meta を使わない場合は hash を自前で作ることになるのでコードが煩雑になると思われます。
app/controllers/users_controller.rb
class UsersController < Api::ApplicationController
def show
hash = { user: ActiveModelSerializers::SerializableResource.new(current_user,
serializer: UserSerializer, adapter: :attributes).serializable_hash,
meta: {time: Time.zone.now}
}
render json: hash
end
end
ディレクトリの切り方はコントローラーに準じた方が良さそう
使いたい user serializer がコントローラーのメソッドごとに違う場合は
app/controllers/users_controller.rb
class UsersController < Api::ApplicationController
def index
...
end
def show
...
end
end
serializers 配下にフラットにシリアライザファイルを作成するよりも
app/serializers/index_user_serializer.rb
app/serializers/show_user_serializer.rb
返却されるオブジェクトはAPIのエンドポイントに依存しているので、コントローラーのクラス名の名前空間、メソッドに応じてディレクトリを切った方がわかりやすいと思っています。
app/serializers/users/index/user_serializer.rb
app/serializers/users/show/user_serializer.rb
汎用的ではなく1つのエンドポイントでしか使われないシリアライザはシリアライザ内にクラス定義した方が良さそう
1つのエンドポイントでしか使われない特殊なシリアライザならばファイルに切り出して、コードを参照するときに別ファイルを見にいくよりも
app/serializers/special_post_serializer.rb
シリアライザクラス内にシリアライザを定義した方がコードを追いやすいかもしれません。
app/serializers/user_serializer.rb
class UserSerializer < ApplicationSerializer
attribute :special_post do
ActiveModelSerializers::SerializableResource.new(
object.posts,
serializer: UserSerializer::SpecialPostSerializer,
adapter: :attributes
).serializable_hash
end
class SpecialPostSerializer < ApplicationSerializer
attribute :special_post_key do
'value'
end
end
end
名前空間が変わらないので基本的には type を指定した方が良さそう
type メソッドを使うことでルートキーの名前を固定することができます。
app/serializers/user_serializer.rb
class UserSerializer < ApplicationSerializer
type 'user'
attribute :name
end
ActiveModelSerializers::SerializableResource.new(
user,
serializer: UserSerializer,
).serializable_hash
=> {user: {...}}
type を指定しないとシリアライザの名前空間によってルートキーの名前が変わってしまいます。
app/serializers/v2/user_serializer.rb
class V2::UserSerializer < ApplicationSerializer
attribute :name
end
ActiveModelSerializers::SerializableResource.new(
user,
serializer: V2::UserSerializer,
).serializable_hash
=> {'v2/user' => {...}}
名前空間に関わらず一意のリソースの名前を使うことの方が多いと思うので type は指定した方が良いと思われます。
# ルートキーが変わるとこのようなコードがデグレを起こす可能性がある
user_res = ActiveModelSerializers::SerializableResource.new(...).serializable_hash
user_res[:user][:name]
シリアライザ内でデータの出どころがわかりにくいので scope は使わない方が良さそう
scope を使用すると指定した情報をそのままシリアライザに持っていくことができますが、下記の例の場合 current_post がどこから来たのかがわかりにくいため
app/controllers/users_controller.rb
class UsersController < Api::ApplicationController
def show
render json: current_user,
serializer: UserSerializer,
scope: current_user.current_post,
scope_name: :current_post
end
end
app/serializers/user_serializer.rb
class UserSerializer < ApplicationSerializer
attribute :post_name do
current_post.name
end
end
通常のパラメーターで渡した方がわかりやすいと思っています。
app/controllers/users_controller.rb
class UsersController < Api::ApplicationController
def show
render json: current_user,
serializer: UserSerializer,
current_post: current_user.current_post
end
end
app/serializers/user_serializer.rb
class UserSerializer < ApplicationSerializer
attribute :post_name do
instance_options[:current_post].name
end
end
has_many のアソシエーションで order を使うときはそれ専用のアソシエーションをモデルで定義した方が良さそう
下記記事を参照してください。
has_manyで紐づくモデルをSerializerの中でorderするとSQLキャッシュのN+1が起きる問題をなんとかしたい
